extract code

Author: Pearl1594

api/src/main/java/org/apache/cloudstack/backup/BackupManager.java

@@ -47,16 +47,7 @@ public interface BackupManager extends BackupService, Configurable, PluggableSer
             "backup.framework.provider.plugin",
             "dummy",
             "The backup and recovery provider plugin. Valid plugin values: dummy, veeam, networker and nas",
-            true, ConfigKey.Scope.Zone, BackupFrameworkEnabled.key(),
-            value -> {
-                if (value != null && ((String)value).contains(",") || ((String)value).contains(" ")) {
-                    throw new IllegalArgumentException("Multiple backup provider plugins are not supported. Please provide a single plugin value.");
-                }
-                List<String> validPlugins = List.of("dummy", "veeam", "networker", "nas");
-                if (!validPlugins.contains(((String) value).toLowerCase())) {
-                    throw new IllegalArgumentException("Invalid backup provider plugin: " + value + ". Valid plugin values are: " + String.join(", ", validPlugins));
-                }
-            });
+            true, ConfigKey.Scope.Zone, BackupFrameworkEnabled.key(), value -> validateBackupProviderConfig((String)value));
 
     ConfigKey<Long> BackupSyncPollingInterval = new ConfigKey<>("Advanced", Long.class,
             "backup.framework.sync.interval",
@@ -159,4 +150,14 @@ public interface BackupManager extends BackupService, Configurable, PluggableSer
     boolean deleteBackup(final Long backupId, final Boolean forced);
 
     BackupOffering updateBackupOffering(UpdateBackupOfferingCmd updateBackupOfferingCmd);
+
+    static void validateBackupProviderConfig(String value) {
+        if (value != null && (value.contains(",") || value.contains(" "))) {
+            throw new IllegalArgumentException("Multiple backup provider plugins are not supported. Please provide a single plugin value.");
+        }
+        List<String> validPlugins = List.of("dummy", "veeam", "networker", "nas");
+        if (value != null && !validPlugins.contains(value.toLowerCase())) {
+            throw new IllegalArgumentException("Invalid backup provider plugin: " + value + ". Valid plugin values are: " + String.join(", ", validPlugins));
+        }
+    }
 }