Manca una parentesi in quella riga... e se avessi postato il messaggio di errore sarebbe stato decisamente più facile capire cosa non andava
Il blocco è però eccessivamente complicato a mio parere, tanto che si fatica a capire cosa tu voglia fare. Direi che attendi una condizione positiva, cioé se la directory esiste, o se sei riuscito a crearla, allora lo script va avanti; altrimenti, messaggio di errore. Quindi:
Codice PHP:
$dir = $path . "/" . $this->pathThumbs;
if ( is_dir($dir) || mkdir($dir, 0755) ) {
// la directory c'è, lavoriamoci
} else {
// non c'è, o non siamo riusciti a crearla: messaggio di errore
}
Tutto qua. Non serve ripetere is_dir() negata, perché se la prima parte di un or ( || ) è vera, la seconda non viene neanche valutata.
nessun problema in generale ad usare funzione nelle condizioni dell'if, anzi, è un'ottima pratica per rendere il codice più leggibile (senza esagerare però )