Improved logic for HHVM/Pagespeed update

ee/cli/plugins/site.py

@@ -326,7 +326,7 @@ class EESiteCreateController(CementBaseController):
         if not ee_domain.strip():
             Log.error("Invalid domain name, "
                       "Provide valid domain name")
-                      
+
         ee_site_webroot = EEVariables.ee_webroot + ee_domain
 
         if check_domain_exists(self, ee_domain):
@@ -718,22 +718,27 @@ class EESiteUpdateController(CementBaseController):
                 pagespeed = False
 
         if self.app.pargs.pagespeed:
-            if pagespeed == old_pagespeed:
+            if pagespeed is old_pagespeed:
                 if pagespeed is False:
-                    Log.error(self, "Pagespeed is allready disabled for given "
-                              "site")
+                    Log.info(self, "Pagespeed is allready disabled for given "
+                             "site")
                 elif pagespeed is True:
-                    Log.error(self, "Pagespeed is allready enabled for given "
-                              "site")
+                    Log.info(self, "Pagespeed is allready enabled for given "
+                             "site")
 
         if self.app.pargs.hhvm:
-            if hhvm == old_hhvm:
+            if hhvm is old_hhvm:
                 if hhvm is False:
-                    Log.error(self, "HHVM is allready disabled for given "
-                              "site")
+                    Log.info(self, "HHVM is allready disabled for given "
+                             "site")
                 elif hhvm is True:
-                    Log.error(self, "HHVM is allready enabled for given "
-                              "site")
+                    Log.info(self, "HHVM is allready enabled for given "
+                             "site")
+
+        if self.app.pargs.pagespeed and self.app.pargs.hhvm:
+            if ((hhvm is old_hhvm) and (pagespeed is old_pagespeed) and
+               (stype == oldsitetype and cache == oldcachetype)):
+                self.app.close(0)
 
         if data and (not self.app.pargs.hhvm):
             if old_hhvm is True: