Page MenuHomePhabricator

Improve PHP7 error reporting for include-time syntax errors
ClosedPublic

Authored by epriestley on Nov 28 2017, 2:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 17, 8:19 AM
Unknown Object (File)
Sat, Mar 16, 2:50 AM
Unknown Object (File)
Fri, Mar 8, 5:25 PM
Unknown Object (File)
Fri, Mar 1, 3:28 AM
Unknown Object (File)
Wed, Feb 28, 3:46 PM
Unknown Object (File)
Feb 21 2024, 11:05 AM
Unknown Object (File)
Feb 21 2024, 8:37 AM
Unknown Object (File)
Feb 6 2024, 10:25 PM
Subscribers
None

Details

Summary

Ref T12101. Ref T12855. Ref T12190.

In PHP7, include_once may raise a Throwable which has a useless message and stack trace (neither point at the actual file or line where the error occurs).

Catch and mangle these so they're useful. This upgrades the message from syntax error: unexpected X so it has a file and line number.

Test Plan

Screen Shot 2017-11-28 at 6.03.12 AM.png (1×1 px, 220 KB)

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Nov 28 2017, 5:54 PM

The specific error / reproduction case here was something like $this->someMethod(): (instead of ...();) if that's helpful to some future archaeologist.

This revision was automatically updated to reflect the committed changes.