Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Squiz/MethodScope: bugfix for unconventional spacing
The `Squiz.Scope.MethodScope` sniff is intended to check whether each method has visibility declared, but would in actual fact also enforce that the visibility should be on the same line as the `function` keyword as it stopped searching for the visibility keyword once it reached the start of the line. Fixed now by using the `File::getMethodProperties()` method for determining visibility instead of searching with sniff specific logic. Includes unit test.
- Loading branch information