From 491455286747cb100caee9ca12d9b5a94077b85e Mon Sep 17 00:00:00 2001
From: 이민희 <mhlee@maprex.co.kr>
Date: 화, 18 1월 2022 10:07:11 +0900
Subject: [PATCH] 이슈 추가/수정 시 ISP, 호스팅 이름 못 불러오는 문제 해결 (custom-input)

---
 src/main/java/kr/wisestone/owl/service/impl/IssueIspServiceImpl.java |  123 ++++++++++++++++++++++++----------------
 1 files changed, 74 insertions(+), 49 deletions(-)

diff --git a/src/main/java/kr/wisestone/owl/service/impl/IssueIspServiceImpl.java b/src/main/java/kr/wisestone/owl/service/impl/IssueIspServiceImpl.java
index a87e858..955cb24 100644
--- a/src/main/java/kr/wisestone/owl/service/impl/IssueIspServiceImpl.java
+++ b/src/main/java/kr/wisestone/owl/service/impl/IssueIspServiceImpl.java
@@ -44,6 +44,43 @@
         return this.issueIspRepository;
     }
 
+    /**
+     * �깉濡쒖슫 ISP 吏곸젒 異붽�
+     * @param issueForm IssueForm
+     * @param issue Issue
+     */
+    private void CreateIspField(IssueForm issueForm, Issue issue) {
+        // issueIsp �븘�뱶 �뤌 留뚮뱾湲�
+        IspFieldForm ispFieldForm = new IspFieldForm();
+        ispFieldForm.setName(issueForm.getIspName());
+        ispFieldForm.setCode(issueForm.getIspCode());
+        ispFieldForm.setEmail(issueForm.getIspEmail());
+        ispFieldForm.setUrl(issueForm.getIspUrl());
+        ispFieldForm.setManager(issueForm.getIspManager());
+        ispFieldForm.setTel(issueForm.getIspTel());
+        ispFieldForm.setMemo(issueForm.getIspMemo());
+
+        IssueIsp newIssueIsp = CreateIssueIsp(ispFieldForm, issue);
+        IspField ispField = new IspField();
+        StringBuilder sb = new StringBuilder();
+
+        // �궗�슜�옄媛� 吏곸젒 �엯�젰�떆�뿉 ISP 紐⑸줉�뿉 異붽�
+        if (newIssueIsp.getIspField() == null) {
+            IssueIsp oldIssueIsp = this.issueIspRepository.findByIssueId(issue.getId());
+            ispField = this.ispFieldService.add(ispFieldForm);
+            ispFieldForm.setId(ispField.getId());
+            if (oldIssueIsp != null) {
+                issueHistoryService.detectIssueIsp(IssueHistoryType.MODIFY, null, ispFieldForm, oldIssueIsp, sb);
+                this.issueIspRepository.deleteById(oldIssueIsp.getId());
+            } else {
+                newIssueIsp.setIspField(ispField);
+                issueHistoryService.detectIssueIsp(IssueHistoryType.ADD, null, ispFieldForm, newIssueIsp, sb);
+            }
+        }
+        // 異붽� �씠�젰
+        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
+        this.issueIspRepository.saveAndFlush(newIssueIsp);
+    }
 
     /**
      * �씠�뒋 ISP 留뚮뱾湲�
@@ -54,8 +91,8 @@
     private IssueIsp CreateIssueIsp(IspFieldForm ispFieldForm, Issue issue) {
         IssueIsp issueIsp = ConvertUtil.copyProperties(ispFieldForm, IssueIsp.class);
         issueIsp.setIssue(issue);
-        IspField ispField = this.ispFieldService.getIsp(ispFieldForm.getId());
-        if (ispField != null) {
+        if (ispFieldForm.getId() != null && ispFieldForm.getId() != -1) {
+            IspField ispField = this.ispFieldService.getIsp(ispFieldForm.getId());
             issueIsp.setIspField(ispField);
         }
 
@@ -67,7 +104,7 @@
      *  �씠�뒋 ISP 留뚮뱾湲�
      * @param ispFieldMap Map<String, Object> IspFieldForm map
      * @param issue �씠�뒋
-     * @return ISPIssue
+     * @return IssueIsp CreateIssueIsp
      */
     private IssueIsp CreateIssueIsp(Map<String, Object> ispFieldMap, Issue issue) {
         IspFieldForm ispFieldForm = ConvertUtil.convertMapToClass(ispFieldMap, IspFieldForm.class);
@@ -81,59 +118,47 @@
     @Transactional
     public void modifyIssueIspField(Issue issue, IssueForm issueForm) {
         if (issue != null) {
+            StringBuilder sb = new StringBuilder();
+            IssueIsp issueIsp = this.issueIspRepository.findByIssueId(issue.getId());
             List<Map<String, Object>> issueIspFields = issueForm.getIssueIspFields();
-            Map<String, Object> param = issueIspFields.get(0);
-
             if (issueIspFields != null && issueIspFields.size() > 0) {
-                IssueIsp issueIsp = this.issueIspRepository.findByIssueId(issue.getId());
+                Map<String, Object> param = issueIspFields.get(0);
 
-                if (issueIsp != null) {//�닔�젙 �떆
-                    issueIsp.setName(MapUtil.getString(param, "name"));
-                    issueIsp.setEmail(MapUtil.getString(param, "email"));
-                    issueIsp.setCode(MapUtil.getString(param, "code"));
-                    issueIsp.setUrl(MapUtil.getString(param, "url"));
-                    issueIsp.setManager(MapUtil.getString(param, "manager"));
-                    issueIsp.setTel(MapUtil.getString(param, "tel"));
-                    issueIsp.setMemo(MapUtil.getString(param, "memo"));
-                    // 蹂�寃� �씠�젰
-                    StringBuilder sb = new StringBuilder();
-                    issueHistoryService.detectIssueIsp(IssueHistoryType.MODIFY, param, null, issueIsp, sb);
-                    issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-                    this.issueIspRepository.saveAndFlush(issueIsp);
+                    if (issueIsp != null) {//�닔�젙 �떆
+                        // 蹂�寃� �씠�젰 �궓湲곌퀬 issueIsp�뿉 set�빐二쇨린
+                        issueHistoryService.detectIssueIsp(IssueHistoryType.MODIFY, param, null, issueIsp, sb);
+                        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
 
-                } else { //異붽� �떆
-                    IssueIsp newIssueIsp = CreateIssueIsp(param, issue);
-                    // 異붽� �씠�젰
-                    StringBuilder sb = new StringBuilder();
-                    issueHistoryService.detectIssueIsp(IssueHistoryType.ADD, param, null, newIssueIsp, sb);
-                    issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-                    this.issueIspRepository.saveAndFlush(newIssueIsp);
-                }
+                        issueIsp.setName(MapUtil.getString(param, "name"));
+                        issueIsp.setEmail(MapUtil.getString(param, "email"));
+                        issueIsp.setCode(MapUtil.getString(param, "code"));
+                        issueIsp.setUrl(MapUtil.getString(param, "url"));
+                        issueIsp.setManager(MapUtil.getString(param, "manager"));
+                        issueIsp.setTel(MapUtil.getString(param, "tel"));
+                        issueIsp.setMemo(MapUtil.getString(param, "memo"));
+
+                        this.issueIspRepository.saveAndFlush(issueIsp);
+
+                    } else { //異붽� �떆
+                        IssueIsp newIssueIsp = CreateIssueIsp(param, issue);
+                        // 異붽� �씠�젰
+                        issueHistoryService.detectIssueIsp(IssueHistoryType.ADD, param, null, newIssueIsp, sb);
+                        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
+                        this.issueIspRepository.saveAndFlush(newIssueIsp);
+                    }
+            } // �궗�슜�옄媛� 吏곸젒 �엯�젰�떆
+            else if (issueForm.getIspName() != null && !issueForm.getIspName().equals("")) {
+                //ISP�젙蹂� 吏곸젒 異붽�
+                CreateIspField(issueForm, issue);
             }
-            // �궗�슜�옄媛� 吏곸젒 �엯�젰�떆
-            else if(issueForm.getIspName() != null && !issueForm.getIspName().equals("")) {
-                // issueIsp �븘�뱶 �뤌 留뚮뱾湲�
-                IspFieldForm ispFieldForm = new IspFieldForm();
-                ispFieldForm.setName(issueForm.getIspName());
-                ispFieldForm.setCode(issueForm.getIspCode());
-                ispFieldForm.setEmail(issueForm.getIspEmail());
-                ispFieldForm.setUrl(issueForm.getIspUrl());
-                ispFieldForm.setManager(issueForm.getIspManager());
-                ispFieldForm.setTel(issueForm.getIspTel());
-                ispFieldForm.setMemo(issueForm.getIspMemo());
+            else {
+                this.issueIspRepository.deleteByIssueId(issue.getId());
+                this.issueIspRepository.flush();
 
-                IssueIsp newIssueIsp = CreateIssueIsp(ispFieldForm, issue);
-                IspField ispField = new IspField();
-                // �궗�슜�옄媛� 吏곸젒 �엯�젰�떆�뿉 ISP 紐⑸줉�뿉 異붽�
-                if (newIssueIsp.getIspField() == null) {
-                    ispField = this.ispFieldService.add(ispFieldForm);
-                    newIssueIsp.setIspField(ispField);
+                if (issueIsp != null) {
+                    issueHistoryService.detectIssueIsp(IssueHistoryType.DELETE, null, null, issueIsp, sb);
+                    issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
                 }
-                // 異붽� �씠�젰
-                StringBuilder sb = new StringBuilder();
-                issueHistoryService.detectIssueIsp(IssueHistoryType.ADD, param, null, newIssueIsp, sb);
-                issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-                this.issueIspRepository.saveAndFlush(newIssueIsp);
             }
         }
     }

--
Gitblit v1.8.0