Skip to content

Commit 2b12616

Browse files
jo-elimujo-elimu
authored
feat: edit storybook chapter image (#2334)
2 parents 57e7be6 + 72667ac commit 2b12616

File tree

6 files changed

+179
-69
lines changed

6 files changed

+179
-69
lines changed

‎pom-dependency-tree.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
ai.elimu:webapp:war:2.6.125-SNAPSHOT
1+
ai.elimu:webapp:war:2.6.128-SNAPSHOT
22
+- ai.elimu:model:jar:model-2.0.120:compile
33
| \- com.google.code.gson:gson:jar:2.13.1:compile
44
| \- com.google.errorprone:error_prone_annotations:jar:2.38.0:compile

‎src/main/java/ai/elimu/web/content/storybook/chapter/StoryBookChapterDeleteController.java‎

Lines changed: 17 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,13 @@
11
package ai.elimu.web.content.storybook.chapter;
22

3-
import ai.elimu.dao.ImageContributionEventDao;
4-
import ai.elimu.dao.ImageDao;
53
import ai.elimu.dao.StoryBookChapterDao;
64
import ai.elimu.dao.StoryBookContributionEventDao;
75
import ai.elimu.dao.StoryBookDao;
86
import ai.elimu.dao.StoryBookParagraphDao;
97
import ai.elimu.entity.content.StoryBook;
108
import ai.elimu.entity.content.StoryBookChapter;
119
import ai.elimu.entity.content.StoryBookParagraph;
12-
import ai.elimu.entity.content.multimedia.Image;
1310
import ai.elimu.entity.contributor.Contributor;
14-
import ai.elimu.entity.contributor.ImageContributionEvent;
1511
import ai.elimu.entity.contributor.StoryBookContributionEvent;
1612
import ai.elimu.entity.enums.PeerReviewStatus;
1713
import ai.elimu.entity.enums.Role;
@@ -40,9 +36,6 @@ public class StoryBookChapterDeleteController {
4036
private final StoryBookParagraphDao storyBookParagraphDao;
4137
private final StoryBookContributionEventDao storyBookContributionEventDao;
4238

43-
private final ImageDao imageDao;
44-
private final ImageContributionEventDao imageContributionEventDao;
45-
4639
private final StoryBooksJsonService storyBooksJsonService;
4740

4841
@GetMapping
@@ -72,41 +65,35 @@ public String handleRequest(HttpSession session, @PathVariable Long storyBookId,
7265
log.info("Deleting StoryBookChapter with ID " + storyBookChapterToBeDeleted.getId());
7366
storyBookChapterDao.delete(storyBookChapterToBeDeleted);
7467

75-
// Delete the chapter's image (if any)
76-
Image chapterImage = storyBookChapterToBeDeleted.getImage();
77-
log.info("chapterImage: " + chapterImage);
78-
if (chapterImage != null) {
79-
// Remove content labels
80-
chapterImage.setLiteracySkills(null);
81-
chapterImage.setNumeracySkills(null);
82-
chapterImage.setLetters(null);
83-
chapterImage.setNumbers(null);
84-
chapterImage.setWords(null);
85-
imageDao.update(chapterImage);
86-
87-
// Remove contribution events
88-
for (ImageContributionEvent imageContributionEvent : imageContributionEventDao.readAll(chapterImage)) {
89-
log.warn("Deleting ImageContributionEvent from the database");
90-
imageContributionEventDao.delete(imageContributionEvent);
91-
}
92-
93-
log.warn("Deleting the chapter image from the database");
94-
imageDao.delete(chapterImage);
95-
}
96-
9768
// Update the StoryBook's metadata
9869
StoryBook storyBook = storyBookChapterToBeDeleted.getStoryBook();
9970
storyBook.setRevisionNumber(storyBook.getRevisionNumber() + 1);
10071
storyBook.setPeerReviewStatus(PeerReviewStatus.PENDING);
10172
storyBookDao.update(storyBook);
10273

74+
// Update the sorting order of the remaining chapters
75+
List<StoryBookChapter> storyBookChapters = storyBookChapterDao.readAll(storyBook);
76+
log.info("storyBookChapters.size(): " + storyBookChapters.size());
77+
for (StoryBookChapter storyBookChapter : storyBookChapters) {
78+
log.info("storyBookChapter.getId(): " + storyBookChapter.getId() + ", storyBookChapter.getSortOrder(): " + storyBookChapter.getSortOrder());
79+
if (storyBookChapter.getSortOrder() > storyBookChapterToBeDeleted.getSortOrder()) {
80+
// Reduce sort order by 1
81+
storyBookChapter.setSortOrder(storyBookChapter.getSortOrder() - 1);
82+
storyBookChapterDao.update(storyBookChapter);
83+
log.info("storyBookChapter.getSortOrder() (after update): " + storyBookChapter.getSortOrder());
84+
}
85+
}
86+
87+
// Refresh the REST API cache
88+
storyBooksJsonService.refreshStoryBooksJSONArray();
89+
10390
// Store contribution event
10491
StoryBookContributionEvent storyBookContributionEvent = new StoryBookContributionEvent();
10592
storyBookContributionEvent.setContributor(contributor);
10693
storyBookContributionEvent.setTimestamp(Calendar.getInstance());
10794
storyBookContributionEvent.setStoryBook(storyBook);
10895
storyBookContributionEvent.setRevisionNumber(storyBook.getRevisionNumber());
109-
storyBookContributionEvent.setComment("Deleted storybook chapter " + (storyBookChapterToBeDeleted.getSortOrder() + 1) + " (🤖 auto-generated comment)");
96+
storyBookContributionEvent.setComment("Deleted storybook chapter " + (storyBookChapterToBeDeleted.getSortOrder() + 1) + "/" + storyBookChapters.size() + " (🤖 auto-generated comment)");
11097
storyBookContributionEventDao.create(storyBookContributionEvent);
11198

11299
String contentUrl = DomainHelper.getBaseUrl() + "/content/storybook/edit/" + storyBook.getId();
@@ -123,22 +110,6 @@ public String handleRequest(HttpSession session, @PathVariable Long storyBookId,
123110
embedThumbnailUrl
124111
);
125112

126-
// Update the sorting order of the remaining chapters
127-
List<StoryBookChapter> storyBookChapters = storyBookChapterDao.readAll(storyBook);
128-
log.info("storyBookChapters.size(): " + storyBookChapters.size());
129-
for (StoryBookChapter storyBookChapter : storyBookChapters) {
130-
log.info("storyBookChapter.getId(): " + storyBookChapter.getId() + ", storyBookChapter.getSortOrder(): " + storyBookChapter.getSortOrder());
131-
if (storyBookChapter.getSortOrder() > storyBookChapterToBeDeleted.getSortOrder()) {
132-
// Reduce sort order by 1
133-
storyBookChapter.setSortOrder(storyBookChapter.getSortOrder() - 1);
134-
storyBookChapterDao.update(storyBookChapter);
135-
log.info("storyBookChapter.getSortOrder() (after update): " + storyBookChapter.getSortOrder());
136-
}
137-
}
138-
139-
// Refresh the REST API cache
140-
storyBooksJsonService.refreshStoryBooksJSONArray();
141-
142113
return "redirect:/content/storybook/edit/" + storyBookId;
143114
}
144115
}

‎src/main/java/ai/elimu/web/content/storybook/chapter/StoryBookChapterEditController.java‎

Lines changed: 105 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,105 @@
1+
package ai.elimu.web.content.storybook.chapter;
2+
3+
import ai.elimu.dao.ImageDao;
4+
import ai.elimu.dao.StoryBookChapterDao;
5+
import ai.elimu.dao.StoryBookContributionEventDao;
6+
import ai.elimu.dao.StoryBookDao;
7+
import ai.elimu.entity.content.StoryBook;
8+
import ai.elimu.entity.content.StoryBookChapter;
9+
import ai.elimu.entity.content.multimedia.Image;
10+
import ai.elimu.entity.contributor.Contributor;
11+
import ai.elimu.entity.contributor.StoryBookContributionEvent;
12+
import ai.elimu.entity.enums.PeerReviewStatus;
13+
import ai.elimu.rest.v2.service.StoryBooksJsonService;
14+
import ai.elimu.util.DiscordHelper;
15+
import ai.elimu.util.DiscordHelper.Channel;
16+
import ai.elimu.util.DomainHelper;
17+
import jakarta.servlet.http.HttpSession;
18+
import jakarta.validation.Valid;
19+
import lombok.RequiredArgsConstructor;
20+
import lombok.extern.slf4j.Slf4j;
21+
22+
import java.util.Calendar;
23+
import java.util.List;
24+
25+
import org.springframework.stereotype.Controller;
26+
import org.springframework.ui.Model;
27+
import org.springframework.validation.BindingResult;
28+
import org.springframework.web.bind.annotation.GetMapping;
29+
import org.springframework.web.bind.annotation.PathVariable;
30+
import org.springframework.web.bind.annotation.PostMapping;
31+
import org.springframework.web.bind.annotation.RequestMapping;
32+
33+
@Controller
34+
@RequestMapping("/content/storybook/edit/{storyBookId}/chapter/edit/{id}")
35+
@RequiredArgsConstructor
36+
@Slf4j
37+
public class StoryBookChapterEditController {
38+
39+
private final StoryBookDao storyBookDao;
40+
private final StoryBookChapterDao storyBookChapterDao;
41+
private final StoryBookContributionEventDao storyBookContributionEventDao;
42+
43+
private final ImageDao imageDao;
44+
45+
private final StoryBooksJsonService storyBooksJsonService;
46+
47+
@GetMapping
48+
public String handleRequest(Model model, @PathVariable Long storyBookId, @PathVariable Long id) {
49+
log.info("handleRequest");
50+
51+
StoryBookChapter storyBookChapter = storyBookChapterDao.read(id);
52+
model.addAttribute("storyBookChapter", storyBookChapter);
53+
54+
List<Image> images = imageDao.readAllOrdered();
55+
model.addAttribute("images", images);
56+
57+
return "content/storybook/chapter/edit";
58+
}
59+
60+
@PostMapping
61+
public String handleSubmit(
62+
HttpSession session,
63+
@PathVariable Long storyBookId,
64+
@Valid StoryBookChapter storyBookChapter,
65+
BindingResult result,
66+
Model model
67+
) {
68+
log.info("handleSubmit");
69+
70+
Contributor contributor = (Contributor) session.getAttribute("contributor");
71+
72+
if (result.hasErrors()) {
73+
model.addAttribute("storyBookChapter", storyBookChapter);
74+
75+
List<Image> images = imageDao.readAllOrdered();
76+
model.addAttribute("images", images);
77+
78+
return "content/storybook/chapter/edit";
79+
} else {
80+
storyBookChapterDao.update(storyBookChapter);
81+
82+
// Update the storybook's metadata
83+
StoryBook storyBook = storyBookChapter.getStoryBook();
84+
storyBook.setRevisionNumber(storyBook.getRevisionNumber() + 1);
85+
storyBook.setPeerReviewStatus(PeerReviewStatus.PENDING);
86+
storyBookDao.update(storyBook);
87+
88+
// Refresh the REST API cache
89+
storyBooksJsonService.refreshStoryBooksJSONArray();
90+
91+
// Store contribution event
92+
StoryBookContributionEvent storyBookContributionEvent = new StoryBookContributionEvent();
93+
storyBookContributionEvent.setContributor(contributor);
94+
storyBookContributionEvent.setTimestamp(Calendar.getInstance());
95+
storyBookContributionEvent.setStoryBook(storyBook);
96+
storyBookContributionEvent.setRevisionNumber(storyBook.getRevisionNumber());
97+
storyBookContributionEvent.setComment("Edited storybook chapter " + (storyBookChapter.getSortOrder() + 1) + " (🤖 auto-generated comment)");
98+
storyBookContributionEventDao.create(storyBookContributionEvent);
99+
100+
DiscordHelper.postToChannel(Channel.CONTENT, "Storybook chapter edited: " + DomainHelper.getBaseUrl() + "/content/storybook/edit/" + storyBook.getId() + "#ch-id-" + storyBookChapter.getId());
101+
102+
return "redirect:/content/storybook/edit/" + storyBookId + "#ch-id-" + storyBookChapter.getId();
103+
}
104+
}
105+
}

‎src/main/java/ai/elimu/web/content/storybook/paragraph/StoryBookParagraphDeleteController.java‎

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,28 @@ public String handleRequest(HttpSession session, @PathVariable Long id) {
5555
log.info("Deleting StoryBookParagraph with ID " + storyBookParagraphToBeDeleted.getId());
5656
storyBookParagraphDao.delete(storyBookParagraphToBeDeleted);
5757

58+
// Update the sorting order of the remaining paragraphs
59+
List<StoryBookParagraph> storyBookParagraphs = storyBookParagraphDao.readAll(storyBookParagraphToBeDeleted.getStoryBookChapter());
60+
log.info("storyBookParagraphs.size(): " + storyBookParagraphs.size());
61+
for (StoryBookParagraph storyBookParagraph : storyBookParagraphs) {
62+
log.info("storyBookParagraph.getId(): " + storyBookParagraph.getId() + ", storyBookParagraph.getSortOrder(): " + storyBookParagraph.getSortOrder());
63+
if (storyBookParagraph.getSortOrder() > storyBookParagraphToBeDeleted.getSortOrder()) {
64+
// Reduce sort order by 1
65+
storyBookParagraph.setSortOrder(storyBookParagraph.getSortOrder() - 1);
66+
storyBookParagraphDao.update(storyBookParagraph);
67+
log.info("storyBookParagraph.getSortOrder() (after update): " + storyBookParagraph.getSortOrder());
68+
}
69+
}
70+
5871
// Update the storybook's metadata
5972
StoryBook storyBook = storyBookParagraphToBeDeleted.getStoryBookChapter().getStoryBook();
6073
storyBook.setRevisionNumber(storyBook.getRevisionNumber() + 1);
6174
storyBook.setPeerReviewStatus(PeerReviewStatus.PENDING);
6275
storyBookDao.update(storyBook);
6376

77+
// Refresh the REST API cache
78+
storyBooksJsonService.refreshStoryBooksJSONArray();
79+
6480
// Store contribution event
6581
StoryBookContributionEvent storyBookContributionEvent = new StoryBookContributionEvent();
6682
storyBookContributionEvent.setContributor(contributor);
@@ -85,22 +101,6 @@ public String handleRequest(HttpSession session, @PathVariable Long id) {
85101
embedThumbnailUrl
86102
);
87103

88-
// Update the sorting order of the remaining paragraphs
89-
List<StoryBookParagraph> storyBookParagraphs = storyBookParagraphDao.readAll(storyBookParagraphToBeDeleted.getStoryBookChapter());
90-
log.info("storyBookParagraphs.size(): " + storyBookParagraphs.size());
91-
for (StoryBookParagraph storyBookParagraph : storyBookParagraphs) {
92-
log.info("storyBookParagraph.getId(): " + storyBookParagraph.getId() + ", storyBookParagraph.getSortOrder(): " + storyBookParagraph.getSortOrder());
93-
if (storyBookParagraph.getSortOrder() > storyBookParagraphToBeDeleted.getSortOrder()) {
94-
// Reduce sort order by 1
95-
storyBookParagraph.setSortOrder(storyBookParagraph.getSortOrder() - 1);
96-
storyBookParagraphDao.update(storyBookParagraph);
97-
log.info("storyBookParagraph.getSortOrder() (after update): " + storyBookParagraph.getSortOrder());
98-
}
99-
}
100-
101-
// Refresh the REST API cache
102-
storyBooksJsonService.refreshStoryBooksJSONArray();
103-
104104
return "redirect:/content/storybook/edit/" + storyBook.getId() + "#ch-id-" + storyBookParagraphToBeDeleted.getStoryBookChapter().getId();
105105
}
106106
}

‎src/main/java/ai/elimu/web/content/storybook/paragraph/StoryBookParagraphEditController.java‎

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,8 @@
3434
public class StoryBookParagraphEditController {
3535

3636
private final StoryBookDao storyBookDao;
37-
38-
private final StoryBookContributionEventDao storyBookContributionEventDao;
39-
4037
private final StoryBookParagraphDao storyBookParagraphDao;
38+
private final StoryBookContributionEventDao storyBookContributionEventDao;
4139

4240
private final StoryBooksJsonService storyBooksJsonService;
4341

@@ -80,6 +78,9 @@ public String handleSubmit(
8078
storyBook.setPeerReviewStatus(PeerReviewStatus.PENDING);
8179
storyBookDao.update(storyBook);
8280

81+
// Refresh the REST API cache
82+
storyBooksJsonService.refreshStoryBooksJSONArray();
83+
8384
// Store contribution event
8485
StoryBookContributionEvent storyBookContributionEvent = new StoryBookContributionEvent();
8586
storyBookContributionEvent.setContributor(contributor);
@@ -107,9 +108,6 @@ public String handleSubmit(
107108
embedThumbnailUrl
108109
);
109110

110-
// Refresh the REST API cache
111-
storyBooksJsonService.refreshStoryBooksJSONArray();
112-
113111
return "redirect:/content/storybook/edit/" +
114112
storyBookParagraph.getStoryBookChapter().getStoryBook().getId() +
115113
"#ch-id-" + storyBookParagraph.getStoryBookChapter().getId();

‎src/main/webapp/WEB-INF/jsp/content/storybook/chapter/edit.jsp‎

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
<content:title>
2+
Edit chapter ${storyBookChapter.sortOrder + 1}
3+
</content:title>
4+
5+
<content:section cssId="storyBookChapterEditPage">
6+
<h4><content:gettitle /></h4>
7+
<div class="card-panel">
8+
<form:form modelAttribute="storyBookChapter">
9+
<tag:formErrors modelAttribute="storyBookChapter" />
10+
11+
<form:hidden path="storyBook" value="${storyBookChapter.storyBook.id}" />
12+
<form:hidden path="sortOrder" value="${storyBookChapter.sortOrder}" />
13+
14+
<div class="row">
15+
<div class="input-field col s12">
16+
<select id="image" name="image">
17+
<option value="">-- Select --</option>
18+
<c:forEach var="image" items="${images}">
19+
<option value="${image.id}" <c:if test="${image.id == storyBookChapter.image.id}">selected="selected"</c:if>>${image.title}</option>
20+
</c:forEach>
21+
</select>
22+
<label for="image">Image</label>
23+
<c:if test="${not empty storyBookChapter.image}">
24+
<a href="<spring:url value='/content/multimedia/image/edit/${storyBookChapter.image.id}' />">
25+
<img class="checksumGitHub-${storyBookChapter.image.checksumGitHub != null}" src="<spring:url value='${storyBookChapter.image.url}' />" alt="${storyBook.title}" />
26+
</a>
27+
</c:if>
28+
</div>
29+
</div>
30+
31+
<button id="submitButton" class="btn-large waves-effect waves-light" type="submit" <c:if test="${empty contributor}">disabled</c:if>>
32+
Edit <i class="material-icons right">send</i>
33+
</button>
34+
</form:form>
35+
</div>
36+
</content:section>

0 commit comments

Comments
 (0)