AlabLan commented on PR #516: URL: https://github.com/apache/casbin-jcasbin/pull/516#issuecomment-4321380717
## Background Issue: https://github.com/apache/casbin/issues/1621 Sync GetAllUsers API from Go Casbin (PR #1652) to Java version. ## Changes - Add `getAllUsers()` API: returns subjects minus roles (users = subjects from "p" - roles from "g") - Rename `getUsers()` to `getAllUsers()` to align with Go Casbin API - Remove `getNamedUsers(ptype)` as Go version doesn't have it - Fix `Model.getFieldIndex()`: returns `-1` when not found (was `0`), add null-check for NPE prevention - Expand test coverage for `getAllUsers()` with edge cases ## Testing - ManagementAPIUnitTest passes - Behavior consistent with Go's GetAllUsers -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
