Include error in trigger arguments

This commit is contained in:
Kevin Sawicki
2013-05-22 15:41:03 -07:00
parent 681bc18248
commit 6f54250ee5
2 changed files with 6 additions and 6 deletions

View File

@@ -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

View File

@@ -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()