From 0d7097a682f3be90a6a2790fbca412a68a8dafca Mon Sep 17 00:00:00 2001
From: Mark McDowall <markus.mcd5@gmail.com>
Date: Mon, 7 Oct 2013 20:35:27 -0700
Subject: [PATCH] Deleting logs will refresh on completion

---
 src/UI/System/Logs/Table/LogsTableLayout.js | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/src/UI/System/Logs/Table/LogsTableLayout.js b/src/UI/System/Logs/Table/LogsTableLayout.js
index 1eb1ef132..9512e4105 100644
--- a/src/UI/System/Logs/Table/LogsTableLayout.js
+++ b/src/UI/System/Logs/Table/LogsTableLayout.js
@@ -1,6 +1,7 @@
 'use strict';
 define(
     [
+        'app',
         'marionette',
         'backgrid',
         'System/Logs/Table/LogTimeCell',
@@ -9,7 +10,7 @@ define(
         'System/Logs/LogsCollection',
         'Shared/Toolbar/ToolbarLayout',
         'Shared/LoadingView'
-    ], function (Marionette, Backgrid, LogTimeCell, LogLevelCell, GridPager, LogCollection, ToolbarLayout, LoadingView) {
+    ], function (App, Marionette, Backgrid, LogTimeCell, LogLevelCell, GridPager, LogCollection, ToolbarLayout, LoadingView) {
         return Marionette.Layout.extend({
             template: 'System/Logs/Table/LogsTableLayoutTemplate',
 
@@ -57,6 +58,8 @@ define(
             initialize: function () {
                 this.collection = new LogCollection();
                 this.collectionPromise = this.collection.fetch();
+
+                App.vent.on(App.Events.CommandComplete, this._commandComplete, this);
             },
 
             onRender: function () {
@@ -72,8 +75,6 @@ define(
                 });
             },
 
-
-
             _showTable: function () {
                 this.grid.show(new Backgrid.Grid({
                     row       : Backgrid.Row,
@@ -104,11 +105,7 @@ define(
                         {
                             title          : 'Clear Logs',
                             icon           : 'icon-trash',
-                            command        : 'clearLog',
-                            successMessage : 'Logs have been cleared',
-                            errorMessage   : 'Failed to clear logs',
-                            ownerContext   : this,
-                            onSuccess      : this._refreshLogs
+                            command        : 'clearLog'
                         }
                     ]
                 };
@@ -126,6 +123,12 @@ define(
                 this.collection.state.currentPage = 1;
                 this.collection.fetch({ reset: true });
                 this._showTable();
+            },
+
+            _commandComplete: function (options) {
+                if (options.command.get('name') === 'clearlog') {
+                    this._refreshLogs();
+                }
             }
         });
     });