Details
-
Type: Bug
-
Status: Closed
-
Priority: Minor
-
Resolution: Fixed
-
Affects Version/s: 1.7.1
-
Fix Version/s: 1.7.2
-
Component/s: Documentation, Framework
-
Labels:None
-
Environment:all
-
Affects:Documentation (User Guide, Ref. Guide, etc.)
-
Workaround Exists:Yes
-
Workaround Description:Include or remove the context parameter entirely to activate/deactivate it.
Description
We use this as a marker to include the openajax.js library as one of the scripts on the page. The code looks like this:
if (context.getExternalContext().getInitParameter(D2DViewHandler.INCLUDE_OPEN_AJAX_HUB) != null) {
libs.add("/xmlhttp/openajax.js");
}
This might be slightly problematic since the code simply looks for the value to be non-null. Which means that setting to true/false makes no difference. The only way to turn it off is to remove the entry. I think we should change the behaviour and then document it as a true/false setting.
if (context.getExternalContext().getInitParameter(D2DViewHandler.INCLUDE_OPEN_AJAX_HUB) != null) {
libs.add("/xmlhttp/openajax.js");
}
This might be slightly problematic since the code simply looks for the value to be non-null. Which means that setting to true/false makes no difference. The only way to turn it off is to remove the entry. I think we should change the behaviour and then document it as a true/false setting.
Enable OpenAJAX integration only when context parameter is specifically set to 'true'.