mirror of
https://github.com/JLiscom/OpenNote.git
synced 2026-04-25 16:25:49 +03:00
[GH-ISSUE #26] Unable to Register on fresh install #22
Labels
No labels
bug
enhancement
pull-request
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/OpenNote-JLiscom#22
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @dnovotny on GitHub (Dec 18, 2013).
Original GitHub issue: https://github.com/JLiscom/OpenNote/issues/26
I have downloaded the latest release last night, commit
aea61fb3cf, and on install I can't create a new account. The register button does not appear. I was trying to test out the search function, I am not sure if that's there or not, but it wasn't working in the demo site before that commit.@JLiscom commented on GitHub (Dec 19, 2013):
Sorry about that. Did you you have it installed already?
That button is controlled by line 51 in the config file:
https://github.com/FoxUSA/OpenNote/blob/master/OpenNote/Config.php#L51
setting the registrationEnabled to true should make that button appear.
Can you confirm that is what its set to?
@dnovotny commented on GitHub (Dec 19, 2013):
No, I did not have it installed previously, and yes, that is what it was set to by default.
I just installed it on a different server, and had the same result. However, if I comment out the check:
https://github.com/FoxUSA/OpenNote/blob/master/OpenNote/controller/modules/login/index.php#L38
Then it works correctly, and now the copyright statement shows up as well, though the registration doesn't actually complete. I will try to look more into it tonight.
@dnovotny commented on GitHub (Dec 19, 2013):
Found it, https://github.com/FoxUSA/OpenNote/blob/master/OpenNote/controller/modules/login/Config.php#L15 it is still referencing Common.php instead of common.php after commit
8c030acd3b.@JLiscom commented on GitHub (Dec 19, 2013):
I fixed it in the head.
I try to use camel case for script and pascal case for types. It causes issues on some systems if I didnt type it in an includes statement correctly. Thank you for reporting this.