Skip to content

Commit d4f1025

Browse files
committed
SeriesController.processImage(): prepare common attributes only when we're redisplaying page.
No functional changes.
1 parent 0f47d6e commit d4f1025

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/main/java/ru/mystamps/web/controller/SeriesController.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -238,12 +238,13 @@ public String processImage(
238238
boolean maxQuantityOfImagesExceeded = !isAdmin() && !isAllowedToAddingImages(series);
239239
model.addAttribute("maxQuantityOfImagesExceeded", maxQuantityOfImagesExceeded);
240240

241-
Map<String, ?> commonAttrs = prepareCommonAttrsForSeriesInfo(series, currentUserId);
242-
model.addAllAttributes(commonAttrs);
243-
244241
if (result.hasErrors() || maxQuantityOfImagesExceeded) {
242+
Map<String, ?> commonAttrs = prepareCommonAttrsForSeriesInfo(series, currentUserId);
243+
model.addAllAttributes(commonAttrs);
244+
245245
// don't try to re-display file upload field
246246
form.setImage(null);
247+
247248
return "series/info";
248249
}
249250

0 commit comments

Comments
 (0)