rajujith commented on code in PR #7857:
URL: https://github.com/apache/cloudstack/pull/7857#discussion_r2303042105
##########
api/src/main/java/org/apache/cloudstack/api/command/admin/acl/CreateRoleCmd.java:
##########
@@ -101,11 +101,11 @@ public void execute() {
private void validateRoleParameters() {
if (getRoleType() == null && getRoleId() == null) {
- throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Neither
role type nor role ID is provided");
+ throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Neither
role type nor role id is provided");
}
if (getRoleType() != null && getRoleId() != null) {
- throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Both role
type and role ID should not be specified");
+ throw new ServerApiException(ApiErrorCode.PARAM_ERROR, "Both role
type and role id should not be specified");
Review Comment:
Retain 'ID'?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
