Comment 5 for bug 340662

Revision history for this message
Adi Roiban (adiroiban) wrote :

I apologies for all the inconveniences created by this branch. I have created the MP, thinking that such stupid mistakes will be raised during the reviewer.

I have reevaluated the usage of IPOTemplateSet and IPOTemplateSubset and I realized my foolish error.

Since we allow Launchpad.Edit to disable templates, I assume we should also allow them to enable them back.
This is why I had to change the traversal for POTemplateSubset.

If the permission were no change, once a person disabled the template he/she will not be able to enable it back.

I have created a new branch, removing „name” from Edit page, but fixing and keeping Launchapd.Edit for IPOTemplateSubset.

Please let me know if a person should be able to enable a template, once he/she had disable it, and I will do the required changes.