Honour Required Fields & Administrator Approves Public Registration
Permalink Browser Info Environment
Hi Nicola,
It doesn't seem like this add on honours Required Fields & Administrator Approves Public Registration.
Normally users are forced to complete required fields after signing in, however Social Login seems to log the user straight in. Regarding Admin approves Public Registration, may be as simple as creating the user as inactive and showing text explaining that an Admin needs to approve their account.
Any ideas on how I could get this happening?
Cheers,
Tim
It doesn't seem like this add on honours Required Fields & Administrator Approves Public Registration.
Normally users are forced to complete required fields after signing in, however Social Login seems to log the user straight in. Regarding Admin approves Public Registration, may be as simple as creating the user as inactive and showing text explaining that an Admin needs to approve their account.
Any ideas on how I could get this happening?
Cheers,
Tim
Type: | Discussion |
---|---|
Status: | In Progress |
Hi Nicola,
Thanks for the response!
So I've tried both options in isolation, and neither seem to work.
For the mandatory fields, I tried a few things and couldn't seem to make it work. Firstly I suspected that there was missing ';' in the if statements, but there was still a problem after adding them. So I tried copying the login.php as instructed, and not changing the code. This seems to result in a 'Page not found' error when the do_login function is called.
Re the second option to register users as inactive, I tried out your solution, however the user is still active after registering.
Let me know if you have any other ideas, I'm keen to get this working as it's an awesome addon!
In case it's relevant, I'm running Concrete5 5.6.3.2
Thanks,
Tim
Thanks for the response!
So I've tried both options in isolation, and neither seem to work.
For the mandatory fields, I tried a few things and couldn't seem to make it work. Firstly I suspected that there was missing ';' in the if statements, but there was still a problem after adding them. So I tried copying the login.php as instructed, and not changing the code. This seems to result in a 'Page not found' error when the do_login function is called.
Re the second option to register users as inactive, I tried out your solution, however the user is still active after registering.
Let me know if you have any other ideas, I'm keen to get this working as it's an awesome addon!
In case it's relevant, I'm running Concrete5 5.6.3.2
Thanks,
Tim
Hi
This should work correctly with 5.6.3.2
Please replace social_login package with the attached one and add the login.php file in the controllers/ directory.
Let me know if you have any problem
Thanks
Regards
Nicola
This should work correctly with 5.6.3.2
Please replace social_login package with the attached one and add the login.php file in the controllers/ directory.
Let me know if you have any problem
Thanks
Regards
Nicola
Awesome, that did the job after clearing the cache & running the automated jobs. Looking back, I think your original instructions may have worked after just performing these tasks.
Thanks for your help!
Thanks for your help!
There is a small workaround to honour Required Fields:
To apply this fix you have to copy
concrete/core/controllers/single_pages/login.php
to
/controllers/login.php
Then, you have to add after
this code:
Then, you need to edit social_login/models/social_login.php
and replace
with
For Admin approval, as you said you can add the user inactive, please edit:
social_login/models/social_login.php
replace
with
This should work correctly.
Let me know if you have any problem with this solution or in case you prefer a refund.
Thanks
Regards
Nicola