diff --git a/ecrire/inc/math.php b/ecrire/inc/math.php
index 01470ef6bf3e641e04f2fd58298e9797c56ad552..df96768caf5879dbd6d42f186eba9fd990238167 100644
--- a/ecrire/inc/math.php
+++ b/ecrire/inc/math.php
@@ -41,13 +41,11 @@ function produire_image_math($tex) {
$fichier = $dir_tex . md5(trim($tex)) . $ext;
- if (!@file_exists($fichier)) {
- // Aller chercher l'image sur le serveur
- if ($server) {
- spip_log($url = $server . '?' . rawurlencode($tex));
- include_spip('inc/distant');
- recuperer_url($url, ['file' => $fichier]);
- }
+ // Aller chercher l'image sur le serveur
+ if (!@file_exists($fichier) && $server) {
+ spip_log($url = $server . '?' . rawurlencode($tex));
+ include_spip('inc/distant');
+ recuperer_url($url, ['file' => $fichier]);
}
diff --git a/ecrire/inc/minipres.php b/ecrire/inc/minipres.php
index af74b975c1e368a2f8d8af81ae76f7c0d210419a..1d3f9dbc2f60543a94770d32c5eeed566061bc8e 100644
--- a/ecrire/inc/minipres.php
+++ b/ecrire/inc/minipres.php
@@ -102,7 +102,7 @@ function minipres($titre = '', $corps = '', $options = []) {
// compat signature old
// minipres($titre='', $corps="", $onload='', $all_inline = false)
$args = func_get_args();
- if (isset($args[2]) and is_string($args[2])) {
+ if (isset($args[2]) && is_string($args[2])) {
$options = ['onload' => $args[2]];
}
if (isset($args[3])) {
diff --git a/ecrire/inc/nfslock.php b/ecrire/inc/nfslock.php
index e9e37a043137d42363cfbc0c1b8d61e01669491f..e66921892c08b0ad09958b20eb489798f75173eb 100644
--- a/ecrire/inc/nfslock.php
+++ b/ecrire/inc/nfslock.php
@@ -155,7 +155,7 @@ function spip_nfslock($fichier, $max_age = 0) {
*/
$old_stat = lstat($lock_file);
- if (@fputs($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
+ if (@fwrite($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
break;
} /* something bogus is going on */
@@ -243,11 +243,7 @@ function spip_nfsunlock($fichier, $birth, $max_age = 0, $test = false) {
$tpath = _DIR_TMP . "stime.$id";
$tmpfd = @fopen($tpath, 'w');
- if (
- (!$tmpfd)
- or (@fputs($tmpfd, 'zz', 2) != 2)
- or !($our_tmp = fstat($tmpfd))
- ) {
+ if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
/* The open failed, or we can't write the file, or we can't stat it */
@fclose($tmpfd);
spip_unlink($tpath);
diff --git a/ecrire/inc/prepare_recherche.php b/ecrire/inc/prepare_recherche.php
index f685cbc7f08fd7cae64c33cdbc21f60c72a89fee..c8cf9e5bdfd2c6b41759e4b1b3ac01d1dee7e7d9 100644
--- a/ecrire/inc/prepare_recherche.php
+++ b/ecrire/inc/prepare_recherche.php
@@ -65,12 +65,12 @@ function inc_prepare_recherche_dist(
// si recherche n'est pas dans le contexte, on va prendre en globals
// ca permet de faire des inclure simple.
- if (!isset($recherche) and isset($GLOBALS['recherche'])) {
+ if (!isset($recherche) && isset($GLOBALS['recherche'])) {
$recherche = $GLOBALS['recherche'];
}
// traiter le cas {recherche?}
- if ($cond and !strlen($recherche)) {
+ if ($cond && !strlen($recherche)) {
return [
'0 as points' /* as points */, /* where */
''
@@ -93,10 +93,7 @@ function inc_prepare_recherche_dist(
'',
'0,1'
);
- if (
- !$row
- or (defined('_VAR_MODE') and _VAR_MODE == 'recalcul')
- ) {
+ if (!$row || (defined('_VAR_MODE') && _VAR_MODE == 'recalcul')) {
$rechercher = true;
}
}
@@ -187,7 +184,7 @@ function inc_prepare_recherche_dist(
*/
function generer_select_where_explicites($table, $primary, $rows, $serveur) {
# calculer le {id_article IN()} et le {... as points}
- if (!count($rows)) {
+ if ($rows === []) {
return ["''", '0=1'];
} else {
$listes_ids = [];
diff --git a/ecrire/inc/preselectionner_parent_nouvel_objet.php b/ecrire/inc/preselectionner_parent_nouvel_objet.php
index 523b2fe05f122a6d243faa0e28ff05be6694a0ec..5ead975ffff300adfaf99db9a9f34a189d4c26c0 100644
--- a/ecrire/inc/preselectionner_parent_nouvel_objet.php
+++ b/ecrire/inc/preselectionner_parent_nouvel_objet.php
@@ -41,7 +41,7 @@ function inc_preselectionner_parent_nouvel_objet_dist($objet, $row) {
$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
} elseif (
is_int(_AUTO_SELECTION_RUBRIQUE)
- and sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . intval(_AUTO_SELECTION_RUBRIQUE))
+ && sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
) {
$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
} else {
@@ -54,7 +54,7 @@ function inc_preselectionner_parent_nouvel_objet_dist($objet, $row) {
$id_rubrique = '';
// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
- while (!$id_rubrique and $row_rub = sql_fetch($res)) {
+ while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
$id_rubrique = $row_rub['id_rubrique'];
}
diff --git a/ecrire/inc/presenter_enfants.php b/ecrire/inc/presenter_enfants.php
index 8aa38a9da3afcf11ce59593fd331bff7e0b8dcd3..db11cecf12349341a87d49793d37127527bbe000 100644
--- a/ecrire/inc/presenter_enfants.php
+++ b/ecrire/inc/presenter_enfants.php
@@ -29,7 +29,7 @@ include_spip('inc/presentation');
* Un tableau des sous rubriques
*/
function enfant_rub($collection, $debut = 0, $limite = 500) {
- $voir_logo = (isset($GLOBALS['meta']['image_process']) and $GLOBALS['meta']['image_process'] != 'non');
+ $voir_logo = (isset($GLOBALS['meta']['image_process']) && $GLOBALS['meta']['image_process'] != 'non');
$logo = '';
if ($voir_logo) {
@@ -42,7 +42,7 @@ function enfant_rub($collection, $debut = 0, $limite = 500) {
$result = sql_select(
'id_rubrique, id_parent, titre, descriptif, lang',
'spip_rubriques',
- 'id_parent=' . intval($collection),
+ 'id_parent=' . (int) $collection,
'',
'0+titre,titre',
$debut == -1 ? '' : "$debut,$limite"
@@ -63,23 +63,17 @@ function enfant_rub($collection, $debut = 0, $limite = 500) {
$lang_dir = lang_dir($row['lang']);
$descriptif = propre($row['descriptif']);
- if ($voir_logo) {
- if ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on')) {
- [$fid, $dir, $nom, $format] = $logo;
- $logo = image_recadre_avec_fallback("<img src='$fid' alt='' />", 70, 70);
- if ($logo) {
- $logo = wrap(inserer_attribut($logo, 'class', 'logo'), '<span class="logo-carre">');
- }
- }
- }
-
- $lib_bouton = (!acces_restreint_rubrique($id_rubrique) ? '' :
- http_img_pack(
- 'auteur-0minirezo-16.png',
- '',
- " width='16' height='16'",
- _T('image_administrer_rubrique')
- )) .
+ if ($voir_logo && ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on'))) {
+ [$fid, $dir, $nom, $format] = $logo;
+ $logo = image_recadre_avec_fallback("<img src='$fid' alt='' />", 70, 70);
+ if ($logo) {
+ $logo = wrap(inserer_attribut($logo, 'class', 'logo'), '<span class="logo-carre">');
+ }
+ }
+
+ $lib_bouton = (acces_restreint_rubrique($id_rubrique)
+ ? http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))
+ : '') .
" <a class='titremlien' dir='$lang_dir'" .
($row['lang'] !== $GLOBALS['spip_lang'] ? " hreflang='" . $row['lang'] . "'" : '') .
" href='" .
@@ -91,7 +85,7 @@ function enfant_rub($collection, $debut = 0, $limite = 500) {
. '</a>';
$titre = bouton_block_depliable($lib_bouton, $les_sous_enfants ? false : -1, "enfants$id_rubrique")
- . (!$descriptif ? '' : "\n<div class='descriptif'>$descriptif</div>")
+ . ($descriptif ? "\n<div class='descriptif'>$descriptif</div>" : '')
;
$res[] =
@@ -115,7 +109,7 @@ function enfant_rub($collection, $debut = 0, $limite = 500) {
* Le contenu du bloc dépliable
*/
function sous_enfant_rub($collection2) {
- $nb = sql_countsel('spip_rubriques', 'id_parent=' . intval($collection2));
+ $nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $collection2);
$retour = '';
$pagination = '';
@@ -143,7 +137,7 @@ function sous_enfant_rub($collection2) {
$result = sql_select(
'id_rubrique, id_parent, titre, lang',
'spip_rubriques',
- 'id_parent=' . intval($collection2),
+ 'id_parent=' . (int) $collection2,
'',
'0+titre,titre',
$debut == -1 ? '' : "$debut,$limite"
@@ -176,7 +170,7 @@ function sous_enfant_rub($collection2) {
return '';
}
- return debut_block_depliable($debut > 0 ? true : false, "enfants$collection2")
+ return debut_block_depliable($debut > 0, "enfants$collection2")
. "\n<ul class='liste-items sous-sous-rub'>\n"
. $retour
. "</ul>\n" . fin_block() . "\n\n";
@@ -198,7 +192,7 @@ function afficher_enfant_rub($id_rubrique = 0) {
$debut = 0;
$limite = 500;
- $nb = sql_countsel('spip_rubriques', 'id_parent=' . intval($id_rubrique));
+ $nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $id_rubrique);
if ($nb > $limite) {
$debut = _request('debut_rubrique' . $id_rubrique) ?: $debut;
@@ -223,8 +217,7 @@ function afficher_enfant_rub($id_rubrique = 0) {
$les_enfants = implode('', array_slice($les_enfants, 0, $n));
}
- $res =
- $pagination
+ return $pagination
. "<div class='gauche'>"
. $les_enfants
. '</div>'
@@ -232,6 +225,4 @@ function afficher_enfant_rub($id_rubrique = 0) {
. $les_enfants2
. '</div>'
. $pagination;
-
- return $res;
}
diff --git a/ecrire/inc/puce_statut.php b/ecrire/inc/puce_statut.php
index 9f3369515cb3680362ddb6b6ec48832c12bad8ba..68df951038cc29e3e08ffa161d723f59453c6cbf 100644
--- a/ecrire/inc/puce_statut.php
+++ b/ecrire/inc/puce_statut.php
@@ -66,7 +66,7 @@ function inc_puce_statut_dist(
static $f_puce_statut = [];
$type = objet_type($type);
// cas prioritaire : fonction perso, qui permet aussi de gerer les cas historiques
- if (!isset($f_puce_statut[$type]) or is_null($f_puce_statut[$type])) {
+ if (!isset($f_puce_statut[$type]) || is_null($f_puce_statut[$type])) {
$f_puce_statut[$type] = charger_fonction($type, 'puce_statut', true);
}
if ($f_puce_statut[$type]) {
@@ -334,15 +334,7 @@ function puce_statut_changement_rapide(
return $src;
}
- if (
- !$id
- or !_SPIP_AJAX
- or !$menu_rapide
- ) {
- $ajax_node = '';
- } else {
- $ajax_node = " class='imgstatut$type$id'";
- }
+ $ajax_node = (!$id || !_SPIP_AJAX || !$menu_rapide) ? '' : " class='imgstatut$type$id'";
$t = statut_titre($type, $statut);
@@ -396,9 +388,8 @@ function puce_statut_changement_rapide(
foreach ($desc['statut_textes_instituer'] as $s => $t) {
$out .= afficher_script_statut($id, $type, -$zero - $i++ * $unit, statut_image($type, $s), $s, _T($t));
}
- $out .= '</span>';
- return $out;
+ return $out . '</span>';
} else {
$nom = 'puce_statut_';
$action = generer_url_ecrire('puce_statut', '', true);
diff --git a/ecrire/inc/recherche_to_array.php b/ecrire/inc/recherche_to_array.php
index 864b554659374b42294ec4d516591b5746522a3a..b36f08343b8c07a3d3371f4c57bdb5fa1ce90cd6 100644
--- a/ecrire/inc/recherche_to_array.php
+++ b/ecrire/inc/recherche_to_array.php
@@ -61,7 +61,7 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
// c'est un pis-aller : ca a peu de chance de marcher, mais mieux quand meme que en conservant la ','
// (aka ca marche au moins dans certains cas comme avec spip_formulaires_reponses_champs)
- if (strpos($_id_table, ',') !== false) {
+ if (str_contains($_id_table, ',')) {
$_id_table = explode(',', $_id_table);
$_id_table = reset($_id_table);
}
@@ -73,7 +73,7 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
if (is_array($champ)) {
spip_log('requetes imbriquees interdites');
} else {
- if (strpos($champ, '.') === false) {
+ if (!str_contains($champ, '.')) {
$champ = "t.$champ";
}
$requete['SELECT'][] = $champ;
@@ -81,7 +81,7 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
}
}
if ($a) {
- $requete['WHERE'][] = join(' OR ', $a);
+ $requete['WHERE'][] = implode(' OR ', $a);
}
$requete['FROM'][] = table_objet_sql($table) . ' AS t';
@@ -99,15 +99,12 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
);
while (
- $t = sql_fetch($s, $serveur)
- and (!isset($t['score']) or $t['score'] > 0)
+ ($t = sql_fetch($s, $serveur))
+ && (!isset($t['score']) || $t['score'] > 0)
) {
- $id = intval($t[$_id_table]);
+ $id = (int) $t[$_id_table];
- if (
- $options['toutvoir']
- or autoriser('voir', $table, $id)
- ) {
+ if ($options['toutvoir'] || autoriser('voir', $table, $id)) {
// indiquer les champs concernes
$champs_vus = [];
$score = 0;
@@ -139,11 +136,7 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
$matches[$champ] = $regs;
}
- if (
- !$options['champs']
- and !$options['score']
- and !$options['matches']
- ) {
+ if (!$options['champs'] && !$options['score'] && !$options['matches']) {
break;
}
}
@@ -173,11 +166,11 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
// on ne sait passer que par table de laison en 1 coup
if (
isset($jointures[$table])
- and $joints = recherche_en_base(
+ && ($joints = recherche_en_base(
$recherche,
$jointures[$table],
array_merge($options, ['jointures' => false])
- )
+ ))
) {
include_spip('action/editer_liens');
$trouver_table = charger_fonction('trouver_table', 'base');
@@ -189,9 +182,9 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
// on peut definir une fonction de recherche jointe pour regler les cas particuliers
if (
!(
- $rechercher_joints = charger_fonction("rechercher_joints_{$table}_{$table_liee}", 'inc', true)
- or $rechercher_joints = charger_fonction("rechercher_joints_objet_{$table_liee}", 'inc', true)
- or $rechercher_joints = charger_fonction("rechercher_joints_{$table}_objet_lie", 'inc', true)
+ ($rechercher_joints = charger_fonction("rechercher_joints_{$table}_{$table_liee}", 'inc', true))
+ || ($rechercher_joints = charger_fonction("rechercher_joints_objet_{$table_liee}", 'inc', true))
+ || ($rechercher_joints = charger_fonction("rechercher_joints_{$table}_objet_lie", 'inc', true))
)
) {
$cle_arrivee = id_table_objet($table_liee);
@@ -251,8 +244,8 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
);
} // cas table de liaison generique spip_xxx_yyy
elseif (
- $t = $trouver_table($table_arrivee . '_' . $table_depart, $serveur)
- or $t = $trouver_table($table_depart . '_' . $table_arrivee, $serveur)
+ ($t = $trouver_table($table_arrivee . '_' . $table_depart, $serveur))
+ || ($t = $trouver_table($table_depart . '_' . $table_arrivee, $serveur))
) {
$s = sql_select(
"$cle_depart,$cle_arrivee",
@@ -283,18 +276,18 @@ function inc_recherche_to_array_dist($recherche, $options = []) {
if (!isset($results[$id])) {
$results[$id] = [];
}
- if (isset($joint['score']) and $joint['score']) {
+ if (isset($joint['score']) && $joint['score']) {
if (!isset($results[$id]['score'])) {
$results[$id]['score'] = 0;
}
$results[$id]['score'] += $joint['score'];
}
- if (isset($joint['champs']) and $joint['champs']) {
+ if (isset($joint['champs']) && $joint['champs']) {
foreach ($joint['champs'] as $c => $val) {
$results[$id]['champs'][$table_liee . '.' . $c] = $val;
}
}
- if (isset($joint['matches']) and $joint['matches']) {
+ if (isset($joint['matches']) && $joint['matches']) {
foreach ($joint['matches'] as $c => $val) {
$results[$id]['matches'][$table_liee . '.' . $c] = $val;
}
diff --git a/ecrire/inc/rechercher.php b/ecrire/inc/rechercher.php
index a6e951a5aeb1b6a7f8777abdddf2de71d7880771..6a3d5a35b29024ce90d81ec2bf8a6f95633297b9 100644
--- a/ecrire/inc/rechercher.php
+++ b/ecrire/inc/rechercher.php
@@ -84,7 +84,7 @@ function expression_recherche($recherche, $options) {
}
$u = $GLOBALS['meta']['pcre_u'];
- if ($u and !str_contains($options['preg_flags'], (string) $u)) {
+ if ($u && !str_contains($options['preg_flags'], (string) $u)) {
$options['preg_flags'] .= $u;
}
include_spip('inc/charsets');
@@ -95,7 +95,7 @@ function expression_recherche($recherche, $options) {
$recherche = preg_replace(',(\w)\*($|\s),Uims', '$1$2', $recherche);
$is_preg = false;
- if (substr($recherche, 0, 1) == '/' and substr($recherche, -1, 1) == '/' and strlen($recherche) > 2) {
+ if (str_starts_with($recherche, '/') && str_ends_with($recherche, '/') && strlen($recherche) > 2) {
// c'est une preg
$recherche_trans = translitteration($recherche);
$preg = $recherche_trans . $options['preg_flags'];
@@ -137,7 +137,7 @@ function expression_recherche($recherche, $options) {
// mais on cherche quand même l'expression complète, même si elle
// comporte des mots de moins de quatre lettres
$recherche = trim(preg_replace(',\s+,' . $u, '|', $recherche_inter), '|');
- if (!$recherche or $petits_mots) {
+ if (!$recherche || $petits_mots) {
$recherche = preg_quote($recherche_org);
}
$recherche_trans = translitteration($recherche);
@@ -148,10 +148,7 @@ function expression_recherche($recherche, $options) {
// Si la chaine est inactive, on va utiliser LIKE pour aller plus vite
// ou si l'expression reguliere est invalide
- if (
- !$is_preg
- or (@preg_match($preg, '') === false)
- ) {
+ if (!$is_preg || @preg_match($preg, '') === false) {
$methode = 'LIKE';
$u = $GLOBALS['meta']['pcre_u'];
@@ -200,8 +197,8 @@ function expression_recherche($recherche, $options) {
$char = spip_substr($q, $i, 1);
if (
!is_ascii($char)
- and $char_t = translitteration($char)
- and $char_t !== $char
+ && ($char_t = translitteration($char))
+ && $char_t !== $char
) {
// on utilise ..?.? car le char utf peut etre encode sur 1, 2 ou 3 bytes
// mais c'est un pis aller cf #4354
@@ -216,7 +213,7 @@ function expression_recherche($recherche, $options) {
// (oui c'est tres dicustable...)
if (
isset($GLOBALS['connexions'][$options['serveur'] ?: 0]['type'])
- and strncmp($GLOBALS['connexions'][$options['serveur'] ?: 0]['type'], 'sqlite', 6) == 0
+ && str_starts_with($GLOBALS['connexions'][$options['serveur'] ?: 0]['type'], 'sqlite')
) {
$q_t = strtr($q, 'aeuioc', $is_preg ? '......' : '______');
// si il reste au moins un char significatif...
@@ -257,10 +254,7 @@ function recherche_en_base($recherche = '', $tables = null, $options = [], $serv
if (!is_array($tables)) {
$liste = liste_des_champs();
- if (
- is_string($tables)
- and $tables != ''
- ) {
+ if (is_string($tables) && $tables != '') {
$toutes = [];
foreach (explode(',', $tables) as $t) {
$t = trim($t);
@@ -275,7 +269,7 @@ function recherche_en_base($recherche = '', $tables = null, $options = [], $serv
}
}
- if (!strlen($recherche) or !count($tables)) {
+ if (!strlen($recherche) || $tables === []) {
return [];
}
@@ -363,17 +357,14 @@ function remplace_en_base($recherche = '', $remplace = null, $tables = null, $op
foreach ($results as $table => $r) {
$_id_table = id_table_objet($table);
foreach ($r as $id => $x) {
- if (
- $options['toutmodifier']
- or autoriser('modifier', $table, $id)
- ) {
+ if ($options['toutmodifier'] || autoriser('modifier', $table, $id)) {
$modifs = [];
foreach ($x['champs'] as $key => $val) {
if ($key == $_id_table) {
continue;
}
$repl = preg_replace($preg, $remplace, $val);
- if ($repl <> $val) {
+ if ($repl != $val) {
$modifs[$key] = $repl;
}
}
diff --git a/ecrire/inc/roles.php b/ecrire/inc/roles.php
index 163e5320731af0063d2e38bd1ff4346eda406ea2..15787f12b045d35bc1beeef849660502d26a1f53 100644
--- a/ecrire/inc/roles.php
+++ b/ecrire/inc/roles.php
@@ -50,7 +50,7 @@ function roles_presents($objet, $objet_destination = '') {
$desc = lister_tables_objets_sql(table_objet_sql($objet));
// pas de liste de roles, on sort
- if (!isset($desc['roles_titres']) or !($titres = $desc['roles_titres'])) {
+ if (!isset($desc['roles_titres']) || !($titres = $desc['roles_titres'])) {
return false;
}
@@ -142,10 +142,7 @@ function roles_trouver_dans_qualif($objet, $objet_destination, $qualif = []) {
if ($roles = roles_presents($objet, $objet_destination)) {
$colonne_role = $roles['colonne'];
// qu'il n'est pas défini
- if (
- !isset($qualif[$colonne_role])
- or !($role = $qualif[$colonne_role])
- ) {
+ if (!isset($qualif[$colonne_role]) || !($role = $qualif[$colonne_role])) {
$role = $roles['roles']['defaut'];
}
// where
@@ -180,12 +177,9 @@ function roles_creer_condition_role($objet_source, $objet, $cond, $tous_si_absen
// chercher d'eventuels rôles transmis
$role = ($cond['role'] ?? ($tous_si_absent ? '*' : $role_defaut));
unset($cond['role']); // cette condition est particuliere...
-
- if ($colonne_role) {
- // on ajoute la condition du role aux autres conditions.
- if ($role != '*') {
- $cond[] = "$colonne_role=" . sql_quote($role);
- }
+ // on ajoute la condition du role aux autres conditions.
+ if ($colonne_role && $role != '*') {
+ $cond[] = "$colonne_role=" . sql_quote($role);
}
return [$cond, $colonne_role, $role];
diff --git a/ecrire/inc/selectionner.php b/ecrire/inc/selectionner.php
index 1b3a4e83a33a60b0d3d4342883c6571b16b538ae..12ee60e43541e0c44da65f50a26116cec28a3fc7 100644
--- a/ecrire/inc/selectionner.php
+++ b/ecrire/inc/selectionner.php
@@ -176,7 +176,7 @@ function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name,
function mini_hier($id_rubrique) {
$liste = $id_rubrique;
- $id_rubrique = intval($id_rubrique);
+ $id_rubrique = (int) $id_rubrique;
while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
$liste = $id_rubrique . ",$liste";
}
diff --git a/ecrire/inc/simplexml_to_array.php b/ecrire/inc/simplexml_to_array.php
index 0e6e0622a96044a19591b29ef2b40ce41ec8cbc8..2962e30477181ad05951c3742ef601f0f5bb30c0 100644
--- a/ecrire/inc/simplexml_to_array.php
+++ b/ecrire/inc/simplexml_to_array.php
@@ -67,7 +67,7 @@ function xmlObjToArr($obj, $utiliser_namespace = false) {
$attributes = [];
// get info for all namespaces
- foreach ($namespace as $ns => $nsUrl) {
+ foreach (array_keys($namespace) as $ns) {
// attributes
$objAttributes = $obj->attributes($ns, true);
foreach ($objAttributes as $attributeName => $attributeValue) {
diff --git a/ecrire/inc/svg.php b/ecrire/inc/svg.php
index cb4321b20de00298e943fd787789df709d3a5fbb..b3c7a4cae6c4b8badb6e26a894bf156b6f442319 100644
--- a/ecrire/inc/svg.php
+++ b/ecrire/inc/svg.php
@@ -38,20 +38,20 @@ if (!defined('IMG_SVG')) {
* false si on a pas pu charger l'image
*/
function svg_charger($fichier, $maxlen = null) {
- if (strpos($fichier, 'data:image/svg+xml') === 0) {
+ if (str_starts_with($fichier, 'data:image/svg+xml')) {
$image = explode(';', $fichier, 2);
$image = end($image);
- if (strpos($image, 'base64,') === 0) {
+ if (str_starts_with($image, 'base64,')) {
$image = base64_decode(substr($image, 7));
}
- if (strpos($image, '<svg') !== false) {
+ if (str_contains($image, '<svg')) {
return $image;
}
// encodage inconnu ou autre format d'image ?
return false;
}
// c'est peut etre deja une image svg ?
- if (strpos($fichier, '<svg') !== false) {
+ if (str_contains($fichier, '<svg')) {
return $fichier;
}
if (!file_exists($fichier)) {
@@ -60,14 +60,9 @@ function svg_charger($fichier, $maxlen = null) {
return false;
}
}
- if (is_null($maxlen)) {
- $image = file_get_contents($fichier);
- }
- else {
- $image = file_get_contents($fichier, false, null, 0, $maxlen);
- }
+ $image = is_null($maxlen) ? file_get_contents($fichier) : file_get_contents($fichier, false, null, 0, $maxlen);
// est-ce bien une image svg ?
- if (strpos($image, '<svg') !== false) {
+ if (str_contains($image, '<svg')) {
return $image;
}
return false;
@@ -126,35 +121,16 @@ function svg_lire_attributs($img) {
*/
function svg_dimension_to_pixels($dimension, $precision = 2) {
if (preg_match(',^(-?\d+(\.\d+)?)([^\d]*),i', trim($dimension), $m)) {
- switch (strtolower($m[2])) {
- case '%':
- // on ne sait pas faire :(
- return false;
- break;
- case 'em':
- return round($m[1] * 16, $precision); // 16px font-size par defaut
- break;
- case 'ex':
- return round($m[1] * 16, $precision); // 16px font-size par defaut
- break;
- case 'pc':
- return round($m[1] * 16, $precision); // 1/6 inch = 96px/6 in CSS
- break;
- case 'cm':
- return round($m[1] * 96 / 2.54, $precision); // 96px / 2.54cm;
- break;
- case 'mm':
- return round($m[1] * 96 / 25.4, $precision); // 96px / 25.4mm;
- break;
- case 'in':
- return round($m[1] * 96, $precision); // 1 inch = 96px in CSS
- break;
- case 'px':
- case 'pt':
- default:
- return $m[1];
- break;
- }
+ return match (strtolower($m[2])) {
+ '%' => false,
+ 'em' => round($m[1] * 16, $precision),
+ 'ex' => round($m[1] * 16, $precision),
+ 'pc' => round($m[1] * 16, $precision),
+ 'cm' => round($m[1] * 96 / 2.54, $precision),
+ 'mm' => round($m[1] * 96 / 25.4, $precision),
+ 'in' => round($m[1] * 96, $precision),
+ default => $m[1],
+ };
}
return false;
}
@@ -174,8 +150,7 @@ function svg_change_balise_svg($svg, $old_balise_svg, $attributs) {
$new_balise_svg .= '>';
$p = strpos($svg, $old_balise_svg);
- $svg = substr_replace($svg, $new_balise_svg, $p, strlen($old_balise_svg));
- return $svg;
+ return substr_replace($svg, $new_balise_svg, $p, strlen($old_balise_svg));
}
/**
@@ -187,7 +162,7 @@ function svg_change_balise_svg($svg, $old_balise_svg, $attributs) {
*/
function svg_insert_shapes($svg, $shapes, $start = true) {
- if ($start === false or $start === 'end') {
+ if ($start === false || $start === 'end') {
$svg = str_replace('</svg>', $shapes . '</svg>', $svg);
}
else {
@@ -213,8 +188,7 @@ function svg_clip_in_box($svg, $x, $y, $width, $height) {
$clippath = "<clipPath id=\"$id\">$rect</clipPath>";
$g = "<g clip-path=\"url(#$id)\">";
$svg = svg_insert_shapes($svg, $clippath . $g);
- $svg = svg_insert_shapes($svg, '</g>', false);
- return $svg;
+ return svg_insert_shapes($svg, '</g>', false);
}
/**
@@ -226,18 +200,16 @@ function svg_clip_in_box($svg, $x, $y, $width, $height) {
*/
function svg_redimensionner($img, $new_width, $new_height) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
[$balise_svg, $attributs] = $svg_infos;
if (!isset($attributs['viewBox'])) {
$attributs['viewBox'] = '0 0 ' . $attributs['width'] . ' ' . $attributs['height'];
}
- $attributs['width'] = strval($new_width);
- $attributs['height'] = strval($new_height);
-
- $svg = svg_change_balise_svg($svg, $balise_svg, $attributs);
- return $svg;
+ $attributs['width'] = (string) $new_width;
+ $attributs['height'] = (string) $new_height;
+ return svg_change_balise_svg($svg, $balise_svg, $attributs);
}
return $img;
@@ -249,15 +221,14 @@ function svg_redimensionner($img, $new_width, $new_height) {
* @return string
*/
function svg_couleur_to_hexa($couleur) {
- if (strpos($couleur, 'rgb(') === 0) {
+ if (str_starts_with($couleur, 'rgb(')) {
$c = explode(',', substr($couleur, 4));
- $couleur = _couleur_dec_to_hex(intval($c[0]), intval($c[1]), intval($c[2]));
+ $couleur = _couleur_dec_to_hex((int) $c[0], (int) $c[1], (int) $c[2]);
}
else {
$couleur = couleur_html_to_hex($couleur);
}
- $couleur = '#' . ltrim($couleur, '#');
- return $couleur;
+ return '#' . ltrim($couleur, '#');
}
/**
@@ -266,9 +237,9 @@ function svg_couleur_to_hexa($couleur) {
* @return array
*/
function svg_couleur_to_rgb($couleur) {
- if (strpos($couleur, 'rgb(') === 0) {
+ if (str_starts_with($couleur, 'rgb(')) {
$c = explode(',', substr($couleur, 4));
- return ['red' => intval($c[0]),'green' => intval($c[1]),'blue' => intval($c[2])];
+ return ['red' => (int) $c[0],'green' => (int) $c[1],'blue' => (int) $c[2]];
}
return _couleur_hex_to_dec($couleur);
}
@@ -301,7 +272,7 @@ function svg_getimagesize_from_attr($attributs) {
$coeff = 1;
if (
isset($attributs['width'])
- and $w = svg_dimension_to_pixels($attributs['width'])
+ && ($w = svg_dimension_to_pixels($attributs['width']))
) {
$width = $w;
// si on avait pas de viewBox, la construire a partir de ce width
@@ -326,7 +297,7 @@ function svg_getimagesize_from_attr($attributs) {
}
if (
isset($attributs['height'])
- and $h = svg_dimension_to_pixels($attributs['height'])
+ && ($h = svg_dimension_to_pixels($attributs['height']))
) {
$height = $h;
// si on avait pas de viewBox, la construire a partir de ce height
@@ -370,8 +341,8 @@ function svg_getimagesize_from_attr($attributs) {
*/
function svg_force_viewBox_px($img, $force_width_and_height = false) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
[$balise_svg, $attributs] = $svg_infos;
@@ -383,9 +354,7 @@ function svg_force_viewBox_px($img, $force_width_and_height = false) {
}
$attributs['viewBox'] = $viewBox;
-
- $svg = svg_change_balise_svg($svg, $balise_svg, $attributs);
- return $svg;
+ return svg_change_balise_svg($svg, $balise_svg, $attributs);
}
return $img;
}
@@ -396,10 +365,11 @@ function svg_force_viewBox_px($img, $force_width_and_height = false) {
* @return array|mixed
*/
function svg_extract_couleurs($img) {
- if ($svg = svg_charger($img)) {
- if (preg_match_all('/(#[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f])|(rgb\([\s\d]+,[\s\d]+,[\s\d]+\))|(#[0-9a-f][0-9a-f][0-9a-f])/imS', $svg, $matches)) {
- return $matches[0];
- }
+ if (
+ ($svg = svg_charger($img))
+ && preg_match_all('/(#[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f])|(rgb\([\s\d]+,[\s\d]+,[\s\d]+\))|(#[0-9a-f][0-9a-f][0-9a-f])/imS', $svg, $matches)
+ ) {
+ return $matches[0];
}
return [];
}
@@ -413,8 +383,8 @@ function svg_extract_couleurs($img) {
*/
function svg_recadrer($img, $new_width, $new_height, $offset_width, $offset_height, $background_color = '') {
if (
- $svg = svg_force_viewBox_px($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_force_viewBox_px($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
[$balise_svg, $attributs] = $svg_infos;
$viewBox = explode(' ', $attributs['viewBox']);
@@ -427,9 +397,9 @@ function svg_recadrer($img, $new_width, $new_height, $offset_width, $offset_heig
// si on a un width/height qui rescale, il faut rescaler
if (
isset($attributs['width'])
- and $w = svg_dimension_to_pixels($attributs['width'])
- and isset($attributs['height'])
- and $h = svg_dimension_to_pixels($attributs['height'])
+ && ($w = svg_dimension_to_pixels($attributs['width']))
+ && isset($attributs['height'])
+ && ($h = svg_dimension_to_pixels($attributs['height']))
) {
$xscale = $viewBox[2] / $w;
$viewport_w = round($viewport_w * $xscale, 2);
@@ -439,7 +409,7 @@ function svg_recadrer($img, $new_width, $new_height, $offset_width, $offset_heig
$viewport_oy = round($viewport_oy * $yscale, 2);
}
- if ($viewport_w > $viewBox[2] or $viewport_h > $viewBox[3]) {
+ if ($viewport_w > $viewBox[2] || $viewport_h > $viewBox[3]) {
$svg = svg_clip_in_box($svg, $viewBox[0], $viewBox[1], $viewBox[2], $viewBox[3]);
}
@@ -450,13 +420,13 @@ function svg_recadrer($img, $new_width, $new_height, $offset_width, $offset_heig
$viewBox[3] = $viewport_h;
$attributs['viewBox'] = implode(' ', $viewBox);
- $attributs['width'] = strval($new_width);
- $attributs['height'] = strval($new_height);
+ $attributs['width'] = (string) $new_width;
+ $attributs['height'] = (string) $new_height;
$svg = svg_change_balise_svg($svg, $balise_svg, $attributs);
// ajouter un background
- if ($background_color and $background_color !== 'transparent') {
+ if ($background_color && $background_color !== 'transparent') {
$svg = svg_ajouter_background($svg, $background_color);
}
@@ -474,10 +444,10 @@ function svg_recadrer($img, $new_width, $new_height, $offset_width, $offset_heig
*/
function svg_ajouter_background($img, $background_color) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
- if ($background_color and $background_color !== 'transparent') {
+ if ($background_color && $background_color !== 'transparent') {
[$balise_svg, $attributs] = $svg_infos;
$background_color = svg_couleur_to_hexa($background_color);
@@ -504,10 +474,10 @@ function svg_ajouter_background($img, $background_color) {
*/
function svg_ajouter_voile($img, $background_color, $opacity) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
- if ($background_color and $background_color !== 'transparent') {
+ if ($background_color && $background_color !== 'transparent') {
[$balise_svg, $attributs] = $svg_infos;
$background_color = svg_couleur_to_hexa($background_color);
@@ -534,8 +504,8 @@ function svg_ajouter_voile($img, $background_color, $opacity) {
*/
function svg_transformer($img, $attributs) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
if ($attributs) {
[$balise_svg, ] = $svg_infos;
@@ -565,8 +535,8 @@ function svg_transformer($img, $attributs) {
*/
function svg_apply_filter($img, $filter_def) {
if (
- $svg = svg_charger($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_charger($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
if ($filter_def) {
[$balise_svg, ] = $svg_infos;
@@ -588,7 +558,7 @@ function svg_apply_filter($img, $filter_def) {
* @return string
*/
function svg_filter_blur($img, $blur_width) {
- $blur_width = intval($blur_width);
+ $blur_width = (int) $blur_width;
return svg_apply_filter($img, "<feGaussianBlur stdDeviation=\"$blur_width\"/>");
}
@@ -624,8 +594,8 @@ function svg_filter_sepia($img, $intensity) {
*/
function svg_flip($img, $HorV) {
if (
- $svg = svg_force_viewBox_px($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_force_viewBox_px($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
[$balise_svg, $atts] = $svg_infos;
$viewBox = explode(' ', $atts['viewBox']);
@@ -633,19 +603,18 @@ function svg_flip($img, $HorV) {
if (!in_array($HorV, ['h', 'H'])) {
$transform = 'scale(-1,1)';
- $x = intval($viewBox[0]) + intval($viewBox[2] / 2);
+ $x = (int) $viewBox[0] + (int) ($viewBox[2] / 2);
$mx = -$x;
$transform = "translate($x, 0) $transform translate($mx, 0)";
}
else {
$transform = 'scale(1,-1)';
- $y = intval($viewBox[1]) + intval($viewBox[3] / 2);
+ $y = (int) $viewBox[1] + (int) ($viewBox[3] / 2);
$my = -$y;
$transform = "translate(0, $y) $transform translate(0, $my)";
}
- $svg = svg_transformer($svg, ['transform' => $transform]);
- return $svg;
+ return svg_transformer($svg, ['transform' => $transform]);
}
return $img;
}
@@ -662,17 +631,16 @@ function svg_flip($img, $HorV) {
*/
function svg_rotate($img, $angle, $center_x, $center_y) {
if (
- $svg = svg_force_viewBox_px($img)
- and $svg_infos = svg_lire_balise_svg($svg)
+ ($svg = svg_force_viewBox_px($img))
+ && ($svg_infos = svg_lire_balise_svg($svg))
) {
[$balise_svg, $atts] = $svg_infos;
$viewBox = explode(' ', $atts['viewBox']);
$center_x = round($viewBox[0] + $center_x * $viewBox[2]);
$center_y = round($viewBox[1] + $center_y * $viewBox[3]);
- $svg = svg_transformer($svg, ['transform' => "rotate($angle $center_x $center_y)"]);
- return $svg;
+ return svg_transformer($svg, ['transform' => "rotate($angle $center_x $center_y)"]);
}
return $img;
}
@@ -687,8 +655,8 @@ function svg_rotate($img, $angle, $center_x, $center_y) {
*/
function svg_filtrer_couleurs($img, $callback_filter) {
if (
- $svg = svg_force_viewBox_px($img)
- and $colors = svg_extract_couleurs($svg)
+ ($svg = svg_force_viewBox_px($img))
+ && ($colors = svg_extract_couleurs($svg))
) {
$colors = array_unique($colors);
@@ -718,9 +686,8 @@ function svg_filtrer_couleurs($img, $callback_filter) {
}
$svg = str_replace(array_keys($colors), array_values($colors), $svg);
- $svg = str_replace(array_keys($new_colors), array_values($new_colors), $svg);
- return $svg;
+ return str_replace(array_keys($new_colors), array_values($new_colors), $svg);
}
return $img;
}
diff --git a/ecrire/inc/traduire.php b/ecrire/inc/traduire.php
index f239cfbe4b67622a094c8f332293a23090b73919..e171c6fbad928d153bdb0ddb219b4120d15464ce 100644
--- a/ecrire/inc/traduire.php
+++ b/ecrire/inc/traduire.php
@@ -45,10 +45,8 @@ function find_langs_in_path($file, $dirname = 'lang') {
if (!isset($dirs[$a = $dir . $dirname])) {
$dirs[$a] = (is_dir($a) || !$a);
}
- if ($dirs[$a]) {
- if (is_readable($a .= $file)) {
- $liste[] = $a;
- }
+ if ($dirs[$a] && is_readable($a .= $file)) {
+ $liste[] = $a;
}
}
@@ -81,7 +79,7 @@ function chercher_module_lang($module, $lang = '') {
}
// 2) directement dans le chemin (old style, uniquement pour local)
- return (($module == 'local') or strpos($module, '/'))
+ return ($module == 'local' || strpos($module, '/'))
? (($f = find_in_path($module . $lang . '.php')) ? [$f] : false)
: false;
}
@@ -111,7 +109,7 @@ function charger_langue($lang, $module = 'spip') {
$langs[$lang] = [];
if ($lang) {
$langs[$lang][] = $lang;
- if (strpos($lang, '_') !== false) {
+ if (str_contains($lang, '_')) {
$l = explode('_', $lang);
$langs[$lang][] = reset($l);
}
@@ -150,7 +148,7 @@ function lire_fichier_langue(string $fichier): array {
$idx_lang = include $fichier;
$GLOBALS['idx_lang'] = $idx_lang_before;
if (!is_array($idx_lang)) {
- if (isset($GLOBALS[$idx_lang_tmp]) and is_array($GLOBALS[$idx_lang_tmp])) {
+ if (isset($GLOBALS[$idx_lang_tmp]) && is_array($GLOBALS[$idx_lang_tmp])) {
$idx_lang = $GLOBALS[$idx_lang_tmp];
} else {
$idx_lang = [];
@@ -186,7 +184,7 @@ function surcharger_langue($fichiers) {
if (!is_array($fichiers)) {
$fichiers = [$fichiers];
}
- if (!count($fichiers)) {
+ if ($fichiers === []) {
return;
}
foreach ($fichiers as $fichier) {
@@ -244,7 +242,7 @@ function inc_traduire_dist($ori, $lang, $raw = false) {
static $deja_vu = [];
static $local = [];
- if (isset($deja_vu[$lang][$ori]) and (_request('var_mode') != 'traduction')) {
+ if (isset($deja_vu[$lang][$ori]) && _request('var_mode') != 'traduction') {
return $raw ? $deja_vu[$lang][$ori] : $deja_vu[$lang][$ori]->texte;
}
@@ -302,7 +300,7 @@ function inc_traduire_dist($ori, $lang, $raw = false) {
// on essaie d'abord la langue du site, puis a defaut la langue fr
if (
($desc->texte === null || !strlen($desc->texte))
- and $lang !== _LANGUE_PAR_DEFAUT
+ && $lang !== _LANGUE_PAR_DEFAUT
) {
if ($lang !== $GLOBALS['meta']['langue_site']) {
$desc = inc_traduire_dist($ori, $GLOBALS['meta']['langue_site'], true);
@@ -312,15 +310,15 @@ function inc_traduire_dist($ori, $lang, $raw = false) {
}
// Supprimer la mention <NEW> ou <MODIF>
- if ($desc->texte && substr($desc->texte, 0, 1) === '<') {
+ if ($desc->texte && str_starts_with($desc->texte, '<')) {
$desc->texte = str_replace(['<NEW>', '<MODIF>'], [], $desc->texte);
}
// Si on n'est pas en utf-8, la chaine peut l'etre...
// le cas echeant on la convertit en entites html &#xxx;
if (
- (!isset($GLOBALS['meta']['charset']) or $GLOBALS['meta']['charset'] !== 'utf-8')
- and preg_match(',[\x7f-\xff],S', $desc->texte)
+ (!isset($GLOBALS['meta']['charset']) || $GLOBALS['meta']['charset'] !== 'utf-8')
+ && preg_match(',[\x7f-\xff],S', $desc->texte)
) {
include_spip('inc/charsets');
$desc->texte = charset2unicode($desc->texte, 'utf-8');
@@ -345,7 +343,7 @@ function inc_traduire_dist($ori, $lang, $raw = false) {
* @return Description
*/
function definir_details_traduction($desc, $modules) {
- if (!$desc->mode and $desc->texte) {
+ if (!$desc->mode && $desc->texte) {
// ne pas modifier 2 fois l'affichage
$desc->mode = 'traduction';
$classe = 'debug-traduction' . ($desc->module == 'ecrire' ? '-prive' : '');