[FIXED JENKINS-13197] Saving job config with managed script builder c…

…auses json error
jenkinsci · Mar 24, 2012 · acc8050 · acc8050
1 parent b22d541
commit acc8050
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/managedscripts/ScriptBuildStep.java b/src/main/java/org/jenkinsci/plugins/managedscripts/ScriptBuildStep.java
@@ -297,8 +297,8 @@ public ScriptBuildStep newInstance(StaplerRequest req, JSONObject json) {
 			logger.log(Level.FINE, json.toString(2));
 
 			String id = json.getString("buildStepId");
-			final JSONObject definedArgs = json.getJSONObject("defineArgs");
-			if (!definedArgs.isNullObject()) {
+			final JSONObject definedArgs = json.optJSONObject("defineArgs");
+			if (definedArgs != null && !definedArgs.isNullObject()) {
 				JSONObject argsObj = definedArgs.optJSONObject("buildStepArgs");
 				if (argsObj == null) {
 					JSONArray argsArrayObj = definedArgs.optJSONArray("buildStepArgs");