Hi,
I was cursing the name of this module's author all evening, until I realized the module works as intended and the problem I was having was simply that nowhere in the documentation does it mention you have to click the "Add" button on the Intervals page.
Although this is really stupid from a UI perspective (Don't make it look like a config form has default values when those values require an action to activate!), probably half the users who simply think this module is non-functional are experiencing what I did — I.e., any user that can't figure out why the "Refresh stats" page doesn't show anything and appears to hang.
Probably a better long-term solution would be to make those values actually default values — install them by default, let the user remove them if he or she dislikes having that level of granularity.
See attached patch for documentation.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 1801448-2-documentation_update.patch | 1.29 KB | aendra |
| #1 | 1801448-2-documentation_update.patch | 1.28 KB | aendra |
| documentation_fix_describing_intervals_page.patch | 1.28 KB | aendra |
Comments
Comment #1
aendra commentedI think I got it backwards — "Add" doesn't do anything, it's "Save" that makes everything work. Which makes sense, but is still pretty unintuitive. Doc patch updated to reflect that.
Comment #2
aendra commentedClarified a line. Sorry this patch is such a dog's lunch — haven't done too many of these.
Comment #3
seanrPatch looks OK, but would it be better to set the defaults on install? Maybe we should patch mostpopular.install to set those.
Comment #4
aendra commentedApologies for the delay. Yes, patching the .install file makes the most sense. Would write one myself but am super busy right now...