Thanks. Can we remove the instance of the static utility class please?
-Adrian
On 12/14/2012 8:53 AM, jler...@apache.org wrote:
Author: jleroux
Date: Fri Dec 14 08:53:09 2012
New Revision: 1421738
URL: http://svn.apache.org/viewvc?rev=1421738&view=rev
Log:
Adrian remarked there was no need to use class fields
Modified:
ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java
Modified:
ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java?rev=1421738&r1=1421737&r2=1421738&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java
(original)
+++ ofbiz/trunk/framework/common/src/org/ofbiz/common/login/LoginServices.java
Fri Dec 14 08:53:09 2012
@@ -64,8 +64,6 @@ public class LoginServices {
public static final String module = LoginServices.class.getName();
public static final String resource = "SecurityextUiLabels";
- public boolean usePasswordPattern =
UtilProperties.getPropertyAsBoolean("security.properties",
"security.login.password.pattern.enable", true);
- public String passwordPattern = UtilProperties.getPropertyValue("security.properties",
"security.login.password.pattern", "^.*(?=.{5,}).*$");
/** Login service to authenticate username and password
* @return Map of results including (userLogin) GenericValue object
@@ -961,6 +959,8 @@ public class LoginServices {
if (newPassword != null) {
// Matching password with pattern
+ String passwordPattern = UtilProperties.getPropertyValue("security.properties",
"security.login.password.pattern", "^.*(?=.{5,}).*$");
+ boolean usePasswordPattern =
UtilProperties.getPropertyAsBoolean("security.properties",
"security.login.password.pattern.enable", true);
if (usePasswordPattern) {
Pattern pattern = Pattern.compile(passwordPattern);
Matcher matcher = pattern.matcher(newPassword);