Comment 1 for bug 422128

Revision history for this message
Brad Crittenden (bac) wrote :

The reason for this failure is that in the view code for products changing the owner calls _reassignProductDependencies which changes the owner of the product translation import queues, product series, and product releases. The way this change is done is problematic for a number of reasons:

1) The reassignment happens in the browser code so changes via other mechanisms, such as the web service or direct manipulation in a test, have different affect than doing it through the web.

2) There is no reason to change the owner of a product series or product release. In fact, that data element is shown in the UI as 'registrant', which has historical implications. It is unclear whether changing the import queue 'importer' is necessary or desired.

So the view method _reassignProductDependencies should either go away (if not required for translation import queue) or be moved to the model as part of a setter for product.owner.

Either way, the owner of product releases and product series should not be changed, which will solve this bug since a team will never be used as the owner of those objects.