java/code/src/com/redhat/rhn/frontend/action/user/UserEditActionHelper.java | 17 +++++-----
1 file changed, 10 insertions(+), 7 deletions(-)
New commits:
commit 6ba702882de04a73360b6472d37c7916dfca68f0
Author: Tomas Lestach <tlestach(a)redhat.com>
Date: Thu Oct 13 15:17:18 2011 +0200
fixing checkstyle issues
diff --git a/java/code/src/com/redhat/rhn/frontend/action/user/UserEditActionHelper.java b/java/code/src/com/redhat/rhn/frontend/action/user/UserEditActionHelper.java
index 98e3619..e7726ad 100644
--- a/java/code/src/com/redhat/rhn/frontend/action/user/UserEditActionHelper.java
+++ b/java/code/src/com/redhat/rhn/frontend/action/user/UserEditActionHelper.java
@@ -39,7 +39,8 @@ public abstract class UserEditActionHelper extends RhnAction {
* @param form The form we're grabbing the info from
* @return Returns an ActionErrors object containing the errors (if any) that occurred.
*/
- public ActionErrors updateDetails(User loggedInUser, User targetUser, DynaActionForm form) {
+ public ActionErrors updateDetails(User loggedInUser, User targetUser,
+ DynaActionForm form) {
//get validation errors
ActionErrors errors = RhnValidationHelper.validateDynaActionForm(this, form);
@@ -82,14 +83,16 @@ public abstract class UserEditActionHelper extends RhnAction {
* @param targetUser The user that will be updated
* @param form The form containing the attribute value to use
*/
- protected void updatePamAttribute(User loggedInUser, User targetUser, DynaActionForm form) {
+ protected void updatePamAttribute(User loggedInUser, User targetUser,
+ DynaActionForm form) {
String pamAuthService = Config.get().getString(ConfigDefaults.WEB_PAM_AUTH_SERVICE);
- if (pamAuthService != null && pamAuthService.trim().length() > 0
- && loggedInUser.hasRole(RoleFactory.ORG_ADMIN)) {
- if (form.get("usepam") != null
- && ((Boolean) form.get("usepam")).booleanValue()) {
+ if (pamAuthService != null && pamAuthService.trim().length() > 0 &&
+ loggedInUser.hasRole(RoleFactory.ORG_ADMIN)) {
+ if (form.get("usepam") != null &&
+ ((Boolean) form.get("usepam")).booleanValue()) {
targetUser.setUsePamAuthentication(true);
- } else {
+ }
+ else {
targetUser.setUsePamAuthentication(false);
}
}