weizhouapache commented on PR #13456:
URL: https://github.com/apache/cloudstack/pull/13456#issuecomment-5104825946
> @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
--
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]
