Skip to content
Extraits de code Groupes Projets
Valider 69fc6182 rédigé par marcimat@rezo.net's avatar marcimat@rezo.net
Parcourir les fichiers

Meilleure compatibilité avec PSR-2 et nos règles d'écriture, en appliquant 

différents fix avec php-cs-fixers. Fixers appliqués ici :

	'encoding',                // utf8
	'eof_ending',              // un saut de ligne en fin de fichier
	'elseif',                  // elseif plutôt que else if
	'function_call_space',     // espaces sur fonctions
	'function_declaration',    // espaces sur fonctions
	'function_typehint_space', // espaces sur fonctions
	'linefeed',                // sauts de ligne \n uniquement
	'lowercase_constants',     // true, false, null en munuscule
	'lowercase_keywords',      // mots clés PHP en lowercase
	'method_argument_space',   // espaces sur appels de fonctions
	'multiple_use',            // use unique sur fonctions anonymes
	'newline_after_open_tag',  // ouverture de php… et c'est tout sur cette ligne
	'operators_spaces',        // espaces de part et d'autres des opérateurs binaires
	'parenthesis',             // pas d'espace juste après parenthèse ouvrante, ou avant parenthèse fermante
	'php_closing_tag',         // pas de fermeture de php
	'short_tag',               // tag PHP corrects
	'trailing_spaces',         // pas d'espace qui traîne en fin de ligne
	'visibility',              // déclarer 'public / private / protected' sur les méthodes
parent b117f0c5
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 1032 ajouts et 1073 suppressions
action/acceder_document.php
+ 4
− 6
Voir le fichier @ 69fc6182
......@@ -32,11 +32,11 @@ function action_acceder_document_dist() {
$status = $dcc = false;
if (strpos($f, '../') !== false
OR preg_match(',^\w+://,', $f)
or preg_match(',^\w+://,', $f)
) {
$status = 403;
} else {
if (!file_exists($file) OR !is_readable($file)) {
if (!file_exists($file) or !is_readable($file)) {
$status = 404;
} else {
$where = "D.fichier=" . sql_quote(set_spip_doc($file))
......@@ -51,7 +51,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
and $_SERVER['HTTP_IF_NONE_MATCH'] == $ETag
) {
http_status(304); // Not modified
exit;
......@@ -102,7 +102,7 @@ function action_acceder_document_dist() {
$f = basename($file);
if (isset($doc['titre'])
AND (preg_match('/^\w+[.]\w+$/', $doc['titre']) OR $doc['titre'] == 'Makefile')
and (preg_match('/^\w+[.]\w+$/', $doc['titre']) or $doc['titre'] == 'Makefile')
) {
$f = $doc['titre'];
}
......@@ -129,5 +129,3 @@ function action_acceder_document_dist() {
}
}
?>
action/ajouter_documents.php
+ 19
− 22
Voir le fichier @ 69fc6182
......@@ -101,10 +101,10 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
}
$titrer = isset($file['titrer']) ? $file['titrer'] : _TITRER_DOCUMENTS;
$mode = ((isset($file['mode']) AND $file['mode']) ? $file['mode'] : $mode);
$mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode);
include_spip('inc/modifier');
if (isset($file['distant']) AND $file['distant'] AND !in_array($mode, array('choix', 'auto', 'image', 'document'))) {
if (isset($file['distant']) and $file['distant'] and !in_array($mode, array('choix', 'auto', 'image', 'document'))) {
include_spip('inc/distant');
$file['tmp_name'] = _DIR_RACINE . copie_locale($source);
$source = $file['tmp_name'];
......@@ -114,7 +114,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']) and $file['distant']) {
include_spip('inc/distant');
if (is_array($a = renseigner_source_distante($source))) {
......@@ -197,7 +197,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
}
// lier le parent si necessaire
if ($id_objet = intval($id_objet) AND $objet) {
if ($id_objet = intval($id_objet) and $objet) {
$champs['parents'][] = "$objet|$id_objet";
}
......@@ -308,7 +308,7 @@ function verifier_upload_autorise($source, $mode = '') {
$infos = array('fichier' => $source);
$res = false;
if (preg_match(",\.([a-z0-9]+)(\?.*)?$,i", $source, $match)
AND $ext = $match[1]
and $ext = $match[1]
) {
$ext = corriger_extension(strtolower($ext));
......@@ -326,7 +326,7 @@ function verifier_upload_autorise($source, $mode = '') {
$res['autozip'] = true;
}
}
if ($mode AND $res) {
if ($mode and $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
......@@ -336,7 +336,7 @@ function verifier_upload_autorise($source, $mode = '') {
}
}
if (!$res OR is_string($res)) {
if (!$res or is_string($res)) {
spip_log("Upload $source interdit ($res)", _LOG_INFO_IMPORTANTE);
}
......@@ -447,10 +447,10 @@ function verifier_taille_document_acceptable(&$infos) {
// si ce n'est pas une image
if (!$infos['type_image']) {
if (defined('_DOC_MAX_SIZE') AND _DOC_MAX_SIZE > 0 AND $infos['taille'] > _DOC_MAX_SIZE*1024) {
if (defined('_DOC_MAX_SIZE') and _DOC_MAX_SIZE > 0 and $infos['taille'] > _DOC_MAX_SIZE * 1024) {
return _T('medias:info_doc_max_poids',
array(
'maxi' => taille_en_octets(_DOC_MAX_SIZE*1024),
'maxi' => taille_en_octets(_DOC_MAX_SIZE * 1024),
'actuel' => taille_en_octets($infos['taille'])
)
);
......@@ -458,20 +458,20 @@ function verifier_taille_document_acceptable(&$infos) {
} // si c'est une image
else {
if ((defined('_IMG_MAX_WIDTH') AND _IMG_MAX_WIDTH AND $infos['largeur'] > _IMG_MAX_WIDTH)
OR (defined('_IMG_MAX_HEIGHT') AND _IMG_MAX_HEIGHT AND $infos['hauteur'] > _IMG_MAX_HEIGHT)
if ((defined('_IMG_MAX_WIDTH') and _IMG_MAX_WIDTH and $infos['largeur'] > _IMG_MAX_WIDTH)
or (defined('_IMG_MAX_HEIGHT') and _IMG_MAX_HEIGHT and $infos['hauteur'] > _IMG_MAX_HEIGHT)
) {
$max_width = (defined('_IMG_MAX_WIDTH') AND _IMG_MAX_WIDTH) ? _IMG_MAX_WIDTH : '*';
$max_height = (defined('_IMG_MAX_HEIGHT') AND _IMG_MAX_HEIGHT) ? _IMG_MAX_HEIGHT : '*';
$max_width = (defined('_IMG_MAX_WIDTH') and _IMG_MAX_WIDTH) ? _IMG_MAX_WIDTH : '*';
$max_height = (defined('_IMG_MAX_HEIGHT') and _IMG_MAX_HEIGHT) ? _IMG_MAX_HEIGHT : '*';
// pas la peine d'embeter le redacteur avec ca si on a active le calcul des miniatures
// on met directement a la taille maxi a la volee
if (isset($GLOBALS['meta']['creer_preview']) AND $GLOBALS['meta']['creer_preview'] == 'oui') {
if (isset($GLOBALS['meta']['creer_preview']) and $GLOBALS['meta']['creer_preview'] == 'oui') {
include_spip('inc/filtres');
$img = filtrer('image_reduire', $infos['fichier'], $max_width, $max_height);
$img = extraire_attribut($img, 'src');
$img = supprimer_timestamp($img);
if (@file_exists($img) AND $img !== $infos['fichier']) {
if (@file_exists($img) and $img !== $infos['fichier']) {
spip_unlink($infos['fichier']);
@rename($img, $infos['fichier']);
$size = @getimagesize($infos['fichier']);
......@@ -481,8 +481,8 @@ function verifier_taille_document_acceptable(&$infos) {
}
}
if ((defined('_IMG_MAX_WIDTH') AND _IMG_MAX_WIDTH AND $infos['largeur'] > _IMG_MAX_WIDTH)
OR (defined('_IMG_MAX_HEIGHT') AND _IMG_MAX_HEIGHT AND $infos['hauteur'] > _IMG_MAX_HEIGHT)
if ((defined('_IMG_MAX_WIDTH') and _IMG_MAX_WIDTH and $infos['largeur'] > _IMG_MAX_WIDTH)
or (defined('_IMG_MAX_HEIGHT') and _IMG_MAX_HEIGHT and $infos['hauteur'] > _IMG_MAX_HEIGHT)
) {
return _T('medias:info_image_max_taille',
......@@ -503,10 +503,10 @@ function verifier_taille_document_acceptable(&$infos) {
}
}
if (defined('_IMG_MAX_SIZE') AND _IMG_MAX_SIZE > 0 AND $infos['taille'] > _IMG_MAX_SIZE*1024) {
if (defined('_IMG_MAX_SIZE') and _IMG_MAX_SIZE > 0 and $infos['taille'] > _IMG_MAX_SIZE * 1024) {
return _T('medias:info_image_max_poids',
array(
'maxi' => taille_en_octets(_IMG_MAX_SIZE*1024),
'maxi' => taille_en_octets(_IMG_MAX_SIZE * 1024),
'actuel' => taille_en_octets($infos['taille']
)
)
......@@ -522,6 +522,3 @@ function verifier_taille_document_acceptable(&$infos) {
return true;
}
?>
action/changer_mode_document.php
+ 4
− 6
Voir le fichier @ 69fc6182
......@@ -25,7 +25,7 @@ if (!defined("_ECRIRE_INC_VERSION")) {
* @return void
*/
function action_changer_mode_document_dist($id_document = null, $mode = null) {
if (is_null($id_document) OR is_null($mode)) {
if (is_null($id_document) or is_null($mode)) {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
......@@ -38,8 +38,8 @@ 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)
and include_spip('inc/autoriser')
and autoriser('modifier', 'document', $id_document)
) {
action_changer_mode_document_post($id_document, $mode);
}
......@@ -50,11 +50,9 @@ 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, array('vignette', 'image', 'document'))
and in_array($mode, array('vignette', 'image', 'document'))
) {
include_spip('action/editer_document');
document_modifier($id_document, array('mode' => $mode));
}
}
?>
action/copier_local.php
+ 0
− 2
Voir le fichier @ 69fc6182
......@@ -81,5 +81,3 @@ function action_copier_local_post($id_document) {
return _T('medias:erreur_copie_fichier', array('nom' => $source));
}
?>
action/dissocier_document.php
+ 6
− 8
Voir le fichier @ 69fc6182
......@@ -50,16 +50,16 @@ function action_dissocier_document_dist($arg = null) {
}
$suppr = $check = false;
if (count($arg) > 3 AND $arg[3] == 'suppr') {
if (count($arg) > 3 and $arg[3] == 'suppr') {
$suppr = true;
}
if (count($arg) > 4 AND $arg[4] == 'safe') {
if (count($arg) > 4 and $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)
and (
($id_objet < 0 and $id_objet == -$GLOBALS['visiteur_session']['id_auteur'])
or autoriser('dissocierdocuments', $objet, $id_objet)
)
) {
dissocier_document($document, $objet, $id_objet, $suppr, $check);
......@@ -133,7 +133,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 and !sql_countsel('spip_documents_liens', "objet!='document' AND id_document=" . $id_document)) {
$supprimer_document = charger_fonction('supprimer_document', 'action');
return $supprimer_document($id_document);
......@@ -182,5 +182,3 @@ function dissocier_document($document, $objet, $id_objet, $supprime = false, $ch
depublier_branche_rubrique_if($id_objet);
}
}
?>
action/editer_document.php
+ 12
− 14
Voir le fichier @ 69fc6182
......@@ -138,10 +138,10 @@ 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 @file_exists($f = get_spip_doc($ancien_fichier))
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 @file_exists($f = get_spip_doc($ancien_fichier))
) {
spip_unlink($f);
}
......@@ -195,21 +195,21 @@ function document_instituer($id_document, $champs = array()) {
"objet!='document' AND id_document=" . intval($id_document));
// dans 10 ans, ca nous fera un bug a corriger vers 2018
// penser a ouvrir un ticket d'ici la :p
$date_publication = time()+10*365*24*3600;
$date_publication = time() + 10 * 365 * 24 * 3600;
include_spip('base/objets');
while ($row = sql_fetch($res)) {
if (
// 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'])
or 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('date', 'spip_articles',
and $row2 = sql_fetsel('date', 'spip_articles',
'id_article=' . intval($row['id_objet']) . " AND statut='publie'")
) {
$statut = 'publie';
......@@ -217,15 +217,15 @@ function document_instituer($id_document, $champs = array()) {
}
}
$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' and $statut_ancien == 'publie' and $date_publication == $date_publication_ancienne) {
return false;
}
if ($statut != 'publie' AND $statut_ancien != 'publie' AND $statut_ancien != '0') {
if ($statut != 'publie' and $statut_ancien != 'publie' and $statut_ancien != '0') {
return false;
}
}
if ($statut !== $statut_ancien
OR $date_publication != $date_publication_ancienne
or $date_publication != $date_publication_ancienne
) {
sql_updateq('spip_documents', array('statut' => $statut, 'date_publication' => $date_publication),
'id_document=' . intval($id_document));
......@@ -267,7 +267,7 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
foreach ($parents as $p) {
$p = explode('|', $p);
if (preg_match('/^[a-z0-9_]+$/i', $objet = $p[0])
AND $p[1] = intval($p[1])
and $p[1] = intval($p[1])
) { // securite
$objets_parents[$p[0]][] = $p[1];
}
......@@ -280,7 +280,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']]) or !in_array($lien['id_objet'], $objets_parents[$lien['objet']])) {
if (autoriser('dissocierdocuments', $lien['objet'], $lien['id_objet'])) {
objet_dissocier(array('document' => $id_document), array($lien['objet'] => $lien['id_objet']));
}
......@@ -353,5 +353,3 @@ function instituer_document($id_document, $champs = array()) {
function revision_document($id_document, $c = false) {
return document_modifier($id_document, $c);
}
?>
action/supprimer_document.php
+ 0
− 2
Voir le fichier @ 69fc6182
......@@ -87,5 +87,3 @@ function action_supprimer_document_dist($id_document = 0) {
return true;
}
?>
action/supprimer_tous_orphelins.php
+ 58
− 60
Voir le fichier @ 69fc6182
<?php
/***************************************************************************\
* SPIP, Systeme de publication pour l'internet *
* *
* Copyright (c) 2001-2014 *
* Arnaud Martin, Antoine Pitrou, Philippe Riviere, Emmanuel Saint-James *
* *
* Ce programme est un logiciel libre distribue sous licence GNU/GPL. *
* Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne. *
\***************************************************************************/
if (!defined("_ECRIRE_INC_VERSION")) {
return;
}
include_spip('base/abstract_sql');
function action_supprimer_tous_orphelins() {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
//on recupere le contexte pour ne supprimer les orphelins que de ce dernier
list($media, $distant, $statut, $sanstitre) = explode('/', $arg);
//critere sur le media
if ($media) {
$select = "media=" . sql_quote($media);
}
//critere sur le distant
if ($distant) {
$where[] = "distant=" . sql_quote($distant);
}
//critere sur le statut
if ($statut) {
$where[] = "statut REGEXP " . sql_quote("($statut)");
}
//critere sur le sanstitre
if ($sanstitre) {
$where[] = "titre=''";
}
//on isole les orphelins
$select = sql_get_select("DISTINCT id_document", "spip_documents_liens as oooo");
$cond = "spip_documents.id_document NOT IN ($select)";
$where[] = $cond;
$ids_doc_orphelins = sql_select("id_document", "spip_documents", $where);
$supprimer_document = charger_fonction('supprimer_document', 'action');
while ($row = sql_fetch($ids_doc_orphelins)) {
$supprimer_document($row['id_document']); // pour les orphelins du contexte, on traite avec la fonction existante
}
}
?>
\ No newline at end of file
<?php
/***************************************************************************\
* SPIP, Systeme de publication pour l'internet *
* *
* Copyright (c) 2001-2014 *
* Arnaud Martin, Antoine Pitrou, Philippe Riviere, Emmanuel Saint-James *
* *
* Ce programme est un logiciel libre distribue sous licence GNU/GPL. *
* Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne. *
\***************************************************************************/
if (!defined("_ECRIRE_INC_VERSION")) {
return;
}
include_spip('base/abstract_sql');
function action_supprimer_tous_orphelins() {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
//on recupere le contexte pour ne supprimer les orphelins que de ce dernier
list($media, $distant, $statut, $sanstitre) = explode('/', $arg);
//critere sur le media
if ($media) {
$select = "media=" . sql_quote($media);
}
//critere sur le distant
if ($distant) {
$where[] = "distant=" . sql_quote($distant);
}
//critere sur le statut
if ($statut) {
$where[] = "statut REGEXP " . sql_quote("($statut)");
}
//critere sur le sanstitre
if ($sanstitre) {
$where[] = "titre=''";
}
//on isole les orphelins
$select = sql_get_select("DISTINCT id_document", "spip_documents_liens as oooo");
$cond = "spip_documents.id_document NOT IN ($select)";
$where[] = $cond;
$ids_doc_orphelins = sql_select("id_document", "spip_documents", $where);
$supprimer_document = charger_fonction('supprimer_document', 'action');
while ($row = sql_fetch($ids_doc_orphelins)) {
$supprimer_document($row['id_document']); // pour les orphelins du contexte, on traite avec la fonction existante
}
}
action/tourner.php
+ 7
− 9
Voir le fichier @ 69fc6182
......@@ -28,7 +28,7 @@ if (!defined("_ECRIRE_INC_VERSION")) {
* @return void
*/
function action_tourner_dist($id_document = null, $angle = null) {
if (is_null($id_document) OR is_null($angle)) {
if (is_null($id_document) or is_null($angle)) {
$securiser_action = charger_fonction('securiser_action', 'inc');
$arg = $securiser_action();
......@@ -40,7 +40,7 @@ function action_tourner_dist($id_document = null, $angle = null) {
}
}
if ($id_document AND autoriser('modifier', 'document', $id_document)) {
if ($id_document and autoriser('modifier', 'document', $id_document)) {
action_tourner_post($id_document, $angle);
}
}
......@@ -75,7 +75,7 @@ function action_tourner_post($id_document, $angle) {
$src = $match[1] . '.' . $match[3];
$var_rot += intval($match[2]);
}
$var_rot = ((360+$var_rot)%360); // 0, 90, 180 ou 270
$var_rot = ((360 + $var_rot) % 360); // 0, 90, 180 ou 270
if ($var_rot > 0) {
$dest = preg_replace(',\.[^.]+$,', '-r' . $var_rot . '$0', $src);
......@@ -99,7 +99,7 @@ function action_tourner_post($id_document, $angle) {
}
// succes !
if ($largeur > 0 AND $hauteur > 0) {
if ($largeur > 0 and $hauteur > 0) {
$set = array(
'fichier' => set_spip_doc($dest),
'largeur' => $largeur,
......@@ -139,10 +139,10 @@ function action_tourner_post($id_document, $angle) {
function tourner_selon_exif_orientation($id_document, $fichier) {
if (function_exists('exif_read_data')
AND $exif = exif_read_data($fichier)
AND (
and $exif = exif_read_data($fichier)
and (
$ort = $exif['IFD0']['Orientation']
OR $ort = $exif['Orientation'])
or $ort = $exif['Orientation'])
) {
spip_log("rotation: $ort");
$rot = null;
......@@ -159,5 +159,3 @@ function tourner_selon_exif_orientation($id_document, $fichier) {
}
}
}
?>
action/verifier_documents_brises.php
+ 0
− 2
Voir le fichier @ 69fc6182
......@@ -34,5 +34,3 @@ function action_verifier_documents_brises_dist() {
}
}
}
?>
\ No newline at end of file
action/verifier_documents_liens.php
+ 0
− 2
Voir le fichier @ 69fc6182
......@@ -39,5 +39,3 @@ function action_verifier_documents_liens_dist($id_document = null) {
objet_optimiser_liens(array('document' => $id_document), '*');
}
?>
base/medias.php
+ 2
− 4
Voir le fichier @ 69fc6182
......@@ -265,11 +265,11 @@ function creer_base_types_doc($serveur = '', $champ_media = "media_defaut") {
// type de media
$media = "file";
if (preg_match(",^image/,", $type_mime) OR in_array($type_mime, array('application/illustrator'))) {
if (preg_match(",^image/,", $type_mime) or in_array($type_mime, array('application/illustrator'))) {
$media = "image";
} elseif (preg_match(",^audio/,", $type_mime)) {
$media = "audio";
} elseif (preg_match(",^video/,", $type_mime) OR in_array($type_mime,
} elseif (preg_match(",^video/,", $type_mime) or in_array($type_mime,
array('application/ogg', 'application/x-shockwave-flash', 'application/mp4'))
) {
$media = "video";
......@@ -322,5 +322,3 @@ function medias_optimiser_base_disparus($flux) {
return $flux;
}
?>
base/typedoc.php
+ 406
− 408
Voir le fichier @ 69fc6182
Ce diff est replié.
formulaires/changer_fichier_document.php
+ 1
− 3
Voir le fichier @ 69fc6182
......@@ -51,7 +51,7 @@ function formulaires_changer_fichier_document_traiter_dist($id_document) {
} 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 @file_exists($f = get_spip_doc($ancien_fichier))
and @file_exists($f = get_spip_doc($ancien_fichier))
) {
spip_unlink($f);
}
......@@ -61,5 +61,3 @@ function formulaires_changer_fichier_document_traiter_dist($id_document) {
return $res;
}
?>
\ No newline at end of file
formulaires/configurer_documents.php
+ 0
− 1
Voir le fichier @ 69fc6182
......@@ -57,4 +57,3 @@ function formulaires_configurer_documents_traiter_dist() {
return $res;
}
formulaires/editer_document.php
+ 15
− 17
Voir le fichier @ 69fc6182
......@@ -36,7 +36,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'], array('article', 'rubrique')) AND $p['id_objet'] > 0) {
if (in_array($p['objet'], array('article', 'rubrique')) and $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'] . "' />";
......@@ -68,10 +68,10 @@ 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 and is_array($infos) and isset($infos['taille'])) {
if ($infos['taille'] != $valeurs['taille']
OR ($infos['type_image'] && ($infos['largeur'] != $valeurs['largeur']))
OR ($infos['type_image'] && ($infos['hauteur'] != $valeurs['hauteur']))
or ($infos['type_image'] && ($infos['largeur'] != $valeurs['largeur']))
or ($infos['type_image'] && ($infos['hauteur'] != $valeurs['hauteur']))
) {
$valeurs['_taille_modif'] = $infos['taille'];
$valeurs['_largeur_modif'] = $infos['largeur'];
......@@ -121,7 +121,7 @@ function formulaires_editer_document_verifier_dist(
is_numeric($id_document) ? array() : array('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') or _request('joindre_ftp') or _request('joindre_distant')) {
if (_request('copier_local')) {
} else {
$verifier = charger_fonction('verifier', 'formulaires/joindre_document');
......@@ -132,7 +132,7 @@ 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]))
or !($date = mktime($date[3], $date[4], 0, $date[1], $date[2], $date[0]))
) {
$erreurs['saisie_date'] = _T('medias:format_date_incorrect');
} else {
......@@ -162,7 +162,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 (array('taille', 'largeur', 'hauteur') as $c) {
if (($v = _request("_{$c}_modif")) AND !_request($c)) {
if (($v = _request("_{$c}_modif")) and !_request($c)) {
set_request($c, $v);
}
}
......@@ -172,10 +172,10 @@ function formulaires_editer_document_traiter_dist(
set_request('parents');
$autoclose = "<script type='text/javascript'>if (window.jQuery) jQuery.modalboxclose();</script>";
if (_request('copier_local')
OR _request('joindre_upload')
OR _request('joindre_ftp')
OR _request('joindre_distant')
OR _request('joindre_zip')
or _request('joindre_upload')
or _request('joindre_ftp')
or _request('joindre_distant')
or _request('joindre_zip')
) {
$autoclose = "";
if (_request('copier_local')) {
......@@ -190,7 +190,7 @@ 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 @file_exists($rename = get_spip_doc($ancien_fichier))
and @file_exists($rename = get_spip_doc($ancien_fichier))
) {
@rename($rename, "$rename--.old");
......@@ -215,13 +215,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') or _request('tournerL90_x')) {
$angle = -90;
}
if (_request('tournerR90') OR _request('tournerR90_x')) {
if (_request('tournerR90') or _request('tournerR90_x')) {
$angle = 90;
}
if (_request('tourner180') OR _request('tourner180_x')) {
if (_request('tourner180') or _request('tourner180_x')) {
$angle = 180;
}
if ($angle) {
......@@ -244,5 +244,3 @@ function formulaires_editer_document_traiter_dist(
return $res;
}
?>
formulaires/illustrer_document.php
+ 2
− 4
Voir le fichier @ 69fc6182
......@@ -55,7 +55,7 @@ function formulaires_illustrer_document_traiter_dist($id_document) {
$res = array('editable' => true);
if (_request('supprimer')) {
$supprimer_document = charger_fonction('supprimer_document', 'action');
if ($id_vignette AND $supprimer_document($id_vignette)) {
if ($id_vignette and $supprimer_document($id_vignette)) {
$res['message_ok'] = _T('medias:vignette_supprimee');
} else {
$res['message_erreur'] = _T('medias:erreur_suppression_vignette');
......@@ -69,7 +69,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)
and $id_vignette = reset($ajoute)
) {
include_spip('action/editer_document');
document_modifier($id_document, array("id_vignette" => $id_vignette, 'mode' => 'document'));
......@@ -85,5 +85,3 @@ function formulaires_illustrer_document_traiter_dist($id_document) {
return $res;
}
?>
\ No newline at end of file
formulaires/joindre_document.php
+ 484
− 486
Voir le fichier @ 69fc6182
Ce diff est replié.
inc/choisir_mode_document.php
+ 3
− 5
Voir le fichier @ 69fc6182
......@@ -34,12 +34,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'] or !$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 and isset($GLOBALS['meta']["documents_$objet"]) and ($GLOBALS['meta']["documents_$objet"] == 'non')) {
return 'image';
}
......@@ -58,12 +58,10 @@ function inc_choisir_mode_document($infos, $type_inclus_image, $objet) {
}
if ($infos['largeur'] > 0
AND $infos['largeur'] < _LARGEUR_MODE_IMAGE
and $infos['largeur'] < _LARGEUR_MODE_IMAGE
) {
return 'image';
} else {
return 'document';
}
}
?>
inc/documents.php
+ 9
− 12
Voir le fichier @ 69fc6182
......@@ -53,7 +53,7 @@ function contenu_document($arg, $charset = '') {
include_spip('inc/charset');
if ($charset !== 'auto') {
$r = importer_charset($r, $charset);
} elseif ($GLOBALS['meta']['charset'] == 'utf-8' AND !is_utf8($r)) {
} elseif ($GLOBALS['meta']['charset'] == 'utf-8' and !is_utf8($r)) {
$r = importer_charset($r, CHARSET_JOINT);
}
}
......@@ -84,7 +84,7 @@ function generer_url_document_dist($id_document, $args = '', $ancre = '') {
// Si droit de voir tous les docs, pas seulement celui-ci
// il est inutilement couteux de rajouter une protection
$r = (autoriser('voir', 'document'));
if (($r AND $r !== 'htaccess')) {
if (($r and $r !== 'htaccess')) {
return get_spip_doc($f);
}
......@@ -96,8 +96,7 @@ function generer_url_document_dist($id_document, $args = '', $ancre = '') {
. 'arg=' . $id_document
. ($ancre ? "&ancre=$ancre" : '')
. '&cle=' . calculer_cle_action($id_document . ',' . $f)
. '&file=' . rawurlencode($f)
, true, true);
. '&file=' . rawurlencode($f), true, true);
}
//
......@@ -117,7 +116,7 @@ function vignette_automatique($img, $doc, $lien, $x = 0, $y = 0, $align = '', $c
$e = $doc['extension'];
if (!$img) {
if ($img = image_du_document($doc)) {
if (!$x AND !$y) // eviter une double reduction
if (!$x and !$y) // eviter une double reduction
{
$img = image_reduire($img);
}
......@@ -134,7 +133,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 or $y) {
$img = image_reduire($img, $x, $y);
}
$img = inserer_attribut($img, 'alt', '');
......@@ -167,10 +166,10 @@ function vignette_automatique($img, $doc, $lien, $x = 0, $y = 0, $align = '', $c
function image_du_document($document) {
if ($e = $document['extension']
AND isset($GLOBALS['meta']['formats_graphiques'])
AND (strpos($GLOBALS['meta']['formats_graphiques'], $e) !== false)
AND (!test_espace_prive() OR $GLOBALS['meta']['creer_preview'] == 'oui')
AND $document['fichier']
and isset($GLOBALS['meta']['formats_graphiques'])
and (strpos($GLOBALS['meta']['formats_graphiques'], $e) !== false)
and (!test_espace_prive() or $GLOBALS['meta']['creer_preview'] == 'oui')
and $document['fichier']
) {
if ($document['distant'] == 'oui') {
$image = _DIR_RACINE . copie_locale($document['fichier']);
......@@ -237,5 +236,3 @@ function affiche_raccourci_doc($doc, $id, $align) {
return "\n<div style='text-align: $align'$onclick>&lt;$doc$id$pipe&gt;</div>\n";
}
?>
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