Fixed for Issue 14

#1
4

rolluptool/src/classes/RollupService.cls

@@ -179,8 +179,10 @@ global with sharing class RollupService
 					// Add both old and new value to master record Id list for relationship fields to ensure old and new parent master records are updated (re-parenting)
 					if(relationshipFields.contains(fieldToSearch))
 					{
-						masterRecordIds.add((Id) newValue);
-						masterRecordIds.add((Id) oldValue);		
+						if(newValue!=null)
+							masterRecordIds.add((Id) newValue);
+						if(oldValue!=null)
+							masterRecordIds.add((Id) oldValue);		
 					}
 				}
 			}

rolluptool/src/classes/RollupServiceTest.cls

@@ -275,6 +275,13 @@ private with sharing class RollupServiceTest
 		opp.Amount = 100;
 		insert opp;
 
+		// Assert rollup
+		System.assertEquals(0, [select AnnualRevenue from Account where Id = :account.Id].AnnualRevenue);
+
+		// Do an update
+		opp.Amount = 101;
+		update opp;		
+
 		// Assert rollup
 		System.assertEquals(0, [select AnnualRevenue from Account where Id = :account.Id].AnnualRevenue);					
 	}