MantisBT - ATutor
View Issue Details
0003185ATutorAdminpublic2007-10-13 02:332008-08-21 08:57
IndieRect 
greg 
normalminoralways
closedfixed 
1.5.5 
1.6 
SVN
0003185: Imperfect message when deleting admin
When deleting an administrator, a message about instructors which own courses is shown.
Additionally, the admin's name is output in the same line.

Solution:
1. Create a separate language variable for deleting an admin.
and
2. Change line 54 (as of 1.5.5) of /admin/admins/delete.php to:
    $name_html = '
  • '.$row['login'].'</il>
';
    $confirm = array('DELETE_USER', $name_html);
No tags attached.
Issue History
2007-10-13 02:33IndieRectNew Issue
2007-10-13 02:33IndieRectAffects version => SVN
2007-12-09 04:36gregStatusnew => resolved
2007-12-09 04:36gregFixed in Version => 1.6
2007-12-09 04:36gregResolutionopen => fixed
2007-12-09 04:36gregAssigned To => greg
2007-12-09 04:36gregNote Added: 0002592
2008-01-22 00:16IndieRectStatusresolved => feedback
2008-01-22 00:16IndieRectResolutionfixed => reopened
2008-01-22 00:16IndieRectNote Added: 0002760
2008-01-22 00:17IndieRectNote Edited: 0002760
2008-01-22 04:05gregStatusfeedback => resolved
2008-01-22 04:05gregResolutionreopened => fixed
2008-01-22 04:05gregNote Added: 0002761
2008-08-21 08:57gregStatusresolved => closed

Notes
(0002592)
greg   
2007-12-09 04:36   
Replaced DELETE_USER with DELETE_ADMIN lang var, and added a BR to move listed admins to the next line
(0002760)
IndieRect   
2008-01-22 00:16   
(edited on: 2008-01-22 00:17)
ATutor version: 1.6 RC1

When deleting an admin, the confirmation message reads:
"Are you sure you want to delete the following administrator(s)?"
The admins that are going to be deleted are not listed.

Supposed reason:
In the AT_CONFIRM_DELETE_ADMIN language var, "%s" is absent.

More info:
Compare AT_CONFIRM_DELETE_ADMIN to AT_CONFIRM_DELETE_USER.

(0002761)
greg   
2008-01-22 04:05   
Language string missing the replacement variable %s. Language db for en has been updated.