🗑️ Deprecate ModifyConfig without identifier
authorDavid ‘Bombe’ Roden <bombe@freenetproject.org>
Fri, 3 Jan 2025 15:13:00 +0000 (16:13 +0100)
committerDavid ‘Bombe’ Roden <bombe@freenetproject.org>
Fri, 3 Jan 2025 15:13:00 +0000 (16:13 +0100)
src/main/java/net/pterodactylus/fcp/ModifyConfig.java
src/main/java/net/pterodactylus/fcp/highlevel/FcpClient.java

index 0a18b81..224f598 100644 (file)
@@ -26,11 +26,19 @@ public class ModifyConfig extends FcpMessage {
 
        /**
         * Creates a new “ModifyConfig” message.
+        *
+        * @deprecated Use {@link #ModifyConfig(String)} instead
         */
+       @Deprecated
        public ModifyConfig() {
                super("ModifyConfig");
        }
 
+       /**
+        * Creates a new “ModifyConfig” message.
+        *
+        * @param identifier The identifier of the message
+        */
        public ModifyConfig(String identifier) {
                this();
                setField("Identifier", identifier);
index 3fecbed..a3ee4ca 100644 (file)
@@ -1220,7 +1220,7 @@ public class FcpClient implements Closeable {
                new ExtendedFcpAdapter() {
                        @Override
                        public void run() throws IOException {
-                               ModifyConfig modifyConfig = new ModifyConfig();
+                               ModifyConfig modifyConfig = new ModifyConfig(createIdentifier("modify-config"));
                                options.forEach(modifyConfig::setOption);
                                sendMessage(modifyConfig);
                        }