----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8682/#review14753 -----------------------------------------------------------
Closing the review as only partial patch was applicable. Rest of the stuff needs to be applied on after we've acl adapter ready. - Rohit Yadav On Dec. 19, 2012, 5:19 a.m., Fang Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8682/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2012, 5:19 a.m.) > > > Review request for cloudstack and Rohit Yadav. > > > Description > ------- > > Move some validation code to API layer and modify the @Validator. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/Validator.java ca2c241 > api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java > a1461fb > server/src/com/cloud/user/AccountManagerImpl.java 593d295 > > Diff: https://reviews.apache.org/r/8682/diff/ > > > Testing > ------- > > > Thanks, > > Fang Wang > >
