From 94bc0a6bad49e818e54a5438a6f202448de05ee0 Mon Sep 17 00:00:00 2001
From: 이민희 <mhlee@maprex.co.kr>
Date: 수, 26 1월 2022 15:51:16 +0900
Subject: [PATCH] - 이슈 수정 시 대분류만 선택하고 수정 시 오류 해결 - 전체 댓글 + 메일 보기 / 하위 이슈 댓글 + 메일 보기

---
 src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java |   63 +++++++++++++++----------------
 1 files changed, 30 insertions(+), 33 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 b1460f5..932da15 100644
--- a/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java
+++ b/src/main/java/kr/wisestone/owl/service/impl/IssueHostingServiceImpl.java
@@ -52,8 +52,7 @@
      * @param issueForm IssueForm
      * @param issue Issue
      */
-    private void CreateHostingField(IssueForm issueForm, Issue issue) {
-        StringBuilder sb = new StringBuilder();
+    private void CreateHostingField(IssueForm issueForm, Issue issue, StringBuilder sb) {
         // issueHosting �븘�뱶 �뤌 留뚮뱾湲�
         HostingFieldForm hostingFieldForm = new HostingFieldForm();
         hostingFieldForm.setName(issueForm.getHostingName());
@@ -80,8 +79,6 @@
                 issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, null, hostingFieldForm, newIssueHosting, sb);
             }
         }
-        // 異붽� �씠�젰
-        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
         this.issueHostingRepository.saveAndFlush(newIssueHosting);
     }
 
@@ -117,46 +114,46 @@
     //  �씠�뒋�뿉�꽌 �궗�슜�릺�뒗 �뾽泥� 媛믪쓣 �뾽�뜲�씠�듃�븳�떎.
     @Override
     @Transactional
-    public void modifyIssueHostingField(Issue issue, IssueForm issueForm) {
+    public void modifyIssueHostingField(Issue issue, IssueForm issueForm, StringBuilder sb) {
         if (issue != null) {
-            StringBuilder sb = new StringBuilder();
+            IssueHosting issueHosting = this.issueHostingRepository.findByIssueId(issue.getId());
             List<Map<String, Object>> issueHostingFields = issueForm.getIssueHostingFields();
             if (issueHostingFields != null && issueHostingFields.size() > 0) {
                 Map<String, Object> param = issueHostingFields.get(0);
 
-                if (issueHostingFields != null && issueHostingFields.size() > 0) {
-                    IssueHosting issueHosting = this.issueHostingRepository.findByIssueId(issue.getId());
+                if (issueHosting != null) { //�닔�젙 �떆
+                    // 蹂�寃� �씠�젰 �궓湲곌퀬 issueHosting�뿉 set �빐二쇨린
+                    issueHistoryService.detectIssueHosting(IssueHistoryType.MODIFY, param, null, issueHosting, sb);
 
-                    if (issueHosting != null) { //�닔�젙 �떆
-                        // 蹂�寃� �씠�젰 �궓湲곌퀬 issueHosting�뿉 set �빐二쇨린
-                        issueHistoryService.detectIssueHosting(IssueHistoryType.MODIFY, param, null, issueHosting, sb);
-                        issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
+                    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"));
 
-                        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"));
+                    this.issueHostingRepository.saveAndFlush(issueHosting);
 
-                        this.issueHostingRepository.saveAndFlush(issueHosting);
+                } else { //異붽� �떆
+                    IssueHosting newIssueHosting = CreateIssueHosting(param, issue);
+                    // 異붽� �씠�젰
+                    issueHistoryService.detectIssueHosting(IssueHistoryType.ADD, param, null, newIssueHosting, sb);
+                    this.issueHostingRepository.saveAndFlush(newIssueHosting);
+                }
 
-                    } else { //異붽� �떆
-                        IssueHosting newIssueHosting = CreateIssueHosting(param, issue);
-                        // 異붽� �씠�젰
-                        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("")) {
-                    // �샇�뒪�똿 �젙蹂� 吏곸젒 異붽�
-                    CreateHostingField(issueForm, issue);
-                } else {
-                    this.issueHostingRepository.deleteByIssueId(issue.getId());
-                    this.issueHostingRepository.flush();
+            }else if (issueForm.getHostingName() != null && !issueForm.getHostingName().equals("")) {
+                // �샇�뒪�똿 �젙蹂� 吏곸젒 異붽�
+                CreateHostingField(issueForm, issue, sb);
+            } else {
+                this.issueHostingRepository.deleteByIssueId(issue.getId());
+                this.issueHostingRepository.flush();
+
+                if (issueHosting != null) {
+                    issueHistoryService.detectIssueHosting(IssueHistoryType.DELETE, null, null, issueHosting, sb);
                 }
             }
+            //issueHistoryService.addIssueHistory(issue, IssueHistoryType.MODIFY, sb.toString());
         }
     }
 

--
Gitblit v1.8.0