Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ModuleController.Instance.UpdateModuleSetting Not Updating #2868

Closed
4 tasks done
mitchelsellers opened this issue Jun 23, 2019 · 3 comments
Closed
4 tasks done

ModuleController.Instance.UpdateModuleSetting Not Updating #2868

mitchelsellers opened this issue Jun 23, 2019 · 3 comments

Comments

@mitchelsellers
Copy link
Contributor

Description of the bug

When working with the ModuleController object and attempting to update a module setting the setting can be initially set, but subsequent calls to UpdateModuleSetting does not result in the value being changed

Example Code:

ModuleController.Instance.UpdateModuleSetting(moduleId, "MyName", "MyValue");

Steps to reproduce

  1. Have a module settings action, or otherwise, that will set a value for a Module Setting using the above snippet.
  2. After updating validate that the database has the proper value in the ModuleSettings table
  3. Try to update the setting value with a new value

Current result

The value in the DNN Database contains the older value, with the older timestamp. It is not possible to overwrite the setting value.

Expected result

The value in the database, and exposed to the code, should be the new value set. With the timestamp of when the last action was completed.

Additional context

This appears to only impact ModuleSettings, TabModuleSettings appear to be working as desired.

Affected version

  • 9.3.2
  • 9.3.1
  • 9.2.2
  • 9.2.1

Earlier versions were not verified

@valadas
Copy link
Contributor

valadas commented Jun 24, 2019

Is this always the case or only under some conditions, I am surprised this was not noticed earlier...

@mitchelsellers
Copy link
Contributor Author

I have tested both an MVC module and a WebForms module and the behavior is consistent.

@david-poindexter
Copy link
Contributor

Closing per #3360

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants