From e0db51a08e6c058abc394a385011aa011487fb4d Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Tue, 18 Oct 2005 04:13:46 +0000 Subject: [PATCH] =?utf8?q?Bug=20302835:=20The=20"confirm=20delete"=20page?= =?utf8?q?=20when=20deleting=20a=20user=20account=20should=20distinguish?= =?utf8?q?=20between=20risky=20and=20no=20risky=20actions=20-=20Patch=20by?= =?utf8?q?=20Fr=C3=A9d=C3=A9ric=20Buclin=20=20r=3Dwurbl?= =?utf8?q?zap=20a=3Djustdave?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- skins/standard/admin.css | 12 +- .../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 1f266b792d..ef02f69d9a 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 e8bfe8609d..afe464697a 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.

-- 2.47.2