Page MenuHomeDevCentral

Optimize Handler::report
ClosedPublic

Authored by dereckson on Mar 2 2016, 10:34.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 16:01
Unknown Object (File)
Tue, Dec 10, 04:30
Unknown Object (File)
Tue, Dec 10, 04:16
Unknown Object (File)
Wed, Dec 4, 11:39
Unknown Object (File)
Wed, Dec 4, 09:58
Unknown Object (File)
Tue, Dec 3, 03:45
Unknown Object (File)
Wed, Nov 27, 09:39
Unknown Object (File)
Tue, Nov 26, 16:57
Subscribers

Details

Summary

Parent class method check the $dontReport array, then
log the exception.

As we have more logic, and this logic also require to
check what we want to report, we would check twice this
condition.

So we handle all the report logic in our class method,
without calling the parent class method anymore.

Test Plan

Run tests.

A new test for an unreportable exception has been added.

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson retitled this revision from to Optimize Handler::report.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson edited edge metadata.
This revision is now accepted and ready to land.Mar 2 2016, 10:48

Class is fully tested according report coverage, so it's fine.

This revision was automatically updated to reflect the committed changes.