I am sure Scott is right. @Jacques: the meaning of demand in English is: "an insistent and peremptory request, made as if by right" and it is different from the French "demande" (I guess) and the Italian "domanda" that simply mean "request". I am pretty sure that Jacques meant to use "demand(e)" as a synonym of "request".
Regards, Jacopo On Dec 15, 2012, at 3:18 AM, Scott Gray wrote: > I think Jacques tends to use "demand" where most would use "request", just a > language issue. > > Regards > Scott > > On 14/12/2012, at 9:36 PM, Adrian Crum wrote: > >> Please don't do this. >> >> There was no demand - Adam and I suggested a better way to do things, and >> this is not a better way. There is no need to store the property values in >> class fields. >> >> -Adrian >> >> On 12/14/2012 3:27 AM, jler...@apache.org wrote: >>> Author: jleroux >>> Date: Fri Dec 14 03:27:40 2012 >>> New Revision: 1421651 >>> >>> URL: http://svn.apache.org/viewvc?rev=1421651&view=rev >>> Log: >>> At Adrian and Adam's demand allow to use cache here, use default constructor >>> Also refactored things a bit: >>> * uses getPropertyAsBoolean >>> * set default property values >>> >>> 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=1421651&r1=1421650&r2=1421651&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 03:27:40 2012 >>> @@ -64,8 +64,8 @@ public class LoginServices { >>> public static final String module = LoginServices.class.getName(); >>> public static final String resource = "SecurityextUiLabels"; >>> - public static boolean usePasswordPattern = >>> "true".equals(UtilProperties.getPropertyValue("security.properties", >>> "security.login.password.pattern.enable")); >>> - public static String passwordPattern = >>> UtilProperties.getPropertyValue("security.properties", >>> "security.login.password.pattern"); >>> + 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 >>> @@ -517,7 +517,8 @@ public class LoginServices { >>> } >>> } >>> - checkNewPassword(null, null, currentPassword, >>> currentPasswordVerify, passwordHint, errorMessageList, true, locale); >>> + LoginServices loginServices = new LoginServices(); >>> + loginServices.checkNewPassword(null, null, currentPassword, >>> currentPasswordVerify, passwordHint, errorMessageList, true, locale); >>> GenericValue userLoginToCreate = delegator.makeValue("UserLogin", >>> UtilMisc.toMap("userLoginId", userLoginId)); >>> userLoginToCreate.set("externalAuthId", externalAuthId); >>> @@ -656,7 +657,8 @@ public class LoginServices { >>> List<String> errorMessageList = FastList.newInstance(); >>> if (newPassword != null) { >>> - checkNewPassword(userLoginToUpdate, currentPassword, >>> newPassword, newPasswordVerify, >>> + LoginServices loginServices = new LoginServices(); >>> + loginServices.checkNewPassword(userLoginToUpdate, >>> currentPassword, newPassword, newPasswordVerify, >>> passwordHint, errorMessageList, adminUser, locale); >>> } >>> @@ -887,7 +889,7 @@ public class LoginServices { >>> return result; >>> } >>> - public static void checkNewPassword(GenericValue userLogin, String >>> currentPassword, String newPassword, String newPasswordVerify, String >>> passwordHint, List<String> errorMessageList, boolean ignoreCurrentPassword, >>> Locale locale) { >>> + public void checkNewPassword(GenericValue userLogin, String >>> currentPassword, String newPassword, String newPasswordVerify, String >>> passwordHint, List<String> errorMessageList, boolean ignoreCurrentPassword, >>> Locale locale) { >>> boolean useEncryption = >>> "true".equals(UtilProperties.getPropertyValue("security.properties", >>> "password.encrypt")); >>> String errMsg = null; >>> >>> >> >