Browse Source

refactoring

Shinsuke Sugaya 9 năm trước cách đây
mục cha
commit
e3a37938a4
20 tập tin đã thay đổi với 91 bổ sung151 xóa
  1. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/badword/AdminBadwordAction.java
  2. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/boostdoc/AdminBoostdocAction.java
  3. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/dataconfig/AdminDataconfigAction.java
  4. 2 5
      src/main/java/org/codelibs/fess/app/web/admin/dict/kuromoji/AdminDictKuromojiAction.java
  5. 2 5
      src/main/java/org/codelibs/fess/app/web/admin/dict/synonym/AdminDictSynonymAction.java
  6. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/duplicatehost/AdminDuplicatehostAction.java
  7. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/elevateword/AdminElevatewordAction.java
  8. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/fileauth/AdminFileauthAction.java
  9. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/fileconfig/AdminFileconfigAction.java
  10. 4 7
      src/main/java/org/codelibs/fess/app/web/admin/group/AdminGroupAction.java
  11. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/keymatch/AdminKeymatchAction.java
  12. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/labeltype/AdminLabeltypeAction.java
  13. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/pathmap/AdminPathmapAction.java
  14. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/reqheader/AdminReqheaderAction.java
  15. 4 7
      src/main/java/org/codelibs/fess/app/web/admin/role/AdminRoleAction.java
  16. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/roletype/AdminRoletypeAction.java
  17. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/scheduler/AdminSchedulerAction.java
  18. 4 7
      src/main/java/org/codelibs/fess/app/web/admin/user/AdminUserAction.java
  19. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/webauth/AdminWebauthAction.java
  20. 5 8
      src/main/java/org/codelibs/fess/app/web/admin/webconfig/AdminWebconfigAction.java

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/badword/AdminBadwordAction.java

@@ -285,14 +285,11 @@ public class AdminBadwordAction extends FessAdminAction {
     private OptionalEntity<BadWord> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new BadWord()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new BadWord()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return badWordService.getBadWord(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/boostdoc/AdminBoostdocAction.java

@@ -210,14 +210,11 @@ public class AdminBoostdocAction extends FessAdminAction {
     private OptionalEntity<BoostDocumentRule> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new BoostDocumentRule()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new BoostDocumentRule()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return boostDocumentRuleService.getBoostDocumentRule(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/dataconfig/AdminDataconfigAction.java

@@ -224,14 +224,11 @@ public class AdminDataconfigAction extends FessAdminAction {
     private OptionalEntity<DataConfig> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new DataConfig()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new DataConfig()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return dataConfigService.getDataConfig(((EditForm) form).id);

+ 2 - 5
src/main/java/org/codelibs/fess/app/web/admin/dict/kuromoji/AdminDictKuromojiAction.java

@@ -321,11 +321,8 @@ public class AdminDictKuromojiAction extends FessAdminAction {
     private OptionalEntity<KuromojiItem> getEntity(final CreateForm form) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                final KuromojiItem entity = new KuromojiItem(0, StringUtil.EMPTY, StringUtil.EMPTY, StringUtil.EMPTY, StringUtil.EMPTY);
-                return OptionalEntity.of(entity);
-            }
-            break;
+            final KuromojiItem entity = new KuromojiItem(0, StringUtil.EMPTY, StringUtil.EMPTY, StringUtil.EMPTY, StringUtil.EMPTY);
+            return OptionalEntity.of(entity);
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return kuromojiService.getKuromojiItem(form.dictId, ((EditForm) form).id);

+ 2 - 5
src/main/java/org/codelibs/fess/app/web/admin/dict/synonym/AdminDictSynonymAction.java

@@ -325,11 +325,8 @@ public class AdminDictSynonymAction extends FessAdminAction {
     private OptionalEntity<SynonymItem> getEntity(final CreateForm form) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                final SynonymItem entity = new SynonymItem(0, StringUtil.EMPTY_STRINGS, StringUtil.EMPTY_STRINGS);
-                return OptionalEntity.of(entity);
-            }
-            break;
+            final SynonymItem entity = new SynonymItem(0, StringUtil.EMPTY_STRINGS, StringUtil.EMPTY_STRINGS);
+            return OptionalEntity.of(entity);
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return synonymService.getSynonymItem(form.dictId, ((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/duplicatehost/AdminDuplicatehostAction.java

@@ -210,14 +210,11 @@ public class AdminDuplicatehostAction extends FessAdminAction {
     private OptionalEntity<DuplicateHost> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new DuplicateHost()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new DuplicateHost()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return duplicateHostService.getDuplicateHost(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/elevateword/AdminElevatewordAction.java

@@ -296,14 +296,11 @@ public class AdminElevatewordAction extends FessAdminAction {
     private OptionalEntity<ElevateWord> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new ElevateWord()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new ElevateWord()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return elevateWordService.getElevateWord(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/fileauth/AdminFileauthAction.java

@@ -227,14 +227,11 @@ public class AdminFileauthAction extends FessAdminAction {
     private OptionalEntity<FileAuthentication> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new FileAuthentication()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new FileAuthentication()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return fileAuthenticationService.getFileAuthentication(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/fileconfig/AdminFileconfigAction.java

@@ -220,14 +220,11 @@ public class AdminFileconfigAction extends FessAdminAction {
     private OptionalEntity<FileConfig> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new FileConfig()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new FileConfig()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return fileConfigService.getFileConfig(((EditForm) form).id);

+ 4 - 7
src/main/java/org/codelibs/fess/app/web/admin/group/AdminGroupAction.java

@@ -212,13 +212,10 @@ public class AdminGroupAction extends FessAdminAction {
     private OptionalEntity<Group> getEntity(final CreateForm form) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new Group()).map(entity -> {
-                    entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new Group()).map(entity -> {
+                entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return groupService.getGroup(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/keymatch/AdminKeymatchAction.java

@@ -215,14 +215,11 @@ public class AdminKeymatchAction extends FessAdminAction {
     private OptionalEntity<KeyMatch> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new KeyMatch()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new KeyMatch()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return keyMatchService.getKeyMatch(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/labeltype/AdminLabeltypeAction.java

@@ -219,14 +219,11 @@ public class AdminLabeltypeAction extends FessAdminAction {
     private OptionalEntity<LabelType> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new LabelType()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new LabelType()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return labelTypeService.getLabelType(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/pathmap/AdminPathmapAction.java

@@ -211,14 +211,11 @@ public class AdminPathmapAction extends FessAdminAction {
     private OptionalEntity<PathMapping> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new PathMapping()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new PathMapping()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return pathMappingService.getPathMapping(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/reqheader/AdminReqheaderAction.java

@@ -227,14 +227,11 @@ public class AdminReqheaderAction extends FessAdminAction {
     private OptionalEntity<RequestHeader> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new RequestHeader()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new RequestHeader()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return requestHeaderService.getRequestHeader(((EditForm) form).id);

+ 4 - 7
src/main/java/org/codelibs/fess/app/web/admin/role/AdminRoleAction.java

@@ -212,13 +212,10 @@ public class AdminRoleAction extends FessAdminAction {
     private OptionalEntity<Role> getEntity(final CreateForm form) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new Role()).map(entity -> {
-                    entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new Role()).map(entity -> {
+                entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return roleService.getRole(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/roletype/AdminRoletypeAction.java

@@ -213,14 +213,11 @@ public class AdminRoletypeAction extends FessAdminAction {
     private OptionalEntity<RoleType> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new RoleType()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new RoleType()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return roleTypeService.getRoleType(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/scheduler/AdminSchedulerAction.java

@@ -298,14 +298,11 @@ public class AdminSchedulerAction extends FessAdminAction {
     private OptionalEntity<ScheduledJob> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new ScheduledJob()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new ScheduledJob()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return scheduledJobService.getScheduledJob(((EditForm) form).id);

+ 4 - 7
src/main/java/org/codelibs/fess/app/web/admin/user/AdminUserAction.java

@@ -234,13 +234,10 @@ public class AdminUserAction extends FessAdminAction {
     private OptionalEntity<User> getEntity(final CreateForm form) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new User()).map(entity -> {
-                    entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new User()).map(entity -> {
+                entity.setId(Base64.getEncoder().encodeToString(form.name.getBytes(Constants.CHARSET_UTF_8)));
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return userService.getUser(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/webauth/AdminWebauthAction.java

@@ -226,14 +226,11 @@ public class AdminWebauthAction extends FessAdminAction {
     private OptionalEntity<WebAuthentication> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new WebAuthentication()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new WebAuthentication()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return webAuthenticationService.getWebAuthentication(((EditForm) form).id);

+ 5 - 8
src/main/java/org/codelibs/fess/app/web/admin/webconfig/AdminWebconfigAction.java

@@ -220,14 +220,11 @@ public class AdminWebconfigAction extends FessAdminAction {
     private OptionalEntity<WebConfig> getEntity(final CreateForm form, final String username, final long currentTime) {
         switch (form.crudMode) {
         case CrudMode.CREATE:
-            if (form instanceof CreateForm) {
-                return OptionalEntity.of(new WebConfig()).map(entity -> {
-                    entity.setCreatedBy(username);
-                    entity.setCreatedTime(currentTime);
-                    return entity;
-                });
-            }
-            break;
+            return OptionalEntity.of(new WebConfig()).map(entity -> {
+                entity.setCreatedBy(username);
+                entity.setCreatedTime(currentTime);
+                return entity;
+            });
         case CrudMode.EDIT:
             if (form instanceof EditForm) {
                 return webConfigService.getWebConfig(((EditForm) form).id);