Forum post delay setting only sticks after you clicked "Save"
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Mahara |
Low
|
Aaron Wells | ||
| 1.6 |
Low
|
Unassigned | ||
| 1.7 |
Low
|
Unassigned | ||
| 1.8 |
Low
|
Unassigned |
Bug Description
When you have a fresh install of Mahara (tested it on a 1.5 and a master), the forum post delay setting which defaults to 30 minutes is not taken into consideration when you post a forum message.
Only when you go into the plugin itself /admin/
Changed in mahara: | |
status: | New → Triaged |
importance: | Undecided → Low |
tags: | added: bite-sized |
tags: |
added: forums removed: forum |
Robert Lyon (robertl-9) wrote : | #1 |
Aaron Wells (u-aaronw) wrote : | #2 |
So currently, as Robert mentioned, it doesn't *actually* set the interaction_
So in addition to *actually* setting the config to 30 at installation time, we should also change that form so that it *always* shows what the value really is.
Aaron Wells (u-aaronw) wrote : | #3 |
Changed in mahara: | |
milestone: | none → 1.9.0 |
status: | Triaged → In Progress |
Reviewed: https:/
Committed: http://
Submitter: Robert Lyon (<email address hidden>)
Branch: master
commit 5fb2dd021c471c6
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 17:59:30 2013 +1300
Be honest about the forum post delay's current value
Bug 996337: We always treat it as an integer, so it makes the most sense
to just show the user this config's integer value.
Change-Id: Ia4b198b0f642b9
Mahara Bot (dev-mahara) wrote : | #5 |
Reviewed: https:/
Committed: http://
Submitter: Robert Lyon (<email address hidden>)
Branch: master
commit dedacfa40326e64
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 18:07:10 2013 +1300
Set the new install default forum post delay to 30 minutes
Change-Id: I9a6f5b94fb0551
Changed in mahara: | |
status: | In Progress → Fix Committed |
assignee: | nobody → Aaron Wells (u-aaronw) |
Robert Lyon (robertl-9) wrote : | #6 |
Patches for v1.6
https:/
https:/
Patches for v1.7
https:/
https:/
Patches for v1.8
https:/
https:/
Mahara Bot (dev-mahara) wrote : | #7 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.6_STABLE
commit a26f631134d5404
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 17:59:30 2013 +1300
Be honest about the forum post delay's current value
Bug 996337: We always treat it as an integer, so it makes the most sense
to just show the user this config's integer value.
Change-Id: Ia4b198b0f642b9
Mahara Bot (dev-mahara) wrote : | #8 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.6_STABLE
commit 72adce8604eeced
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 18:07:10 2013 +1300
Set the new install default forum post delay to 30 minutes
Change-Id: I9a6f5b94fb0551
Mahara Bot (dev-mahara) wrote : | #9 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.7_STABLE
commit 719069721f6b77f
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 17:59:30 2013 +1300
Be honest about the forum post delay's current value
Bug 996337: We always treat it as an integer, so it makes the most sense
to just show the user this config's integer value.
Change-Id: Ia4b198b0f642b9
Mahara Bot (dev-mahara) wrote : | #10 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.7_STABLE
commit 743e52f1fae4ff4
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 18:07:10 2013 +1300
Set the new install default forum post delay to 30 minutes
Change-Id: I9a6f5b94fb0551
Mahara Bot (dev-mahara) wrote : | #11 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.8_STABLE
commit 9859f2622228c66
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 17:59:30 2013 +1300
Be honest about the forum post delay's current value
Bug 996337: We always treat it as an integer, so it makes the most sense
to just show the user this config's integer value.
Change-Id: Ia4b198b0f642b9
Mahara Bot (dev-mahara) wrote : | #12 |
Reviewed: https:/
Committed: http://
Submitter: Son Nguyen (<email address hidden>)
Branch: 1.8_STABLE
commit a6af1b8f98f12b6
Author: Aaron Wells <email address hidden>
Date: Wed Nov 20 18:07:10 2013 +1300
Set the new install default forum post delay to 30 minutes
Change-Id: I9a6f5b94fb0551
Changed in mahara: | |
status: | Fix Committed → Fix Released |
Changed in mahara: | |
status: | Fix Released → Fix Committed |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
Looks as though the variable is not getting set in the database on install - it was getting set if you upgraded from a site before 2009-08-17
Really need to have the setting of db value as a post install option in core_install_ lastcoredata_ defaults( ) [or similar] like how the flexible layouts changes (view_layout) are done.