From 8cb2b9b97af5512d791fe4bb40cf3960ba395dba Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@designcreateplay.com>
Date: Wed, 16 Apr 2014 14:45:47 -0400
Subject: [PATCH] fixed #1351 -- themes and plugins no longer auto-restart the
 server

---
 public/src/forum/admin/plugins.js | 14 ++++++++++----
 public/src/forum/admin/themes.js  | 11 +++++++----
 src/socket.io/admin.js            |  2 --
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/public/src/forum/admin/plugins.js b/public/src/forum/admin/plugins.js
index a494f3053d..01b4e4589f 100644
--- a/public/src/forum/admin/plugins.js
+++ b/public/src/forum/admin/plugins.js
@@ -1,3 +1,6 @@
+"use strict";
+/* global define, app, socket */
+
 define(function() {
 	var Plugins = {
 		init: function() {
@@ -19,10 +22,13 @@ define(function() {
 					app.alert({
 						alert_id: 'plugin_toggled',
 						title: 'Plugin ' + (status.active ? 'Enabled' : 'Disabled'),
-						message: 'Restarting your NodeBB <i class="fa fa-refresh fa-spin"></i>',
-						type: 'warning',
-						timeout: 5000
-					})
+						message: 'Please restart your NodeBB to fully ' + (status.active ? 'activate' : 'deactivate') + ' this plugin',
+						type: 'info',
+						timeout: 5000,
+						clickfn: function() {
+							socket.emit('admin.restart');
+						}
+					});
 				});
 			} else {
 				pluginsList.append('<li><p><i>No plugins found.</i></p></li>');
diff --git a/public/src/forum/admin/themes.js b/public/src/forum/admin/themes.js
index 4dd38c1d92..e74887d8f2 100644
--- a/public/src/forum/admin/themes.js
+++ b/public/src/forum/admin/themes.js
@@ -39,10 +39,13 @@ define(['forum/admin/settings'], function(Settings) {
 
 					app.alert({
 						alert_id: 'admin:theme',
-						type: 'success',
+						type: 'info',
 						title: 'Theme Changed',
-						message: 'Restarting your NodeBB <i class="fa fa-refresh fa-spin"></i>',
-						timeout: 3500
+						message: 'Please restart your NodeBB to fully activate this theme',
+						timeout: 5000,
+						clickfn: function() {
+							socket.emit('admin.restart');
+						}
 					});
 				});
 			}
@@ -251,7 +254,7 @@ define(['forum/admin/settings'], function(Settings) {
 							timeout: 2500
 						});
 					}
-					
+
 				});
 			});
 		}
diff --git a/src/socket.io/admin.js b/src/socket.io/admin.js
index 64fbb841ba..05286f616b 100644
--- a/src/socket.io/admin.js
+++ b/src/socket.io/admin.js
@@ -79,7 +79,6 @@ SocketAdmin.themes.set = function(socket, data, callback) {
 	widgets.reset(function(err) {
 		meta.themes.set(data, function() {
 			callback();
-			meta.restart();
 		});
 	});
 };
@@ -87,7 +86,6 @@ SocketAdmin.themes.set = function(socket, data, callback) {
 SocketAdmin.plugins.toggle = function(socket, plugin_id) {
 	plugins.toggleActive(plugin_id, function(status) {
 		socket.emit('admin.plugins.toggle', status);
-		meta.restart();
 	});
 };