Closed
Bug 679836
Opened 13 years ago
Closed 13 years ago
Clicking "add" on missing entries in admin UI does nothing
Categories
(Socorro :: General, task)
Socorro
General
Tracking
(Not tracked)
VERIFIED
FIXED
2.2.2
People
(Reporter: kairo, Assigned: brandon)
References
()
Details
Attachments
(2 files)
7.50 KB,
patch
|
rhelmer
:
review+
|
Details | Diff | Splinter Review |
47.08 KB,
image/png
|
Details |
https://crash-stats.mozilla.com/admin/branch_data_sources has a "Missing Entries" list but clicking "add" on any of them does not do anything. Previously, that would show the form with some data to actually click a button to add the entry, but the form doesn't appear now.
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → bsavage
Updated•13 years ago
|
Target Milestone: --- → 2.2.2
Assignee | ||
Comment 1•13 years ago
|
||
This patch does two things: * Reinstates code that was removed from 2.2, but is required for the proper functioning of the "add" behavior * Makes use of jberkus' new stored procedure for adding items to the product_info table.
Attachment #555515 -
Flags: review?(rhelmer)
Comment 2•13 years ago
|
||
Comment on attachment 555515 [details] [diff] [review] Correct admin section >Index: webapp-php/application/models/branch.php >=================================================================== >+ $rv = $this->db->query("/* soc.web branch.add */ >+ SELECT * FROM edit_product_info(null, ?, ?, ?, ?, ?, ?, ?)", $product, $version, $release, $start_date, $end_date, $featured, $throttle); Just a nit - I didn't notice that this was a function so was scratching my head at first. Not having the "* FROM" is more typical, like: SELECT edit_product_info(a,b,c,d,); Especially if it only returns one value (as this function does) and/or you don't care about the return value. Anyway works/looks fine otherwise as-is, so I'll leave that up to you to change or not as you see fit.
Attachment #555515 -
Flags: review?(rhelmer) → review+
Updated•13 years ago
|
Flags: in-testsuite?
Flags: in-litmus?
Assignee | ||
Comment 3•13 years ago
|
||
Committed revision 3474.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 4•13 years ago
|
||
The dialog for adding missing products to the branches is back and functioning again.
Comment 5•13 years ago
|
||
QA verified. Kairo when you have a moment please take a look at the re-instated behavior on stage. If need be please re-open.
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 6•13 years ago
|
||
Yup, verified by satisfying bug 683107 on stage ;-)
Updated•13 years ago
|
Component: Socorro → General
Product: Webtools → Socorro
You need to log in
before you can comment on or make changes to this bug.
Description
•