[phpMyAdmin Developers] PHPMYADMIN constant check
Marc Delisle
marc at infomarc.info
Sat Aug 1 22:33:35 CEST 2015
Le 2015-08-01 16:10, Dan Ungureanu a écrit :
> Hello,
>
> I see the following snippet in almost every file of phpMyAdmin, but I
> believe that in most files it is not required.
>
> if (! defined('PHPMYADMIN')) {
> exit;
> }
>
> I think that it was used to prevent execution if a direct request was
> made to the file, but nowadays, most files contain only procedural code.
> In most cases, with or without this, the code behaves the same so it may
> be considered dead code.
>
> I propose removing it where possible. What is your opinion?
>
> Best regards,
> Dan Ungureanu
I agree that if the file only contains functions and classes, there is
no point having this snippet, but I may be missing something.
--
Marc Delisle | phpMyAdmin
More information about the Developers
mailing list