484878 - Add application_id to collections

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Collections, defect, P1)

Description

[Justin Scott [:fligtar]]

Collections should be associated with a particular application, e.g., a Firefox collection or Thunderbird collection. Add-ons in the collection might support multiple apps, but an add-on shouldn't be in a collection if it *doesn't* support that application.

On the website when making a collection, the app will default to whatever app the user's currently browsing the AMO site in, and can be changed from the manage collections page. From the API, the application should always be Firefox.

Comment 1

[Fred Wenzel [:wenzel]]

Attachment: SQL: Add application_id to collections table

Comment 2

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 1

Saves the application ID when creating a new one. Adds the app id to the API's XML results (tests updated, too).

Note: When creating a collection from the API, it is possible to choose the app, though it *defaults* to Firefox.

Comment 3

[Les Orchard [:lorchard]]

Comment on attachment 372867 [details] [diff] [review]
Patch, rev. 1

One issue in the remora.sql:

CONSTRAINT `collections_ibfk_1` FOREIGN KEY (`application_id`) REFERENCES `remora_prod`.`applications` (`id`)

Once I removed the "`remora_prod`." in the REFERENCES, all the tests passed and things look good - r+ with that caveat.

Comment 4

[Fred Wenzel [:wenzel]]

Thanks, I missed that.

Comment 5

[Fred Wenzel [:wenzel]]

Attachment: Fixed SQL

Comment 6

[Fred Wenzel [:wenzel]]

Thanks, I committed this to r24446 and added the SQL to be ran to the wiki page.