271267 - [Submission] Detect whether uploaded file is theme or extension

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, enhancement, P5)

Description

[alanjstr]

When someone is uploading a file with an install.rdf, we can tell whether it is
a theme or not, at least for FX and TB.

Themes have an instance of em:internalName.

If someone tries to upload a new item incorrectly, this will catch it and
prevent us from having to go back and fix it manually.

Comment 1

[Wolf [:wolf]]

The question is not whether we can, the question is, do we really *need* to? and
is it practical to?

For once, let's wait and see what real authors who use the system actually
want/need, before filling buglists with minor bugs like this.

Comment 2

[Wolf [:wolf]]

Bulk Moving Developer Control Panel bugs to new component.
(Filter: massdevcpspam)

Comment 3

[alanjstr]

Attachment: Allow user to not have to specify theme or extension

This way we don't have to guess Theme or Extension based on the URL.

Comment 4

[Wolf [:wolf]]

Comment on attachment 169129 [details] [diff] [review]
Allow user to not have to specify theme or extension

Question: if bug 271272 is moving this to RDF parsing and not regexs. why is
this patch using a regexp? This probably should be done right, if it's going to
be done at all and be built on top of the pending patches in Bug 271272.

Aside from that, I'm still unconvinced that this gains us anything besides
unexpected trouble if em:internalName is ever fubared by an author.
Particularly considering this patch removes the previous code and doesn't allow
for author error.

Comment 5

[alanjstr]

if em:internalName is fubarred, then the theme jar is invalid.

I patched against current source, since you hadn't reviewed or checked in the
RDF parsing patch.

Comment 6

[Mike Shaver (:shaver emeritus)]

Comment on attachment 169129 [details] [diff] [review]
Allow user to not have to specify theme or extension

we use the XML parser now

Comment 7

[Michael Morgan [:morgamic]]

Mass change - bugs to be read / (re)confirmed.

Comment 8

[Cameron]

AMO bugspam. Correcting QA contacts on OLD bugs ([e-mail])

-> Correct QA contact ([e-mail])

Filtermeplzkthx

Comment 9

[Mike Shaver (:shaver emeritus)]

Fixed in Remora (though I wonder about things like the Joga extension, which also had a theme component to them).