修正prepare接口在新版MVC上的bug，在createForm提交时，如果有hidden input value且值为空时出错，因此…

…将required=false 改为 defaultValue="-1"
kaven-G · Jul 3, 2013 · a70cb10 · a70cb10
1 parent e5b8e30
commit a70cb10
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/...start/src/main/java/org/springside/examples/quickstart/web/account/ProfileController.java b/...start/src/main/java/org/springside/examples/quickstart/web/account/ProfileController.java
@@ -45,8 +45,8 @@ public String update(@Valid @ModelAttribute("user") User user) {
 	 * 因为仅update()方法的form中有id属性，因此仅在update时实际执行.
 	 */
 	@ModelAttribute
-	public void getUser(@RequestParam(value = "id", required = false) Long id, Model model) {
-		if (id != null) {
+	public void getUser(@RequestParam(value = "id", defaultValue = "-1") Long id, Model model) {
+		if (id != -1) {
 			model.addAttribute("user", accountService.getUser(id));
 		}
 	}

diff --git a/...art/src/main/java/org/springside/examples/quickstart/web/account/UserAdminController.java b/...art/src/main/java/org/springside/examples/quickstart/web/account/UserAdminController.java
@@ -62,8 +62,8 @@ public String delete(@PathVariable("id") Long id, RedirectAttributes redirectAtt
 	 * 因为仅update()方法的form中有id属性，因此仅在update时实际执行.
 	 */
 	@ModelAttribute
-	public void getUser(@RequestParam(value = "id", required = false) Long id, Model model) {
-		if (id != null) {
+	public void getUser(@RequestParam(value = "id", defaultValue = "-1") Long id, Model model) {
+		if (id != -1) {
 			model.addAttribute("user", accountService.getUser(id));
 		}
 	}

diff --git a/.../quickstart/src/main/java/org/springside/examples/quickstart/web/task/TaskController.java b/.../quickstart/src/main/java/org/springside/examples/quickstart/web/task/TaskController.java
@@ -51,7 +51,7 @@ public class TaskController {
 	@Autowired
 	private TaskService taskService;
 
-	@RequestMapping(value = "")
+	@RequestMapping(method = RequestMethod.GET)
 	public String list(@RequestParam(value = "page", defaultValue = "1") int pageNumber,
 			@RequestParam(value = "page.size", defaultValue = PAGE_SIZE) int pageSize,
 			@RequestParam(value = "sortType", defaultValue = "auto") String sortType, Model model,
@@ -112,9 +112,9 @@ public String delete(@PathVariable("id") Long id, RedirectAttributes redirectAtt
 	 * 所有RequestMapping方法调用前的Model准备方法, 实现Struts2 Preparable二次部分绑定的效果,先根据form的id从数据库查出Task对象,再把Form提交的内容绑定到该对象上。
 	 * 因为仅update()方法的form中有id属性，因此仅在update时实际执行.
 	 */
-	@ModelAttribute()
-	public void getTask(@RequestParam(value = "id", required = false) Long id, Model model) {
-		if (id != null) {
+	@ModelAttribute
+	public void getTask(@RequestParam(value = "id", defaultValue = "-1") Long id, Model model) {
+		if (id != -1) {
 			model.addAttribute("task", taskService.getTask(id));
 		}
 	}

diff --git a/examples/showcase/src/main/java/org/springside/examples/showcase/web/UserController.java b/examples/showcase/src/main/java/org/springside/examples/showcase/web/UserController.java
@@ -103,8 +103,8 @@ public String checkLoginName(@RequestParam("oldLoginName") String oldLoginName,
 	 * 因为仅update()方法的form中有id属性，因此仅在update时实际执行.
 	 */
 	@ModelAttribute
-	public void getUser(@RequestParam(value = "id", required = false) Long id, Model model) {
-		if (id != null) {
+	public void getUser(@RequestParam(value = "id", defaultValue = "-1") Long id, Model model) {
+		if (id != -1) {
 			model.addAttribute("user", accountService.getUser(id));
 		}
 	}