summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorgerv%gerv.net <>2003-01-06 16:53:11 +0100
committergerv%gerv.net <>2003-01-06 16:53:11 +0100
commitef85a567fdb1c5d58223902316351d57f062897d (patch)
tree15ce0756f457f0d78ad97a7fa1dd973831872557
parented5513d9d67a8b789cfa20e29316cf3d025df0d5 (diff)
downloadbugzilla-ef85a567fdb1c5d58223902316351d57f062897d.tar.gz
bugzilla-ef85a567fdb1c5d58223902316351d57f062897d.tar.xz
Bug 187837 - Unify showing and editing of quips. Patch by gerv; r=timeless, a=justdave.
-rwxr-xr-xquips.cgi19
-rw-r--r--template/en/default/list/quips.html.tmpl77
2 files changed, 42 insertions, 54 deletions
diff --git a/quips.cgi b/quips.cgi
index f1783cd0a..d152234ac 100755
--- a/quips.cgi
+++ b/quips.cgi
@@ -46,23 +46,6 @@ my $action = $::FORM{'action'} || "";
if ($action eq "show") {
# Read in the entire quip list
- SendSQL("SELECT quip FROM quips");
-
- my @quips;
- while (MoreSQLData()) {
- my ($quip) = FetchSQLData();
- push(@quips, $quip);
- }
-
- $vars->{'quips'} = \@quips;
- $vars->{'show_quips'} = 1;
-}
-
-if ($action eq "edit") {
- if (!UserInGroup('admin')) {
- ThrowUserError("quips_edit_denied");
- }
- # Read in the entire quip list
SendSQL("SELECT quipid,userid,quip FROM quips");
my $quips;
@@ -83,7 +66,7 @@ if ($action eq "edit") {
$vars->{'quipids'} = \@quipids;
$vars->{'quips'} = $quips;
$vars->{'users'} = $users;
- $vars->{'edit_quips'} = 1;
+ $vars->{'show_quips'} = 1;
}
if ($action eq "add") {
diff --git a/template/en/default/list/quips.html.tmpl b/template/en/default/list/quips.html.tmpl
index ccb5208b7..c178c5838 100644
--- a/template/en/default/list/quips.html.tmpl
+++ b/template/en/default/list/quips.html.tmpl
@@ -62,46 +62,51 @@
</form>
[% IF show_quips %]
- <h2>
- Existing quips:
- </h2>
- <ul>
- [% FOREACH quip = quips %]
- <li>[% quip FILTER html %]</li>
- [% END %]
- </ul>
-[% ELSIF edit_quips %]
- <h2>Edit existing quips:</h2>
- <table border="1">
- <thead><tr>
- <th>Action</th>
- <th>User</th>
- <th>Quip</th>
- </tr></thead><tbody>
- [% FOREACH quipid = quipids %]
- <tr>
- <td>
- <a href="quips.cgi?action=delete&amp;quipid=[% quipid FILTER uri%]">
- Delete
- </a>
- </td>
- <td>
- [% userid = quips.$quipid.userid %]
- [% users.$userid FILTER html %]
- [% "Unknown" IF NOT users.$userid %]
- </td>
- <td>[% quips.$quipid.quip FILTER html %]</td>
- </tr>
- [% END %]
- </tbody></table>
+ [% IF !UserInGroup('admin') %]
+ <h2>
+ Existing quips:
+ </h2>
+ <ul>
+ [% FOREACH quip = quips %]
+ <li>[% quip FILTER html %]</li>
+ [% END %]
+ </ul>
+ [% ELSE %]
+ <h2>Edit existing quips:</h2>
+ <table border="1">
+ <thead><tr>
+ <th>Quip</th>
+ <th>Author</th>
+ <th>Action</th>
+ </tr></thead><tbody>
+ [% FOREACH quipid = quipids %]
+ <tr>
+ <td>[% quips.$quipid.quip FILTER html %]</td>
+ <td>
+ [% userid = quips.$quipid.userid %]
+ [% users.$userid FILTER html %]
+ [% "Unknown" IF NOT users.$userid %]
+ </td>
+ <td>
+ <a href="quips.cgi?action=delete&amp;quipid=[% quipid FILTER uri%]">
+ Delete
+ </a>
+ </td>
+ </tr>
+ [% END %]
+ </tbody>
+ </table>
+ <br>
+ [% END %]
[% ELSE %]
<p>
Those who like their wisdom in large doses can
- <a href="quips.cgi?action=show">view the whole quip list</a>.
+ <a href="quips.cgi?action=show">view
+ [% IF UserInGroup('admin') %]
+ and edit
+ [% END %]
+ the whole quip list</a>.
</p>
- [% IF UserInGroup('admin') %]
- <p><a href="quips.cgi?action=edit">Edit</a> the quip list.</p>
- [% END %]
[% END %]
[% PROCESS global/footer.html.tmpl %]