Github user mike-jumper commented on a diff in the pull request:
https://github.com/apache/guacamole-client/pull/319#discussion_r221491527
--- Diff:
extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/permission/ModeledObjectPermissionService.java
---
@@ -47,14 +49,14 @@ protected ObjectPermission
getPermissionInstance(ObjectPermissionModel model) {
}
@Override
- protected ObjectPermissionModel getModelInstance(ModeledUser
targetUser,
+ protected ObjectPermissionModel getModelInstance(
+ ModeledPermissions<? extends EntityModel> targetEntity,
ObjectPermission permission) {
ObjectPermissionModel model = new ObjectPermissionModel();
--- End diff --
Only that there is no constructor of `ObjectPermissionModel` which
populates those fields. We could write one.
---
