weizhouapache commented on PR #13456:
URL: https://github.com/apache/cloudstack/pull/13456#issuecomment-5606761819
> > @weizhouapache, I checked for possible regressions by creating a User
account with 2 users: `user1` and `user1.2`. Then, I created a project with the
admin as the owner and added `user1` to it. When I logged in as `user1`, I
could access the project normally, but when I logged in as `user1.2`, I could
not access the project.
> > I also checked the other uses, for example deleting the `user1` of the
project and no regressions occurred.
>
> @Tonitzpp thanks for the checking the case. however, the method
`ProjectAccountSearch` is being used in many places. It would be better to
check more cases which might be broken by the change:
>
> ```
> - ProjectAccountSearch.and("userId",
ProjectAccountSearch.entity().getUserId(), Op.NULL);
> ```
>
> an alternative way is, create a new method instead of changing the
existing method
@Tonitzpp
please let me know if you have received this message
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
