From 74016c928fea3a17769c18b83c5e4066853a5f4d Mon Sep 17 00:00:00 2001
From: alejandro-campos <alejandro.campos@artica.es>
Date: Tue, 30 Oct 2018 10:20:31 +0100
Subject: [PATCH] fixed bug: failing load of post process and unit fields in
 plugin component in module creation

---
 .../agentes/module_manager_editor_common.php  |  2 +-
 pandora_console/include/javascript/pandora.js | 43 ++++++++++++++++++-
 .../include/javascript/pandora_modules.js     | 19 +++++++-
 3 files changed, 59 insertions(+), 5 deletions(-)

diff --git a/pandora_console/godmode/agentes/module_manager_editor_common.php b/pandora_console/godmode/agentes/module_manager_editor_common.php
index 0527bea08d..19dcc94ffa 100644
--- a/pandora_console/godmode/agentes/module_manager_editor_common.php
+++ b/pandora_console/godmode/agentes/module_manager_editor_common.php
@@ -439,7 +439,7 @@ $table_advanced->data[2][1] .= html_print_input_hidden ('moduletype', $moduletyp
 $table_advanced->data[2][3] = __('Post process').' ' . ui_print_help_icon ('postprocess', true);
 $table_advanced->data[2][4] =
 	html_print_extended_select_for_post_process('post_process',
-		$post_process, '', '', '0', false, true, false, true,
+		$post_process, '', '', '0', false, true, false, false,
 		$disabledBecauseInPolicy);
 $table_advanced->colspan[2][4] = 3;
 
diff --git a/pandora_console/include/javascript/pandora.js b/pandora_console/include/javascript/pandora.js
index 2b323a55f1..1ca0759801 100644
--- a/pandora_console/include/javascript/pandora.js
+++ b/pandora_console/include/javascript/pandora.js
@@ -594,12 +594,51 @@ function post_process_select_init_unit(name,selected) {
 function post_process_select_events_unit(name,selected) {
 	
 	$('.' + name + '_toggler').click(function() {
-		$('#' + name + '_select option[value=none]').attr("selected",true);
-		$('#text-' + name + '_text').val("");
+		var value = $('#text-' + name + '_text').val();
+		
+		var count = $('#' + name + '_select option')
+			.filter(function(i, item) {
+				
+				if ($(item).val() == value)
+					return true;
+				else return false;
+			})
+			.length;
+		
+		if (count != 1) {
+			$('#' + name + '_select')
+				.append($("<option>").val(value).text(value));
+			
+		}
+		
+		$('#' + name + '_select option')
+			.filter(function(i, item) {
+				
+				if ($(item).val() == value)
+					return true;
+				else return false;
+			})
+			.prop("selected", true);
+		
 		toggleBoth(name);
+		$('#text-' + name + '_text').focus();
+	});
+	
+	// When select a default period, is setted in seconds
+	$('#' + name + '_select').change(function() {
+		var value = $('#' + name + '_select').val();
+		
+		$('.' + name).val(value); 
+		$('#text-' + name + '_text').val(value);
+	});
+	
+	$('#text-' + name + '_text').keyup (function () {
+		var value = $('#text-' + name + '_text').val();
+		$('.' + name).val(value);
 	});
 }
 
+
 function post_process_select_events(name) {
 	$('.' + name + '_toggler').click(function() {
 		var value = $('#text-' + name + '_text').val();
diff --git a/pandora_console/include/javascript/pandora_modules.js b/pandora_console/include/javascript/pandora_modules.js
index 459776c5b2..06c971f5c0 100644
--- a/pandora_console/include/javascript/pandora_modules.js
+++ b/pandora_console/include/javascript/pandora_modules.js
@@ -320,8 +320,23 @@ function configure_modules_form () {
 				$("#text-ff_event_normal").attr ("value", (data["min_ff_event_normal"] == 0) ? 0 : data["min_ff_event_normal"]);
 				$("#text-ff_event_warning").attr ("value", (data["min_ff_event_warning"] == 0) ? 0 : data["min_ff_event_warning"]);
 				$("#text-ff_event_critical").attr ("value", (data["min_ff_event_critical"] == 0) ? 0 : data["min_ff_event_critical"]);
-				$("#text-post_process").attr("value", (data["post_process"] == 0) ? 0 : data["post_process"]);
-				$("#text-unit").attr("value", (data["unit"] == '') ? '' : data["unit"]);
+
+				// Shows manual input if post_process field is setted
+				if (data["post_process"] != 0) {
+					$('#post_process_manual').show();
+					$('#post_process_default').hide();
+				}
+
+				$("#text-post_process_text").attr("value", (data["post_process"] == 0) ? 0 : data["post_process"]);
+
+				// Shows manual input if unit field is setted
+				if (data["unit"] != '') {
+					$('#unit_manual').show();
+					$('#unit_default').hide();
+				}
+
+				$("#text-unit_text").attr("value", (data["unit"] == '') ? '' : data["unit"]);
+
 				$("#checkbox-critical_inverse").prop ("checked", data["critical_inverse"]);
 				$("#checkbox-warning_inverse").prop ("checked", data["warning_inverse"]);
 				$("#component_loading").hide ();