Improve source formatter to flag swallowed(caught but not processing it) Portal/System exception

• Type: Story
• Status: Closed
• Priority: Minor
• Resolution: Completed
• Affects Version/s: 6.1.30 EE GA3, 6.2.0 CE M6
• Fix Version/s: 6.1.30 EE GA3, 6.2.0 CE M6
• Component/s: Core Infrastructure, Core Infrastructure > ORM, Performance
• Labels: None
• Branch Version/s:
  6.1.x
• Backported to Branch:
  Committed

Using exception throwing to do execution flow control is a very bad practice. Because creating exception needs to capture the current thread's call stack info, when the execution stack is very deep, capturing this info could cause significant overhead.

For Portal/System Exception, things could get even worse, because we by default doing a transaction rollback at service layer on seeing them. You may actually rollback a tx even without knowing it, until later you can not see the data you just tried to save to database.

This source formatting change won't be smart enough to catch all bad usage of throwing exception, but it can handle one typical scenario, that is you catch a portal/system exception, without any processing logic, just swallow it. This bad usage after this ticket is forbidden.