There is a bug with new user registration which causes the password to not be set and appears to be caused by the location field. The addressfield module recently fixed a bug allowing the field to be optional (https://www.drupal.org/node/968112). Although COD updated to the 1.0 release of the addressfield module, which included the fix, the location field in the cod_profile_base feature needs to be re-exported. Patch to follow...

CommentFileSizeAuthor
#1 cod_support-update-profile_location-field-2446727-1.patch1.47 KBtimwood
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

timwood’s picture

timwood
Primary language English
Location Rockville, Maryland
CreditAttribution: timwood commented
Related issues: +#968112: Allow addressfield to be optional
FileSize
cod_support-update-profile_location-field-2446727-1.patch1.47 KB
japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Status: Needs review » Fixed

Looks good! Thanks for pointing that out. I did not include an update hook to revert the view, mainly because I think each site needs to manually review profile data before reverting. But this works great for creating a starting point for new installs.

Fixed!

timwood’s picture

timwood
Primary language English
Location Rockville, Maryland
CreditAttribution: timwood commented
Status: Fixed » Needs work

After further testing and found that the patch does not resolve the fact that there is some kind of validation error. To test, apply the previously provided patch to a new COD site. Enable verify email during account registration. Follow one-time password reset link from registration email. Fill out required fields, including password/verify password, but NOT address. Keep address field collapsed. This works, password gets set. Account fully provisioned.

Now try again but this time add a country and fill in all the required fields in the address field form and then save. A validation error is thrown "Your current password is missing or incorrect; it's required to change the Password." The password doesn't get set. You can not set it because it never got set. You are stuck needing to go get another password reset link sent to you.

  • japerry committed 5d69379 on 7.x-1.x authored by timwood 
    Issue #2446727 by timwood: Update profile_location field for new...
japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry at Drupal Association commented
Status: Needs work » Needs review
Issue tags: +cod 7.x-1.0 radar

Needs re-testing for new installs.

CountPacMan’s picture

CountPacMan CreditAttribution: CountPacMan at Drupal Association commented
Status: Needs review » Reviewed & tested by the community

New installs work fine.

timwood’s picture

timwood
Primary language English
Location Rockville, Maryland
CreditAttribution: timwood commented

CountPanMan, did you follow steps to recreate the issue? Register a ticket & account and try using/not using the address field on profile?

CountPacMan’s picture

CountPacMan CreditAttribution: CountPacMan at Drupal Association commented
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

Yes - I wasn't able to replicate the issue. I'll have a co-worker double check on their new install.

BabaYaga64’s picture

BabaYaga64 CreditAttribution: BabaYaga64 commented
Status: Postponed (maintainer needs more info) » Needs review

Hi Tim,

I retraced Daniel's steps and was not able to recreate the issue.

  • japerry committed 5d69379 on 7.x-2.x authored by timwood 
    Issue #2446727 by timwood: Update profile_location field for new...

  • japerry committed 5d69379 on 7.x-1.x authored by timwood 
    Issue #2446727 by timwood: Update profile_location field for new...