search

apex-enterprise-patterns / fflib-apex-common

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

Readable adjust #446

Closed clebrsonn closed 1 year ago

clebrsonn commented 1 year ago

removed chained ifs and added map to return messages by operation type

This change is Reviewable

daveespo commented 1 year ago

We have a general rule around not merging PRs that are purely "just different, not better". That is, there is no need for this change: It doesn't fix a bug, it doesn't improve performance materially, etc. Making changes for the sake of change introduces risk unnecessarily.

Additionally, with the introduction of native User Mode support in #419 , the legacy methods for testing CRUD and FLS should be considered deprecated and so any enhancements to that code will likely be rejected.