From 1a84cc52fea5f653e51a6ec43c778d4452351964 Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Tue, 18 Oct 2005 04:10:40 +0000 Subject: Bug 302835: The "confirm delete" page when deleting a user account should distinguish between risky and no risky actions - Patch by Frédéric Buclin r=wurblzap a=justdave MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- skins/standard/admin.css | 12 +- .../default/admin/users/confirm-delete.html.tmpl | 502 +++++++++++---------- 2 files changed, 274 insertions(+), 240 deletions(-) diff --git a/skins/standard/admin.css b/skins/standard/admin.css index 1f266b792..ef02f69d9 100644 --- a/skins/standard/admin.css +++ b/skins/standard/admin.css @@ -13,12 +13,20 @@ * Contributor(s): Marc Schumann */ -ul.warningmessages { +.warningmessages, .criticalmessages { background-color: white; border-style: solid; border-width: 1px; - border-color: yellow; padding: 1ex 1ex 1ex 4ex; + margin: 1ex; +} + +.warningmessages { + border-color: yellow; +} + +.criticalmessages { + border-color: red; } p.areyoureallyreallysure { diff --git a/template/en/default/admin/users/confirm-delete.html.tmpl b/template/en/default/admin/users/confirm-delete.html.tmpl index e8bfe8609..afe464697 100644 --- a/template/en/default/admin/users/confirm-delete.html.tmpl +++ b/template/en/default/admin/users/confirm-delete.html.tmpl @@ -131,252 +131,278 @@

Confirmation

- [% IF reporter || assignee_or_qa || bug_activity || cc || flags.requestee || - flags.setter || longdescs || namedqueries || profiles_activity || series || - votes || watch.watched || watch.watcher || whine_events || whine_schedules %] - + + [% display_warning = 1 %] + [% END %] + + [% IF assignee_or_qa || cc || flags.requestee || namedqueries || + series || votes || watch.watched || watch.watcher || + whine_events || whine_schedules %] +
+

The following deletions are safe and will not generate + referential integrity inconsistencies.

+ +
    + [% IF assignee_or_qa %] +
  • + [% otheruser.login FILTER html %] + is + the assignee or the QA contact of + [% IF assignee_or_qa == 1 %] + one [% terms.bug %] + [% ELSE %] + [%+ assignee_or_qa %] [%+ terms.bugs %] + [% END %]. + If you delete the user account, these roles will fall back to + the default assignee or default QA contact. +
    • + [% END %] + [% IF cc %] +
  • + [% otheruser.login FILTER html %] + is + on the CC list of + [% IF cc == 1 %] + [%+ terms.abug %] + [% ELSE %] + [%+ cc %] [%+ terms.bugs %] + [% END %]. + If you delete the user account, it will be removed from these CC lists. +
    • + [% END %] + [% IF flags.requestee %] +
  • + [% otheruser.login FILTER html %] has been + asked + to set + [% IF flags.requestee == 1 %] + a flag + [% ELSE %] + [% flags.requestee %] flags + [% END %]. + If you delete the user account, + [% IF flags.requestee == 1 %] + this flag + [% ELSE %] + these flags + [% END %] + will change to be unspecifically requested. +
    • + [% END %] + [% IF namedqueries %] +
  • + [% otheruser.login FILTER html %] has + [% IF namedqueries == 1 %] + a named query + [% ELSE %] + [%+ namedqueries %] named queries + [% END %]. + [% IF namedqueries == 1 %] + This named query + [% ELSE %] + These named queries + [% END %] + will be deleted along with the user account. +
    • + [% END %] + [% IF series %] +
  • + [% otheruser.login FILTER html %] has created + [% IF series == 1 %] + a series + [% ELSE %] + [%+ series %] series + [% END %]. + [% IF series == 1 %] + This series + [% ELSE %] + These series + [% END %] + will be deleted along with the user account. +
    • + [% END %] + [% IF votes %] +
  • + [% otheruser.login FILTER html %] has voted on + [% IF votes == 1 %] + [%+ terms.abug %] + [% ELSE %] + [%+ votes %] [%+ terms.bugs %] + [% END %]. + If you delete the user account, + [% IF votes == 1 %] + this vote + [% ELSE %] + these votes + [% END %] + will be deleted along with the user account. +
    • + [% END %] + [% IF watch.watched || watch.watcher %] +
  • + [% otheruser.login FILTER html %] + [% IF watch.watched %] + is being watched by + [% IF watch.watched == 1 %] + a user + [% ELSE %] + [%+ watch.watched %] users + [% END %] + [% END %] + [% IF watch.watcher %] + [%+ 'and' IF watch.watched %] + watches + [% IF watch.watcher == 1 %] + a user + [% ELSE %] + [%+ watch.watcher %] users + [% END %] + [% END %]. + [% IF watch.watched + watch.watcher == 1 %] + This watching + [% ELSE %] + These watchings + [% END %] + will cease along with the deletion of the user account. +
    • + [% END %] + [% IF whine_events || whine_schedules %] +
  • + [% otheruser.login FILTER html %] + [% IF whine_events %] + has scheduled + [% IF whine_events == 1 %] + a whine + [% ELSE %] + [%+ whine_events %] whines + [% END %] + [% END %] + [% IF whine_schedules %] + [%+ 'and' IF whine_events %] + is on the receiving end of + [% IF whine_schedules == 1 %] + a whine + [% ELSE %] + [%+ whine_schedules %] whines + [% END %] + [% END %]. + [% IF whine_events + whine_schedules == 1 %] + This whine + [% ELSE %] + These whines + [% END %] + will be deleted along with the user account. +
    • + [% END %] +
+
+ [% display_warning = 1 %] + [% END %] + + [% IF display_warning %]

Please be aware of the consequences of this before continuing.

-- cgit v1.2.3-24-g4f1b