Mahara

Site options - Event log expiry field not enabled

Bug #1686885 reported by Ghada El-Zoghbi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Pramith Dayananda
Mahara 17.10.0
17.04
Fix Released
High
Unassigned
Mahara 17.04.1
17.10
Fix Released
High
Pramith Dayananda
Mahara 17.10.0

Bug Description

Mahara: 17.04
OS: Linux
DB: Postgres
Browser: FF

After a a vanilla install of mahara 17.04, while setting up the site options, the field 'Event log expiry' under the Logging setting is not enabled when 'Days', 'Weeks' or 'Months' is selected from the drop down.

I have to select 'Months', save, then come back into the screen. The field is then enabled and I'm allowed to enter a number in the field.

Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

Marked as "High" as you can't select a time period.

tags: added: regression
Kristina Hoeppner (kris-hoeppner)
tags: added: bite-sized
Revision history for this message
Pramith Dayananda (pramithkm) wrote :

Hi All,

There are other form elements affected by this issue. I identify them as

Site Options -> Event log expiry
Account Settings -> Default registration expiry lifetime
Account Settings -> Default account lifetime
Account Settings -> Default account inactivity time
Account Settings -> Warning time for inactivity / expiry

There's a javascript code on admin options which could be obsolete.

Thanks,
Pramith.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/7715

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "master" branch: https://reviews.mahara.org/7735

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7715
Committed: https://git.mahara.org/mahara/mahara/commit/36abf4b9cab3094b5760ca94c7964b50579f8722
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 36abf4b9cab3094b5760ca94c7964b50579f8722
Author: Pramith Dayananda <email address hidden>
Date: Fri May 5 13:04:04 2017 +1200

Bug 1686885: Site options, fix jquery on fields

Site options > Logging Settings > Event log expiry
Site options > Account Settings > Def reg expiry
Site options > Account Settings > Def acc lifetime
Site options > Account Settings > Def acc inactive time
Site options > Account Settings > Warn time for inactive

behatnotneeded

Change-Id: Ia4596cc6da1842ddedf170d2f707412323dda0c9

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/7744

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7744
Committed: https://git.mahara.org/mahara/mahara/commit/3d1a24fd1aa1a065fb605b74a07dee38f25d1af1
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit 3d1a24fd1aa1a065fb605b74a07dee38f25d1af1
Author: Pramith Dayananda <email address hidden>
Date: Fri May 5 13:04:04 2017 +1200

Bug 1686885: Site options, fix jquery on fields

Site options > Logging Settings > Event log expiry
Site options > Account Settings > Def reg expiry
Site options > Account Settings > Def acc lifetime
Site options > Account Settings > Def acc inactive time
Site options > Account Settings > Warn time for inactive

behatnotneeded

Change-Id: Ia4596cc6da1842ddedf170d2f707412323dda0c9
(cherry picked from commit 36abf4b9cab3094b5760ca94c7964b50579f8722)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/7735
Committed: https://git.mahara.org/mahara/mahara/commit/f9fb633089775fef663fa61df8983973eb837c5b
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit f9fb633089775fef663fa61df8983973eb837c5b
Author: Robert Lyon <email address hidden>
Date: Tue May 9 12:50:42 2017 +1200

Bug 1686885: Behat test for 'expiry' field

Change-Id: I105ad1b7f791e312b65ad9b4eaf9a1eac0f24aca
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/7745

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/7745
Committed: https://git.mahara.org/mahara/mahara/commit/01d4ffd50b7fc6116edb36115ae1e30e7e3c4d43
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit 01d4ffd50b7fc6116edb36115ae1e30e7e3c4d43
Author: Robert Lyon <email address hidden>
Date: Tue May 9 12:50:42 2017 +1200

Bug 1686885: Behat test for 'expiry' field

Change-Id: I105ad1b7f791e312b65ad9b4eaf9a1eac0f24aca
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit f9fb633089775fef663fa61df8983973eb837c5b)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.