Closed
Bug 1269549
Opened 9 years ago
Closed 9 years ago
Component watching prefix matches too much
Categories
(bugzilla.mozilla.org :: Extensions, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: dylan, Assigned: dylan)
References
Details
Attachments
(1 file)
|
933 bytes,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
I received a bug report from a user that is getting notifications for things they are not watching. I believe the prefix watching added in bug 720832 doesn't work.
This is part of the query used, and it should return nothing.
SELECT
user_id, profiles.login_name
FROM
component_watch
JOIN
profiles ON userid = user_id
INNER JOIN
components ON components.product_id = component_watch.product_id
WHERE
component_prefix IS NOT NULL
AND (component_watch.product_id = 55
OR component_watch.product_id = 55)
AND components.name LIKE CONCAT(component_prefix, '%')
|
Assignee | |
Comment 1•9 years ago
|
||
Attachment #8748018 -
Flags: review?(dkl)
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8748018 [details] [diff] [review]
1269549_1.patch
Review of attachment 8748018 [details] [diff] [review]:
-----------------------------------------------------------------
r=dkl
Attachment #8748018 -
Flags: review?(dkl) → review+
|
|
Assignee | |
Comment 3•9 years ago
|
||
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
991b48e..6381dc7 master -> master
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Component: Extensions: ComponentWatching → Extensions
You need to log in
before you can comment on or make changes to this bug.
Description
•