482792 - Provide a wipeClient, wipeRemote, resetService for weave service

Status: RESOLVED FIXED

(Cloud Services :: General, defect, P1)

Description

[Ed Lee :Mardak]

For the reset UI, it'll need ways to get rid of all data on the client, on the remote clients. A useful helper resetService would be nice to get rid of non-engine service caches, etc.

Comment 1

[Ed Lee :Mardak]

Attachment: v1

Refactor out resetService from resetClient (for clearing logs, cache, etc).

Add wipeClient that resets services stuff and calls wipe on each engine (which will call its own reset then wipe data).

Add wipeRemote that just sends the wipeAll command. (Originally I had it wipe the server... but that caused syncIDs to mismatch when resyncing after the wipe which would cause local data to be reset.. including the wipeAll command.)

Comment 2

[Dan Mills [:thunder]]

Comment on attachment 366900 [details] [diff] [review]
v1

Looks good, except:

>+  /**
>+   * Wipe all remote user data by wiping the server then telling each remote
>+   * client to wipe itself.
>+   *
>+   * @param onComplete
>+   *        Callback when this method completes
>+   */
>+  wipeRemote: function WeaveSvc_wipeRemote(onComplete) {

wipeRemote needs an optional engine list to use instead of sending wipeAll.  Split this to a followup bug if you prefer, though.

Comment 3

[Ed Lee :Mardak]

http://hg.mozilla.org/labs/weave/rev/72029d49cd62