Undeclared variable lint doesn't work at top-level scope
Open, NormalPublic
Actions

Assigned To

Authored By

	joshuaspence
	Mar 5 2015, 5:51 AM

Description

No linter errors are raised for the following code:

<?php
echo $x;

joshuaspence claimed this task.

joshuaspence raised the priority of this task from to Needs Triage.

joshuaspence updated the task description. (Show Details)

joshuaspence added a project: Lint.

In the general case, there are approximately one hundred trillion PHP files out there which do this:

<?php

include 'lib.php';
echo $g_coolGlobal;

...so we should consider require/include to be scope poison (i.e., make following scope unknowable) if we don't already.

It seems that LINT_NAMING_CONVENTIONS doesn't work at the top level scope either.

joshuaspence triaged this task as Normal priority.May 14 2015, 8:56 AM