Fix di alcuni warning (#229)

This commit is contained in:
Thomas Zilio 2018-05-03 17:29:10 +02:00
parent d9f9461e9f
commit 95aa649ee1
2 changed files with 25 additions and 7 deletions

View File

@ -75,7 +75,8 @@ class API extends \Util\Singleton
$select = !empty($request['display']) ? explode(',', substr($request['display'], 1, -1)) : $select; $select = !empty($request['display']) ? explode(',', substr($request['display'], 1, -1)) : $select;
// Ricerca personalizzata // Ricerca personalizzata
foreach ((array) $request['filter'] as $key => $value) { $values = isset($request['filter']) ? (array) $request['filter'] : [];
foreach ($values as $key => $value) {
// Rimozione delle parentesi // Rimozione delle parentesi
$value = substr($value, 1, -1); $value = substr($value, 1, -1);
@ -84,13 +85,14 @@ class API extends \Util\Singleton
} }
// Ordinamento personalizzato // Ordinamento personalizzato
foreach ((array) $request['order'] as $value) { $values = isset($request['order']) ? (array) $request['order'] : [];
foreach ($values as $value) {
$pieces = explode('|', $value); $pieces = explode('|', $value);
$order[] = empty($pieces[1]) ? $pieces[0] : [$pieces[0] => $pieces[1]]; $order[] = empty($pieces[1]) ? $pieces[0] : [$pieces[0] => $pieces[1]];
} }
// Paginazione automatica dell'API // Paginazione automatica dell'API
$page = (int) $request['page'] ?: 0; $page = isset($request['page']) ? (int) $request['page'] : 0;
$length = Settings::get('Lunghezza pagine per API'); $length = Settings::get('Lunghezza pagine per API');
$database = Database::getConnection(); $database = Database::getConnection();
@ -105,12 +107,19 @@ class API extends \Util\Singleton
// Esecuzione delle operazioni personalizzate // Esecuzione delle operazioni personalizzate
$filename = DOCROOT.'/modules/'.$resources[$resource].'/api/'.$kind.'.php'; $filename = DOCROOT.'/modules/'.$resources[$resource].'/api/'.$kind.'.php';
include $filename; include $filename;
} elseif (!in_array($resource, explode(',', Settings::get('Tabelle escluse per la sincronizzazione API automatica')))) { } elseif (
!in_array($resource, explode(',', Settings::get('Tabelle escluse per la sincronizzazione API automatica')))
&& $database->fetchNum('SHOW TABLES WHERE `Tables_in_'.$database->getDatabaseName().'` = '.prepare($resource))
) {
$table = $resource; $table = $resource;
// Individuazione della colonna AUTO_INCREMENT per l'ordinamento automatico // Individuazione della colonna AUTO_INCREMENT per l'ordinamento automatico
if (empty($order)) { if (empty($order)) {
$order[] = $database->fetchArray('SELECT COLUMN_NAME FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_NAME = '.prepare($table)." AND EXTRA LIKE '%AUTO_INCREMENT%' AND TABLE_SCHEMA = ".prepare($database->getDatabaseName()))[0]['COLUMN_NAME']; $column = $database->fetchArray('SELECT COLUMN_NAME FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_NAME = '.prepare($table)." AND EXTRA LIKE '%AUTO_INCREMENT%' AND TABLE_SCHEMA = ".prepare($database->getDatabaseName()));
if (!empty($column)) {
$order[] = $column[0]['COLUMN_NAME'];
}
} }
} }
@ -250,6 +259,9 @@ class API extends \Util\Singleton
if (!is_array(self::$resources)) { if (!is_array(self::$resources)) {
$resources = []; $resources = [];
// Ignore dei warning
$resource = '';
// File nativi // File nativi
$files = glob(DOCROOT.'/modules/*/api/{retrieve,create,update,delete}.php', GLOB_BRACE); $files = glob(DOCROOT.'/modules/*/api/{retrieve,create,update,delete}.php', GLOB_BRACE);
@ -272,7 +284,7 @@ class API extends \Util\Singleton
$module = basename(dirname(dirname($operation))); $module = basename(dirname(dirname($operation)));
$kind = basename($operation, '.php'); $kind = basename($operation, '.php');
$resources[$kind] = (array) $resources[$kind]; $resources[$kind] = isset($resources[$kind]) ? (array) $resources[$kind] : [];
// Individuazione delle operazioni // Individuazione delle operazioni
$api = include $operation; $api = include $operation;
@ -371,6 +383,10 @@ class API extends \Util\Singleton
if ($_SERVER['REQUEST_METHOD'] == 'GET' && empty($request)) { if ($_SERVER['REQUEST_METHOD'] == 'GET' && empty($request)) {
$request = Filter::getGET(); $request = Filter::getGET();
} }
if (empty($request['token'])) {
$request['token'] = '';
}
} }
} }

View File

@ -55,7 +55,9 @@ class Auth extends \Util\Singleton
if (API::isAPIRequest()) { if (API::isAPIRequest()) {
$token = API::getRequest()['token']; $token = API::getRequest()['token'];
$id = $database->fetchArray('SELECT `id_utente` FROM `zz_tokens` WHERE `enabled` = 1 AND `token` = '.prepare($token))[0]['id_utente']; $user = $database->fetchArray('SELECT `id_utente` FROM `zz_tokens` WHERE `enabled` = 1 AND `token` = '.prepare($token));
$id = !empty($user) ? $user[0]['id_utente'] : null;
} }
// Controllo sulla sessione attiva // Controllo sulla sessione attiva
elseif (!empty($_SESSION['id_utente'])) { elseif (!empty($_SESSION['id_utente'])) {