----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8691/#review14754 -----------------------------------------------------------
Holding review on this one, we may have multiple adapters this patch assumes only one adapter and not a list and we have to change interface to accept method(s) instead of strings. - Rohit Yadav On Dec. 20, 2012, 12:41 a.m., Fang Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8691/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2012, 12:41 a.m.) > > > Review request for cloudstack and Rohit Yadav. > > > Description > ------- > > Do the domain access check on createUserCmd. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/command/admin/user/CreateUserCmd.java > a1461fb > server/src/com/cloud/api/ApiDispatcher.java e75a15c > server/src/com/cloud/user/AccountManagerImpl.java 593d295 > > Diff: https://reviews.apache.org/r/8691/diff/ > > > Testing > ------- > > > Thanks, > > Fang Wang > >
