From 6f54250ee5bcf1425d7054138d8a3366ef2861fa Mon Sep 17 00:00:00 2001 From: Kevin Sawicki Date: Wed, 22 May 2013 15:41:03 -0700 Subject: [PATCH] Include error in trigger arguments --- src/app/installed-packages-config-panel.coffee | 6 ++++-- src/app/package-config-view.coffee | 6 ++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/app/installed-packages-config-panel.coffee b/src/app/installed-packages-config-panel.coffee index 12c7c9c26..e119a7e28 100644 --- a/src/app/installed-packages-config-panel.coffee +++ b/src/app/installed-packages-config-panel.coffee @@ -20,8 +20,10 @@ class InstalledPackagesConfigPanel extends ConfigPanel @packagesArea.append(new PackageConfigView(pack, @packageEventEmitter)) @packageEventEmitter.trigger 'installed-packages-loaded', [@packages] - @packageEventEmitter.on 'package-installed', (pack) => @addPackage(pack) - @packageEventEmitter.on 'package-uninstalled', (pack) => @removePackage(pack) + @packageEventEmitter.on 'package-installed', (error, pack) => + @addPackage(pack) unless error? + @packageEventEmitter.on 'package-uninstalled', (error, pack) => + @removePackage(pack) unless error? removePackage: ({name}) -> @packages = _.reject @packages, (pack) -> pack.name is name diff --git a/src/app/package-config-view.coffee b/src/app/package-config-view.coffee index 7b807e9d6..b0b386a94 100644 --- a/src/app/package-config-view.coffee +++ b/src/app/package-config-view.coffee @@ -104,14 +104,12 @@ class PackageConfigView extends View else @defaultAction.text('Uninstalling\u2026') packageManager.uninstall @pack, (error) => - unless error? - @packageEventEmitter.trigger('package-uninstalled', @pack) + @packageEventEmitter.trigger('package-uninstalled', error, @pack) packageManagerCallback() else @defaultAction.text('Installing\u2026') packageManager.install @pack, (error) => - unless error? - @packageEventEmitter.trigger('package-installed', @pack) + @packageEventEmitter.trigger('package-installed', error, @pack) packageManagerCallback() @updateDefaultAction()