- don't set password via remoting if field is empty

Marius Cramer

2015-08-06 37b29231e47a0c4458dc1c15d98588f16f07e1e2

 server/plugins-available/firewall_plugin.inc.php

@@ -38,7 +38,7 @@
   public function onInstall() {
      global $conf;

      if($conf['bastille']['installed'] = true && $conf['services']['firewall'] == true) {
      if($conf['bastille']['installed'] == true && $conf['services']['firewall'] == true) {
         return true;
      } else {
         return false;
@@ -76,29 +76,31 @@
      global $app, $conf;

      //* load the server configuration options
      $app->uses('getconf');
      $server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
      if($server_config['firewall'] == 'ufw') {
         $this->ufw_update($event_name, $data);
      } else {
         $this->bastille_update($event_name, $data);
      if(!$data['mirrored']) {
         $app->uses('getconf');
         $server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
         if($server_config['firewall'] == 'ufw') {
            $this->ufw_update($event_name, $data);
         } else {
            $this->bastille_update($event_name, $data);
         }
      }

   }

   public function delete($event_name, $data) {
      global $app, $conf;

      //* load the server configuration options
      $app->uses('getconf');
      $server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
      if(!$data['mirrored']) {
         $app->uses('getconf');
         $server_config = $app->getconf->get_server_config($conf['server_id'], 'server');

      if($server_config['firewall'] == 'ufw') {
         $this->ufw_delete($event_name, $data);
      } else {
         $this->bastille_delete($event_name, $data);
         if($server_config['firewall'] == 'ufw') {
            $this->ufw_delete($event_name, $data);
         } else {
            $this->bastille_delete($event_name, $data);
         }
      }

   }

   private function ufw_update($event_name, $data) {