[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-10344?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16462303#comment-16462303
 ] 

ASF GitHub Bot commented on CLOUDSTACK-10344:
---------------------------------------------

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-386265921
 
 
   @rafaelweingartner instead of catering to individual change, a fix could be 
to simply provide the update API the new list and old list, on the backend a 
comparison could be done to check and update the list only if old list is the 
same as in db. The same logic was used in Roles (drag+drop for order). Would 
you like to reconsider the implemenation and avoid adding a new API?

----------------------------------------------------------------
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:
us...@infra.apache.org


> Sometimes a bug happens when moving ACL rules (changing their order with drag 
> and drop) 
> ----------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-10344
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-10344
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>            Reporter: Rafael Weingärtner
>            Assignee: Rafael Weingärtner
>            Priority: Major
>             Fix For: 4.12
>
>
> An error is happening in certain conditions, such as when you have only 2 ACL 
> rules and you move the last one to the top. There are other conditions, for 
> instance, when moving ACLs that are in a sequence of numbers without gaps.
> Example, rules:
> number | rule
> 1 - rule A
> 2 - rule D
> 3 - rule B
> 4 - rule C
> 5 - rule E
> It is not possible to move "rule C" in position 2, 1, and 3.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to