Closed Bug 619682 Opened 14 years ago Closed 14 years ago

Addons Manager "Available Updates" pane shown even if no updates, if check for updates performed from "Recent Updates" section

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
minor

Tracking

()

VERIFIED FIXED
mozilla2.0b10

People

(Reporter: stanley.appel, Assigned: Unfocused)

References

Details

(Keywords: regression)

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows NT 5.1; rv:2.0b9pre) Gecko/20101216 Firefox/4.0b9pre
Build Identifier: Mozilla/5.0 (Windows NT 5.1; rv:2.0b9pre) Gecko/20101216 Firefox/4.0b9pre

Sometimes it shows the "available updates" tab on add-on manager page, when there is no update available.


Reproducible: Always

Steps to Reproduce:
in the add-ons manager page
1. click on the "recent updates" on the left bar
2. then press the "wheely button" en click "check for updates"
3. now a new item appears on the left: "available updates"
4. click it and see: no updates found

this does not appear when you click on "check for updates" when you do not goto the recent updates page first.
Actual Results:  
the tab

Expected Results:  
no extra tab
Component: Toolbars → Add-ons Manager
Product: Firefox → Toolkit
Version: unspecified → Trunk
If i recall correctly this can be triggered by some Extensions, no?
Until recently, certain addons (eg Scriptish) triggered an underlying bug that meant the "available updates" section showed even if no updates. This was fixed in bug 591024.

Have you definitely seen the issue since bug 591024 landed? (~12th Dec)
	
If so, can you try with a new profile (ie: so empty of addons) and see if it still occurs:
http://support.mozilla.com/en-US/kb/Basic%20Troubleshooting#w_make-a-new-profile
QA Contact: toolbars → add-ons.manager
Have you been able to try with a clean profile, per comment 2?

Thanks!
Whiteboard: [closeme 2011-01-10]
Yes, also in a clean profile, even a clean install.
steps:
- install windows xp
- install firefox 4.0 beta 8
- start firefox 4.0 beta 8
- close firefox
- install firefox 4.0b9pre nightly 2010-12-30 (http://hg.mozilla.org/mozilla-central/rev/ec3e4786877d)
- start firefox 4.0b9pre
- goto add-ons manager
- on the top right click on the button and select: view recent updates
- on the left click in the new item: recent updates
- on the top right click on the button and select: check for updates
- on the left there will be a new item: available updates
- click on it and it shows: no updates found
Attached image no addons available
Thanks for the clarification, have been able to follow the STR and can confirm with Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20101230 Firefox/4.0b9pre ID:20101230030351

For reference, reduced STR:
1) Start Firefox/Minefield with clean profile.
2) Open addons manager.
3) Click the cog icon dropdown top right of the addons mangager, then "view recent updates" (no addons need to be installed).
4) Click the cog again and choose "check for updates".
5) Observe sections in left-hand sidebar of addons manager.

Expected:
- The "Available Updates" entry in the left hand sidebar only appears if updates are available.

Actual:
- Even with no addons installed/no updates downloaded in step #4, "Available Updates" appears in the left hand sidebar, which when clicked on shows "no updates found".
- This only occurs if the check for updates was performed whilst in the "recent updates" section.

Regression range...

Last good nightly: 2010-12-08 
First bad nightly: 2010-12-09

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=95452499f3d6&tochange=11e328a49e0a

In that range is:
http://hg.mozilla.org/mozilla-central/rev/c44ffda23deb
Dave Townsend — Bug 591024: Only show "Available Updates" pane when pending updates are available.

...so blocking bug 591024.

Thanks for the report! :-)
Blocks: 591024
Severity: trivial → minor
Status: UNCONFIRMED → NEW
blocking2.0: --- → ?
Ever confirmed: true
Keywords: regression
Summary: invalid "available updates" tab on add-on manager → Addons Manager "Available Updates" pane shown even if no updates, if check for updates performed from "Recent Updates" section
Whiteboard: [closeme 2011-01-10]
Attached patch Patch v1Splinter Review
updateAvailableCount() didn't take into account that gUpdatesView is really two views handed by one object. Sadly, can't seem to test this without a bunch of setTimeout() calls - which would be horribly unreliable.
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Attachment #500718 - Flags: review?(dtownsend)
OS: Windows XP → All
Hardware: x86 → All
Attachment #500718 - Flags: review?(dtownsend)
Attachment #500718 - Flags: review+
Attachment #500718 - Flags: approval2.0+
blocking2.0: ? → -
Status: ASSIGNED → NEW
blocking2.0: - → ?
OS: All → Windows XP
Hardware: All → x86
Whiteboard: [has patch][needs landing]
blocking2.0: ? → ---
OS: Windows XP → All
Hardware: x86 → All
http://hg.mozilla.org/mozilla-central/rev/c22c9543a449
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [has patch][needs landing]
Target Milestone: --- → mozilla2.0b10
Verified fixed with Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b10pre) Gecko/20110119 Firefox/4.0b10pre ID:20110119030331

Blair, do we have an automated test or is a manual test necessary?
Status: RESOLVED → VERIFIED
Flags: in-testsuite?
Flags: in-litmus?
(In reply to comment #9)
> Blair, do we have an automated test or is a manual test necessary?

Manual test for this one. Sadly, any automated test would be unreliable (see comment 7).
Flags: in-testsuite? → in-testsuite-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: