Closed
Bug 564550
Opened 15 years ago
Closed 14 years ago
[Browse] Omit nameless add-ons
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect, P3)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
Future
People
(Reporter: fligtar, Assigned: jbalogh)
Details
(Whiteboard: [z])
When sorting by name, there are a ton of nameless add-ons: https://preview.addons.mozilla.org/z/en-US/firefox/extensions/?sort=name&unreviewed=on
We should not show these. I don't think we do in Remora (the same sort doesn't show them there)
Updated•15 years ago
|
Assignee: nobody → jbalogh
Comment 1•15 years ago
|
||
I don't think these are nameless:
https://preview.addons.mozilla.org/en-US/firefox/addon/160845/
has a only 'de' translation of it's name, but we don't pull that through.
Hope we don't take a huge performance hit for the fix.
Comment 2•15 years ago
|
||
If it doesn't have an English name and you're looking at en-US, we shouldn't be showing the add-on, right?
Target Milestone: 5.11 → 5.11.1
Assignee | ||
Comment 3•15 years ago
|
||
One of the add-ons I looked at had a translation with locale=zh-tw and english text in the string, and the default locale was en-US. So the default locale didn't have a row in the trans table. It's bad data.
Updated•15 years ago
|
Target Milestone: 5.11.1 → 5.11
Reporter | ||
Comment 4•15 years ago
|
||
We should fix this before pushing these pages live -- it's a regression from Remora and makes the Name sort (the most popular one) pretty worthless.
Assignee | ||
Comment 5•15 years ago
|
||
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Comment 6•15 years ago
|
||
When the add-ons are sorted by "name" we show 8638(including unreviewed)as opposed to other sorts where we show 8757 add-ons.That's a difference of 119 add-ons just based on your sort option.
Fligtar,are you okay with this?
Reporter | ||
Comment 7•15 years ago
|
||
We shouldn't show nameless add-ons anywhere, regardless of sort. Unless those 119 add-ons are showing up in other sorts, sounds like a separate issue.
Comment 8•15 years ago
|
||
(In reply to comment #7)
> We shouldn't show nameless add-ons anywhere, regardless of sort. Unless those
> 119 add-ons are showing up in other sorts, sounds like a separate issue.
We show the nameless add-ons in other sorts.Reopening.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 9•15 years ago
|
||
We should add a cron that checks the name and valid version and other vagaries and drops a boolean in something like addons.ready_for_battle.
Target Milestone: 5.11 → Future
Comment 10•15 years ago
|
||
Rules:
- If an add-on doesn't have a name in the locale you're browsing in, we show the add-on info in the add-on's default locale
If there are blanks outside of that, those are other bugs and things we should fix. So, this shouldn't be "omit nameless add-ons" so much as "show default locales for add-ons without current locale"
Assignee | ||
Comment 11•15 years ago
|
||
(In reply to comment #10)
> If there are blanks outside of that, those are other bugs and things we should
> fix. So, this shouldn't be "omit nameless add-ons" so much as "show default
> locales for add-ons without current locale"
We're already doing the right thing with the default locale in queries.
Assignee | ||
Updated•14 years ago
|
Status: REOPENED → RESOLVED
Closed: 15 years ago → 14 years ago
Resolution: --- → WORKSFORME
Updated•9 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•