marcellustavares/liferay-plugins — pull request #187 by lipusz (2014-07-08 12:43:14+00:00)
Title: [TECHNICAL-SUPPORT | July 18] LPS-48296 Web Form portlet shows field Validation options even with validation.script.enabled=false

https://issues.liferay.com/browse/LPS-48296

Hey Marcellus,

Property "validation.script.enabled" is already checked in the backend (WebFormPortlet#validate), so it doesn't make sense to display the "Validation" fields in the configuration view when the value is set to false.

Regarding the 1st commit (lipusz@ba2f9f4): the field editing is disabled (previous form data exist) we display the current fields as a non-editable elements, so I do no see any reasons for adding the script to the response in this case.

Let me know if you have any concerns,
Tibor