From 716586b849dd2368fb59a54adeda2d65122f643b Mon Sep 17 00:00:00 2001
From: 이민희 <mhlee@maprex.co.kr>
Date: 금, 14 1월 2022 21:07:07 +0900
Subject: [PATCH] - 이슈 수정 시 파트너 정보 수정 오류 - 이슈 상세 페이지에서 연관/하위 이슈의 파트너 정보 보여주기

---
 src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java |  106 ++++++++++++++++++++++++++++++++---------------------
 1 files changed, 64 insertions(+), 42 deletions(-)

diff --git a/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java b/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java
index 0817429..5e55c83 100644
--- a/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java
+++ b/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java
@@ -11,6 +11,7 @@
 import kr.wisestone.owl.vo.IssueHostingVo;
 import kr.wisestone.owl.web.condition.IssueCondition;
 import kr.wisestone.owl.web.form.HostingFieldForm;
+import kr.wisestone.owl.web.form.IspFieldForm;
 import kr.wisestone.owl.web.form.IssueForm;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -33,6 +34,9 @@
     private HostingFieldService hostingFieldService;
 
     @Autowired
+    private IssueService issueService;
+
+    @Autowired
     private IssueHistoryService issueHistoryService;
 
     @Autowired
@@ -43,52 +47,70 @@
         return this.issueHostingRepository;
     }
 
+    /**
+     * �씠�뒋 �샇�뒪�똿 留뚮뱾湲�
+     * @param hostingFieldForm HostingFieldForm
+     * @param issue �씠�뒋
+     * @return IssueHosting
+     */
+    private IssueHosting CreateIssueHosting(HostingFieldForm hostingFieldForm, Issue issue) {
+        IssueHosting issueHosting = ConvertUtil.copyProperties(hostingFieldForm, IssueHosting.class);
+        issueHosting.setIssue(issue);
+        HostingField hostingField = this.hostingFieldService.getHosting(hostingFieldForm.getId());
+        if (hostingField != null) {
+            issueHosting.setHostingField(hostingField);
+        }
+        return issueHosting;
+    }
+
+
+    /**
+     *  �씠�뒋 �샇�뒪�똿 留뚮뱾湲�
+     * @param hostingFieldMap Map<String, Object> hostingFieldMap
+     * @param issue �씠�뒋
+     * @return IssueHosting
+     */
+    private IssueHosting CreateIssueHosting(Map<String, Object> hostingFieldMap, Issue issue) {
+        HostingFieldForm hostingFieldForm = ConvertUtil.convertMapToClass(hostingFieldMap, HostingFieldForm.class);
+        hostingFieldForm.setId(MapUtil.getLong(hostingFieldMap, "hostingId"));
+        return CreateIssueHosting(hostingFieldForm, issue);
+    }
+
     //  �씠�뒋�뿉�꽌 �궗�슜�릺�뒗 �뾽泥� 媛믪쓣 �뾽�뜲�씠�듃�븳�떎.
     @Override
     @Transactional
     public void modifyIssueHostingField(Issue issue, IssueForm issueForm) {
         if (issue != null) {
             List<Map<String, Object>> issueHostingFields = issueForm.getIssueHostingFields();
+            Map<String, Object> param = issueHostingFields.get(0);
+
             if (issueHostingFields != null && issueHostingFields.size() > 0) {
-                Map<String, Object> param = issueHostingFields.get(0);
+                IssueHosting issueHosting = this.issueHostingRepository.findByIssueId(issue.getId());
 
-                if (param != null && param.get("hostingId") != null  && param.get("hostingId") != "") {
-                    HostingField hostingField = this.hostingFieldService.getHosting(MapUtil.getLong(param, "hostingId"));
-
-                    Set<IssueHosting> issueHostings = issue.getIssueHostingFields();
-                    IssueHosting issueHosting = new IssueHosting();
-
-                    if (issueHostings != null && issueHostings.size() >0){//�닔�젙 �븷 寃쎌슦
-                        issueHosting = issueHostings.iterator().next();
-                        // 蹂�寃� �씠�젰
-                        StringBuilder sb = new StringBuilder();
-                        issueHistoryService.detectIssueHosting(IssueHistoryType.MODIFY, param, null, issueHosting, sb);
-                        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-
-                        issueHosting.setHostingField(hostingField);
-                        issueHosting.setName(MapUtil.getString(param, "name"));
-                        issueHosting.setEmail(MapUtil.getString(param, "email"));
-                        issueHosting.setCode(MapUtil.getString(param, "code"));
-                        issueHosting.setUrl(MapUtil.getString(param, "url"));
-                        issueHosting.setManager(MapUtil.getString(param, "manager"));
-                        issueHosting.setTel(MapUtil.getString(param, "tel"));
-                        issueHosting.setMemo(MapUtil.getString(param, "memo"));
-                    }else{//異붽� �븷 寃쎌슦
-                        issueHosting = ConvertUtil.convertMapToClass(param, IssueHosting.class);
-                        issueHosting.setIssue(issue);
-                        if (hostingField != null) {
-                            issueHosting.setHostingField(hostingField);
-                        }
-                        // 異붽� �씠�젰
-                        StringBuilder sb = new StringBuilder();
-                        issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, param, null, issueHosting, sb);
-                        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-                    }
+                if (issueHosting != null) { //�닔�젙 �떆
+                    issueHosting.setName(MapUtil.getString(param, "name"));
+                    issueHosting.setEmail(MapUtil.getString(param, "email"));
+                    issueHosting.setCode(MapUtil.getString(param, "code"));
+                    issueHosting.setUrl(MapUtil.getString(param, "url"));
+                    issueHosting.setManager(MapUtil.getString(param, "manager"));
+                    issueHosting.setTel(MapUtil.getString(param, "tel"));
+                    issueHosting.setMemo(MapUtil.getString(param, "memo"));
+                    // 蹂�寃� �씠�젰
+                    StringBuilder sb = new StringBuilder();
+                    issueHistoryService.detectIssueHosting(IssueHistoryType.MODIFY, param, null, issueHosting, sb);
+                    issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
                     this.issueHostingRepository.saveAndFlush(issueHosting);
-                }else {
-                    this.issueHostingRepository.deleteById(MapUtil.getLong(param, "id"));
+
+                } else { //異붽� �떆
+                    IssueHosting newIssueHosting = CreateIssueHosting(param, issue);
+                    // 異붽� �씠�젰
+                    StringBuilder sb = new StringBuilder();
+                    issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, param, null, newIssueHosting, sb);
+                    issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
+                    this.issueHostingRepository.saveAndFlush(newIssueHosting);
                 }
             } else if(issueForm.getHostingName() != null && !issueForm.getHostingName().equals("")) {
+                // issueHosting �븘�뱶 �뤌 留뚮뱾湲�
                 HostingFieldForm hostingFieldForm = new HostingFieldForm();
                 hostingFieldForm.setName(issueForm.getHostingName());
                 hostingFieldForm.setCode(issueForm.getHostingCode());
@@ -98,17 +120,17 @@
                 hostingFieldForm.setTel(issueForm.getHostingTel());
                 hostingFieldForm.setMemo(issueForm.getHostingMemo());
 
-                HostingField hostingField = this.hostingFieldService.add(hostingFieldForm);
-                IssueHosting issueHosting = ConvertUtil.copyProperties(hostingFieldForm, IssueHosting.class);
-                issueHosting.setIssue(issue);
-                if (hostingField != null) {
-                    issueHosting.setHostingField(hostingField);
+                IssueHosting newIssueHosting = CreateIssueHosting(hostingFieldForm, issue);
+                HostingField hostingField = new HostingField();
+                if (newIssueHosting.getHostingField() == null) { // �궗�슜�옄媛� 吏곸젒 �엯�젰�떆�뿉 �샇�뒪�똿 紐⑸줉�뿉 異붽�
+                    hostingField = this.hostingFieldService.add(hostingFieldForm);
+                    newIssueHosting.setHostingField(hostingField);
                 }
                 // 異붽� �씠�젰
                 StringBuilder sb = new StringBuilder();
-                issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, null, hostingField, issueHosting, sb);
+                issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, param, null, newIssueHosting, sb);
                 issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
-                this.issueHostingRepository.saveAndFlush(issueHosting);
+                this.issueHostingRepository.saveAndFlush(newIssueHosting);
             }
         }
     }

--
Gitblit v1.8.0