necouchman commented on code in PR #673:
URL: https://github.com/apache/guacamole-client/pull/673#discussion_r1550637394
##########
guacamole-ext/src/main/java/org/apache/guacamole/net/auth/Identifiable.java:
##########
@@ -43,5 +43,26 @@ public interface Identifiable {
* The identifier to assign.
*/
public void setIdentifier(String identifier);
+
+ /**
+ * Returns true if this object is disabled, otherwise false.
+ *
+ * @return
+ * True if this user account is disabled, otherwise false.
+ */
+ default public boolean isDisabled() {
+ return false;
+ }
+
+ /**
+ * Set the disabled status of this object to the boolean value provided,
+ * true if the object should be disabled, otherwise false.
+ *
+ * @param disabled
+ * True if the object should be disabled, otherwise false.
+ */
+ default public void setDisabled(boolean disabled) {
Review Comment:
Okay - I'm fine creating a new interface for this. Is `Disableable` what we
want to go with, or are there other things besides just the ability to disable
something that would fit in the new Interface? I can't think of anything
necessarily, just wondering before we make it official...
--
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: dev-unsubscr...@guacamole.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
