feat(Events): Add newAttributes and originalAttributes to the `pr…

…eUpdate` event
coldbox-modules · May 31, 2024 · 4dd8496 · 4dd8496
1 parent 084a089
commit 4dd8496
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/models/BaseEntity.cfc b/models/BaseEntity.cfc
@@ -1100,7 +1100,14 @@ component accessors="true" {
 		variables._saving = true;
 		var builder       = newQuery();
 		if ( variables._loaded ) {
-			fireEvent( "preUpdate", { entity : this } );
+			fireEvent(
+				"preUpdate",
+				{
+					"entity"             : this,
+					"newAttributes"      : get_data(),
+					"originalAttributes" : get_originalAttributes()
+				}
+			);
 			builder
 				.where( function( q ) {
 					arrayZipEach( [ keyNames(), keyValues() ], function( keyName, keyValue ) {

diff --git a/models/Relationships/HasOneOrMany.cfc b/models/Relationships/HasOneOrMany.cfc
@@ -321,9 +321,7 @@ component
 	 * @return      quick.models.BaseEntity
 	 */
 	public any function create( struct attributes = {} ) {
-		return newEntity()
-			.fill( arguments.attributes )
-			.save();
+		return newEntity().fill( arguments.attributes ).save();
 	}
 
 	/**

diff --git a/tests/specs/integration/BaseEntity/Events/PreUpdateSpec.cfc b/tests/specs/integration/BaseEntity/Events/PreUpdateSpec.cfc
@@ -29,6 +29,11 @@ component extends="tests.resources.ModuleIntegrationSpec" {
 				expect( request.quickPreUpdateCalled[ 1 ].entity.getDownloadUrl() ).toBe(
 					"https://open.spotify.com/track/0GHGd3jYqChGNxzjqgRZSv"
 				);
+				expect( request.quickPreUpdateCalled[ 1 ] ).toHaveKey( "originalAttributes" );
+				expect( request.quickPreUpdateCalled[ 1 ].originalAttributes.download_url ).toBe( "https://open.spotify.com/track/4Nd5HJn4EExnLmHtClk4QV" );
+				expect( request.quickPreUpdateCalled[ 1 ] ).toHaveKey( "newAttributes" );
+				expect( request.quickPreUpdateCalled[ 1 ].newAttributes.download_url ).toBe( "https://open.spotify.com/track/0GHGd3jYqChGNxzjqgRZSv" );
+
 				structDelete( request, "quickPreUpdateCalled" );
 			} );