ravening commented on pull request #4200:
URL: https://github.com/apache/cloudstack/pull/4200#issuecomment-671869938


   > > Same applies for child1
   > 
   > I think this is a bug, `isrecursive` should not traverse up, only down.
   > 
   > > if you dont pass isrecursive=true and try listing offerings for child11 
then you will see all the parent domain offerings.
   
   @DaanHoogland `isrecursive` is traversing down as expected.
   Problem happens when this flag is true and function is called with child 
domain id as first parameter and parent domain id as second parameter.
   
   If `isrecursive` is true, it checks if the second parameter is the child 
domain of first parameter. Since we are passing parent domain as the second 
parameter, the condition will be false and it will never display the offerings 
belonging to parent domain


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to