Page MenuHomeDevCentral

Improve code guidelines for exception handler
ClosedPublic

Authored by dereckson on Dec 31 2016, 20:56.
Tags
None
Referenced Files
F2830490: D783.diff
Fri, Apr 19, 12:30
Unknown Object (File)
Thu, Apr 18, 02:42
Unknown Object (File)
Mon, Apr 15, 05:16
Unknown Object (File)
Sun, Apr 14, 14:22
Unknown Object (File)
Sun, Apr 14, 04:54
Unknown Object (File)
Sun, Apr 14, 02:44
Unknown Object (File)
Sat, Apr 13, 18:37
Unknown Object (File)
Wed, Apr 10, 13:20
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.