[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-server-cvs
Subject:    [Server-cvs] admin/web/src/srvprxy config_ports.html.wasm, 1.15, 1.16
From:       yijil () helixcommunity ! org
Date:       2012-10-22 10:18:21
[Download RAW message or body]

Update of /cvsroot/server/admin/web/src/srvprxy
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv14268/src/srvprxy

Modified Files:
	config_ports.html.wasm 
Log Message:
Committed to : HEAD, SERVER_15_0_RN

Reviewed by: Xiaocheng Li

Synopsis: 
===============
Bug 270784: Invalid error message when attempting to change control port security in \
Helix Admin

Branch: SERVER_15_0_RN, HEAD

Suggested Reviewers: Anyone

Description: 
===============
When attempting to enable "Control Port Security" and clicking save, an invalid error \
message appears as in the attached image saying that the control port and <> cannot \
be shared.

ROOT CAUSE:
When change "Control Port Security", the page proceed it like other port property. So \
all "YES" is treat as the same value. And alert warm message.

My Fix:
When change "Enable Control Port Security", "Enable Ramgen Port Hinting URLs" and \
"Enable HTTP Fail Over URL for ASXGen", it does not need treat it like port property.

Files affected:
===============
server/admin/web/build/mproxy/config_ports.html
server/admin/web/build/mserver/config_ports.html
server/admin/web/build/proxy/config_ports.html
server/admin/web/build/server/config_ports.html
server/admin/web/src/srvprxy/config_ports.html.wasm

Testing Performed: 
===============
Integration Tests:
N/A

Performance Tests: 
N/A

QA Hints
========
None.



Index: config_ports.html.wasm
===================================================================
RCS file: /cvsroot/server/admin/web/src/srvprxy/config_ports.html.wasm,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -d -r1.15 -r1.16
--- config_ports.html.wasm	27 Sep 2012 08:08:42 -0000	1.15
+++ config_ports.html.wasm	22 Oct 2012 10:17:59 -0000	1.16
@@ -178,13 +178,17 @@
 								msg = "The " + prop.label + " is in the " + prop2.label + ".";
 							}
 						}
+						else if (propName.bSearch( "CloakingHint" ) || propName.bSearch( \
"EnableControlPortSecurity" ) || propName.bSearch( "HaveAltHTTPURL" )) +						{
+							break;
+						}
 						else if ( prop.getValue() && (prop.getValue() == prop2.getValue()) )
 						{
-                                                	var strSharePorts = \
                CanSharePortsList.toString();
-                                                	if ( \
                !(strSharePorts.bSearch(propName)) ||  \
                !(strSharePorts.bSearch(propName2)) )
-                                                	{
-                                                            msg = "The " + \
                prop.label + " and " + prop2.label + " cannot share the same port \
                value.";
-                                                        }
+							var strSharePorts = CanSharePortsList.toString();
+							if ( !(strSharePorts.bSearch(propName)) ||  \
!(strSharePorts.bSearch(propName2)) ) +							{
+								msg = "The " + prop.label + " and " + prop2.label + " cannot share the same \
port value."; +							}
 						}
 					}
 				}


_______________________________________________
Server-cvs mailing list
Server-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/server-cvs


[prev in list] [next in list] [prev in thread] [next in thread]