Gervase Markham [:gerv] Assignee
	Description • 22 years ago

Bugzilla currently has the following static HTML files in the root directory:
bug_status.html      helpemailquery.html  notargetmilestone.html  votehelp.html
bugwritinghelp.html  help.html            quicksearchhack.html
confirmhelp.html     how_to_mail.html     quicksearch.html

Translating these files is hard where they are, because there is no possibility
for multiple versions. We should make them into page.cgi pages; this gives them
proper Bugzilla headers and footers, and makes them translatable.

This will also involve changing quite a few links in these documents and others.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 1 • 22 years ago

Plan:

Eliminated as irrelevant/unused (to be CVS removed):
help.html
how_to_mail.html
helpemailquery.html
confirmhelp.html
notargetmilestone.html

Ones I'd like to kill off:
votehelp.html 
(on the basis that this sort of thing should be in the Guide.)

Leave alone for the moment:
quicksearch.html
quicksearchhack.html
on the basis that quicksearch is being rewritten and consultation is necessary.

Templatise:
bug_status.html
bugwritinghelp.html

How does that sound?

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 2 • 22 years ago

Matty: bbaetz says you've looked at this issue as part of custres...

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 3 • 22 years ago

Myk, bbaetz, dave: I plan to CVS remove the list of files above marked
"Eliminated" quite soon, unless any of you want to step in to save them. 

I also have page.cgi-isations of bug_status.html and bugwritinghelp.html, and
I'll write up that patch in a minute. OK, bug_status may not be part of page.cgi
for ever, but people are wanting to translate it now.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 4 • 22 years ago

This patch page.cgi-ises the Bug Writing Guidelines and the status description
page.

Gerv

	Bradley Baetz (:bbaetz)
	Comment 5 • 22 years ago

Comment on attachment 100520 [details] [diff] [review]
Patch v.1

The 'bug writing help' one is way to bmo specific. I realise that you're just
copying the file in 'as-is', but I don't really see the point, if we're not
going to have a more generic version, or something.

	Gervase Markham [:gerv] Assignee
	Comment 6 • 22 years ago

> The 'bug writing help' one is way to bmo specific.

The examples relate to Mozilla - but they have to relate to something, otherwise
they wouldn't be examples. Many sites use only-slightly-modified versions of
these; and putting them in the template directory actually makes this easier,
because they won't get CVS conflicts.

Gerv

	Ran
	Comment 7 • 22 years ago

Template names fit with new naming scheme?

	Gervase Markham [:gerv] Assignee
	Comment 8 • 22 years ago

Yes, they do. I wrote the new (now rather old) naming scheme. :-)

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 9 • 22 years ago

help.html, how_to_mail.html, helpemailquery.html, confirmhelp.html and
notargetmilestone.html have been CVS removed.

Gerv

	Matthew Tuck [:CodeMachine]
	Comment 10 • 22 years ago

bug_status is going to become a CGI by customised resolutions.

bugwritinghelp was going to become a part of the user documentation that was
planned for bug #109311.

	Gervase Markham [:gerv] Assignee
	Comment 11 • 22 years ago

> bug_status is going to become a CGI by customised resolutions.

I'm sure it is - but people want to translate it now. The current system can
always be changed again when the CGI comes along. In fact, there's a pretty
strong chance that by the time we get there, a page.cgi-based solution will work
for enumerating bug statuses.

> bugwritinghelp was going to become a part of the user documentation that was
> planned for bug #109311.

Hmm. We need to discuss whether a Bugzilla installation's doc tree should be
exposed via HTTP and, if so, whether it's appropriate for Bugzilla to link into it.

Gerv

	Dave Miller [:justdave]
	Comment 12 • 22 years ago

see also bug 191478

	Oscar Manuel Gómez Senovilla
	Comment 13 • 21 years ago

In bug 215149, I'm working in a kind of, apart what is about here (I would have
created a duplicate if not writing right now). Taking that, where there are
several descs like fields, bugs, resolutions, etc. ready for localization, I
have templatised bug_status.html into bug_status.html.tmpl, and it's only
missing the initial legal stuff.

Any chances for you to take a look? Should I create an attachment?

	Dave Miller [:justdave]
	Comment 14 • 21 years ago

*** Bug 106612 has been marked as a duplicate of this bug. ***

	Dave Miller [:justdave]
	Comment 15 • 21 years ago

picking up the dependencies from the duplicate...

	Oscar Manuel Gómez Senovilla
	Comment 20 • 21 years ago

These templates work for me. Maybe only some "legal" stuff should be added.

	Oscar Manuel Gómez Senovilla
	Comment 21 • 21 years ago

Well, I've corrected several links inside the attachtments, but to be precise, I
have that at least index.html.tmpl has some links pointing to <file>.html, and
it seems the index.html.tmpl supplied in the 2.17 (and previous) in the tarball
doesn't match the one existing in bmo. I'll keep searching while somebody makes
a suggestion about how to handle this and then I'll provide a patch for the
attachments above, making them obsolete. If it's the first time, just take a
look at them, but be aware they need one more review.

	Gervase Markham [:gerv] Assignee
	Comment 22 • 21 years ago

> it seems the index.html.tmpl supplied in the 2.17 (and previous) in the tarball
> doesn't match the one existing in bmo.

Indeed. b.m.o. has a customised front page, and the admins are responsible for
updating it when they move to a version containing this change.

Your attachments are a good start, but templatisation is also an opportunity to
reformat, fix HTML, make pages validate, remove cruft like the "last mod time"
at the bottom of bug_status, make the page names fit with our naming scheme, and
so on. Is there any chance you could give the pages a spring clean? I believe my
patch in this bug did some of that - feel free to use it as a starting point.

For the names, I suggest fields.html.tmpl (as it doesn't just cover statuses),
guidelines.html.tmpl for the bug writing guidelines, quicksearch.html.tmpl and
advanced-quicksearch.html.tmpl . But feel free to suggest other possibilities.

Gerv

	Oscar Manuel Gómez Senovilla
	Comment 23 • 21 years ago

Gerv: I've applied your patch, and it's good (well, some rejects, but it could
be my fault).

I'm claiming for a review of code style and variable names of bug 215149, where
I templatize the bug descriptions and other common "key" fields for bz in
global/field-descs.none.tmpl, so I'd rewrite your bug-status-writing.html.tmpl
(for instance) and where it's said (it's an example):

"The bug has the status NEW"

I replace with

"The bug has the status [% bug_status_descs.NEW %]"

which centralizes the status (and other descriptions) avoiding typos and work
time when translating.

This somehow blocks my further work, until somebody corrects or grants the work.

	Oscar Manuel Gómez Senovilla
	Comment 24 • 21 years ago

Details of template are in bug 215148

	Oscar Manuel Gómez Senovilla
	Comment 25 • 21 years ago

Since bug 215148 seems to be stopped, we could center in just moving the html
files to the pages directory, regardless they need to be rewritten or not.

	Oscar Manuel Gómez Senovilla
	Comment 27 • 21 years ago

Gerv: I've moved the four files to the pages directory, have searched for all
occurrences in the template tree and modified for the page.cgi. I have created
the patch using "cvs -q diff -uN", as indicated by Dave, and that's what I've
attached to this bug.

Please do the requests you find necessary if the patch works for you, because
that step wasn't given before and the work has got lost. Theorically, this patch
should make the former obsolete, but since I don't have checkin privs, there
could be something wrong.

If you want me to do something else to fix this bug, please tell me.

	Oscar Manuel Gómez Senovilla
	Comment 28 • 21 years ago

Forgot to rename bug_status.html

	Oscar Manuel Gómez Senovilla
	Comment 29 • 21 years ago

i just forgot to mention that there's a fifth file, votehelp.html, but it has
been always included in the patch, for if you worry about it.

	Gervase Markham [:gerv] Assignee
	Comment 30 • 21 years ago

Hmm. The "N" option to diff does not appear to have worked - the new templates
are not in the patch file that you have attached.

The issue is that some of these files really need a rewrite or at least a good
edit. But this also should not hold up this process.

If you email me a tarball of the new files, I will make a proper patch, review
it and check it in after approval has been granted.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 31 • 21 years ago

This patch, based on Oscar's work, converts bugwritinghelp.html, votehelp.html
and bug_status.html to page.cgi pages. It also changes all the links to them,
as Oscar's patch did.

It doesn't convert the quicksearch pages, as their future is currently under
discussion.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 32 • 21 years ago

Comment on attachment 141095 [details] [diff] [review]
Patch v.2

Dave: could you review and even approve this? No scary code - it's all simple
stuff. I've made almost no changes to the content - it does need fixing up, but
that's not the point of this bug.

Gerv

	Christian Reis
	Comment 33 • 21 years ago

Comment on attachment 141095 [details] [diff] [review]
Patch v.2


General comments: I didn't read through the whole text, but through a good deal
of it. I would *really* appreciate it if you wrapped the text at 72 to 78 chars
while you're doing this, because long lines are *very* difficult to review for
me (my browser window is 640px wide). 

You might consider renaming describe-states to describe-status -- for
consistency's sake with the use of "status" pretty much everywhere, and also
because I've found the word "states" tends to remit to geographical "states",
and status is less ambiguous, but it might not be important here.

I assume we're not going to change the keywords (bug, bugzilla, etc) in these
pages, given it't probably not worth the hassle?

Have you ensured that the tests still run with these new templates?

I haven't checked if any links to the old pages are left over, but I guess
you've already grepped for them.

>Index: template/en/default/pages/bug-writing-help.html.tmpl

Check if Eli still maintains this file?

Long lines in this one.

>+[%# INTERFACE:
>+  # This template uses the pages.cgi semi-static display system.
>+  #%]

s/pages.cgi/page.cgi right?

>+++ template/en/default/pages/describe-states.html.tmpl	>+[%# INTERFACE:
>+  # This template uses the pages.cgi semi-static display system.
>+  #%]

Same as in the file above.

>+      The <b>status</B> field indicates the general health 

This is funny, and there are a lot of </B>s and a </DL> or two in this file. If
you feel like lowercasing them, cool.

>Index: docs/sgml/using.sgml
>===================================================================
>RCS file: /cvsroot/mozilla/webtools/bugzilla/docs/sgml/using.sgml,v
>retrieving revision 1.11
>diff -u -r1.11 using.sgml
>--- docs/sgml/using.sgml	27 Jul 2002 22:01:08 -0000	1.11
>+++ docs/sgml/using.sgml	25 Sep 2002 07:25:17 -0000
>@@ -339,7 +339,7 @@
>       <para>Years of bug writing experience has been distilled for your

s/has/have/ been distilled, right?

>Index: docs/html/how.html
>-      While some of the advice is Mozilla-specific, the basic principles of
>-      reporting Reproducible, Specific bugs, isolating the Product you are
>+      These basic principles of
>+      reporting reproducible, specific bugs, isolating the Product you are
>       using, the Version of the Product, the Component which failed, the
>       Hardware Platform, and Operating System you were using at the time of
>       the failure go a long way toward ensuring accurate, responsible fixes

I think the correct wording here would be s/These/The/ making "The basic
principles... go a long way"

>-    Resolve bugs, changing <a href="bug_status.html">resolution</a> to
>+    Resolve bugs, changing <a href="page.cgi?id=describe-states.html">resolution</a> to

shouldn't this have a #status here? I see the anchor in the template at least.

>Index: template/en/default/bug/edit.html.tmpl
>-          <a href="bug_status.html">Status</a>:
>+          <a href="page.cgi?id=describe-states.html">Status</a>:

Same here

>-          <a href="bug_status.html">Resolution</a>:
>+          <a href="page.cgi?id=describe-states.html">Resolution</a>:

Same here

>-      Resolve bug, changing <a href="bug_status.html">resolution</a> to
>+      Resolve bug, changing <a href="page.cgi?id=describe-states.html">resolution</a> to

Same here

>Index: quicksearchhack.html
>-  <td><a href="bug_status.html">Status</a> 
>+  <td><a href="page.cgi?id=describe-states.html">Status</a> 

Same here

>-  <td><a href="bug_status.html">Resolution</a></td>
>+  <td><a href="page.cgi?id=describe-states.html">Resolution</a></td>

And here

>-    print "  <TH ALIGN=\"right\">Number of votes a bug in this product needs to automatically get out of the <A HREF=\"bug_status.html#status\">UNCONFIRMED</A> state:</TH>\n";
>+    print "  <TH ALIGN=\"right\">Number of votes a bug in this product needs to automatically get out of the <A HREF=\"page.cgi?id=describe-states.html#status\">UNCONFIRMED</A> state:</TH>\n";

... maybe only because we do it here. :-) I guess this is only important if you
want to take advantage of the fact that you already have your hands dirty
anyway.

Assuming this is the way we want to go (templatizing all static HTML), r=kiko
and address the above as you see fit.

	Gervase Markham [:gerv] Assignee
	Comment 34 • 21 years ago

Oh, pants.

I've attached the wrong patch. This is the one from six months ago! Aargh!

I feel a right idiot.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 35 • 21 years ago

<looks sheepish> The correct patch.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 36 • 21 years ago

Comment on attachment 141223 [details] [diff] [review]
Patch C v.1

Sorry again...

Gerv

	Christian Reis
	Comment 37 • 21 years ago

REDRUM REDRUM REDRUM

	Gervase Markham [:gerv] Assignee
	Comment 38 • 21 years ago

Red Rum the racehorse? Or Re-drum as in "hit the drum again"? 

Gerv

	Christian Reis
	Comment 39 • 21 years ago

REDRUM as in MURDER backwards. I take it you've never seen The Shining?

	Gervase Markham [:gerv] Assignee
	Comment 40 • 21 years ago

No; my cultural education is obviously sorely lacking.

What can I do but apologise again? :-|

Gerv

	Christian Reis
	Comment 41 • 21 years ago

Oh, I was joking -- it's obviously not a big deal. I'll try looking at this
later today, back in office (finally).

	Christian Reis
	Comment 42 • 21 years ago

Comment on attachment 141223 [details] [diff] [review]
Patch C v.1


This came out a bit fussier than I hoped, so feel free to take into account as
much as you like here. You might want to read this comment backwards as I
started at the last file and worked up.

Issues:

  - #bug_severity is linked to, but doesn't exist (#severity does though)
  - #status isn't used in some places
  - #resolution doesn't exist, what to do?

If you want to run the pages through the validator before checking them in, you
might catch an error or two. I don't think HTML fixes of the sort require
additional reviews given that these pages are really non-critical :-)

Feel free to request approval when you've banged this up as much as you think
is necessary.

>Index: template/en/default/bug/edit.html.tmpl
>===================================================================
>@@ -205,14 +205,14 @@
>     <tr>
>       <td align="right">
>         <b>
>-          <a href="bug_status.html">Status</a>:
>+          <a href="page.cgi?id=fields.html">Status</a>:
>         </b>
>       </td>
>       <td>[% bug.bug_status FILTER html 

#status?

>@@ -220,7 +220,7 @@
>     <tr>
>       <td align="right">
>         <b>
>-          <a href="bug_status.html">Resolution</a>:
>+          <a href="page.cgi?id=fields.html">Resolution</a>:
>         </b>
>       </td>
>       <td>

#status?

>Index: template/en/default/bug/knob.html.tmpl
>===================================================================
>RCS file: /cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/knob.html.tmpl,v
>retrieving revision 1.2
>diff -u -r1.2 knob.html.tmpl
>--- template/en/default/bug/knob.html.tmpl	18 Jan 2004 18:39:18 -0000	1.2
>+++ template/en/default/bug/knob.html.tmpl	10 Feb 2004 23:31:54 -0000
>@@ -59,7 +59,8 @@
>       [% END %]
> 
>       <input type="radio" name="knob" value="resolve">
>-      Resolve [% terms.bug %], changing <a href="bug_status.html">resolution</a> to
>+      Resolve [% terms.bug %], changing 
>+      <a href="page.cgi?id=fields.html">resolution</a> to

Want #status?

>Index: template/en/default/bug/create/create.html.tmpl
>       </strong>
>     </td>
>     <td colspan="3">
>@@ -331,7 +331,8 @@
>   [% IF sel.description %]
>   <td align="right">
>     <strong>
>-      <a href="bug_status.html#[% sel.name %]">[% sel.description %]</a>:
>+      <a href="page.cgi?id=fields.html#[% sel.name %]">
>+        [% sel.description %]</a>:
>     </strong>
>   </td>
>   [% END %]

I checked and there's a problem here. The name of the anchor that will be used
here is bug_severity, but the anchor in fields.html is severity. I think the
easiest is to fix occurences/links from severity to bug_severity..

>Index: template/en/default/list/edit-multiple.html.tmpl
>@@ -266,7 +266,7 @@
>   [% knum = knum + 1 %]
>   <input id="knob-resolve" type="radio" name="knob" value="resolve">
>   <label for="knob-resolve">
>-    Resolve [% terms.bugs %], changing <a href="bug_status.html">resolution</a> to
>+    Resolve [% terms.bugs %], changing <a href="page.cgi?id=fields.html">resolution</a> to

Does this want #status?

>Index: editproducts.cgi

Five bucks to whoever templatizes this..

>Index: quicksearchhack.html
>@@ -79,7 +79,7 @@
>   <td rowspan="2"><tt>UNCO,NEW,...,CLOS,<br>FIX,DUP,...<i>(as first word)</i></tt></td>
>   <td><tt>status</tt></td>
>   <td>&nbsp;</td>
>-  <td><a href="bug_status.html">Status</a> 
>+  <td><a href="page.cgi?id=fields.html">Status</a> 
>       <i>("bug_status")</i>
>   </td>
> </tr>
>@@ -87,35 +87,35 @@
>   <td>&nbsp;</td>
>   <td><tt>resolution</tt></td>
>   <td>&nbsp;</td>
>-  <td><a href="bug_status.html">Resolution</a></td>
>+  <td><a href="page.cgi?id=fields.html">Resolution</a></td>


These links should have at least a #status here. I know we don't have a
#resolution -- should we?

I'll repost my question: I assume we're not going to change the keywords (bug,
bugzilla, etc) in these pages, given it't probably not worth the hassle?

	Gervase Markham [:gerv] Assignee
	Comment 43 • 21 years ago

Kiko's issues addressed. It also now passes tests.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 44 • 21 years ago

Requesting approval.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 45 • 21 years ago

Fixed.

Checking in template/en/default/pages/voting.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/voting.html.tmpl,v
 <--  voting.html.tmpl
initial revision: 1.1
done
RCS file:
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/fields.html.tmpl,v
done
Checking in template/en/default/pages/fields.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/fields.html.tmpl,v
 <--  fields.html.tmpl
initial revision: 1.1
done
RCS file:
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/bug-writing.html.tmpl,v
done
Checking in template/en/default/pages/bug-writing.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/pages/bug-writing.html.tmpl,v
 <--  bug-writing.html.tmpl
initial revision: 1.1
done
Checking in template/en/default/bug/edit.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/edit.html.tmpl,v  <--
 edit.html.tmpl
new revision: 1.40; previous revision: 1.39
done
Checking in template/en/default/bug/knob.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/knob.html.tmpl,v  <--
 knob.html.tmpl
new revision: 1.3; previous revision: 1.2
done
Checking in template/en/default/bug/create/create.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/create.html.tmpl,v
 <--  create.html.tmpl
new revision: 1.27; previous revision: 1.26
done
Checking in template/en/default/bug/create/user-message.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/user-message.html.tmpl,v
 <--  user-message.html.tmpl
new revision: 1.4; previous revision: 1.3
done
Checking in template/en/default/bug/votes/list-for-user.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/votes/list-for-user.html.tmpl,v
 <--  list-for-user.html.tmpl
new revision: 1.13; previous revision: 1.12
done
Checking in template/en/default/list/edit-multiple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl,v
 <--  edit-multiple.html.tmpl
new revision: 1.14; previous revision: 1.13
done
Checking in editproducts.cgi;
/cvsroot/mozilla/webtools/bugzilla/editproducts.cgi,v  <--  editproducts.cgi
new revision: 1.45; previous revision: 1.44
done
Checking in quicksearchhack.html;
/cvsroot/mozilla/webtools/bugzilla/quicksearchhack.html,v  <--  quicksearchhack.html
new revision: 1.5; previous revision: 1.4
done

Gerv

	Oscar Manuel Gómez Senovilla
	Comment 46 • 21 years ago

Didn't you forget to remove the html files from the root dir?

	Gervase Markham [:gerv] Assignee
	Comment 47 • 21 years ago

Yep. I was thinking of adding redirects, but no-one externally links to the
b.m.o. versions, so there's no need.

Gerv

	Gervase Markham [:gerv] Assignee
	Comment 48 • 21 years ago

Removing bugwritinghelp.html;
/cvsroot/mozilla/webtools/bugzilla/bugwritinghelp.html,v  <--  bugwritinghelp.html
new revision: delete; previous revision: 1.3
done
Removing votehelp.html;
/cvsroot/mozilla/webtools/bugzilla/votehelp.html,v  <--  votehelp.html
new revision: delete; previous revision: 1.10
done
Removing bug_status.html;
/cvsroot/mozilla/webtools/bugzilla/bug_status.html,v  <--  bug_status.html
new revision: delete; previous revision: 1.16
done

Gerv

	Oscar Manuel Gómez Senovilla
	Comment 49 • 21 years ago

Are you leaving quicksearchhack.html and quicksearch.html for a later stage? At
this time (not sure if because of this bug) cvs is broken because
global/descs.none.tmpl is missing.

	Gervase Markham [:gerv] Assignee
	Comment 50 • 21 years ago

The breakage isn't this bug.

Yes, I've left QuickSearch; it's currently being redesigned in a whole bunch of
bugs filed by afranke, so it wasn't worth it.

Gerv