fix: When no dirty fields provided, overwrite the registered record

JonnyPower · JonnyPower · commit 7db6291513a1 · 2018-03-11T21:30:29.000-07:00
diff --git a/fflib/src/classes/fflib_SObjectUnitOfWork.cls b/fflib/src/classes/fflib_SObjectUnitOfWork.cls
@@ -260,7 +260,7 @@ public virtual class fflib_SObjectUnitOfWork
             throw new UnitOfWorkException(String.format('SObject type {0} is not supported by this unit of work', new String[] { sObjectType }));
 
         // If record isn't registered as dirty
-        if (!m_dirtyMapByType.get(sObjectType).containsKey(record.Id))
+        if (!m_dirtyMapByType.get(sObjectType).containsKey(record.Id) || dirtyFields.isEmpty())
         {
             // Register the record as dirty
             m_dirtyMapByType.get(sObjectType).put(record.Id, record);

Original file line number	Diff line number	Diff line change
`@@ -260,7 +260,7 @@ public virtual class fflib_SObjectUnitOfWork`
`260`	`260`	`throw new UnitOfWorkException(String.format('SObject type {0} is not supported by this unit of work', new String[] { sObjectType }));`
`261`	`261`
`262`	`262`	`// If record isn't registered as dirty`
`263`		`- if (!m_dirtyMapByType.get(sObjectType).containsKey(record.Id))`
	`263`	`+ if (!m_dirtyMapByType.get(sObjectType).containsKey(record.Id) \|\| dirtyFields.isEmpty())`
`264`	`264`	`{`
`265`	`265`	`// Register the record as dirty`
`266`	`266`	`m_dirtyMapByType.get(sObjectType).put(record.Id, record);`