#307 Only removing ID property for databases for 3.9.1

Author: rjrudin

gradle.properties

@@ -1,6 +1,6 @@
 group=com.marklogic
 javadocsDir=../gh-pages-marklogic-java/javadocs
-version=3.9.0
+version=3.9.1
 mlJavaclientUtilVersion=3.8.1
 mlJunitVersion=3.1.0
 

src/main/java/com/marklogic/mgmt/AbstractManager.java

@@ -4,6 +4,7 @@
 import com.marklogic.mgmt.ManageClient;
 import com.marklogic.mgmt.resource.forests.ForestManager;
 import com.marklogic.rest.util.Fragment;
+import org.springframework.http.ResponseEntity;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -20,7 +21,28 @@ public DatabaseManager(ManageClient manageClient) {
         super(manageClient);
     }
 
-    /**
+	/**
+	 * This is being added in 3.9.1 to add support for when a database is updated by a user that only has privileges
+	 * to update indexes. In such a scenario, the database-name property cannot exist, even if it's not changing. It's
+	 * also not needed, because the URL for the request specifies the database to update.
+	 *
+	 * It may be safe to make this change for all resource property updates, but it's only being applied for databases
+	 * since that's the immediate need for the Data Hub Framework.
+	 *
+	 * @param client
+	 * @param path
+	 * @param payload
+	 * @return
+	 */
+	@Override
+	protected ResponseEntity<String> putPayload(ManageClient client, String path, String payload) {
+		if (payloadParser.isJsonPayload(payload)) {
+			payload = payloadParser.excludeProperties(payload, getIdFieldName());
+		}
+		return super.putPayload(client, path, payload);
+	}
+
+	/**
      * This will catch and log any exception by default, as the most frequent reason why this fails is because the
      * database doesn't exist yet.
      *

src/main/java/com/marklogic/mgmt/resource/databases/DatabaseManager.java

@@ -0,0 +1,28 @@
+package com.marklogic.appdeployer.command.databases;
+
+import com.marklogic.appdeployer.AbstractAppDeployerTest;
+import org.junit.Test;
+
+import java.io.File;
+
+public class CreateAndUpdateDatabaseTest extends AbstractAppDeployerTest {
+
+	/**
+	 * TODO Update this test to perform the second deployment as a user that only has privileges to update indexes.
+	 *
+	 */
+	@Test
+	public void test() {
+		appConfig.getFirstConfigDir().setBaseDir(new File("src/test/resources/sample-app/db-only-config"));
+
+		DeployContentDatabasesCommand command = new DeployContentDatabasesCommand(1);
+		initializeAppDeployer(command);
+
+		try {
+			deploySampleApp();
+			deploySampleApp();
+		} finally {
+			undeploySampleApp();
+		}
+	}
+}