Skip to content
Extraits de code Groupes Projets
Valider d8fd1860 rédigé par JamesRezo's avatar JamesRezo :tada: Validation de marcimat
Parcourir les fichiers

chore: rector config the right way

parent 3e0c1ca0
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
Tout étendre Masquer les modifications d'espaces
En ligne Côte à côte
Affichage de
avec 116 ajouts et 968 suppressions
.php_cs.cache supprimé 100644 → 0
+ 0
− 1
Voir le fichier @ 3e0c1ca0
Impossible d'afficher diff de source : il est trop volumineux. Options pour résoudre ce problème : voir le blob.
.php_cs.txt supprimé 100644 → 0
+ 0
− 824
Voir le fichier @ 3e0c1ca0
Ce diff est replié.
action/acceder_document.php
+ 4
− 7
Voir le fichier @ d8fd1860
......@@ -31,12 +31,11 @@ function action_acceder_document_dist() {
$status = false;
if (
str_contains($f, '../')
or preg_match(',^\w+://,', $f)
str_contains($f, '../') || preg_match(',^\w+://,', $f)
) {
$status = 403;
} else {
if (!file_exists($file) or !is_readable($file)) {
if (!file_exists($file) || !is_readable($file)) {
$status = 404;
} else {
$where = 'D.fichier=' . sql_quote(set_spip_doc($file))
......@@ -53,8 +52,7 @@ function action_acceder_document_dist() {
// ETag pour gerer le status 304
$ETag = md5($file . ': ' . filemtime($file));
if (
isset($_SERVER['HTTP_IF_NONE_MATCH'])
and $_SERVER['HTTP_IF_NONE_MATCH'] == $ETag
isset($_SERVER['HTTP_IF_NONE_MATCH']) && $_SERVER['HTTP_IF_NONE_MATCH'] == $ETag
) {
http_response_code(304); // Not modified
exit;
......@@ -97,8 +95,7 @@ function action_acceder_document_dist() {
// ou si c'est un nom bien connu d'Unix, le prendre
// sinon l'ignorer car certains navigateurs pataugent
if (
isset($doc['titre'])
and (preg_match('/^\w+[.]\w+$/', (string) $doc['titre']) or $doc['titre'] == 'Makefile')
isset($doc['titre']) && (preg_match('/^\w+[.]\w+$/', (string) $doc['titre']) || $doc['titre'] == 'Makefile')
) {
$options['attachment'] = $doc['titre'];
}
......
action/ajouter_documents.php
+ 10
− 12
Voir le fichier @ d8fd1860
......@@ -113,12 +113,11 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
}
$titrer = $file['titrer'] ?? _TITRER_DOCUMENTS;
$mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode);
$mode = ((isset($file['mode']) && $file['mode']) ? $file['mode'] : $mode);
include_spip('inc/modifier');
if (
isset($file['distant']) and $file['distant']
and !in_array($mode, ['choix', 'auto', 'image', 'document'])
isset($file['distant']) && $file['distant'] && !in_array($mode, ['choix', 'auto', 'image', 'document'])
) {
spip_log("document distant $source accepte sans verification, mode=$mode", 'medias' . _LOG_INFO_IMPORTANTE);
include_spip('inc/distant');
......@@ -130,7 +129,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// Documents distants : pas trop de verifications bloquantes, mais un test
// via une requete HEAD pour savoir si la ressource existe (non 404), si le
// content-type est connu, et si possible recuperer la taille, voire plus.
if (isset($file['distant']) and $file['distant']) {
if (isset($file['distant']) && $file['distant']) {
if (!tester_url_absolue($source)) {
return _T('medias:erreur_chemin_distant', ['nom' => $source]);
}
......@@ -142,7 +141,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
# $a['copie_locale'] est une copie locale du fichier
// voir si le document a besoin d'un nettoyage et le cas echeant relire ses infos apres
if (!empty($champs['copie_locale']) and file_exists($champs['copie_locale'])) {
if (!empty($champs['copie_locale']) && file_exists($champs['copie_locale'])) {
$res_sanitize = sanitizer_document($champs['copie_locale'], $champs['extension']);
$infos = renseigner_taille_dimension_image($champs['copie_locale'], $champs['extension']);
} else {
......@@ -152,7 +151,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// on ignore erreur eventuelle sur $infos car on est distant, ca ne marche pas forcement
if (is_array($infos)) {
foreach ($infos as $k => $v) {
if (!empty($v) or empty($champs[$k])) {
if (!empty($v) || empty($champs[$k])) {
$champs[$k] = $v;
}
}
......@@ -210,7 +209,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// lorsqu’une image arrive avec une mauvaise extension par rapport au mime type, adapter.
// Exemple : si extension .jpg mais le contenu est un png
if (!empty($infos['type_image']) and $infos['type_image'] !== $champs['extension']) {
if (!empty($infos['type_image']) && $infos['type_image'] !== $champs['extension']) {
spip_log(
'Image `' . $file['name'] . '` mauvaise extension. Correcte : ' . $infos['type_image'],
'medias' . _LOG_INFO
......@@ -280,7 +279,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// lier le parent si necessaire
// attention au cas particulier du site 0 utilisé pour le logo du site
if ($objet and (($id_objet = intval($id_objet)) or in_array($objet, ['site', 'rubrique']))) {
if ($objet && ($id_objet = intval($id_objet) || in_array($objet, ['site', 'rubrique']))) {
$champs['parents'][] = "$objet|$id_objet";
}
......@@ -374,8 +373,7 @@ function verifier_upload_autorise($source, $mode = '') {
$infos = ['fichier' => $source];
$res = false;
if (
preg_match(',\.([a-z0-9]+)(\?.*)?$,i', $source, $match)
and $ext = $match[1]
preg_match(',\.([a-z0-9]+)(\?.*)?$,i', $source, $match) && $ext = $match[1]
) {
$ext = corriger_extension(strtolower($ext));
if (
......@@ -400,7 +398,7 @@ function verifier_upload_autorise($source, $mode = '') {
$res['autozip'] = true;
}
}
if ($mode and $res) {
if ($mode && $res) {
// verifier en fonction du mode si une fonction est proposee
if ($verifier_document_mode = charger_fonction('verifier_document_mode_' . $mode, 'inc', true)) {
$check = $verifier_document_mode($infos); // true ou message d'erreur sous forme de chaine
......@@ -410,7 +408,7 @@ function verifier_upload_autorise($source, $mode = '') {
}
}
if (!$res or is_string($res)) {
if (!$res || is_string($res)) {
spip_log("Upload $source interdit ($res)", _LOG_INFO_IMPORTANTE);
}
......
action/changer_mode_document.php
+ 4
− 6
Voir le fichier @ d8fd1860
......@@ -20,7 +20,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
* @param string $mode
*/
function action_changer_mode_document_dist($id_document = null, $mode = null) {
if ($id_document === null or $mode === null) {
if ($id_document === null || $mode === null) {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
......@@ -33,9 +33,7 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) {
}
if (
$id_document
and include_spip('inc/autoriser')
and autoriser('modifier', 'document', $id_document)
$id_document && include_spip('inc/autoriser') && autoriser('modifier', 'document', $id_document)
) {
action_changer_mode_document_post($id_document, $mode);
}
......@@ -45,8 +43,8 @@ function action_changer_mode_document_post($id_document, $mode) {
// - id_document le doc a modifier
// - mode le mode a lui donner
if (
$id_document = intval($id_document)
and in_array($mode, ['vignette', 'image', 'document'])
($id_document = intval($id_document))
&& in_array($mode, ['vignette', 'image', 'document'])
) {
include_spip('action/editer_document');
document_modifier($id_document, ['mode' => $mode]);
......
action/copier_local.php
+ 2
− 4
Voir le fichier @ d8fd1860
......@@ -55,15 +55,13 @@ function action_copier_local_post($id_document) {
// sinon c'est une donnee moisie, on ne fait rien
include_spip('inc/distant');
if (
tester_url_absolue($source)
and valider_url_distante($source)
tester_url_absolue($source) && valider_url_distante($source)
) {
// on fait une copie locale en verifiant aussi l'URL finale qui a été récupérée
$fichier = copie_locale($source, 'auto', null, null, 'valider_url_distante');
// on revalide la source *apres* copie car si elle est controlee par un serveur dns malicieux elle peut etre changeante
if (
$fichier
and valider_url_distante($source)
$fichier && valider_url_distante($source)
) {
$fichier = _DIR_RACINE . $fichier;
$files = [];
......
action/desordonner_liens_documents.php
+ 2
− 3
Voir le fichier @ d8fd1860
......@@ -38,12 +38,11 @@ function action_desordonner_liens_documents_dist($arg = null) {
[$id_objet, $objet, $document] = $arg;
if (
$id_objet = intval($id_objet)
and autoriser('desordonnerdocuments', $objet, $id_objet)
($id_objet = intval($id_objet)) && autoriser('desordonnerdocuments', $objet, $id_objet)
) {
desordonner_liens_documents($document, $objet, $id_objet);
} else {
spip_log("Interdit de désordonner les documents sur : $objet $id_objet", 'spip');
spip_logger()->info("Interdit de désordonner les documents sur : $objet $id_objet");
}
}
......
action/dissocier_document.php
+ 11
− 9
Voir le fichier @ d8fd1860
......@@ -45,18 +45,20 @@ function action_dissocier_document_dist($arg = null) {
}
$suppr = $check = false;
if (count($arg) > 3 and $arg[3] == 'suppr') {
if (count($arg) > 3 && $arg[3] == 'suppr') {
$suppr = true;
}
if (count($arg) > 4 and $arg[4] == 'safe') {
if (count($arg) > 4 && $arg[4] == 'safe') {
$check = true;
}
if (
$id_objet = intval($id_objet)
and (
($id_objet < 0 and $id_objet == -$GLOBALS['visiteur_session']['id_auteur'])
or autoriser('dissocierdocuments', $objet, $id_objet)
)
($id_objet = intval(
$id_objet
)) && ($id_objet < 0 && $id_objet == -$GLOBALS['visiteur_session']['id_auteur'] || autoriser(
'dissocierdocuments',
$objet,
$id_objet
))
) {
dissocier_document($document, $objet, $id_objet, $suppr, $check);
} else {
......@@ -78,7 +80,7 @@ function action_dissocier_document_dist($arg = null) {
* @return bool
*/
function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = false, $check = false) {
if (!$id_document = intval($id_document)) {
if (!($id_document = intval($id_document))) {
return false;
}
......@@ -130,7 +132,7 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
// On supprime ensuite s'il est orphelin
// et si demande
// ici on ne bloque pas la suppression d'un document rattache a un autre
if ($supprime and !sql_countsel('spip_documents_liens', "objet!='document' AND id_document=" . $id_document)) {
if ($supprime && !sql_countsel('spip_documents_liens', "objet!='document' AND id_document=" . $id_document)) {
$supprimer_document = charger_fonction('supprimer_document', 'action');
return $supprimer_document($id_document);
......
action/editer_document.php
+ 10
− 17
Voir le fichier @ d8fd1860
......@@ -137,12 +137,12 @@ function document_modifier($id_document, $set = null) {
// nettoyer l'ancien fichier si necessaire
if (
isset($champs['fichier']) // un plugin a pu interdire la modif du fichier en virant le champ
and $champs['fichier']
and $ancien_fichier // on avait bien note le nom du fichier avant la modif
and $ancien_fichier !== $champs['fichier'] // et il a ete modifie
and !tester_url_absolue($ancien_fichier)
and @file_exists($f = get_spip_doc($ancien_fichier))
isset($champs['fichier'])
&& $champs['fichier']
&& $ancien_fichier
&& $ancien_fichier !== $champs['fichier']
&& !tester_url_absolue($ancien_fichier)
&& @file_exists($f = get_spip_doc($ancien_fichier))
) {
spip_unlink($f);
}
......@@ -202,9 +202,7 @@ function document_instituer($id_document, $champs = []) {
}
if (
$date_publication !== null
and empty($champs['date_publication'])
and $date_publication != $date_publication_ancienne
$date_publication !== null && empty($champs['date_publication']) && $date_publication != $date_publication_ancienne
) {
$champs['date_publication'] = $date_publication;
}
......@@ -286,8 +284,7 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
foreach ($parents as $p) {
$p = explode('|', (string) $p);
if (
preg_match('/^[a-z0-9_]+$/i', $objet = $p[0])
and (($p[1] = intval($p[1])) or in_array($objet, ['site', 'rubrique']))
preg_match('/^[a-z0-9_]+$/i', $objet = $p[0]) && ($p[1] = intval($p[1]) || in_array($objet, ['site', 'rubrique']))
) { // securite
$objets_parents[$p[0]][] = $p[1];
}
......@@ -300,7 +297,7 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
// si ce n'est pas un ajout, il faut supprimer les liens actuels qui ne sont pas dans $objets_parents
if (!$ajout) {
foreach ($liens as $k => $lien) {
if (!isset($objets_parents[$lien['objet']]) or !in_array($lien['id_objet'], $objets_parents[$lien['objet']])) {
if (!isset($objets_parents[$lien['objet']]) || !in_array($lien['id_objet'], $objets_parents[$lien['objet']])) {
if (autoriser('dissocierdocuments', $lien['objet'], $lien['id_objet'])) {
objet_dissocier(['document' => $id_document], [$lien['objet'] => $lien['id_objet']]);
}
......@@ -315,11 +312,7 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
foreach ($objets_parents as $objet => $ids) {
foreach ($ids as $k => $id) {
if (
(
isset($deja_parents[$objet])
and in_array($id, $deja_parents[$objet])
)
or !autoriser('associerdocuments', $objet, $id)
isset($deja_parents[$objet]) && in_array($id, $deja_parents[$objet]) || !autoriser('associerdocuments', $objet, $id)
) {
unset($objets_parents[$objet][$k]);
}
......
action/ordonner_liens_documents.php
+ 1
− 1
Voir le fichier @ d8fd1860
......@@ -33,7 +33,7 @@ function action_ordonner_liens_dist() {
// ordre des éléments
$ordre = _request('ordre');
if (!$objet or !$objet_lie or !$id_objet_lie or !$ordre or !is_array($ordre) or !objet_associable($objet)) {
if (!$objet || !$objet_lie || !$id_objet_lie || !$ordre || !is_array($ordre) || !objet_associable($objet)) {
return envoyer_json_erreur(_T('medias:erreur_objet_absent') . ' ' . _T('medias:erreur_deplacement_impossible'));
}
......
action/tourner.php
+ 3
− 3
Voir le fichier @ d8fd1860
......@@ -24,7 +24,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
* angle de rotation en degre>0
*/
function action_tourner_dist($id_document = null, $angle = null) {
if ($id_document === null or $angle === null) {
if ($id_document === null || $angle === null) {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
......@@ -35,7 +35,7 @@ function action_tourner_dist($id_document = null, $angle = null) {
[$id_document, $angle] = $r;
}
}
if ($id_document and autoriser('modifier', 'document', $id_document)) {
if ($id_document && autoriser('modifier', 'document', $id_document)) {
action_tourner_post($id_document, $angle);
}
}
......@@ -96,7 +96,7 @@ function action_tourner_post($id_document, $angle) {
}
// succes !
if ($largeur > 0 and $hauteur > 0) {
if ($largeur > 0 && $hauteur > 0) {
$set = [
'fichier' => set_spip_doc($dest),
'largeur' => $largeur,
......
base/medias.php
+ 2
− 2
Voir le fichier @ d8fd1860
......@@ -276,12 +276,12 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') {
// type de media
$media = 'file';
if (preg_match(',^image/,', (string) $type_mime) or in_array($type_mime, ['application/illustrator'])) {
if (preg_match(',^image/,', (string) $type_mime) || in_array($type_mime, ['application/illustrator'])) {
$media = 'image';
} elseif (preg_match(',^audio/,', (string) $type_mime)) {
$media = 'audio';
} elseif (
preg_match(',^video/,', (string) $type_mime) or in_array(
preg_match(',^video/,', (string) $type_mime) || in_array(
$type_mime,
['application/ogg', 'application/x-shockwave-flash', 'application/mp4']
)
......
formulaires/editer_document.php
+ 21
− 21
Voir le fichier @ d8fd1860
......@@ -38,7 +38,7 @@ function formulaires_editer_document_charger_dist(
$hidden
);
if (intval($id_document) and !autoriser('modifier', 'document', intval($id_document))) {
if (intval($id_document) && !autoriser('modifier', 'document', intval($id_document))) {
$valeurs['editable'] = '';
}
......@@ -47,7 +47,7 @@ function formulaires_editer_document_charger_dist(
$valeurs['_hidden'] = '';
$parents = sql_allfetsel('objet,id_objet', 'spip_documents_liens', 'id_document=' . intval($id_document));
foreach ($parents as $p) {
if (in_array($p['objet'], ['article', 'rubrique']) and $p['id_objet'] > 0) {
if (in_array($p['objet'], ['article', 'rubrique']) && $p['id_objet'] > 0) {
$valeurs['parents'][] = $p['objet'] . '|' . $p['id_objet'];
} else {
$valeurs['_hidden'] .= "<input type='hidden' name='parents[]' value='" . $p['objet'] . '|' . $p['id_objet'] . "' />";
......@@ -83,11 +83,9 @@ function formulaires_editer_document_charger_dist(
if ($valeurs['distant'] !== 'oui') {
include_spip('inc/renseigner_document');
$infos = renseigner_taille_dimension_image(get_spip_doc($valeurs['fichier']), $valeurs['extension']);
if ($infos and is_array($infos) and isset($infos['taille'])) {
if ($infos && is_array($infos) && isset($infos['taille'])) {
if (
$infos['taille'] != $valeurs['taille']
or ($infos['type_image'] && ($infos['largeur'] != $valeurs['largeur']))
or ($infos['type_image'] && ($infos['hauteur'] != $valeurs['hauteur']))
$infos['taille'] != $valeurs['taille'] || $infos['type_image'] && $infos['largeur'] != $valeurs['largeur'] || $infos['type_image'] && $infos['hauteur'] != $valeurs['hauteur']
) {
$valeurs['_taille_modif'] = $infos['taille'];
$valeurs['_largeur_modif'] = $infos['largeur'];
......@@ -135,7 +133,7 @@ function formulaires_editer_document_verifier_dist(
$erreurs = formulaires_editer_objet_verifier('document', $id_document, is_numeric($id_document) ? [] : ['titre']);
// verifier l'upload si on a demande a changer le document
if (_request('joindre_upload') or _request('joindre_ftp') or _request('joindre_distant')) {
if (_request('joindre_upload') || _request('joindre_ftp') || _request('joindre_distant')) {
if (_request('copier_local')) {
} else {
$verifier = charger_fonction('verifier', 'formulaires/joindre_document');
......@@ -146,8 +144,14 @@ function formulaires_editer_document_verifier_dist(
// On ne vérifie la date que si on avait le droit de la modifier
if (lire_config('documents_date') == 'oui') {
if (
!$date = recup_date(_request('saisie_date') . ' ' . _request('saisie_heure') . ':00')
or !($date = mktime($date[3], $date[4], 0, $date[1], $date[2], $date[0]))
!($date = recup_date(_request('saisie_date') . ' ' . _request('saisie_heure') . ':00')) || !($date = mktime(
$date[3],
$date[4],
0,
$date[1],
$date[2],
$date[0]
))
) {
$erreurs['saisie_date'] = _T('medias:format_date_incorrect');
} else {
......@@ -177,7 +181,7 @@ function formulaires_editer_document_traiter_dist(
// verifier les infos de taille et dimensions sur les fichiers locaux
// cas des maj de fichier directes par ftp
foreach (['taille', 'largeur', 'hauteur'] as $c) {
if (($v = _request("_{$c}_modif")) and !_request($c)) {
if (($v = _request("_{$c}_modif")) && !_request($c)) {
set_request($c, $v);
}
}
......@@ -196,11 +200,7 @@ function formulaires_editer_document_traiter_dist(
$autoclose = '<script>if (window.jQuery) jQuery.modalboxclose();</script>';
$mode = sql_getfetsel('mode', 'spip_documents', "id_document = $id_document");
if (
_request('copier_local')
or _request('joindre_upload')
or _request('joindre_ftp')
or _request('joindre_distant')
or _request('joindre_zip')
_request('copier_local') || _request('joindre_upload') || _request('joindre_ftp') || _request('joindre_distant') || _request('joindre_zip')
) {
$autoclose = '';
if (_request('copier_local')) {
......@@ -215,9 +215,9 @@ function formulaires_editer_document_traiter_dist(
} else {
// liberer le nom de l'ancien fichier pour permettre le remplacement par un fichier du meme nom
if (
$ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document))
and !tester_url_absolue($ancien_fichier)
and @file_exists($rename = get_spip_doc($ancien_fichier))
($ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document)))
&& !tester_url_absolue($ancien_fichier)
&& @file_exists($rename = get_spip_doc($ancien_fichier))
) {
@rename($rename, "$rename--.old");
}
......@@ -240,13 +240,13 @@ function formulaires_editer_document_traiter_dist(
// regarder si une demande de rotation a eu lieu
// c'est un bouton image, dont on a pas toujours le name en request, on fait avec
$angle = 0;
if (_request('tournerL90') or _request('tournerL90_x')) {
if (_request('tournerL90') || _request('tournerL90_x')) {
$angle = -90;
}
if (_request('tournerR90') or _request('tournerR90_x')) {
if (_request('tournerR90') || _request('tournerR90_x')) {
$angle = 90;
}
if (_request('tourner180') or _request('tourner180_x')) {
if (_request('tourner180') || _request('tourner180_x')) {
$angle = 180;
}
if ($angle) {
......
formulaires/illustrer_document.php
+ 4
− 6
Voir le fichier @ d8fd1860
......@@ -21,8 +21,7 @@ function formulaires_illustrer_document_charger_dist($id_document) {
'id_document=' . intval($id_document)
);
if (
!$valeurs
or in_array($valeurs['mode'], ['logoon', 'logooff'])
!$valeurs || in_array($valeurs['mode'], ['logoon', 'logooff'])
) {
return ['editable' => false, 'id' => $id_document];
}
......@@ -48,7 +47,7 @@ function formulaires_illustrer_document_charger_dist($id_document) {
}
$valeurs['_pipeline'] = ['editer_contenu_objet', ['type' => 'illustrer_document', 'id' => $id_document]];
if (intval($id_document) and !autoriser('modifier', 'document', intval($id_document))) {
if (intval($id_document) && !autoriser('modifier', 'document', intval($id_document))) {
$valeurs['editable'] = '';
}
......@@ -72,7 +71,7 @@ function formulaires_illustrer_document_traiter_dist($id_document) {
$res = ['editable' => true];
if (_request('supprimer')) {
$supprimer_document = charger_fonction('supprimer_document', 'action');
if ($id_vignette and $supprimer_document($id_vignette)) {
if ($id_vignette && $supprimer_document($id_vignette)) {
$res['message_ok'] = _T('medias:vignette_supprimee');
} else {
$res['message_erreur'] = _T('medias:erreur_suppression_vignette');
......@@ -86,8 +85,7 @@ function formulaires_illustrer_document_traiter_dist($id_document) {
$ajoute = $ajouter_documents($id_vignette, $files, '', 0, 'vignette');
if (
is_numeric(reset($ajoute))
and $id_vignette = reset($ajoute)
is_numeric(reset($ajoute)) && ($id_vignette = reset($ajoute))
) {
include_spip('action/editer_document');
document_modifier($id_document, ['id_vignette' => $id_vignette, 'mode' => 'document']);
......
formulaires/joindre_document.php
+ 11
− 15
Voir le fichier @ d8fd1860
......@@ -38,8 +38,7 @@ function joindre_determiner_mode($mode, $id_document, $objet) {
if (!in_array($mode, ['choix', 'document', 'image'])) {
$mode = 'choix';
if (
$objet
and !in_array(table_objet_sql($objet), explode(',', (string) $GLOBALS['meta']['documents_objets']))
$objet && !in_array(table_objet_sql($objet), explode(',', (string) $GLOBALS['meta']['documents_objets']))
) {
$mode = 'image';
}
......@@ -58,7 +57,7 @@ function joindre_determiner_mode($mode, $id_document, $objet) {
*/
function joindre_document_galerie_valide(?string $galerie): bool {
$galeries = [];
if (isset($GLOBALS['medias_deprecated_liste_galeries']) and is_array($GLOBALS['medias_deprecated_liste_galeries'])) {
if (isset($GLOBALS['medias_deprecated_liste_galeries']) && is_array($GLOBALS['medias_deprecated_liste_galeries'])) {
$galeries = $GLOBALS['medias_deprecated_liste_galeries'];
}
return in_array($galerie, $galeries);
......@@ -123,9 +122,7 @@ function formulaires_joindre_document_charger_dist(
# regarder si un choix d'upload FTP est vraiment possible
if (
$valeurs['proposer_ftp']
and test_espace_prive() # ??
and ($mode != 'image') and ($mode != 'vignette') # si c'est pour un document
$valeurs['proposer_ftp'] && test_espace_prive() && $mode != 'image' && $mode != 'vignette' # si c'est pour un document
) {
include_spip('inc/documents');
if ($dir = determine_upload('documents')) {
......@@ -133,22 +130,22 @@ function formulaires_joindre_document_charger_dist(
$valeurs['_options_upload_ftp'] = joindre_options_upload_ftp($dir, $mode);
// s'il n'y en a pas, on affiche un message d'aide
// en mode document, mais pas en mode image
if ($valeurs['_options_upload_ftp'] or ($mode == 'document' or $mode == 'choix')) {
if ($valeurs['_options_upload_ftp'] || ($mode == 'document' || $mode == 'choix')) {
$valeurs['_dir_upload_ftp'] = '<b>' . joli_repertoire($dir) . '</b>';
}
}
}
// On ne propose le FTP que si on a des choses a afficher
$valeurs['proposer_ftp'] = ($valeurs['_options_upload_ftp'] or $valeurs['_dir_upload_ftp']);
$valeurs['proposer_ftp'] = ($valeurs['_options_upload_ftp'] || $valeurs['_dir_upload_ftp']);
/** @deprecated 4.0 (SPIP 4.1). Utiliser ajaxReload('documents') après upload */
if ($galerie and joindre_document_galerie_valide($galerie)) {
if ($galerie && joindre_document_galerie_valide($galerie)) {
# passer optionnellement une galerie jointe au form
# plus utilise nativement, on prefere la mise a jour
# apres upload par ajaxReload('documents')
$valeurs['_galerie'] = $galerie;
}
if ($objet and $id_objet) {
if ($objet && $id_objet) {
$valeurs['id_objet'] = $id_objet;
$valeurs['objet'] = $objet;
$valeurs['refdoc_joindre'] = '';
......@@ -216,8 +213,7 @@ function formulaires_joindre_document_verifier_dist(
// regarder si on a eu une erreur sur l'upload d'un fichier
foreach ($files as $file) {
if (
isset($file['error'])
and $test = joindre_upload_error($file['error'])
isset($file['error']) && $test = joindre_upload_error($file['error'])
) {
if (is_string($test)) {
$erreurs['message_erreur'] = $test;
......@@ -254,7 +250,7 @@ function formulaires_joindre_document_verifier_dist(
}
}
if (count($erreurs) and defined('_TMP_DIR')) {
if (count($erreurs) && defined('_TMP_DIR')) {
effacer_repertoire_temporaire(_TMP_DIR);
}
}
......@@ -376,7 +372,7 @@ function formulaires_joindre_document_traiter_dist(
$res['redirect'] = "#doc$ancre";
}
}
if (count($sel) or isset($res['message_ok'])) {
if (count($sel) || isset($res['message_ok'])) {
$callback = '';
if ($ancre) {
$callback .= "jQuery('#doc$ancre a.editbox').eq(0).focus();";
......@@ -415,7 +411,7 @@ function joindre_options_upload_ftp($dir, $mode = 'document') {
$exts = $dirs = $texte_upload = [];
// en mode "charger une image", ne proposer que les inclus
$inclus = ($mode == 'image' or $mode == 'vignette')
$inclus = ($mode == 'image' || $mode == 'vignette')
? " AND inclus='image'"
: '';
......
inc/choisir_mode_document.php
+ 3
− 4
Voir le fichier @ d8fd1860
......@@ -33,12 +33,12 @@ if (!defined('_ECRIRE_INC_VERSION')) {
function inc_choisir_mode_document($infos, $type_inclus_image, $objet) {
// si ce n'est pas une image, c'est forcement un document
if (!$infos['type_image'] or !$type_inclus_image) {
if (!$infos['type_image'] || !$type_inclus_image) {
return 'document';
}
// si on a pas le droit d'ajouter de document a l'objet, c'est donc un mode image
if ($objet and isset($GLOBALS['meta']["documents_$objet"]) and ($GLOBALS['meta']["documents_$objet"] == 'non')) {
if ($objet && isset($GLOBALS['meta']["documents_$objet"]) && $GLOBALS['meta']["documents_$objet"] == 'non') {
return 'image';
}
......@@ -56,8 +56,7 @@ function inc_choisir_mode_document($infos, $type_inclus_image, $objet) {
}
if (
$infos['largeur'] > 0
and $infos['largeur'] < _LARGEUR_MODE_IMAGE
$infos['largeur'] > 0 && $infos['largeur'] < _LARGEUR_MODE_IMAGE
) {
return 'image';
}
......
inc/determiner_statut_document.php
+ 7
− 11
Voir le fichier @ d8fd1860
......@@ -41,21 +41,17 @@ function inc_determiner_statut_document($id_document, $statut_ancien, $date_publ
while ($row = sql_fetch($res)) {
if (
// si ce n'est pas un logo
!in_array($row['mode'], ['logoon', 'logooff'])
and (
// cas particulier des rubriques qui sont publiees des qu'elles contiennent un document !
$row['objet'] == 'rubrique'
// ou si objet publie selon sa declaration
or objet_test_si_publie($row['objet'], $row['id_objet'])
)
!in_array($row['mode'], ['logoon', 'logooff']) && ($row['objet'] == 'rubrique' || objet_test_si_publie(
$row['objet'],
$row['id_objet']
))
) {
$statut = 'publie';
$date_publication = 0;
continue;
} // si pas publie, et article, il faut checker la date de post-publi eventuelle
elseif (
$row['objet'] == 'article'
and $row2 = sql_fetsel(
$row['objet'] == 'article' && $row2 = sql_fetsel(
'date',
'spip_articles',
'id_article=' . intval($row['id_objet']) . " AND statut='publie'"
......@@ -66,10 +62,10 @@ function inc_determiner_statut_document($id_document, $statut_ancien, $date_publ
}
}
$date_publication = date('Y-m-d H:i:s', $date_publication);
if ($statut == 'publie' and $statut_ancien == 'publie' and $date_publication == $date_publication_ancienne) {
if ($statut == 'publie' && $statut_ancien == 'publie' && $date_publication == $date_publication_ancienne) {
return false;
}
if ($statut != 'publie' and $statut_ancien != 'publie' and $statut_ancien != '0') {
if ($statut != 'publie' && $statut_ancien != 'publie' && $statut_ancien != '0') {
return false;
}
......
inc/documents.php
+ 6
− 6
Voir le fichier @ d8fd1860
......@@ -51,7 +51,7 @@ function contenu_document($arg, $charset = '') {
include_spip('inc/charsets');
if ($charset !== 'auto') {
$r = importer_charset($r, $charset);
} elseif ($GLOBALS['meta']['charset'] == 'utf-8' and !is_utf8($r)) {
} elseif ($GLOBALS['meta']['charset'] == 'utf-8' && !is_utf8($r)) {
$r = importer_charset($r, CHARSET_JOINT);
}
}
......@@ -131,7 +131,7 @@ function vignette_automatique($img, $doc, $lien, $x = 0, $y = 0, $align = '', $c
// on appelle image_reduire independamment de la presence ou non
// des librairies graphiques
// la fonction sait se debrouiller et faire de son mieux dans tous les cas
if ($x or $y) {
if ($x || $y) {
$img = image_reduire($img, $x, $y);
}
$img = inserer_attribut($img, 'alt', '');
......@@ -172,10 +172,10 @@ function vignette_automatique($img, $doc, $lien, $x = 0, $y = 0, $align = '', $c
*/
function image_du_document($document, $connect = null) {
if (
$e = $document['extension'] ?? null
and in_array($e, formats_image_acceptables())
and (!test_espace_prive() or $GLOBALS['meta']['creer_preview'] == 'oui')
and $document['fichier']
$e = $document['extension'] ?? null && in_array(
$e,
formats_image_acceptables()
) && (!test_espace_prive() || $GLOBALS['meta']['creer_preview'] == 'oui') && $document['fichier']
) {
include_spip('inc/quete');
if ($document['distant'] == 'oui') {
......
inc/joindre_document.php
+ 14
− 15
Voir le fichier @ d8fd1860
......@@ -53,7 +53,7 @@ function joindre_trouver_fichier_envoye() {
return $files;
} elseif (_request('joindre_distant')) {
$path = _request('url');
if (!strlen((string) $path) or $path == 'http://') {
if (!strlen((string) $path) || $path == 'http://') {
return _T('medias:erreur_indiquez_un_fichier');
}
include_spip('inc/distant');
......@@ -83,7 +83,7 @@ function joindre_trouver_fichier_envoye() {
include_spip('inc/documents');
include_spip('inc/actions');
$upload = determine_upload();
if ($path != '/' and $path != './') {
if ($path != '/' && $path != './') {
$upload .= $path;
}
......@@ -107,13 +107,11 @@ function joindre_trouver_fichier_envoye() {
return $files;
} elseif (_request('joindre_zip') and $token_zip = _request('chemin_zip')) {
} elseif (_request('joindre_zip') && $token_zip = _request('chemin_zip')) {
$zip_to_clean = (isset($GLOBALS['visiteur_session']['zip_to_clean']) ?
unserialize($GLOBALS['visiteur_session']['zip_to_clean']) : []);
if (
!$zip_to_clean
or !isset($zip_to_clean[$token_zip])
or !$path = $zip_to_clean[$token_zip]
!$zip_to_clean || !isset($zip_to_clean[$token_zip]) || !$path = $zip_to_clean[$token_zip]
) {
return _T('avis_operation_impossible');
}
......@@ -131,7 +129,7 @@ function joindre_trouver_fichier_envoye() {
}
// si le zip doit aussi etre conserve, l'ajouter
if (_request('options_upload_zip') == 'upload' or _request('options_deballe_zip_conserver')) {
if (_request('options_upload_zip') == 'upload' || _request('options_deballe_zip_conserver')) {
$files[] = [
'name' => basename((string) $path),
'tmp_name' => $path,
......@@ -155,7 +153,7 @@ function joindre_trouver_http_post_files($name = null) {
$files = [];
if (is_array($post)) {
foreach ($post as $input_name => $file) {
if ($name and $input_name !== $name) {
if ($name && $input_name !== $name) {
continue;
}
if (is_array($file['name'])) {
......@@ -314,7 +312,7 @@ function joindre_decrire_contenu_zip($infos) {
if (accepte_fichier_upload($f = $file['filename'])) {
$fichiers[$f] = $file;
} else { // pas de message pour les dossiers et fichiers caches
if (!str_ends_with((string) $f, '/') and !str_starts_with(basename((string) $f), '.')) {
if (!str_ends_with((string) $f, '/') && !str_starts_with(basename((string) $f), '.')) {
$erreurs[] = _T('medias:erreur_upload_type_interdit', ['nom' => $f]);
}
}
......@@ -334,8 +332,7 @@ function joindre_deballer_lister_zip($path, $tmp_dir) {
$archive = new SpipArchiver($path);
if (
$infos = $archive->informer()
and $contenu = joindre_decrire_contenu_zip($infos)
($infos = $archive->informer()) && ($contenu = joindre_decrire_contenu_zip($infos))
) {
$fichiers = reset($contenu);
$fichiers = array_column($fichiers, 'filename');
......@@ -370,8 +367,11 @@ if (!function_exists('fixer_extension_document')) {
$extension = '';
$name = $doc['name'];
if (
preg_match(',\.([^.]+)$,', $name, $r)
and $t = sql_fetsel('extension', 'spip_types_documents', 'extension=' . sql_quote(corriger_extension($r[1])))
preg_match(',\.([^.]+)$,', $name, $r) && ($t = sql_fetsel(
'extension',
'spip_types_documents',
'extension=' . sql_quote(corriger_extension($r[1]))
))
) {
$extension = $t['extension'];
$name = preg_replace(',\.[^.]*$,', '', $doc['name']) . '.' . $extension;
......@@ -394,8 +394,7 @@ if (!function_exists('fixer_extension_document')) {
*/
function accepte_fichier_upload($f) {
if (
!preg_match(',.*__MACOSX/,', $f)
and !preg_match(',^\.,', basename($f))
!preg_match(',.*__MACOSX/,', $f) && !preg_match(',^\.,', basename($f))
) {
include_spip('action/ajouter_documents');
$ext = corriger_extension((strtolower(substr(strrchr($f, '.'), 1))));
......
inc/marquer_doublons_doc.php
+ 1
− 1
Voir le fichier @ d8fd1860
......@@ -86,7 +86,7 @@ function inc_marquer_doublons_doc_dist(
// Récupérer chaque champ impactant qui existe dans la table de l'objet et qui nous manque
foreach ($GLOBALS['medias_liste_champs'] as $champ) {
if (isset($desc['field'][$champ]) and !isset($champs[$champ])) {
if (isset($desc['field'][$champ]) && !isset($champs[$champ])) {
$absents[] = $champ;
}
}
......
0% ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site