Page MenuHomeDevCentral

Improve code guidelines for exception handler
ClosedPublic

Authored by dereckson on Dec 31 2016, 20:56.
Tags
None
Referenced Files
F3752930: D783.id1980.diff
Sun, Nov 17, 22:08
F3752434: D783.id.diff
Sun, Nov 17, 19:00
Unknown Object (File)
Sat, Nov 16, 18:09
Unknown Object (File)
Sat, Nov 16, 17:53
Unknown Object (File)
Sat, Nov 16, 17:09
Unknown Object (File)
Sat, Nov 16, 16:21
Unknown Object (File)
Sat, Nov 16, 16:20
Unknown Object (File)
Sat, Nov 16, 14:09
Subscribers

Details

Summary

The class had still the autogenerated style of Laravel
instead to use our code base guidelines:

Quick cleanup:

  • Drop a method only calling parent method
  • Sort by alphabetical and logic order use clauses
  • Add PHP 7 return types
  • Document more richly arrays as type[]
  • Respect Nasqueron code style:
    • Comments are 3rd person
    • Brackets on the same line
Test Plan

Run tests

Diff Detail

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

Event Timeline

dereckson edited the summary of this revision. (Show Details)
dereckson edited edge metadata.
This revision is now accepted and ready to land.Dec 31 2016, 21:00
This revision was automatically updated to reflect the committed changes.