Creating a directory with mkdir php

Question

I wrote a feature on php to upload files by running localhost it works normal creating the correct directory, but on the server it creates wrong, if I step the way: home\banners, it creates a folder with the name bootstrap.. and does not save the file, it is as if the function is not making use of the third parameter of the mkdir(path, mode, recursive = true), follows the function:

// arquivo config.php
define('DP', DIRECTORY_SEPARATOR);
define('UPLOAD_DIRECTORY', __DIR__ . '\..\assets' . DP);

// arquivo controllers.php
public function saveUploadFile(string $uploaddirectory = '', UploadedFile $file)
{
    // PEGA EXTENÇÃO DO ARQUIVO
    $_['ext'] = pathinfo($file->getClientFilename(), PATHINFO_EXTENSION);
    // RENOMEIA PARA NUMERO EXADECIMAL ALEATORIO
    $_['rename'] = bin2hex(random_bytes(16));
    // JUNTA NOME DO ARQUIVO + EXTENSÇÃO
    $image = sprintf('%s.%0.8s', $_['rename'], $_['ext']);
    // CRIA PATH QUE SERA SALVA A IMAGEM        
    $_['path'] = UPLOAD_DIRECTORY . $uploaddirectory;       
    // SE NÃO EXISTE DIRETORIO CRIAR
    if( !is_dir( $_[ 'path' ] ) ) {
        if( !mkdir( $_['path'], 0777, true ) ){
            exit('falha ao criar arquivo no diretorio '. $_['path']);
        }
    }
    // MOVE ARQUIVO PARA O PATH
    $file->moveTo($_['path'] . DP . $image);
    // DEVOLVE CAMINHO ONDE IMAGEM FOI SALVA
    return $_['path'] . DP . $image;
}

Folder structure:

root
  ├── app
  │    └── Controllers
  │            └── Controllers.php
  ├── bootstrap
  │     └── Config.php
  └── assets
       └── // imagens devem ficar aqui

Answer 1

Change:

\..\assets

For:

/../assets

Allies don’t even need to use:

define('DP', DIRECTORY_SEPARATOR);

Both Windows and Linux accept /, then do it like this:

// arquivo config.php
define('UPLOAD_DIRECTORY', __DIR__ . '/../assets/');

// arquivo controllers.php
public function saveUploadFile(string $uploaddirectory = '', UploadedFile $file)
{
    // PEGA EXTENÇÃO DO ARQUIVO
    $_['ext'] = pathinfo($file->getClientFilename(), PATHINFO_EXTENSION);
    // RENOMEIA PARA NUMERO EXADECIMAL ALEATORIO
    $_['rename'] = bin2hex(random_bytes(16));
    // JUNTA NOME DO ARQUIVO + EXTENSÇÃO
    $image = sprintf('%s.%0.8s', $_['rename'], $_['ext']);
    // CRIA PATH QUE SERA SALVA A IMAGEM        
    $_['path'] = UPLOAD_DIRECTORY . $uploaddirectory;       
    // SE NÃO EXISTE DIRETORIO CRIAR
    if( !is_dir( $_[ 'path' ] ) ) {
        if( !mkdir( $_['path'], 0777, true ) ){
            exit('falha ao criar arquivo no diretorio '. $_['path']);
        }
    }
    // MOVE ARQUIVO PARA O PATH
    $file->moveTo($_['path'] . '/' . $image);
    // DEVOLVE CAMINHO ONDE IMAGEM FOI SALVA
    return $_['path'] . '/' . $image;
}

---

extras

Now about the use of $_['ext'], $_['path'], etc, really do not know why to do this, the variables are already isolated in the scope of saveUploadFile, there is no reason to keep creating an array, something else your array or this started, it should have something like this:

public function saveUploadFile(string $uploaddirectory = '', UploadedFile $file)
{
    // PEGA EXTENÇÃO DO ARQUIVO
    $_ = array();

Because if you don’t you’ll probably fill the PHP log with several warnings no need, still yes it would be more practical only to create the vars directly, since this in the scope of the method, has no need of arrays.

Another jealousy that makes no sense are the parameters:

saveUploadFile(string $uploaddirectory = '', UploadedFile $file)

There is no reason why the first parameter should be opitional, in fact it is impossible for it to be opitional, that is, there is no way to pass the second without passing the first, the right would be:

saveUploadFile(string $uploaddirectory, UploadedFile $file)

The end result would be this:

// arquivo config.php
define('UPLOAD_DIRECTORY', __DIR__ . '/../assets/');

// arquivo controllers.php
public function saveUploadFile(string $uploaddirectory, UploadedFile $file)
{
    // PEGA EXTENÇÃO DO ARQUIVO
    $ext = pathinfo($file->getClientFilename(), PATHINFO_EXTENSION);

    // RENOMEIA PARA NUMERO EXADECIMAL ALEATORIO
    $rename = bin2hex(random_bytes(16));

    // JUNTA NOME DO ARQUIVO + EXTENSÇÃO
    $image = sprintf('%s.%0.8s', $rename, $ext);

    // CRIA PATH QUE SERA SALVA A IMAGEM        
    $path = UPLOAD_DIRECTORY . $uploaddirectory;

    // SE NÃO EXISTE DIRETORIO CRIAR
    if( !is_dir( $path ) ) {
        if( !mkdir( $path, 0777, true ) ){
            exit('falha ao criar arquivo no diretorio '. $path);
        }
    }

    // MOVE ARQUIVO PARA O PATH
    $file->moveTo($path . '/' . $image);

    // DEVOLVE CAMINHO ONDE IMAGEM FOI SALVA
    return $path . '/' . $image;
}