204560 - Add alias to long format bug list if enabled

Status: RESOLVED FIXED

(Bugzilla :: Query/Bug List, enhancement)

Description

[Joel Peshkin]

When printing long format listings of bug lists, the alias should be included.

Comment 1

[GavinS]

Attachment: add bug alias to the long listing

This patch puts the alias, if it exists, in square brackets after the bug id

Comment 2

[Dave Miller [:justdave]]

Comment on attachment 124394 [details] [diff] [review]
add bug alias to the long listing

zach: are we bumping template versions yet?

A nit on the IFNULL...	why not just use '' instead of having to backslash the
double-quotes?	From what I've seen MySQL tends to prefer single-quotes anyway.

Comment 3

[GavinS]

I was meaning to clarify about the template version updating in the newsgroup. 

I was just going on the Bugzilla Developers Guide regarding templates, which says:

> Every template shipped with Bugzilla has a version number on the first line
such as
> [e-mail]. You should update this when you edit a template.

> You need to increment the major part if your updates make the interface to
this template
>  incompatible with the interface of previous versions. Otherwise, you need to
increment the
>  minor part.

Comment 4

[GavinS]

Attachment: Add alias to long format bug list patch 2

Without the template version change and nit fixed.

Comment 5

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 127972 [details] [diff] [review]
Add alias to long format bug list patch 2

>Index: template/en/default/bug/show-multiple.html.tmpl

>+          [% IF Param("usebugaliases") %]
>+	    [% IF bug.alias != "" %]  

Nits: 
  * second conditional should be indented additional two spaces;
  * second conditional can be simply [% IF bug.alias %];
  * conditionals can be combined [% IF Param("...") AND bug.alias %].

>+               [ [% bug.alias FILTER html %] ]

Nits:
  * curly braces (());
  * no space between the braces and the alias.

All problems are nits, so r=myk, but consider the nits before opting to check
in current patch.

Comment 6

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 124394 [details] [diff] [review]
add bug alias to the long listing

Removing unneeded review requests.

Comment 7

[GavinS]

Attachment: add alias to long listing patch 3

Patch 3, fixing Myk's nits. I can't commit, so if the review is carried over to
this patch, could someone commit this please?

Comment 8

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 129189 [details] [diff] [review]
add alias to long listing patch 3

This patch fails to apply; I had to apply by hand.  Did you hand-edit this
patch by any chance?  In any case, the code itself works fine.	r=myk

Comment 9

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v3 with patch problems corrected

This patch is like patch #3 except that it applies correctly.

Comment 10

[Myk Melez [:myk] [@mykmelez]]

Checking in long_list.cgi;
/cvsroot/mozilla/webtools/bugzilla/long_list.cgi,v  <--  long_list.cgi
new revision: 1.36; previous revision: 1.35
done
Checking in template/en/default/bug/show-multiple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/show-multiple.html.tmpl,v
 <--  show-multiple.html.tmpl
new revision: 1.12; previous revision: 1.11
done

Comment 11

[Myk Melez [:myk] [@mykmelez]]

Oops, forgot to mark fixed.