Correct capitalization of time registration expiry selector

Bug #1773600 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Unassigned

Bug Description

In Admin -> Configure site -> Site options -> Account settings you can choose the "Default registration expiry lifetime". This drop-down menu displays the correct options with the correct capitalization.

However, when an admin receives the registration request email, the time period is capitalized as it is taken from the drop-down menu, e.g.

"You will need to approve or deny this registration request within 2 Weeks."

However, it should read:

"You will need to approve or deny this registration request within 2 weeks."

A new set of strings should be set up that shows "days", "weeks", "months", "years" not capitalized so that these strings can be used in the HTML and plain text emails sent out for this message. Unfortunately, it's not just a case of renaming the strings as their capitalized versions need to be kept in the pieforms element for the drop-down menu.

To replicate:

1. Activate the "Default registration expiry lifetime" and set it to a number of days / weeks / months / years.
2. turn on "Confirm registration" in an institution on the site and allow self-registration.
3. Self-register for an account in the institution.
4. Check the email you'd receive either as admin of the institution or site admin.

Expected result: "You will need to approve or deny this registration request within 2 weeks." (or days / months / years)
Actual result: "You will need to approve or deny this registration request within 2 Weeks." (or Days / Months / Years)

All options, including the "No end date set" one should be tested as well as the plain text and HTML messages.

Tags: bite-sized
Lisa Seeto (lisaseeto)
Changed in mahara:
status: Confirmed → In Progress
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/9582

Revision history for this message
Lisa Seeto (lisaseeto) wrote :

I have updated lang/en.utf8/pieforms with new lowercase strings for both plural and singular units of time.
Please note that the previous capitlized strings are still in place and have NOT been removed from the file.

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

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

commit 6ce6a4f5ccc47a6060f9524ca0aef57c21eb8936
Author: Lisa Seeto <email address hidden>
Date: Tue Feb 19 17:31:15 2019 +1300

Bug 1773600: Correct captilization of time registration expiry selector

Created new lowercase string arrays for singular and plural time units.
Altered the expirystring to reflect changes.

behatnotneeded

Change-Id: I1555325f24843a628305cd6afe9cc9b45f8d5d94
Signed-off-by: Lisa Seeto <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 19.04.0
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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