#5614 - Error when updating password and smart topic notifications disabled
| Identifier | #5614 |
|---|---|
| Issue type | Major issue (breaks an entire feature) |
| Title | Error when updating password and smart topic notifications disabled |
| Status | Completed |
| Tags |
Roadmap: v11 (custom) |
| Handling member | PDStig |
| Version | 10.0.45 |
| Addon | core |
| Description | When a member tries to reset their password and do not have 'Smart topic notification' checked in their profile they receive the error: "The field cms_smart_topic_notification is required ... |
| Steps to reproduce | |
| Additional information | Patrick: Will also need ported to v11. Make sure admin_version in v11 is bumped to 19, not 18, as v10 had to be bumped to 18. Edit relevant install code that was originally for 18 to be for 19. |
| Funded? | No |
| Commits |
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".


Comments
A resolution is going to be tricky because this involves deleting and re-creating the custom field (with required set to false) while migrating all of the member's values over to the newly-created field. As you mentioned your site has close to 100,000 members, I have to perform careful testing of the upgrade process (and simulate an upgrade on a site with 100,000 dummy members) before I make this fix available.
Smart topic notifications CPF was required when it should not be. This patch fixes the CPF by editing it so it is not required, and thus won't trigger the server-side field validation.
This is a substantial fix! Take great care when applying it. Read all directions below carefully:
1. MAKE A BACKUP of your website and TEST THE BACKUP to make sure it works.
2. Extract the files from this archive into your Composr installation; replace files when prompted.
3. Log in to your site as an admin.
4. Go to the Admin Zone > Structure > Addons > Module management
5. Select Admin Zone
6. Scroll down to admin_version. It should say that it requires an upgrade.
7. Click the action button to upgrade the module.
8. You should get a successful message within a few seconds.
NOTE: This bug fix was only tested on the Conversr forum driver. It might not work with other unsupported drivers.
I went a different approach to fixing the issue which is much more memory / resource friendly and did not require migrating field data. On a test site with 100,000 members, it only took about 2-3 seconds for the module to upgrade.