From: Marcin Haba <marcin.haba@bacula.pl>
Date: Sun, 18 Aug 2019 15:31:26 +0000 (+0200)
Subject: baculum: Fix saving to config empty runscript subresources
X-Git-Tag: Release-9.6.0~132
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=72e91629b33dba22a20c939af4c819731fcc611b;p=thirdparty%2Fbacula.git

baculum: Fix saving to config empty runscript subresources
---

diff --git a/gui/baculum/protected/Web/Portlets/DirectiveRunscript.php b/gui/baculum/protected/Web/Portlets/DirectiveRunscript.php
index 024e55257..e85abe74d 100644
--- a/gui/baculum/protected/Web/Portlets/DirectiveRunscript.php
+++ b/gui/baculum/protected/Web/Portlets/DirectiveRunscript.php
@@ -135,13 +135,6 @@ class DirectiveRunscript extends DirectiveListTemplate {
 				$in_config = $controls[$j]->getInConfig();
 				$index = $controls[$j]->getGroupName();
 
-				if (!isset($directive_values['Runscript'])) {
-					$directive_values = array('Runscript' => array());
-				}
-				if (!isset($directive_values['Runscript'][$index])) {
-					$directive_values['Runscript'][$index] = new stdClass;
-				}
-
 				if (is_null($directive_value)) {
 					// skip not changed values that don't exist in config
 					continue;
@@ -153,6 +146,14 @@ class DirectiveRunscript extends DirectiveListTemplate {
 					// value the same as default value, skip it
 					continue;
 				}
+
+				if (!isset($directive_values['Runscript'])) {
+					$directive_values = array('Runscript' => array());
+				}
+				if (!isset($directive_values['Runscript'][$index])) {
+					$directive_values['Runscript'][$index] = new stdClass;
+				}
+
 				$directive_values['Runscript'][$index]->{$directive_name} = $directive_value;
 			}
 		}