Closed Bug 781946 Opened 7 years ago Closed 7 years ago

Clean up notifications

Categories

(Firefox :: Sync, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: gps, Unassigned)

Details

(Whiteboard: [fixed in services])

Attachments

(1 file)

As part of going through service.js, I cleaned up a bunch of notifications code.

The following notifications were removed because they are unused:

  service:verify-login
  service:changepwd
  service:wipe-server
  service:reset-service
  service:wipe-client
  service:reset-client

I also changed service:changepph to service:change-passphrase

This eliminates a whole lot of Utils.notify() usage and makes the code cleaner. If we need these notifications in the future, they are easy enough to add back in.
Attachment #651011 - Flags: review?(rnewman)
Comment on attachment 651011 [details] [diff] [review]
Clean up notifications, v1

Review of attachment 651011 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good, modulo now not catching. Please fix and land, or let's discuss in the bug.

::: services/sync/modules/service.js
@@ -800,4 @@
>    },
>  
>    changePassphrase: function changePassphrase(newphrase) {
> -    return this._catch(this._notify("changepph", "", function onNotify() {

You killed this._catch here. I'm pretty sure we want to catch.

@@ -1622,5 @@
>     * @param engines [optional]
>     *        Array of engine names to reset. If not given, all engines are used.
>     */
> -  resetClient: function resetClient(engines)
> -    this._catch(this._notify("reset-client", "", function onNotify() {

Same…
Attachment #651011 - Flags: review?(rnewman) → review+
https://hg.mozilla.org/services/services-central/rev/7a7aff2f35d3

With this._catch added back in.
Status: NEW → ASSIGNED
Whiteboard: [fixed in services]
https://hg.mozilla.org/mozilla-central/rev/7a7aff2f35d3
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.