Closed Bug 933752 Opened 11 years ago Closed 11 years ago

[Homescreen] Bug 893800 creates a new objectstore without doing a proper db migration

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(blocking-b2g:-)

RESOLVED FIXED
blocking-b2g -

People

(Reporter: julienw, Assigned: macajc)

References

Details

(Whiteboard: [systemsfe])

Attachments

(1 file)

(deleted), text/x-github-pull-request
julienw
: review+
Details
see https://github.com/mozilla-b2g/gaia/pull/11866/files#diff-4 Bug 893800 does not create properly a new objectstore for pre-existing database. Requesting koi+ as this is a very serious bug preventing migrations.
Blocks: 893800
(In reply to Julien Wajsberg [:julienw] from comment #0) > see https://github.com/mozilla-b2g/gaia/pull/11866/files#diff-4 > > Bug 893800 does not create properly a new objectstore for pre-existing > database. > > Requesting koi+ as this is a very serious bug preventing migrations. Can you explain a bit more about this could happen? Specifically, I'm looking to understand what STR could trigger this bug, what's the impact, etc.
Whiteboard: [systemsfe]
This happens when you move from v1.1 to v1.2. An easy STR using a geeksphone phone could be: * flash a 1.1 firmware * install some apps, install e.me results on your homescreen, add bookmarks to your homescreen * flash a 1.2 firmware while keeping the userdata I don't know what the impact is as I don't know the code. I know that the objectstore svAppsInstalled won't be created, and that I see errors stating this in logcat. I don't know the impact, probably Carmen or Cristian would know. Flagging them. This _could_ be the bug triggering bug 925461 but I'm not sure.
Flags: needinfo?(crdlc)
Flags: needinfo?(cjc)
Please Carmen take a look to this question, thanks
Flags: needinfo?(crdlc)
Assignee: nobody → cjc
Flags: needinfo?(cjc)
You're right, if the bbdd is already created then the objectStore for single variant is not created. But: - Single variant functionality doesn't apply to actualizations, they only apply for new devices. - An exception is launched (NotFoundError) but the functionality is not affected because of the place where the code is. The single variant datas are loaded at the end of homestate.init when the only task that is pending is loading the single variant app list previously installed and, as I said previously, on the actualizations this list is going to be empty anyway. I'm going to fix it because you are truth, it is ugly, but I don't think it is a blocker because the functionality is not affected.
Is FOTA update impacted? How are 1.1 users impacted when upgraded to 1.2? Is their data intact?
Flags: needinfo?(cjc)
No, FOTA updates will not be impacted. The only effect will be that an error will be logged. The users will not see anything and their data will not be affected.
Flags: needinfo?(cjc)
Sounds like we don't need to block on this then.
blocking-b2g: koi? → -
Attached file Patch-v1 (deleted) —
The proposed patch resolves the described problem creating a new version of database
Attachment #828581 - Flags: review?(crdlc)
Comment on attachment 828581 [details] Patch-v1 I added a comment but I would like to Julien review it because he was who detected the issue and I am in SF very busy and I cannot test on my device. Could you take a look? Thanks a lot Julien
Attachment #828581 - Flags: review?(crdlc) → review?(felash)
Comment on attachment 828581 [details] Patch-v1 r=me with adding 2 comments I haven't actually tried it on device but this looks good!
Attachment #828581 - Flags: review?(felash) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 958939
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: