skip Controller中注入属性的setter函数,因为无需unit test
This commit is contained in:
+1
-5
@@ -17,6 +17,7 @@ import org.springside.examples.miniweb.service.account.AccountManager;
|
||||
@RequestMapping(value = "/account/group")
|
||||
public class GroupController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@RequiresPermissions("group:view")
|
||||
@@ -50,9 +51,4 @@ public class GroupController {
|
||||
redirectAttributes.addFlashAttribute("message", "删除权限组成功");
|
||||
return "redirect:/account/group/";
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
}
|
||||
|
||||
+1
-5
@@ -16,6 +16,7 @@ import org.springside.examples.miniweb.service.account.AccountManager;
|
||||
@RequestMapping(value = "/account/group/")
|
||||
public class GroupDetailController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@RequiresPermissions("group:edit")
|
||||
@@ -37,9 +38,4 @@ public class GroupDetailController {
|
||||
public Group getGroup(@PathVariable("id") Long id) {
|
||||
return accountManager.getGroup(id);
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
}
|
||||
|
||||
+2
-6
@@ -18,6 +18,8 @@ import org.springside.modules.utils.Collections3;
|
||||
*/
|
||||
@Component
|
||||
public class GroupListEditor extends PropertyEditorSupport {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
/**
|
||||
@@ -41,10 +43,4 @@ public class GroupListEditor extends PropertyEditorSupport {
|
||||
public String getAsText() {
|
||||
return Collections3.extractToString((List) getValue(), "id", ",");
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
+2
-11
@@ -33,8 +33,10 @@ import org.springside.examples.miniweb.service.account.AccountManager;
|
||||
@RequestMapping(value = "/account/user")
|
||||
public class UserController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@Autowired
|
||||
private GroupListEditor groupListEditor;
|
||||
|
||||
@InitBinder
|
||||
@@ -87,15 +89,4 @@ public class UserController {
|
||||
|
||||
return "false";
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setGroupListEditor(GroupListEditor groupListEditor) {
|
||||
this.groupListEditor = groupListEditor;
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
+2
-11
@@ -25,8 +25,9 @@ import org.springside.examples.miniweb.service.account.AccountManager;
|
||||
@RequestMapping(value = "/account/user/")
|
||||
public class UserDetailController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@Autowired
|
||||
private GroupListEditor groupListEditor;
|
||||
|
||||
@InitBinder
|
||||
@@ -53,14 +54,4 @@ public class UserDetailController {
|
||||
public User getAccount(@PathVariable("id") Long id) {
|
||||
return accountManager.getUser(id);
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setGroupListEditor(GroupListEditor groupListEditor) {
|
||||
this.groupListEditor = groupListEditor;
|
||||
}
|
||||
}
|
||||
|
||||
-1
@@ -21,5 +21,4 @@ public class GeneralController {
|
||||
public String mashupClient() {
|
||||
return "web/mashup-client";
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
+1
-6
@@ -13,6 +13,7 @@ import org.springside.examples.showcase.common.service.AccountManager;
|
||||
@RequestMapping(value = "/common/user")
|
||||
public class UserController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@RequestMapping(value = { "list", "" })
|
||||
@@ -21,10 +22,4 @@ public class UserController {
|
||||
model.addAttribute("users", users);
|
||||
return "common/userList";
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
+1
-6
@@ -26,7 +26,7 @@ import com.google.common.collect.Lists;
|
||||
@Controller
|
||||
@RequestMapping(value = "/common/user/")
|
||||
public class UserDetailController {
|
||||
|
||||
@Autowired
|
||||
private AccountManager accountManager;
|
||||
|
||||
@RequestMapping(value = "update/{id}")
|
||||
@@ -52,9 +52,4 @@ public class UserDetailController {
|
||||
public User getAccount(@PathVariable("id") Long id) {
|
||||
return accountManager.getUser(id);
|
||||
}
|
||||
|
||||
@Autowired
|
||||
public void setAccountManager(AccountManager accountManager) {
|
||||
this.accountManager = accountManager;
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user