Merge pull request #21 from e-line-websolutions/restore-set-reverseFi…

…eld-issue Restore set reverse field issue
e-line-websolutions · May 20, 2021 · 98becce · 98becce
2 parents bb3e66d + 7cf9115
commit 98becce
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/base.cfc b/base.cfc
@@ -987,6 +987,9 @@ component mappedSuperClass=true cacheuse="transactional" defaultSort="sortorder"
         var reverseField = objectToOverride.getReverseField( reverseCFCLookup, property.inverseJoinColumn );
         queueInstruction( objectToOverride, "remove#reverseField#", this );
         arrayAppend( result, "#objectToOverride.getName()#.remove#reverseField#(#this.getName()#)" );
+      } else {
+        var reverseField = objectToOverride.getReverseField( reverseCFCLookup, property.fkcolumn, false );
+        queueInstruction( objectToOverride, "set#reverseField#", "null" );
       }
 
       queueInstruction( this, "remove#propertyName( property )#", objectToOverride );

diff --git a/tests/specs/default.cfc b/tests/specs/default.cfc
@@ -285,6 +285,25 @@ component extends="testbox.system.basespec" {
           expect( savedentitiesinsubfolder[ 1 ].getid() ).tobe( third.getid() );
         } );
 
+        it( 'expects save( {set_data=[]}) to clear all items in data', function() {
+          transaction {
+            var first = entityNew( 'other' ).save( { name = 'first' } );
+            var second = entityNew( 'other' ).save( { name = 'second' } );
+            variables.obj.save( { 'entitiesinsubfolder' = [ first, second ] } );
+          }
+
+          expect( variables.obj.getEntitiesinsubfolder() ).toHaveLength( 2 );
+
+          transaction {
+            variables.obj.save( { 'entitiesinsubfolder' = [] } );
+          }
+
+          expect( variables.obj.getEntitiesinsubfolder() ).toHaveLength( 0 );
+
+          // expect( entityLoad( 'other', { name = 'first' } ) ).toBeEmpty();
+          // expect( entityLoad( 'other', { name = 'second' } ) ).toBeEmpty();
+        } );
+
         it( 'expects remove to work', function() {
           transaction {
             var multiple_1 = entityNew( 'multiple' ).save();