Do not display the username to a person registering for an account

Bug #803747 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Dianne Tennent

Bug Description

The account denial email (master) in htdocs/auth/internal/lang/en.utf8/auth.internal.php shows the username after the name of the administrator. That should not happen.

Accound denial email:
As this is an email that looks like it's bein sent from a direct user, it's weird that it is more impersonal than the other.

Hi Name,

Your registration attempt at Mahara has been denied.

Reason:

"No students accepted"

--
Regards,
Kristina Hoeppner (kristina)

-------------------------------------------

Note: Some of the wording may differ when you look at the text depending on whether the change for bug #803733 was accepted.

Tags: bite-sized
Revision history for this message
François Marier (fmarier) wrote :

This is due to the display_name() function including the username of a person when it's run as admin.

Changed in mahara:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Robert Lyon (robertl-9) wrote :

The display_name() function can take a $nameonly variable

So maybe we could go display_name($USER, null, true) and then change the display_name function's line from

$addusername = $username || !empty($userto->admin) || !empty($userto->staff);

to

$addusername = $username || (!empty($userto->admin) && $nameonly == false) || (!empty($userto->staff) && $nameonly == false);

Robert Lyon (robertl-9)
Changed in mahara:
status: Triaged → Confirmed
milestone: none → 1.9.0
Aaron Wells (u-aaronw)
Changed in mahara:
milestone: 1.9.0 → 1.10.0
Changed in mahara:
milestone: 1.10.0 → none
tags: added: bite-sized
Changed in mahara:
assignee: nobody → Dianne Tennent (dianne-t)
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/12039

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

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

commit c9b221692938a3cbd357dbe06e32e5b3aa357a7d
Author: Dianne Tennent <email address hidden>
Date: Tue Sep 28 09:52:59 2021 +1300

Bug 803747: Remove username from denial email

When user receives denial of registration email
message, no longer has username in brackets, just
the sender's full name.

Change-Id: I427885128ee699eac50ca06a93709ce6dbe256b7

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 21.10.0
Robert Lyon (robertl-9)
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.