search

apex-enterprise-patterns / fflib-apex-common

Common Apex Library supporting Apex Enterprise Patterns and much more!
BSD 3-Clause "New" or "Revised" License
913 stars 517 forks source link

Some cleanup of UnitOfWork #217

Closed wimvelzeboer closed 5 years ago

wimvelzeboer commented 5 years ago

I removed duplications of the validation checks, now the methods are more in one level of abstraction. Also fixed some typos and some minor PDM stuff, mostly spaces and brace positions.

afawcett commented 5 years ago

@agarciaffdc these looks good to me. I added a review as well.

agarciaffdc commented 5 years ago

@dbtavernerffdc FYI

wimvelzeboer commented 5 years ago

@agarciaffdc & @afawcett I have updated this PR and also refactored the commitWork method, to improve readability.

agarciaffdc commented 5 years ago

@wimvelzeboer thanks for your latest contribution. Really appreciate it.