Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why absolute path constants __DIR__ and __FILE__ should not be used in Symfony

I use SensioLabs Insight to control my code quality.

For a simple file upload, I have to get the absolute path of my uploads directory:

protected function getUploadRootDir()
{
    // the absolute directory path where uploaded
    return __DIR__.'/../../../../web/'.$this->getUploadDir();
}

Code directly coming from official documentation (How to handle file uploads with Doctrine)

But SLInsight raises a warning if the code analysed contains __DIR__ or __FILE__ PHP magic constants:

__DIR__ and __FILE__ constants may conflict with the Symfony resource overriding system.

How usage of this constants can causes conflicts with Symfony?

And how can I avoid them in my code?

like image 289
chalasr Avatar asked Dec 17 '15 09:12

chalasr


2 Answers

In the case of the file upload class, you can probably ignore this error message. But in other cases, it's better o use the Symfony file locator instead of hardcoding file paths. For example:

$path = $this->get('kernel')->locateResource('@AppBundle/Resources/config/services.xml');

Instead of:

$path = __DIR__.'/../../../src/Acme/AppBundle/Resources/config/services.xml'
like image 135
Javier Eguiluz Avatar answered Nov 17 '22 04:11

Javier Eguiluz


Well, this is actually something that SensioLabs Insight does not handles properly. It warns against using the constants because of the resource overriding system, but in many cases, these constants are used in places which are unrelated to the resource overriding system (and this is probably the case for your code here). So you can ignore the warning in this case

like image 42
Christophe Coevoet Avatar answered Nov 17 '22 02:11

Christophe Coevoet