Fix issue with optional number params / fields with default values, getting reset if set to empty.

Author: jhuckaby

htdocs/js/pages/PageUtils.class.js

@@ -2572,7 +2572,10 @@ Page.PageUtils = class PageUtils extends Page.Base {
 						autocomplete: 'off',
 						spellcheck: 'false'
 					};
-					if (param.variant == 'number') text_args.step = 'any';
+					if (param.variant == 'number') {
+						text_args.step = 'any';
+						if (!(param.id in params)) text_args.value = '';
+					}
 					if (!param.variant || param.variant.match(/^(password|text|tel)$/)) {
 						// only show explorer icon for non-validating text variants
 						html += explore_start + self.getFormText(text_args) + explore_end;
@@ -4964,7 +4967,10 @@ Page.PageUtils = class PageUtils extends Page.Base {
 						autocomplete: 'off',
 						spellcheck: 'false'
 					};
-					if (param.variant == 'number') text_args.step = 'any';
+					if (param.variant == 'number') {
+						text_args.step = 'any';
+						if (!(param.id in params)) text_args.value = '';
+					}
 					if (!param.variant || param.variant.match(/^(password|text|tel)$/)) {
 						// only show explorer icon for non-validating text variants
 						html += explore_start + self.getFormText(text_args) + explore_end;

lib/job.js

@@ -155,16 +155,6 @@ class Jobs {
 
 			if (plugin.runner) job.runner = true;
 			else delete job.runner;
-			
-			// plugin params may have changed outside of event, so re-copy missing ones
-			if (plugin.params) plugin.params.forEach( function(param) {
-				if (!(param.id in job.params)) {
-					if (param.type == 'select') {
-						job.params[ param.id ] = param.value.replace(/\,.*$/, '').replace(/^.+\[([\w\-\.]+)\]\s*$/, '$1');
-					}
-					else job.params[ param.id ] = param.value;
-				}
-			} );
 		} // plugin
 
 		// apply placeholder substitution on all event params with job as context