Closed Bug 619995 Opened 14 years ago Closed 14 years ago

Sync Options dialog hangs if choose "Replace all data on this computer"

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: jduell.mcbugs, Assigned: philikon)

References

Details

(Whiteboard: [has patch][has review])

Attachments

(1 file)

No description provided.
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20101216 Firefox/4.0b9pre Installed Minefield on brand new Windoze 7 Home Ed. laptop. Trying to hook it up with my existing Sync account via "Set up Sync" | "I already have a Sync Account" | "Sync Options". The Sync Options dialog's "Done" button does nothing if I've changed the default "Recommended: merge data" to "Replace this machine's data with Sync data." If I change the radio button back to the recommended setting, the "Done" button works again. Smells like a blocker to me.
blocking2.0: --- → ?
Yup, there's some places SQL in the UI code (*facepalm*) that we failed to update to the temp table deletion that landed as part of the places branch merge. Yay for no UI tests! We should really move this code out of the UI and into the respective engines. UI code can then use Weave.Engines.get(...).getCount() or something. This should block b9.
Assignee: nobody → philipp
OS: Windows 7 → All
Hardware: x86 → All
Attached patch v1 (deleted) — Splinter Review
This just fixes the issue at hand. Filed bug 620577 for cleaning up this code properly (move the SQL code to services/sync/)
Attachment #498911 - Flags: review?(mconnor)
Whiteboard: [has patch][needs review mconnor]
blocking2.0: ? → beta9+
Blocks: 622134
Blocks: 615950
No longer blocks: 622134
Attachment #498911 - Flags: review?(mconnor) → review+
Whiteboard: [has patch][needs review mconnor] → [has patch][has review]
As per today's meeting, beta 9 will be a time-based release. Marking these all betaN+. Please move it back to beta9+ if you believe it MUST be in the next beta (ie: trunk is in an unshippable state without this)
No longer blocks: 615950
blocking2.0: beta9+ → betaN+
Fixing fields my automated script accidentally blanked. Apologies for the bugspam
Blocks: 615950
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
No longer blocks: 615950
Works on 1.6.1. Will wait for next nightly to verify trunk.
(In reply to comment #11) > Works on 1.6.1. This never affected the add-on in the first place.
Addon was affected with SeaMonkey.
(In reply to comment #13) > Addon was affected with SeaMonkey. Please file a new bug for that. This bug is about the *Firefox* UI.
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: