From 735cbb5f7bb27f096a480dac62138ab27f8734c4 Mon Sep 17 00:00:00 2001
From: Nightblade73 <v-zar@list.ru>
Date: Tue, 16 Apr 2019 09:46:24 +0400
Subject: [PATCH 1/4] #66 added front-end part

---
 .../templates/conferences/conference.html     | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/src/main/resources/templates/conferences/conference.html b/src/main/resources/templates/conferences/conference.html
index c93fc39..13686b3 100644
--- a/src/main/resources/templates/conferences/conference.html
+++ b/src/main/resources/templates/conferences/conference.html
@@ -136,9 +136,11 @@
                                 <div class="form-group">
                                     <label for="papers">Статьи:</label>
                                     <div class="paper-list form-control list-group" id="papers">
-                                        <div class="paper d-flex list-group-item p-0">
-                                            <a class="paper-name" href="/papers/papers">
-                                                <span>
+                                        <div class="paper d-flex list-group-item p-0"
+                                             th:each="paper, rowStat : *{papers}">
+                                            <a class="paper-name"
+                                               th:href="@{'/papers/paper?id=' + *{papers[__${rowStat.index}__].id} + ''}">
+                                                <span th:text="*{papers[__${rowStat.index}__].title}">
                                                     Имя статьи
                                                 </span>
                                                 <img class="icon-paper" src="/img/conference/paper.png"/>
@@ -149,10 +151,13 @@
                                     </div>
                                 </div>
                                 <div class="paper-control form-group d-flex justify-content-end">
-                                    <button id="attach-paper" class="btn btn-primary"
-                                            type="button">
-                                        Прикрепить статью
-                                    </button>
+                                    <select class="selectpicker form-control" multiple="true" data-live-search="true"
+                                            title="Прикрепить статью" data-style="btn-primary" data-size="5"
+                                            th:field="*{paperIds}">
+                                        <option th:each="paper: ${allPapers}" th:value="${paper.id}"
+                                                th:text="${paper.title}">Status
+                                        </option>
+                                    </select>
                                     <button id="add-paper" class="btn btn-primary"
                                             type="button">
                                         Добавить статью

From fbf9a7943b69c4de8a998750659b81d7f56b0d80 Mon Sep 17 00:00:00 2001
From: Nightblade73 <v-zar@list.ru>
Date: Tue, 16 Apr 2019 09:51:31 +0400
Subject: [PATCH 2/4] #66 changed from Set to List

---
 .../ru/ulstu/conference/model/Conference.java |  6 +++---
 .../ulstu/conference/model/ConferenceDto.java | 20 +++++++++----------
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/main/java/ru/ulstu/conference/model/Conference.java b/src/main/java/ru/ulstu/conference/model/Conference.java
index faed15c..b6affcd 100644
--- a/src/main/java/ru/ulstu/conference/model/Conference.java
+++ b/src/main/java/ru/ulstu/conference/model/Conference.java
@@ -60,7 +60,7 @@ public class Conference extends BaseEntity {
     @JoinTable(name = "paper_conference",
             joinColumns = {@JoinColumn(name = "conference_id")},
             inverseJoinColumns = {@JoinColumn(name = "paper_id")})
-    private Set<Paper> papers = new HashSet<>();
+    private List<Paper> papers = new ArrayList<>();
 
     @ManyToMany(fetch = FetchType.EAGER)
     @JoinTable(name = "users_conference",
@@ -124,11 +124,11 @@ public class Conference extends BaseEntity {
         this.deadlines = deadlines;
     }
 
-    public Set<Paper> getPapers() {
+    public List<Paper> getPapers() {
         return papers;
     }
 
-    public void setPapers(Set<Paper> papers) {
+    public void setPapers(List<Paper> papers) {
         this.papers = papers;
     }
 
diff --git a/src/main/java/ru/ulstu/conference/model/ConferenceDto.java b/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
index e5efcde..9e6e26d 100644
--- a/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
+++ b/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
@@ -5,7 +5,7 @@ import com.fasterxml.jackson.annotation.JsonProperty;
 import org.hibernate.validator.constraints.NotEmpty;
 import org.springframework.format.annotation.DateTimeFormat;
 import ru.ulstu.deadline.model.Deadline;
-import ru.ulstu.paper.model.PaperDto;
+import ru.ulstu.paper.model.Paper;
 import ru.ulstu.user.model.UserDto;
 
 import javax.persistence.Temporal;
@@ -39,8 +39,8 @@ public class ConferenceDto {
     private List<Deadline> deadlines = new ArrayList<>();
     private List<Integer> removedDeadlineIds = new ArrayList<>();
     private Set<Integer> userIds = new HashSet<>();
-    private Set<Integer> paperIds = new HashSet<>();
-    private Set<PaperDto> papers = new HashSet<>();
+    private List<Integer> paperIds = new ArrayList<>();
+    private List<Paper> papers = new ArrayList<>();
     private Set<UserDto> users = new HashSet<>();
     private Integer filterUserId;
 
@@ -57,9 +57,9 @@ public class ConferenceDto {
                          @JsonProperty("endDate") Date endDate,
                          @JsonProperty("deadlines") List<Deadline> deadlines,
                          @JsonProperty("userIds") Set<Integer> userIds,
-                         @JsonProperty("paperIds") Set<Integer> paperIds,
+                         @JsonProperty("paperIds") List<Integer> paperIds,
                          @JsonProperty("users") Set<UserDto> users,
-                         @JsonProperty("papers") Set<PaperDto> papers) {
+                         @JsonProperty("papers") List<Paper> papers) {
         this.id = id;
         this.title = title;
         this.description = description;
@@ -86,7 +86,7 @@ public class ConferenceDto {
         this.userIds = convert(conference.getUsers(), user -> user.getId());
         this.paperIds = convert(conference.getPapers(), paper -> paper.getId());
         this.users = convert(conference.getUsers(), UserDto::new);
-        this.papers = convert(conference.getPapers(), PaperDto::new);
+        this.papers = conference.getPapers();
 
     }
 
@@ -162,19 +162,19 @@ public class ConferenceDto {
         this.userIds = userIds;
     }
 
-    public Set<Integer> getPaperIds() {
+    public List<Integer> getPaperIds() {
         return paperIds;
     }
 
-    public void setPaperIds(Set<Integer> paperIds) {
+    public void setPaperIds(List<Integer> paperIds) {
         this.paperIds = paperIds;
     }
 
-    public Set<PaperDto> getPapers() {
+    public List<Paper> getPapers() {
         return papers;
     }
 
-    public void setPapers(Set<PaperDto> papers) {
+    public void setPapers(List<Paper> papers) {
         this.papers = papers;
     }
 

From dca644eba1cca8079f5cfe7bcaa8c94c668d3c4b Mon Sep 17 00:00:00 2001
From: Nightblade73 <v-zar@list.ru>
Date: Tue, 16 Apr 2019 10:42:40 +0400
Subject: [PATCH 3/4] #66 added handling of adding articles not added to the
 picker.

---
 .../controller/ConferenceController.java          | 12 +++++++++++-
 .../conference/service/ConferenceService.java     | 15 ++++++++++++++-
 .../ulstu/paper/repository/PaperRepository.java   |  2 ++
 .../java/ru/ulstu/paper/service/PaperService.java | 13 +++++++++++++
 4 files changed, 40 insertions(+), 2 deletions(-)

diff --git a/src/main/java/ru/ulstu/conference/controller/ConferenceController.java b/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
index 1011c51..c4a2433 100644
--- a/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
+++ b/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
@@ -13,10 +13,13 @@ import ru.ulstu.conference.model.ConferenceDto;
 import ru.ulstu.conference.model.ConferenceFilterDto;
 import ru.ulstu.conference.service.ConferenceService;
 import ru.ulstu.deadline.model.Deadline;
+import ru.ulstu.paper.model.Paper;
 import springfox.documentation.annotations.ApiIgnore;
 
 import javax.validation.Valid;
 import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
 import java.util.stream.Collectors;
 
 import static org.springframework.util.StringUtils.isEmpty;
@@ -44,9 +47,12 @@ public class ConferenceController {
     @GetMapping("/conference")
     public void getConference(ModelMap modelMap, @RequestParam(value = "id") Integer id) {
         if (id != null && id > 0) {
-            modelMap.put("conferenceDto", conferenceService.findOneDto(id));
+            ConferenceDto conferenceDto = conferenceService.findOneDto(id);
+            modelMap.put("conferenceDto", conferenceDto);
+            modelMap.put("allPapers", getAllPapers(conferenceDto.getPaperIds()));
         } else {
             modelMap.put("conferenceDto", new ConferenceDto());
+            modelMap.put("allPapers", getAllPapers(new ArrayList<Integer>()));
         }
     }
 
@@ -87,6 +93,10 @@ public class ConferenceController {
         return CONFERENCE_PAGE;
     }
 
+    public List<Paper> getAllPapers(List<Integer> paperIds) {
+        return conferenceService.getConferencePapers(paperIds);
+    }
+
     private void filterEmptyDeadlines(ConferenceDto conferenceDto) {
         conferenceDto.setDeadlines(conferenceDto.getDeadlines().stream()
                 .filter(dto -> dto.getDate() != null || !isEmpty(dto.getDescription()))
diff --git a/src/main/java/ru/ulstu/conference/service/ConferenceService.java b/src/main/java/ru/ulstu/conference/service/ConferenceService.java
index 84ae923..e95addb 100644
--- a/src/main/java/ru/ulstu/conference/service/ConferenceService.java
+++ b/src/main/java/ru/ulstu/conference/service/ConferenceService.java
@@ -7,6 +7,8 @@ import ru.ulstu.conference.model.Conference;
 import ru.ulstu.conference.model.ConferenceDto;
 import ru.ulstu.conference.repository.ConferenceRepository;
 import ru.ulstu.deadline.service.DeadlineService;
+import ru.ulstu.paper.model.Paper;
+import ru.ulstu.paper.service.PaperService;
 
 import java.io.IOException;
 import java.util.List;
@@ -20,11 +22,14 @@ public class ConferenceService {
 
     private final ConferenceRepository conferenceRepository;
     private final DeadlineService deadlineService;
+    private final PaperService paperService;
 
     public ConferenceService(ConferenceRepository conferenceRepository,
-                             DeadlineService deadlineService) {
+                             DeadlineService deadlineService,
+                             PaperService paperService) {
         this.conferenceRepository = conferenceRepository;
         this.deadlineService = deadlineService;
+        this.paperService = paperService;
     }
 
     public List<Conference> findAll() {
@@ -78,6 +83,10 @@ public class ConferenceService {
         conferenceDto.getDeadlines().remove((int) deadlineIndex);
     }
 
+    public List<Paper> getConferencePapers(List<Integer> paperIds) {
+        return paperService.findAllNotSelect(paperIds);
+    }
+
     private Conference copyFromDto(Conference conference, ConferenceDto conferenceDto) throws IOException {
         conference.setTitle(conferenceDto.getTitle());
         conference.setDescription(conferenceDto.getDescription());
@@ -86,6 +95,10 @@ public class ConferenceService {
         conference.setBeginDate(conferenceDto.getBeginDate());
         conference.setEndDate(conferenceDto.getEndDate());
         conference.setDeadlines(deadlineService.saveOrCreate(conferenceDto.getDeadlines()));
+        if (conferenceDto.getPaperIds() != null && !conferenceDto.getPaperIds().isEmpty()) {
+            conferenceDto.getPaperIds().forEach(paperId ->
+                    conference.getPapers().add(paperService.findEntityById(paperId)));
+        }
         return conference;
     }
 
diff --git a/src/main/java/ru/ulstu/paper/repository/PaperRepository.java b/src/main/java/ru/ulstu/paper/repository/PaperRepository.java
index 3951f53..343e9e2 100644
--- a/src/main/java/ru/ulstu/paper/repository/PaperRepository.java
+++ b/src/main/java/ru/ulstu/paper/repository/PaperRepository.java
@@ -12,4 +12,6 @@ public interface PaperRepository extends JpaRepository<Paper, Integer> {
 
     @Query("SELECT p FROM Paper p WHERE (:author IS NULL OR :author MEMBER OF p.authors) AND (YEAR(p.createDate) = :year OR :year IS NULL)")
     List<Paper> filter(@Param("author") User author, @Param("year") Integer year);
+
+    List<Paper> findByIdNotIn(List<Integer> paperIds);
 }
diff --git a/src/main/java/ru/ulstu/paper/service/PaperService.java b/src/main/java/ru/ulstu/paper/service/PaperService.java
index a93c9c6..6c27c6e 100644
--- a/src/main/java/ru/ulstu/paper/service/PaperService.java
+++ b/src/main/java/ru/ulstu/paper/service/PaperService.java
@@ -215,6 +215,19 @@ public class PaperService {
         return new PaperDto(paperRepository.findOne(paperId));
     }
 
+    public Paper findEntityById(Integer paperId) {
+        return paperRepository.findOne(paperId);
+    }
+
+    public List<Paper> findAllNotSelect(List<Integer> paperIds) {
+        if (!paperIds.isEmpty()) {
+            return sortPapers(paperRepository.findByIdNotIn(paperIds));
+        } else {
+            return sortPapers(paperRepository.findAll());
+        }
+
+    }
+
     public List<User> getPaperAuthors() {
         return userService.findAll();
     }

From d4b38d314fde2cdd828465629eadde48e4392a4c Mon Sep 17 00:00:00 2001
From: Nightblade73 <v-zar@list.ru>
Date: Tue, 16 Apr 2019 13:22:15 +0400
Subject: [PATCH 4/4] #66 fixed bug, when attach-article lose what needed to
 attach after add or remove deadline

---
 .../controller/ConferenceController.java          |  9 +++++----
 .../ru/ulstu/conference/model/ConferenceDto.java  | 15 ++++++++++++++-
 src/main/resources/public/css/conference.css      |  9 ++++++++-
 .../templates/conferences/conference.html         |  9 +++++----
 4 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/src/main/java/ru/ulstu/conference/controller/ConferenceController.java b/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
index c4a2433..0f76881 100644
--- a/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
+++ b/src/main/java/ru/ulstu/conference/controller/ConferenceController.java
@@ -48,11 +48,12 @@ public class ConferenceController {
     public void getConference(ModelMap modelMap, @RequestParam(value = "id") Integer id) {
         if (id != null && id > 0) {
             ConferenceDto conferenceDto = conferenceService.findOneDto(id);
+            conferenceDto.setNotSelectedPapers(getNotSelectPapers(conferenceDto.getPaperIds()));
             modelMap.put("conferenceDto", conferenceDto);
-            modelMap.put("allPapers", getAllPapers(conferenceDto.getPaperIds()));
         } else {
-            modelMap.put("conferenceDto", new ConferenceDto());
-            modelMap.put("allPapers", getAllPapers(new ArrayList<Integer>()));
+            ConferenceDto conferenceDto = new ConferenceDto();
+            conferenceDto.setNotSelectedPapers(getNotSelectPapers(new ArrayList<Integer>()));
+            modelMap.put("conferenceDto", conferenceDto);
         }
     }
 
@@ -93,7 +94,7 @@ public class ConferenceController {
         return CONFERENCE_PAGE;
     }
 
-    public List<Paper> getAllPapers(List<Integer> paperIds) {
+    public List<Paper> getNotSelectPapers(List<Integer> paperIds) {
         return conferenceService.getConferencePapers(paperIds);
     }
 
diff --git a/src/main/java/ru/ulstu/conference/model/ConferenceDto.java b/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
index 9e6e26d..ec9ec92 100644
--- a/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
+++ b/src/main/java/ru/ulstu/conference/model/ConferenceDto.java
@@ -41,6 +41,9 @@ public class ConferenceDto {
     private Set<Integer> userIds = new HashSet<>();
     private List<Integer> paperIds = new ArrayList<>();
     private List<Paper> papers = new ArrayList<>();
+
+
+    private List<Paper> notSelectedPapers = new ArrayList<>();
     private Set<UserDto> users = new HashSet<>();
     private Integer filterUserId;
 
@@ -59,7 +62,8 @@ public class ConferenceDto {
                          @JsonProperty("userIds") Set<Integer> userIds,
                          @JsonProperty("paperIds") List<Integer> paperIds,
                          @JsonProperty("users") Set<UserDto> users,
-                         @JsonProperty("papers") List<Paper> papers) {
+                         @JsonProperty("papers") List<Paper> papers,
+                         @JsonProperty("notSelectedPapers") List<Paper> notSelectedPapers) {
         this.id = id;
         this.title = title;
         this.description = description;
@@ -72,6 +76,7 @@ public class ConferenceDto {
         this.paperIds = paperIds;
         this.users = users;
         this.papers = papers;
+        this.notSelectedPapers = notSelectedPapers;
     }
 
     public ConferenceDto(Conference conference) {
@@ -202,4 +207,12 @@ public class ConferenceDto {
         this.removedDeadlineIds = removedDeadlineIds;
     }
 
+    public List<Paper> getNotSelectedPapers() {
+        return notSelectedPapers;
+    }
+
+    public void setNotSelectedPapers(List<Paper> notSelectedPapers) {
+        this.notSelectedPapers = notSelectedPapers;
+    }
+
 }
diff --git a/src/main/resources/public/css/conference.css b/src/main/resources/public/css/conference.css
index bc63bfe..fa3666e 100644
--- a/src/main/resources/public/css/conference.css
+++ b/src/main/resources/public/css/conference.css
@@ -7,6 +7,10 @@ body {
     border-radius: .25rem;
 }
 
+.filter-option-inner-inner {
+    color: white;
+}
+
 
 
 
@@ -49,6 +53,8 @@ body {
 
 .paper {
     margin: 0;
+    min-height: 40px;
+    height: 40px;
 }
 
 .paper-name {
@@ -104,13 +110,14 @@ body {
     float: right;
 }
 
-@media (max-width: 1199px) {
+@media (max-width: 1199px) and (min-width: 768px){
     .paper-control {
         display: block!important;
     }
 }
 
 @media (max-width: 991px) {
+
     .dates-panel {
         display: block!important;
     }
diff --git a/src/main/resources/templates/conferences/conference.html b/src/main/resources/templates/conferences/conference.html
index 13686b3..1ed2859 100644
--- a/src/main/resources/templates/conferences/conference.html
+++ b/src/main/resources/templates/conferences/conference.html
@@ -62,7 +62,6 @@
                                             <input type="submit" class="icon icon-delete grey-border"
                                                    alt="Удалить" name="removeDeadline" th:value="${rowStat.index}"
                                             />
-                                            <!--th:onclick="|$(this).parent().remove();|"-->
                                         </div>
                                     </div>
                                 </div>
@@ -136,6 +135,7 @@
                                 <div class="form-group">
                                     <label for="papers">Статьи:</label>
                                     <div class="paper-list form-control list-group" id="papers">
+                                        <input th:type="hidden" th:field="*{papers}"/>
                                         <div class="paper d-flex list-group-item p-0"
                                              th:each="paper, rowStat : *{papers}">
                                             <a class="paper-name"
@@ -145,16 +145,17 @@
                                                 </span>
                                                 <img class="icon-paper" src="/img/conference/paper.png"/>
                                             </a>
-                                            <img class="icon grey-border icon-delete" src="/img/conference/delete.png"
-                                                 alt="Удалить"/>
+                                            <input type="submit" class="icon icon-delete grey-border"
+                                                   alt="Удалить"/>
                                         </div>
                                     </div>
                                 </div>
                                 <div class="paper-control form-group d-flex justify-content-end">
+                                    <input th:type="hidden" th:field="*{notSelectedPapers}"/>
                                     <select class="selectpicker form-control" multiple="true" data-live-search="true"
                                             title="Прикрепить статью" data-style="btn-primary" data-size="5"
                                             th:field="*{paperIds}">
-                                        <option th:each="paper: ${allPapers}" th:value="${paper.id}"
+                                        <option th:each="paper: *{notSelectedPapers}" th:value="${paper.id}"
                                                 th:text="${paper.title}">Status
                                         </option>
                                     </select>