----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7130/ -----------------------------------------------------------
(Updated Sept. 25, 2012, 7:56 a.m.) Review request for cloudstack, Abhinandan Prateek, Alena Prokharchyk, and Alex Huang. Changes ------- New patch. A better solution would be to change how UserAccount gets and sets States, which is String now. But, this would possibly affect or break at many places so I fixed it by getting State enum and converting to string and equating (ignore case) against UserAccount.getState() Description ------- Issue: https://issues.apache.org/jira/browse/CLOUDSTACK-121 Download original patch and git am <patch>: http://bhaisaab.org/patches/cloudstack/0001-CLOUDSTACK-121-Fix-NullPointerException-for-incorrec.patch Diffs (updated) ----- api/src/com/cloud/user/UserAccount.java 734e16b server/src/com/cloud/user/AccountManagerImpl.java fa9fafb Diff: https://reviews.apache.org/r/7130/diff/ Testing ------- Thanks, Rohit Yadav