Monitor Reply

Support

Honour Required Fields & Administrator Approves Public Registration

Permalink Browser Info Environment December 03, 2014 at 3:18 PM

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

Type:	Discussion
Status:	In Progress

hanicker replied on Dec 5, 2014 at 3:22 am Permalink Reply

Hi. Sorry for this late reply.

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

public function do_login() {

this code:

if(!empty($_SESSION['social_login_uName']))
   $_POST['uName']=$_SESSION['social_login_uName']
if(!empty($_SESSION['social_login_uPassword']))
   $_POST['uPassword']=$_SESSION['social_login_uPassword']

Then, you need to edit social_login/models/social_login.php
and replace

$u->loginByUserID($ui->uID);
                     $currentPage = Page::getCurrentPage();
                     Loader::helper('navigation');
                     header('Location: '.NavigationHelper::getLinkToCollection($currentPage, true));
                     die();

with

$_SESSION['social_login_uName']=$data['uName'];
                     $_SESSION['social_login_uPassword']=$data['uPassword'];
                     header('Location: /index.php/login/do_login/');
                     die();

For Admin approval, as you said you can add the user inactive, please edit:
social_login/models/social_login.php
replace

$data['uIsValidated'] = '1';

with

$data['uIsValidated'] = '0';

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

viDigital replied on Dec 6, 2014 at 3:46 am Permalink Reply

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

hanicker replied on Dec 7, 2014 at 7:35 pm Permalink Reply 2 Attachments

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

viDigital replied on Dec 16, 2014 at 5:37 am Permalink Reply

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!

Social Login

Support

Honour Required Fields & Administrator Approves Public Registration

Issue Reply

concrete5 Environment Information

Browser User-Agent String

Hide Post Content

Request Refund

Code

Delete Post