327528 - Add an XML format for describekeywords.cgi

Status: RESOLVED WONTFIX

(Bugzilla :: User Interface, enhancement, P4)

Description

[Rémi Zara]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; fr) AppleWebKit/417.9 (KHTML, like Gecko) Safari/417.8
Build Identifier: 

This could help to for instance for AJAXifying the interaction with keywords

Reproducible: Always

Steps to Reproduce:

Comment 1

[Rémi Zara]

Attachment: adopt format/ctype mechanism

adopt format/ctype mechanism

Comment 2

[Vlad Dascalu]

Comment on attachment 212145 [details] [diff] [review]
adopt format/ctype mechanism

It would have been nice to actually have at least another template (besides the default one) in order to be able to use this feature, but yeah, code-wise, it looks good.

Comment 3

[Rémi Zara]

Attachment: Simple XML template

Here is a simple XML template, with the list of keywords and their description.

Comment 4

[Teemu Mannermaa (:wicked)]

Comment on attachment 215806 [details]
Simple XML template

>  <dd><![CDATA[[% keyword.description %]]]></dd>

Sorry, this fails test suite:

not ok 246 - (en/default) template/en/default/reports/keywords.xml.tmpl has unfiltered directives:
#   27: keyword.description
# --ERROR
#     Failed test (t/008filter.t at line 131)

Not to mention it looks funky. :) I assume it's syntactically valid XML because FF doesn't seem to choke on it. As there's no attached style to say what dl, dt and dd are then I must assume they are formatted correctly..

PS. Would be nice if your next attachment of this file would be in a patch format. That way the file would be much easier to add to a test system. :)

Comment 5

[Vlad Dascalu]

LpSolit said in IRC in regard to this:

19:13	LpSolit 	justdave: could you have a look at the patch in bug 327528?
19:13	ssdbot 	LpSolit: Bug https://bugzilla.mozilla.org/show_bug.cgi?id=327528 enh, --, Bugzilla 2.24, remi_zara%mac.com, ASSI, Make describekeywords.cgi adopt the template format/ctype mechanism
19:13	LpSolit 	justdave: the patch having r=vladd on it
19:13	LpSolit 	justdave: shouldn't it have -content_disposition => in it?

(posting here so that everybody can see it next to the patch itself)

Comment 6

[Frédéric Buclin]

Cancelling approval till my question in comment 5 is answered.

Comment 7

[Frédéric Buclin]

This bug is retargetted to Bugzilla 3.2 for one of the following reasons:

- it has no assignee (except the default one)
- we don't expect someone to fix it in the next two weeks (i.e. before we freeze the trunk to prepare Bugzilla 3.0 RC1)
- it's not a blocker

If you are working on this bug and you think you will be able to submit a patch in the next two weeks, retarget this bug to 3.0.

If this bug is something you would like to see implemented in 3.0 but you are not a developer or you don't think you will be able to fix this bug yourself in the next two weeks, please *do not* retarget this bug.

If you think this bug should absolutely be fixed before we release 3.0, either ask on IRC or use the "blocking3.0 flag".

Comment 8

[Max Kanat-Alexander]

Bugzilla 3.2 is now frozen. Only enhancements blocking 3.2 or specifically approved for 3.2 may be checked in to the 3.2 branch. If you would like to nominate your enhancement for Bugzilla 3.2, set "blocking3.2" tp "?", and either the target milestone will be changed back, or the blocking3.2 flag will be granted, if we will accept this enhancement for Bugzilla 3.2.

Comment 9

[Frédéric Buclin]

This bug is no longer relevant to fix comment 0, which is already implemented. Also, I think we should start using WebServices rather than wanting admin pages to be available in XML format. So I'm pretty opposed to this bug.