Missing username from app review listing

VERIFIED FIXED in 2014-02-25

Status

P2
normal
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: krupa.mozbugs, Assigned: mat)

Tracking

2014-02-25
Points:
---

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Created attachment 824848 [details]
reviewer name missing.png

steps to reproduce:
1. Load https://marketplace.firefox.com/app/counters/ratings
2. Notice that one of the reviews is missing the name of the reviewer

See screenshot.

Comment 1

5 years ago
The API is indeed reporting that the user has no display name:

http://cl.ly/image/160b3w1i271e
Component: Consumer Pages → API

Updated

5 years ago
Priority: -- → P2
SELECT users.id, users.display_name FROM reviews INNER JOIN addons ON (reviews.addon_id = addons.id) INNER JOIN users ON (reviews.user_id = users.id) WHERE ("en-us"="en-us" AND addons.app_slug = 'counters' ) ORDER BY reviews.created DESC LIMIT 21;
+---------+--------------------+
| id      | display_name       |
+---------+--------------------+
| 6133935 |                    | 
| 6254801 | andre alves garzia | 
+---------+--------------------+
2 rows in set (0.02 sec)

Likely related to bug 869656. I don't think we have validation in the API here.
(Assignee)

Updated

5 years ago
Assignee: nobody → mpillard
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → 2014-02-18
(Assignee)

Updated

5 years ago
Target Milestone: 2014-02-18 → 2014-02-25
(Assignee)

Comment 3

5 years ago
Fixed in https://github.com/mozilla/zamboni/commit/fcca2025059c433840efb751a747ef81ff5a8a1d

STR:
- Log in in consumer pages on -dev
- Go to settings
- Empty the field with your display name
- Right click and choose Inspect Element on that field
- Remove the "pattern" attribute on that field, close the inspector
- Submit the form
- You should see an error and the changes should not have been saved
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [qa-]
(Reporter)

Comment 4

5 years ago
verified fixed @ https://www.dropbox.com/s/swenhhm74wdhhtf/Screenshot%202014-02-19%2016.58.32.png
Status: RESOLVED → VERIFIED
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.