- Issue created by @ericgsmith
- Merge request !30Issue #3485998: Fix missing . separator between key and property when getting... β (Merged) created by ericgsmith
- First commit to issue fork.
- πΊπΈUnited States tr Cascadia
Thanks for noticing that and figuring out the problem @ericgsmith!
Unfortunately my original tests only checked the data types of the configuration variables before and after the update hook, but did not check the actual values. If the tests checked the values, then that problem would have been caught. So what I did is, in addition to your fix, I updated the tests so that they would check both the data types and values. Notably, as you can see in the test-only output (i.e. the new test run WITHOUT your fix) the test now catches the problem. The test also now proves your fix works.
Fixing
hook_update_10200()
like this will prevent other people from having this same problem, so I'm going to commit this change and push out a new release.Unfortunately, anyone who has already run the old version of
hook_update_10200()
will have already had the values oftotals_per_page
andprecision
both set to 0 - that can't be fixed automatically, and will require anyone experiencing that problem to edit and re-save their Views in order to set those variables how they want. -
tr β
committed c825f21a on 2.1.x authored by
ericgsmith β
Issue #3485998 by ericgsmith, tr: Totals per page and column precision...
-
tr β
committed c825f21a on 2.1.x authored by
ericgsmith β
Automatically closed - issue fixed for 2 weeks with no activity.