329459 - try to use bug_link more

Status: RESOLVED DUPLICATE of bug 115796

(Bugzilla :: User Interface, enhancement)

Description

[timeless]

lpsolit claims we should be using bug_link, but we don't use it much...

Comment 1

[timeless]

Attachment: changes to use bug_link

Comment 2

[Marc Schumann [:Wurblzap]]

See also bug 318265.

Comment 3

[Frédéric Buclin]

Comment on attachment 214139 [details] [diff] [review]
changes to use bug_link

./runtests.pl 008 fails:

filterexceptions.pl has extra members: ...

t/008filter........NOK 148
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 149
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 150
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 155
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 165
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 167
#     Failed test (t/008filter.t at line 142)
t/008filter........NOK 243
#     Failed test (t/008filter.t at line 142)
# Looks like you failed 7 tests of 261.


>Index: mozilla/webtools/bugzilla/template/en/default/bug/navigate.html.tmpl

>+  [%+ First FILTER bug_link(bug_list.first) FILTER none %]
>+  [%+ Last FILTER bug_link(bug_list.last) FILTER none %]

"First" and "Last" must be quoted, because they are text, not variables. Moreover, you should use bug_link() for "Prev" and "Next" too.



>Index: mozilla/webtools/bugzilla/template/en/default/bug/show.html.tmpl

>+    [%+ bug.bug_id" FILTER bug_link(bug.bug_id) FILTER none %]:

Remove the quote after bug.bug_id. This makes checksetup.pl to crash.



>Index: mozilla/webtools/bugzilla/template/en/default/bug/summarize-time.html.tmpl

>+    [% "<b>$terms.Bug&nbsp; $id</b>" FILTER bug_link(id) FILTER none %]

You have to remove the whitespace between &nbsp; and $id, else you get "Bug foo" on two lines.


I didn't check whether there are other places where bug_link() should be used too. I will let you do this check.

Comment 4

[timeless]

Attachment: more changes

i have no idea how filter works when you have foo.$bar. which is why i didn't mess w/ Prev/Next.

I also don't have bugzilla or any intention of installing it in order to get the suite happy. if your suite doesn't like me, give me a list of changes it wants made and i'll make them.

i have hundreds of other things to do with my time.

Comment 5

[Frédéric Buclin]

Comment on attachment 217539 [details] [diff] [review]
more changes

>Index: mozilla/webtools/bugzilla/template/en/default/admin/products/updated.html.tmpl

>+      →removed votes for [% terms.bug %]
>+      [%+ detail.id FILTER bug_link(detail.id) FILTER none %]
>+      from [% detail.name FILTER html %]<br>

>+      &rarr;removed votes for [% terms.bug %]
>+      [%+ detail.id FILTER bug_link(detail.id) FILTER none %]
>+      from [% detail.name FILTER html %]<br>

We should be consistent: in some files, "Bug" is included in the link, sometimes only the bug number is. From a UI point of view, I prefer to include "Bug" as well, everywhere.



>Index: mozilla/webtools/bugzilla/template/en/default/attachment/created.html.tmpl

>+<a href="attachment.cgi?bugid=[% bugid FILTER none %]&amp;action=enter">Create
>+ Another Attachment to 
>+  [%+ "$terms.Bug #$bugid" FILTER bug_link(bugid) FILTER none %]</a>

I don't like having a link in a link. This link should entirely point to the attachment, not the bug ID (we already have another link which points to the bug).



>Index: mozilla/webtools/bugzilla/template/en/default/bug/navigate.html.tmpl
>+  [%+ "First" FILTER bug_link(bug_list.first) FILTER none %]
>+  [%+ "Last" FILTER bug_link(bug_list.last) FILTER none %]
>+        [% "Prev" FILTER bug_link(bug_list.$prev_bug) FILTER none %]
>+        [% "Next" FILTER bug_link(bug_list.$next_bug) FILTER none %]

Technically, that's correct. The problem is that if they point to a closed bug, these URLs will be crossed out, which isn't nice from a UI point of view. I would r+ your patch if we could avoid this behavior. That's the only reason I denied review (I could r+ your patch if bug_link() was fixed accordingly).

Comment 6

[Frédéric Buclin]

*** This bug has been marked as a duplicate of 115796 ***