Default theme left behind in Sunbird's profile directory

RESOLVED FIXED

Status

()

Toolkit
Add-ons Manager
RESOLVED FIXED
12 years ago
10 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

({fixed1.8.1})

Trunk
fixed1.8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

Since the makefile changes from bug 286034 never landed for Sunbird there are now profiles with default theme in their extensions directory for toolkit version 1.8. This caused havoc for Firefox and I think we should just check on all upgrades for the existence of the default theme in the profile's extensions directory instead of only on upgrade from the 1.7 toolkit version.
This depends on bug 335753 since that bug will handle the makefile changes for Sunbird which are required in order for this bug to provide any value
Depends on: 335753
Created attachment 220291 [details] [diff] [review]
patch

Benjamin, if you approve this I promise I won't land it until after bug 335753 lands. I also added a check to see if all add-ons have appManaged="true" so we won't open the compatibility checker when it isn't necessary.
Attachment #220291 - Flags: review?(benjamin)
Attachment #220291 - Attachment is obsolete: true
Attachment #220291 - Flags: review?(benjamin)
Created attachment 220296 [details] [diff] [review]
patch

Turns out earlier versions of Sunbird used a different GUID.

Once again, if you approve this I promise I won't land it until after bug 335753 lands. I also added a check to see if all add-ons have appManaged="true" so we won't open the compatibility checker when it isn't necessary.
Attachment #220296 - Flags: review?(benjamin)
Comment on attachment 220296 [details] [diff] [review]
patch

Obsoleting... this handles upgrading from Sunbird 0.2 but not from 0.3... fun :(
Attachment #220296 - Flags: review?(benjamin)
Attachment #220296 - Attachment is obsolete: true
Comment on attachment 220296 [details] [diff] [review]
patch

bah... nevermind. The reason it didn't fix the 0.3 upgrade is due to it returning early in checkForMismatches since it wasn't a version upgrade. So this does work when there is a version upgrade.
Attachment #220296 - Attachment is obsolete: false
Attachment #220296 - Flags: review?(benjamin)
Comment on attachment 220296 [details] [diff] [review]
patch

This doesn't do the right thing when going from 0.3a1 to 0.3a2. The joys of when not all toolkit apps get updated for changes
Attachment #220296 - Flags: review?(benjamin)
Created attachment 220477 [details] [diff] [review]
patch (more one-off fun - a.k.a. sometimes a sledgehammer is the right tool for the job)
Attachment #220296 - Attachment is obsolete: true
Attachment #220477 - Attachment description: patch (more one-off fun) → patch (more one-off fun - a.k.a. sometimes a sledgehammer is the right tool for the job)
Created attachment 220481 [details] [diff] [review]
patch (w/ new and improved comments)

Benjamin, this will make any profile with a default theme in it remove the toolkit 1.8 extensions datasource to avoid trying to install and uninstall the default theme during the same pass which is a use case that has never been supported or for that matter needed. I also heavily commented it since why this needs to be done is to say the least confusing... hopefully one day these one-offs can be removed.
Attachment #220477 - Attachment is obsolete: true
Attachment #220481 - Flags: review?(benjamin)

Updated

12 years ago
Attachment #220481 - Flags: review?(benjamin) → review+
Fixed on trunk
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Verbal a+ from bsmedberg... checked in to MOZILLA_1_8_BRANCH
Keywords: fixed1.8.1
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.