category/info.html: put to context separate variables instead of single object.

php-coder · php-coder · commit ae3ca6d9ece1 · 2015-12-07T21:43:51.000+01:00
Make view not aware of structure and type of the object (entity/dto/etc) and also allow to easy determine where and what methods of the object were used. Related to #120
diff --git a/src/main/java/ru/mystamps/web/controller/CategoryController.java b/src/main/java/ru/mystamps/web/controller/CategoryController.java
@@ -100,7 +100,9 @@ public String showInfo(
 			return null;
 		}
 		
-		model.addAttribute("category", category);
+		model.addAttribute("categoryId", category.getId());
+		model.addAttribute("categorySlug", category.getSlug());
+		model.addAttribute("categoryName", LocaleUtils.getLocalizedName(userLocale, category));
 		
 		String lang = LocaleUtils.getLanguageOrNull(userLocale);
 		Integer categoryId = category.getId();
diff --git a/src/main/webapp/WEB-INF/views/category/info.html b/src/main/webapp/WEB-INF/views/category/info.html
@@ -12,7 +12,7 @@
 		<link rel="stylesheet" href="http://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.css" th:href="${BOOTSTRAP_CSS}" />
 		<link rel="stylesheet" href="https://cdn.rawgit.com/usrz/bootstrap-languages/3ac2a3d2b27ac43a471cd99e79d378a03b2c6b5f/languages.min.css" th:href="${BOOTSTRAP_LANGUAGE}" />
 		<link rel="stylesheet" href="../../static/styles/main.css" th:href="${MAIN_CSS}" />
-		<link rel="canonical" href="" th:href="@{${PUBLIC_URL} + ${INFO_CATEGORY_PAGE}(id=${category.id},slug=${category.slug})}" />
+		<link rel="canonical" href="" th:href="@{${PUBLIC_URL} + ${INFO_CATEGORY_PAGE}(id=${categoryId},slug=${categorySlug})}" />
 	</head>
 	<body>
 		<div class="container-fluid">
@@ -87,13 +87,13 @@
 			</div>
 			<div class="row">
 				<div id="content" class="col-sm-12">
-					<h3 th:text="${category.getLocalizedName(#locale)}">
+					<h3 th:text="${categoryName}">
 						Animals
 					</h3>
 					
 					<!--/*/
 					<div class="row" th:if="${justAddedCategory}">
-						<div class="alert alert-success text-center col-sm-4 col-sm-offset-4" th:utext="#{t_category_just_added(@{${ADD_SERIES_WITH_CATEGORY_PAGE}(id=${category.id})})}">
+						<div class="alert alert-success text-center col-sm-4 col-sm-offset-4" th:utext="#{t_category_just_added(@{${ADD_SERIES_WITH_CATEGORY_PAGE}(id=${categoryId})})}">
 							Category has been added.<br />
 							Now you could <a href="../series/add.html" class="alert-link">proceed with creating series</a>.
 						</div>

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,9 @@ public String showInfo(`
`100`	`100`	`return null;`
`101`	`101`	`}`
`102`	`102`
`103`		`- model.addAttribute("category", category);`
	`103`	`+ model.addAttribute("categoryId", category.getId());`
	`104`	`+ model.addAttribute("categorySlug", category.getSlug());`
	`105`	`+ model.addAttribute("categoryName", LocaleUtils.getLocalizedName(userLocale, category));`
`104`	`106`
`105`	`107`	`String lang = LocaleUtils.getLanguageOrNull(userLocale);`
`106`	`108`	`Integer categoryId = category.getId();`