Hi!

Great module! I intend to use it to increase usability in the Commons distribution and to solve real names issue together with realname module (#912462: Add Real Names support to Drupal Commons).

But, there is one flaw: module is checking if content_permission module is enabled without checking if content_profile is enabled too.

As I'm not using git yet, I will explain fix here:

Line 218 @ realname_registration.admin.inc file:

if (module_exists('content_permissions')) {

should be changed to:

if (module_exists('content_permissions') && module_exists('content_profile')) {

Thx.

Comments

sgerbino’s picture

sgerbino commented
Assigned: Unassigned » sgerbino
Status: Active » Fixed

Thank you for reporting the issue. In addition, it needed to check if the user is actually using content_profile fields (inside the function).

In case you or anyone has hacked the module to get this desired functionality;
On Line 377, Instead of:

if (isset($form_state['values']['realname_registration_firstname_field'])) {

It needed to be:

if (isset($form_state['values']['realname_registration_firstname_field']) && $form_state['values']['realname_registration_use_content_profile_firstname_field']) {

And so on for each of the fields.

I pushed the fix upstream so it will show up on the development branch soon, I will try to quickly release another RC after I add hook_perm and test it out a bit.

izkreny’s picture

izkreny commented

Thx for the quick response and fix, the dev branch will be just fine for now. :)

sgerbino’s picture

sgerbino commented
Status: Fixed » Closed (fixed)