Page MenuHomeDevCentral

Optimize Handler::report
ClosedPublic

Authored by dereckson on Mar 2 2016, 10:34.
Tags
None
Referenced Files
F3751406: D313.diff
Sun, Nov 17, 08:13
Unknown Object (File)
Wed, Nov 13, 00:49
Unknown Object (File)
Tue, Nov 12, 21:52
Unknown Object (File)
Tue, Nov 12, 21:08
Unknown Object (File)
Sun, Nov 10, 14:44
Unknown Object (File)
Sun, Nov 10, 02:24
Unknown Object (File)
Sat, Nov 9, 15:30
Unknown Object (File)
Thu, Nov 7, 17:16
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 Passed
Unit
Tests Passed
Branch
T765

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.