255304 - Restrict Category list to PET

Status: RESOLVED WONTFIX

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

Description

[alanjstr]

Category List: should be restricted to P/E/T, not allow the creation of new type.

Comment 1

[Wolf [:wolf]]

The site wasn't designed to be inflexable, there's no reason to make that
restriction. It's a table management script anyways so adding items is normal
and expected behavior.

--> wontfix.

Comment 2

[alanjstr]

Leaving the ability creates a new category, in this case G, that has no affect
on the rest of UMO.  Now I have a category named foo for Firefox that is neither
themes nor extensions.  If we do decide to add a category, such as for search
plugins, then we would revise the UMO code to accept it.

Yes, this is for table management.  But creating a new category that goes
nowhere doesn't make sense.

Comment 3

[Wolf [:wolf]]

Having a category that points nowhere, doesn't hurt anything.

Adding a new section, as the example you gave, searchengines, would require new
code for the section itself, but wouldn't require the modification of existing
code for categories, unless this is fixed. You can simply hook into the table as
it stands, and reuse existing code to make the categories extend to your new
section. Why cripple something that isn't broke and force its code to be
modified when the current setup is extendable w/o that work?

My original decision is unchanged, there's no good reason presented here to
break the extensibility of this feature.

Comment 4

[alanjstr]

In order for any new category type to make any sense, you'll have to change PHP.
 You've made it extensible for no reason.  Think of the component dropdowns in
bugzilla.  You don't just add an item without changing code to support it.  If
you want Search Engine categories, you have to change the php to support search
engine categories.  That requires a release, at which time it would be applicable.