Merge pull request #11052 from IQSS/11044-edit-facet-bug

11044 edit facet bug

Author: ofahimIQSS

src/main/java/edu/harvard/iq/dataverse/DataverseFacetServiceBean.java

@@ -42,23 +42,24 @@ public void delete(DataverseFacet dataverseFacet) {
         cache.invalidate();
     }
 
-	public void deleteFacetsFor( Dataverse d ) {
-		em.createNamedQuery("DataverseFacet.removeByOwnerId")
-			.setParameter("ownerId", d.getId())
-				.executeUpdate();
+    public void deleteFacetsFor(Dataverse d) {
+        em.createNamedQuery("DataverseFacet.removeByOwnerId")
+                .setParameter("ownerId", d.getId())
+                .executeUpdate();
         cache.invalidate(d.getId());
-        
-	}
-	
+
+    }
+
     public DataverseFacet create(int displayOrder, DatasetFieldType fieldType, Dataverse ownerDv) {
         DataverseFacet dataverseFacet = new DataverseFacet();
-        
+
         dataverseFacet.setDisplayOrder(displayOrder);
         dataverseFacet.setDatasetFieldType(fieldType);
         dataverseFacet.setDataverse(ownerDv);
-        
         ownerDv.getDataverseFacets().add(dataverseFacet);
         em.persist(dataverseFacet);
+
+        cache.invalidate(ownerDv.getId());
         return dataverseFacet;
     }
 

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/AbstractWriteDataverseCommand.java

@@ -72,7 +72,8 @@ private void processMetadataBlocks() {
     private void processFacets(CommandContext ctxt) {
         if (facets != null) {
             ctxt.facets().deleteFacetsFor(dataverse);
-
+            dataverse.setDataverseFacets(new ArrayList<>());
+           
             if (!facets.isEmpty()) {
                 dataverse.setFacetRoot(true);
             }