rhtyd commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-386270351 @rafaelweingartner I think you're trying to solve the ordering problem. Here's another problem to this: if two admins are trying to change the order in different browsers/cli. I think the simplest solution is that every change in the UI (drag/drop) should send an update to backend which updates when the old list (provided by the api) matches the db, otherwise an error can be shown to the user that the list was changed (by someone else). An alternative could be that in the UI, you drag+drop to change order and there is an apply button that pops up to apply the final order. I faced a similar issue for drag+drop to reorder rule permissions of a role, I used this to solve it: https://github.com/apache/cloudstack/blob/master/ui/scripts/roles.js#L196 https://github.com/apache/cloudstack/blob/master/api/src/main/java/org/apache/cloudstack/api/command/admin/acl/UpdateRolePermissionCmd.java#L55 Therefore, yes it can be done via the update API adding a new arg.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
