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

List:       helix-server-cvs
Subject:    [Server-cvs] admin/web/build/mproxy config_ports.html, 1.10.4.2, 1.10.4.3
From:       yijil () helixcommunity ! org
Date:       2012-10-22 10:15:57
[Download RAW message or body]

Update of /cvsroot/server/admin/web/build/mproxy
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv12278/build/mproxy

Modified Files:
      Tag: SERVER_15_0
	config_ports.html 
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
===================================================================
RCS file: /cvsroot/server/admin/web/build/mproxy/config_ports.html,v
retrieving revision 1.10.4.2
retrieving revision 1.10.4.3
diff -u -d -r1.10.4.2 -r1.10.4.3
--- config_ports.html	27 Sep 2012 07:39:58 -0000	1.10.4.2
+++ config_ports.html	22 Oct 2012 10:15:39 -0000	1.10.4.3
@@ -126,13 +126,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]