Skip to content
Extraits de code Groupes Projets

Comparer les révisions

Les modifications sont affichées comme si la révision source était fusionnée avec la révision cible. En savoir plus sur la comparaison des révisions.

Source

Sélectionner le projet cible
No results found

Cible

Sélectionner le projet cible
  • spip/medias
  • pierre.laszczak/medias
  • cy.altern/medias
  • MathieuAlphamosa/medias
  • taffit/medias
  • RealET/medias
  • johan/medias
  • Yohooo/medias
  • pierretux/medias
  • placido/medias
  • JLuc/medias
11 résultats
Afficher les modifications
Validations sur la source (12)
Masquer les modifications d'espaces
En ligne Côte à côte
Affichage de
avec 296 ajouts et 234 suppressions
.gitignore 0 → 100644
Voir le fichier @ c76b596a
/vendor/
/composer.phar
/composer.lock
/phpcs.xml
/phpstan.neon
/.php_cs.cache
/.php_cs.txt
action/acceder_document.php
Voir le fichier @ c76b596a
...@@ -31,7 +31,8 @@ function action_acceder_document_dist() { ...@@ -31,7 +31,8 @@ function action_acceder_document_dist() {
$arg = rawurldecode(_request('arg')); $arg = rawurldecode(_request('arg'));
$status = false; $status = false;
if (strpos($f, '../') !== false if (
strpos($f, '../') !== false
or preg_match(',^\w+://,', $f) or preg_match(',^\w+://,', $f)
) { ) {
$status = 403; $status = 403;
...@@ -52,7 +53,8 @@ function action_acceder_document_dist() { ...@@ -52,7 +53,8 @@ function action_acceder_document_dist() {
} else { } else {
// ETag pour gerer le status 304 // ETag pour gerer le status 304
$ETag = md5($file . ': ' . filemtime($file)); $ETag = md5($file . ': ' . filemtime($file));
if (isset($_SERVER['HTTP_IF_NONE_MATCH']) 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 http_status(304); // Not modified
...@@ -100,7 +102,8 @@ function action_acceder_document_dist() { ...@@ -100,7 +102,8 @@ function action_acceder_document_dist() {
// sinon l'ignorer car certains navigateurs pataugent // sinon l'ignorer car certains navigateurs pataugent
$f = basename($file); $f = basename($file);
if (isset($doc['titre']) 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']; $f = $doc['titre'];
......
action/ajouter_documents.php
Voir le fichier @ c76b596a
...@@ -44,7 +44,7 @@ include_spip('inc/renseigner_document'); ...@@ -44,7 +44,7 @@ include_spip('inc/renseigner_document');
*/ */
function action_ajouter_documents_dist($id_document, $files, $objet, $id_objet, $mode) { function action_ajouter_documents_dist($id_document, $files, $objet, $id_objet, $mode) {
$ajouter_un_document = charger_fonction('ajouter_un_document', 'action'); $ajouter_un_document = charger_fonction('ajouter_un_document', 'action');
$ajoutes = array(); $ajoutes = [];
// on ne peut mettre qu'un seul document a la place d'un autre ou en vignette d'un autre // on ne peut mettre qu'un seul document a la place d'un autre ou en vignette d'un autre
if (intval($id_document)) { if (intval($id_document)) {
...@@ -104,9 +104,11 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -104,9 +104,11 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
$mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode); $mode = ((isset($file['mode']) and $file['mode']) ? $file['mode'] : $mode);
include_spip('inc/modifier'); include_spip('inc/modifier');
if (isset($file['distant']) and $file['distant'] if (
and !in_array($mode, array('choix', 'auto', 'image', 'document'))) { isset($file['distant']) and $file['distant']
spip_log("document distant $source accepte sans verification, mode=$mode", 'medias'._LOG_INFO_IMPORTANTE); and !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'); include_spip('inc/distant');
$file['tmp_name'] = _DIR_RACINE . copie_locale($source); $file['tmp_name'] = _DIR_RACINE . copie_locale($source);
$source = $file['tmp_name']; $source = $file['tmp_name'];
...@@ -118,16 +120,24 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -118,16 +120,24 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// content-type est connu, et si possible recuperer la taille, voire plus. // 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']) {
if (!tester_url_absolue($source)) { if (!tester_url_absolue($source)) {
return _T('medias:erreur_chemin_distant', array('nom' => $source)); return _T('medias:erreur_chemin_distant', ['nom' => $source]);
} }
include_spip('inc/distant'); include_spip('inc/distant');
$source = str_replace(array("'",'"','<'),array("%27",'%22','%3C'), $source); $source = str_replace(["'",'"','<'], ['%27','%22','%3C'], $source);
if (is_array($a = renseigner_source_distante($source))) { if (is_array($a = renseigner_source_distante($source))) {
$champs = $a; $champs = $a;
# NB: dans les bonnes conditions (fichier autorise et pas trop gros) # NB: dans les bonnes conditions (fichier autorise et pas trop gros)
# $a['fichier'] est une copie locale du fichier # $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'])) {
$res_sanitize = sanitizer_document($champs['copie_locale'], $champs['extension']);
$infos = renseigner_taille_dimension_image($champs['copie_locale'], $champs['extension']);
}
else {
$infos = renseigner_taille_dimension_image($champs['fichier'], $champs['extension'], true);
}
$infos = renseigner_taille_dimension_image($champs['fichier'], $champs['extension'], true);
// on ignore erreur eventuelle sur $infos car on est distant, ca ne marche pas forcement // on ignore erreur eventuelle sur $infos car on est distant, ca ne marche pas forcement
if (is_array($infos)) { if (is_array($infos)) {
foreach ($infos as $k => $v) { foreach ($infos as $k => $v) {
...@@ -145,10 +155,9 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -145,10 +155,9 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
return $a; // message d'erreur return $a; // message d'erreur
} }
} else { // pas distant } else { // pas distant
$champs = [
$champs = array(
'distant' => 'non' 'distant' => 'non'
); ];
$champs['titre'] = ''; $champs['titre'] = '';
if ($titrer) { if ($titrer) {
...@@ -164,7 +173,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -164,7 +173,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
if (!is_array($fichier = fixer_fichier_upload($file, $mode))) { if (!is_array($fichier = fixer_fichier_upload($file, $mode))) {
return is_string($fichier) ? return is_string($fichier) ?
$fichier : _T('medias:erreur_upload_type_interdit', array('nom' => $file['name'])); $fichier : _T('medias:erreur_upload_type_interdit', ['nom' => $file['name']]);
} }
$champs['inclus'] = $fichier['inclus']; $champs['inclus'] = $fichier['inclus'];
...@@ -214,7 +223,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -214,7 +223,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
$champs = array_merge($champs, $infos); $champs = array_merge($champs, $infos);
// Si mode == 'choix', fixer le mode image/document // Si mode == 'choix', fixer le mode image/document
if (in_array($mode, array('choix', 'auto'))) { if (in_array($mode, ['choix', 'auto'])) {
$choisir_mode_document = charger_fonction('choisir_mode_document', 'inc'); $choisir_mode_document = charger_fonction('choisir_mode_document', 'inc');
$mode = $choisir_mode_document($champs, $champs['inclus'] == 'image', $objet); $mode = $choisir_mode_document($champs, $champs['inclus'] == 'image', $objet);
} }
...@@ -276,7 +285,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -276,7 +285,7 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
} }
} }
if (!$id_document) { if (!$id_document) {
return _T('medias:erreur_insertion_document_base', array('fichier' => '<em>' . $file['name'] . '</em>')); return _T('medias:erreur_insertion_document_base', ['fichier' => '<em>' . $file['name'] . '</em>']);
} }
document_modifier($id_document, $champs); document_modifier($id_document, $champs);
...@@ -286,8 +295,8 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -286,8 +295,8 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
// Ce plugin ferait quand même mieux de se placer dans metadata/jpg.php // Ce plugin ferait quand même mieux de se placer dans metadata/jpg.php
pipeline( pipeline(
'post_edition', 'post_edition',
array( [
'args' => array( 'args' => [
'table' => 'spip_documents', // compatibilite 'table' => 'spip_documents', // compatibilite
'table_objet' => 'documents', 'table_objet' => 'documents',
'spip_table_objet' => 'spip_documents', 'spip_table_objet' => 'spip_documents',
...@@ -297,9 +306,9 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet, ...@@ -297,9 +306,9 @@ function action_ajouter_un_document_dist($id_document, $file, $objet, $id_objet,
'serveur' => '', // serveur par defaut, on ne sait pas faire mieux pour le moment 'serveur' => '', // serveur par defaut, on ne sait pas faire mieux pour le moment
'action' => 'ajouter_document', 'action' => 'ajouter_document',
'operation' => 'ajouter_document', // compat <= v2.0 'operation' => 'ajouter_document', // compat <= v2.0
), ],
'data' => $champs 'data' => $champs
) ]
); );
return $id_document; return $id_document;
...@@ -317,7 +326,7 @@ function determiner_sous_dossier_document($ext, $fichier, $mode) { ...@@ -317,7 +326,7 @@ function determiner_sous_dossier_document($ext, $fichier, $mode) {
// si mode un logoxx on met dans logo/ // si mode un logoxx on met dans logo/
if (strncmp($mode, 'logo', 4) === 0) { if (strncmp($mode, 'logo', 4) === 0) {
return "logo"; return 'logo';
} }
return $ext; return $ext;
...@@ -376,26 +385,31 @@ function corriger_extension($ext) { ...@@ -376,26 +385,31 @@ function corriger_extension($ext) {
* - false ou message d'erreur si l'extension est refusée * - false ou message d'erreur si l'extension est refusée
*/ */
function verifier_upload_autorise($source, $mode = '') { function verifier_upload_autorise($source, $mode = '') {
$infos = array('fichier' => $source); $infos = ['fichier' => $source];
$res = false; $res = false;
if (preg_match(',\.([a-z0-9]+)(\?.*)?$,i', $source, $match) if (
preg_match(',\.([a-z0-9]+)(\?.*)?$,i', $source, $match)
and $ext = $match[1] and $ext = $match[1]
) { ) {
$ext = corriger_extension(strtolower($ext)); $ext = corriger_extension(strtolower($ext));
if ($res = sql_fetsel( if (
'extension,inclus,media_defaut as media', $res = sql_fetsel(
'spip_types_documents', 'extension,inclus,media_defaut as media',
'extension=' . sql_quote($ext) . " AND upload='oui'" 'spip_types_documents',
)) { 'extension=' . sql_quote($ext) . " AND upload='oui'"
)
) {
$infos = array_merge($infos, $res); $infos = array_merge($infos, $res);
} }
} }
if (!$res) { if (!$res) {
if ($res = sql_fetsel( if (
'extension,inclus,media_defaut as media', $res = sql_fetsel(
'spip_types_documents', 'extension,inclus,media_defaut as media',
"extension='zip' AND upload='oui'" 'spip_types_documents',
)) { "extension='zip' AND upload='oui'"
)
) {
$infos = array_merge($infos, $res); $infos = array_merge($infos, $res);
$res['autozip'] = true; $res['autozip'] = true;
} }
...@@ -449,7 +463,7 @@ function fixer_fichier_upload($file, $mode = '') { ...@@ -449,7 +463,7 @@ function fixer_fichier_upload($file, $mode = '') {
if ($row['fichier'] && (!$taille = @intval(filesize(get_spip_doc($row['fichier']))))) { if ($row['fichier'] && (!$taille = @intval(filesize(get_spip_doc($row['fichier']))))) {
spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)'); spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)');
spip_unlink(get_spip_doc($row['fichier'])); spip_unlink(get_spip_doc($row['fichier']));
return _T('medias:erreur_copie_fichier', array('nom' => $file['tmp_name'])); return _T('medias:erreur_copie_fichier', ['nom' => $file['tmp_name']]);
} else { } else {
return $row; return $row;
} }
...@@ -497,7 +511,7 @@ function fixer_fichier_upload($file, $mode = '') { ...@@ -497,7 +511,7 @@ function fixer_fichier_upload($file, $mode = '') {
spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)'); spip_log('Echec copie du fichier ' . $file['tmp_name'] . ' (taille de fichier indéfinie)');
spip_unlink(get_spip_doc($row['fichier'])); spip_unlink(get_spip_doc($row['fichier']));
return _T('medias:erreur_copie_fichier', array('nom' => $file['tmp_name'])); return _T('medias:erreur_copie_fichier', ['nom' => $file['tmp_name']]);
} else { } else {
return $row; return $row;
} }
......
action/changer_mode_document.php
Voir le fichier @ c76b596a
...@@ -37,7 +37,8 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) { ...@@ -37,7 +37,8 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) {
} }
} }
if ($id_document if (
$id_document
and include_spip('inc/autoriser') and include_spip('inc/autoriser')
and autoriser('modifier', 'document', $id_document) and autoriser('modifier', 'document', $id_document)
) { ) {
...@@ -49,10 +50,11 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) { ...@@ -49,10 +50,11 @@ function action_changer_mode_document_dist($id_document = null, $mode = null) {
function action_changer_mode_document_post($id_document, $mode) { function action_changer_mode_document_post($id_document, $mode) {
// - id_document le doc a modifier // - id_document le doc a modifier
// - mode le mode a lui donner // - mode le mode a lui donner
if ($id_document = intval($id_document) if (
and in_array($mode, array('vignette', 'image', 'document')) $id_document = intval($id_document)
and in_array($mode, ['vignette', 'image', 'document'])
) { ) {
include_spip('action/editer_document'); include_spip('action/editer_document');
document_modifier($id_document, array('mode' => $mode)); document_modifier($id_document, ['mode' => $mode]);
} }
} }
action/copier_local.php
Voir le fichier @ c76b596a
...@@ -54,22 +54,25 @@ function action_copier_local_dist($id_document = null) { ...@@ -54,22 +54,25 @@ function action_copier_local_dist($id_document = null) {
function action_copier_local_post($id_document) { function action_copier_local_post($id_document) {
// Il faut la source du document pour le copier // Il faut la source du document pour le copier
$row = sql_fetsel('mode,fichier, descriptif, credits', 'spip_documents', 'id_document='.intval($id_document)); $row = sql_fetsel('mode,fichier, descriptif, credits', 'spip_documents', 'id_document=' . intval($id_document));
$source = $row['fichier']; $source = $row['fichier'];
// si la source est bien un fichier distant // si la source est bien un fichier distant
// sinon c'est une donnee moisie, on ne fait rien // sinon c'est une donnee moisie, on ne fait rien
if (tester_url_absolue($source)) { include_spip('inc/distant');
include_spip('inc/distant'); // pour 'copie_locale' if (tester_url_absolue($source)
and valider_url_distante($source)) {
$fichier = copie_locale($source); $fichier = copie_locale($source);
if ($fichier if (
and tester_url_absolue($source)) { $fichier
and valider_url_distante($source)
) {
$fichier = _DIR_RACINE . $fichier; $fichier = _DIR_RACINE . $fichier;
$files = array(); $files = [];
$files[] = array('tmp_name' => $fichier, 'name' => basename($fichier)); $files[] = ['tmp_name' => $fichier, 'name' => basename($fichier)];
$ajouter_documents = charger_fonction('ajouter_documents', 'action'); $ajouter_documents = charger_fonction('ajouter_documents', 'action');
spip_log("convertit doc $id_document en local: $source => $fichier", 'medias'); spip_log("convertit doc $id_document en local: $source => $fichier", 'medias');
$liste = array(); $liste = [];
$ajouter_documents($id_document, $files, '', 0, $row['mode'], $liste); $ajouter_documents($id_document, $files, '', 0, $row['mode'], $liste);
spip_unlink($fichier); spip_unlink($fichier);
...@@ -78,16 +81,19 @@ function action_copier_local_post($id_document) { ...@@ -78,16 +81,19 @@ function action_copier_local_post($id_document) {
include_spip('action/editer_document'); include_spip('action/editer_document');
document_modifier( document_modifier(
$id_document, $id_document,
array('credits' => ($row['credits'] ? $row['credits'] . ', ' : '') . $source) ['credits' => ($row['credits'] ? $row['credits'] . ', ' : '') . $source]
); );
return true; return true;
} else { } else {
spip_log("echec copie locale $source", 'medias' . _LOG_ERREUR); spip_log("echec copie locale $source", 'medias' . _LOG_ERREUR);
if ($fichier) {
@unlink(_DIR_RACINE . $fichier);
}
} }
} else { } else {
spip_log("echec copie locale $source n'est pas une URL distante", 'medias' . _LOG_ERREUR); spip_log("echec copie locale $source n'est pas une URL distante", 'medias' . _LOG_ERREUR);
} }
return _T('medias:erreur_copie_fichier', array('nom' => $source)); return _T('medias:erreur_copie_fichier', ['nom' => $source]);
} }
action/desordonner_liens_documents.php
Voir le fichier @ c76b596a
...@@ -73,7 +73,7 @@ function desordonner_liens_documents($document, $objet, $id_objet) { ...@@ -73,7 +73,7 @@ function desordonner_liens_documents($document, $objet, $id_objet) {
} else { } else {
list($image, $mode) = explode('/', $document); list($image, $mode) = explode('/', $document);
$image = ($image == 'I'); $image = ($image == 'I');
$typdoc = sql_in('docs.extension', array('gif', 'jpg', 'png'), $image ? '' : 'NOT'); $typdoc = sql_in('docs.extension', ['gif', 'jpg', 'png'], $image ? '' : 'NOT');
$obj = 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet); $obj = 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet);
...@@ -101,8 +101,8 @@ function desordonner_liens_documents($document, $objet, $id_objet) { ...@@ -101,8 +101,8 @@ function desordonner_liens_documents($document, $objet, $id_objet) {
*/ */
function desordonner_liens_document($id_document, $objet, $id_objet) { function desordonner_liens_document($id_document, $objet, $id_objet) {
objet_qualifier_liens( objet_qualifier_liens(
array('document' => $id_document), ['document' => $id_document],
array($objet => $id_objet), [$objet => $id_objet],
array('rang_lien' => 0) ['rang_lien' => 0]
); );
} }
\ No newline at end of file
action/dissocier_document.php
Voir le fichier @ c76b596a
...@@ -56,7 +56,8 @@ function action_dissocier_document_dist($arg = null) { ...@@ -56,7 +56,8 @@ function action_dissocier_document_dist($arg = null) {
if (count($arg) > 4 and $arg[4] == 'safe') { if (count($arg) > 4 and $arg[4] == 'safe') {
$check = true; $check = true;
} }
if ($id_objet = intval($id_objet) if (
$id_objet = intval($id_objet)
and ( and (
($id_objet < 0 and $id_objet == -$GLOBALS['visiteur_session']['id_auteur']) ($id_objet < 0 and $id_objet == -$GLOBALS['visiteur_session']['id_auteur'])
or autoriser('dissocierdocuments', $objet, $id_objet) or autoriser('dissocierdocuments', $objet, $id_objet)
...@@ -91,11 +92,11 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa ...@@ -91,11 +92,11 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
// D'abord on ne supprime pas, on dissocie // D'abord on ne supprime pas, on dissocie
include_spip('action/editer_liens'); include_spip('action/editer_liens');
objet_dissocier(array('document' => $id_document), array($objet => $id_objet), array('role' => '*')); objet_dissocier(['document' => $id_document], [$objet => $id_objet], ['role' => '*']);
// Si c'est une vignette, l'eliminer du document auquel elle appartient // Si c'est une vignette, l'eliminer du document auquel elle appartient
// cas tordu peu probable // cas tordu peu probable
sql_updateq('spip_documents', array('id_vignette' => 0), 'id_vignette=' . $id_document); sql_updateq('spip_documents', ['id_vignette' => 0], 'id_vignette=' . $id_document);
// verifier son statut apres une suppression de lien // verifier son statut apres une suppression de lien
include_spip('action/editer_document'); include_spip('action/editer_document');
...@@ -107,17 +108,17 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa ...@@ -107,17 +108,17 @@ function supprimer_lien_document($id_document, $objet, $id_objet, $supprime = fa
pipeline( pipeline(
'post_edition', 'post_edition',
array( [
'args' => array( 'args' => [
'operation' => 'delier_document', // compat v<=2 'operation' => 'delier_document', // compat v<=2
'action' => 'delier_document', 'action' => 'delier_document',
'table' => 'spip_documents', 'table' => 'spip_documents',
'id_objet' => $id_document, 'id_objet' => $id_document,
'objet' => $objet, 'objet' => $objet,
'id' => $id_objet 'id' => $id_objet
), ],
'data' => null 'data' => null
) ]
); );
if ($check) { if ($check) {
...@@ -165,7 +166,7 @@ function dissocier_document($document, $objet, $id_objet, $supprime = false, $ch ...@@ -165,7 +166,7 @@ function dissocier_document($document, $objet, $id_objet, $supprime = false, $ch
} else { } else {
list($image, $mode) = explode('/', $document); list($image, $mode) = explode('/', $document);
$image = ($image == 'I'); $image = ($image == 'I');
$typdoc = sql_in('docs.extension', array('gif', 'jpg', 'png'), $image ? '' : 'NOT'); $typdoc = sql_in('docs.extension', ['gif', 'jpg', 'png'], $image ? '' : 'NOT');
$obj = 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet); $obj = 'id_objet=' . intval($id_objet) . ' AND objet=' . sql_quote($objet);
......
action/editer_document.php
Voir le fichier @ c76b596a
...@@ -34,12 +34,12 @@ function action_editer_document_dist($arg = null) { ...@@ -34,12 +34,12 @@ function action_editer_document_dist($arg = null) {
} }
if (!$id_document) { if (!$id_document) {
return array(0, ''); return [0, ''];
} // erreur } // erreur
$err = document_modifier($id_document); $err = document_modifier($id_document);
return array($id_document, $err); return [$id_document, $err];
} }
/** /**
...@@ -52,10 +52,10 @@ function action_editer_document_dist($arg = null) { ...@@ -52,10 +52,10 @@ function action_editer_document_dist($arg = null) {
*/ */
function document_inserer($id_parent = null, $set = null) { function document_inserer($id_parent = null, $set = null) {
$champs = array( $champs = [
'statut' => 'prop', 'statut' => 'prop',
'date' => 'NOW()', 'date' => 'NOW()',
); ];
if ($set) { if ($set) {
$champs = array_merge($champs, $set); $champs = array_merge($champs, $set);
...@@ -64,23 +64,23 @@ function document_inserer($id_parent = null, $set = null) { ...@@ -64,23 +64,23 @@ function document_inserer($id_parent = null, $set = null) {
// Envoyer aux plugins // Envoyer aux plugins
$champs = pipeline( $champs = pipeline(
'pre_insertion', 'pre_insertion',
array( [
'args' => array( 'args' => [
'table' => 'spip_documents', 'table' => 'spip_documents',
), ],
'data' => $champs 'data' => $champs
) ]
); );
$id_document = sql_insertq('spip_documents', $champs); $id_document = sql_insertq('spip_documents', $champs);
pipeline( pipeline(
'post_insertion', 'post_insertion',
array( [
'args' => array( 'args' => [
'table' => 'spip_documents', 'table' => 'spip_documents',
'id_objet' => $id_document 'id_objet' => $id_document
), ],
'data' => $champs 'data' => $champs
) ]
); );
return $id_document; return $id_document;
...@@ -105,7 +105,7 @@ function document_modifier($id_document, $set = null) { ...@@ -105,7 +105,7 @@ function document_modifier($id_document, $set = null) {
// white list // white list
objet_info('document', 'champs_editables'), objet_info('document', 'champs_editables'),
// black list // black list
array('parents', 'ajout_parents'), ['parents', 'ajout_parents'],
// donnees eventuellement fournies // donnees eventuellement fournies
$set $set
); );
...@@ -127,21 +127,24 @@ function document_modifier($id_document, $set = null) { ...@@ -127,21 +127,24 @@ function document_modifier($id_document, $set = null) {
$ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document)); $ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document));
} }
if ($err = objet_modifier_champs( if (
'document', $err = objet_modifier_champs(
$id_document, 'document',
array( $id_document,
[
'data' => $set, 'data' => $set,
'invalideur' => $invalideur, 'invalideur' => $invalideur,
'indexation' => $indexation 'indexation' => $indexation
), ],
$champs $champs
)) { )
) {
return $err; return $err;
} }
// nettoyer l'ancien fichier si necessaire // nettoyer l'ancien fichier si necessaire
if (isset($champs['fichier']) // un plugin a pu interdire la modif du fichier en virant le champ if (
isset($champs['fichier']) // un plugin a pu interdire la modif du fichier en virant le champ
and $champs['fichier'] and $champs['fichier']
and $ancien_fichier // on avait bien note le nom du fichier avant la modif 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 $ancien_fichier !== $champs['fichier'] // et il a ete modifie
...@@ -153,7 +156,7 @@ function document_modifier($id_document, $set = null) { ...@@ -153,7 +156,7 @@ function document_modifier($id_document, $set = null) {
// Changer le statut du document ? // Changer le statut du document ?
// le statut n'est jamais fixe manuellement mais decoule de celui des objets lies // le statut n'est jamais fixe manuellement mais decoule de celui des objets lies
$champs = collecter_requests(array('parents', 'ajouts_parents'), array(), $set); $champs = collecter_requests(['parents', 'ajouts_parents'], [], $set);
if (document_instituer($id_document, $champs)) { if (document_instituer($id_document, $champs)) {
// //
// Post-modifications // Post-modifications
...@@ -174,7 +177,7 @@ function document_modifier($id_document, $set = null) { ...@@ -174,7 +177,7 @@ function document_modifier($id_document, $set = null) {
* @param array $champs * @param array $champs
* @return bool * @return bool
*/ */
function document_instituer($id_document, $champs = array()) { function document_instituer($id_document, $champs = []) {
$statut = isset($champs['statut']) ? $champs['statut'] : null; $statut = isset($champs['statut']) ? $champs['statut'] : null;
$date_publication = isset($champs['date_publication']) ? $champs['date_publication'] : null; $date_publication = isset($champs['date_publication']) ? $champs['date_publication'] : null;
...@@ -185,11 +188,11 @@ function document_instituer($id_document, $champs = array()) { ...@@ -185,11 +188,11 @@ function document_instituer($id_document, $champs = array()) {
medias_revision_document_parents($id_document, $champs['ajout_parents'], true); medias_revision_document_parents($id_document, $champs['ajout_parents'], true);
} }
$row = sql_fetsel('statut,date_publication', 'spip_documents', 'id_document='.intval($id_document)); $row = sql_fetsel('statut,date_publication', 'spip_documents', 'id_document=' . intval($id_document));
$statut_ancien = $row['statut']; $statut_ancien = $row['statut'];
$date_publication_ancienne = $row['date_publication']; $date_publication_ancienne = $row['date_publication'];
$champs = array(); $champs = [];
/* Autodetermination du statut si non fourni */ /* Autodetermination du statut si non fourni */
if (is_null($statut)) { if (is_null($statut)) {
...@@ -200,7 +203,6 @@ function document_instituer($id_document, $champs = array()) { ...@@ -200,7 +203,6 @@ function document_instituer($id_document, $champs = array()) {
if ($champs === false) { if ($champs === false) {
return false; return false;
} }
} }
else { else {
if ($statut !== $statut_ancien) { if ($statut !== $statut_ancien) {
...@@ -208,24 +210,27 @@ function document_instituer($id_document, $champs = array()) { ...@@ -208,24 +210,27 @@ function document_instituer($id_document, $champs = array()) {
} }
} }
if (!is_null($date_publication) if (
!is_null($date_publication)
and empty($champs['date_publication']) and empty($champs['date_publication'])
and $date_publication != $date_publication_ancienne) { and $date_publication != $date_publication_ancienne
) {
$champs['date_publication'] = $date_publication; $champs['date_publication'] = $date_publication;
} }
// Envoyer aux plugins // Envoyer aux plugins
$champs = pipeline('pre_edition', $champs = pipeline(
array( 'pre_edition',
'args' => array( [
'args' => [
'table' => 'spip_documents', 'table' => 'spip_documents',
'id_objet' => $id_document, 'id_objet' => $id_document,
'action' => 'instituer', 'action' => 'instituer',
'statut_ancien' => $statut_ancien, 'statut_ancien' => $statut_ancien,
'date_ancienne' => $date_publication_ancienne, 'date_ancienne' => $date_publication_ancienne,
), ],
'data' => $champs 'data' => $champs
) ]
); );
if (!count($champs)) { if (!count($champs)) {
...@@ -242,7 +247,7 @@ function document_instituer($id_document, $champs = array()) { ...@@ -242,7 +247,7 @@ function document_instituer($id_document, $champs = array()) {
if (count($publier_rubriques)) { if (count($publier_rubriques)) {
include_spip('inc/rubriques'); include_spip('inc/rubriques');
foreach ($publier_rubriques as $r) { foreach ($publier_rubriques as $r) {
calculer_rubriques_if($r['id_objet'], array('statut' => $champs['statut']), $statut_ancien, false); calculer_rubriques_if($r['id_objet'], ['statut' => $champs['statut']], $statut_ancien, false);
} }
} }
} }
...@@ -251,17 +256,18 @@ function document_instituer($id_document, $champs = array()) { ...@@ -251,17 +256,18 @@ function document_instituer($id_document, $champs = array()) {
include_spip('inc/invalideur'); include_spip('inc/invalideur');
suivre_invalideur("id='document/$id_document'"); suivre_invalideur("id='document/$id_document'");
pipeline('post_edition', pipeline(
array( 'post_edition',
'args' => array( [
'args' => [
'table' => 'spip_documents', 'table' => 'spip_documents',
'id_objet' => $id_document, 'id_objet' => $id_document,
'action' => 'instituer', 'action' => 'instituer',
'statut_ancien' => $statut_ancien, 'statut_ancien' => $statut_ancien,
'date_ancienne' => $date_publication_ancienne, 'date_ancienne' => $date_publication_ancienne,
), ],
'data' => $champs 'data' => $champs
) ]
); );
return true; return true;
...@@ -283,13 +289,14 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout ...@@ -283,13 +289,14 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
return; return;
} }
$insertions = array(); $insertions = [];
$objets_parents = array(); // array('article'=>array(12,23)) $objets_parents = []; // array('article'=>array(12,23))
// au format objet|id_objet // au format objet|id_objet
foreach ($parents as $p) { foreach ($parents as $p) {
$p = explode('|', $p); $p = explode('|', $p);
if (preg_match('/^[a-z0-9_]+$/i', $objet = $p[0]) if (
preg_match('/^[a-z0-9_]+$/i', $objet = $p[0])
and (($p[1] = intval($p[1])) or in_array($objet, ['site', 'rubrique'])) and (($p[1] = intval($p[1])) or in_array($objet, ['site', 'rubrique']))
) { // securite ) { // securite
$objets_parents[$p[0]][] = $p[1]; $objets_parents[$p[0]][] = $p[1];
...@@ -298,14 +305,14 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout ...@@ -298,14 +305,14 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
include_spip('action/editer_liens'); include_spip('action/editer_liens');
// les liens actuels // les liens actuels
$liens = objet_trouver_liens(array('document' => $id_document), '*'); $liens = objet_trouver_liens(['document' => $id_document], '*');
$deja_parents = array(); $deja_parents = [];
// si ce n'est pas un ajout, il faut supprimer les liens actuels qui ne sont pas dans $objets_parents // si ce n'est pas un ajout, il faut supprimer les liens actuels qui ne sont pas dans $objets_parents
if (!$ajout) { if (!$ajout) {
foreach ($liens as $k => $lien) { 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'])) { if (autoriser('dissocierdocuments', $lien['objet'], $lien['id_objet'])) {
objet_dissocier(array('document' => $id_document), array($lien['objet'] => $lien['id_objet'])); objet_dissocier(['document' => $id_document], [$lien['objet'] => $lien['id_objet']]);
} }
unset($liens[$k]); unset($liens[$k]);
} else { } else {
...@@ -317,7 +324,8 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout ...@@ -317,7 +324,8 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
// trier les objets à traiter : ne pas prendre en compte ceux qui sont déjà associés ou qu'on n'a pas le droit d'associer // trier les objets à traiter : ne pas prendre en compte ceux qui sont déjà associés ou qu'on n'a pas le droit d'associer
foreach ($objets_parents as $objet => $ids) { foreach ($objets_parents as $objet => $ids) {
foreach ($ids as $k => $id) { foreach ($ids as $k => $id) {
if (( if (
(
isset($deja_parents[$objet]) isset($deja_parents[$objet])
and in_array($id, $deja_parents[$objet]) and in_array($id, $deja_parents[$objet])
) )
...@@ -327,5 +335,5 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout ...@@ -327,5 +335,5 @@ function medias_revision_document_parents($id_document, $parents = null, $ajout
} }
} }
} }
objet_associer(array('document' => $id_document), $objets_parents); objet_associer(['document' => $id_document], $objets_parents);
} }
action/ordonner_liens_documents.php
Voir le fichier @ c76b596a
...@@ -44,23 +44,25 @@ function action_ordonner_liens_dist() { ...@@ -44,23 +44,25 @@ function action_ordonner_liens_dist() {
list($_id_objet, $table_liens) = objet_associable($objet); list($_id_objet, $table_liens) = objet_associable($objet);
$success = $errors = array(); $success = $errors = [];
$actuels = sql_allfetsel( $actuels = sql_allfetsel(
array($_id_objet . ' AS id', 'rang_lien'), [$_id_objet . ' AS id', 'rang_lien'],
$table_liens, $table_liens,
array( [
sql_in($_id_objet, $ordre), sql_in($_id_objet, $ordre),
'objet = ' . sql_quote($objet_lie), 'objet = ' . sql_quote($objet_lie),
'id_objet = ' . sql_quote($id_objet_lie) 'id_objet = ' . sql_quote($id_objet_lie)
) ]
); );
$futurs = array_flip($ordre); $futurs = array_flip($ordre);
// ordre de 1 à n (pas de 0 à n). // ordre de 1 à n (pas de 0 à n).
array_walk($futurs, function(&$v) { $v++; }); array_walk($futurs, function (&$v) {
$v++;
});
$updates = array(); $updates = [];
foreach ($actuels as $l) { foreach ($actuels as $l) {
if ($futurs[$l['id']] !== $l['rang_lien']) { if ($futurs[$l['id']] !== $l['rang_lien']) {
...@@ -72,21 +74,21 @@ function action_ordonner_liens_dist() { ...@@ -72,21 +74,21 @@ function action_ordonner_liens_dist() {
foreach ($updates as $id => $ordre) { foreach ($updates as $id => $ordre) {
sql_updateq( sql_updateq(
$table_liens, $table_liens,
array('rang_lien' => $ordre), ['rang_lien' => $ordre],
array( [
$_id_objet . ' = ' . $id, $_id_objet . ' = ' . $id,
'objet = ' . sql_quote($objet_lie), 'objet = ' . sql_quote($objet_lie),
'id_objet = ' . sql_quote($id_objet_lie) 'id_objet = ' . sql_quote($id_objet_lie)
) ]
); );
} }
} }
return envoyer_json_envoi(array( return envoyer_json_envoi([
'done' => true, 'done' => true,
'success' => $success, 'success' => $success,
'errors' => $errors, 'errors' => $errors,
)); ]);
} }
function envoyer_json_envoi($data) { function envoyer_json_envoi($data) {
...@@ -95,9 +97,9 @@ function envoyer_json_envoi($data) { ...@@ -95,9 +97,9 @@ function envoyer_json_envoi($data) {
} }
function envoyer_json_erreur($msg) { function envoyer_json_erreur($msg) {
return envoyer_json_envoi(array( return envoyer_json_envoi([
'done' => false, 'done' => false,
'success' => array(), 'success' => [],
'errors' => array($msg) 'errors' => [$msg]
)); ]);
} }
action/supprimer_document.php
Voir le fichier @ c76b596a
...@@ -30,7 +30,7 @@ function action_supprimer_document_dist($id_document = 0) { ...@@ -30,7 +30,7 @@ function action_supprimer_document_dist($id_document = 0) {
// si c'etait une vignette, modifier le document source ! // si c'etait une vignette, modifier le document source !
if ($source = sql_getfetsel('id_document', 'spip_documents', 'id_vignette=' . intval($id_document))) { if ($source = sql_getfetsel('id_document', 'spip_documents', 'id_vignette=' . intval($id_document))) {
include_spip('action/editer_document'); include_spip('action/editer_document');
document_modifier($source, array('id_vignette' => 0)); document_modifier($source, ['id_vignette' => 0]);
} }
include_spip('inc/documents'); include_spip('inc/documents');
...@@ -46,7 +46,7 @@ function action_supprimer_document_dist($id_document = 0) { ...@@ -46,7 +46,7 @@ function action_supprimer_document_dist($id_document = 0) {
// Si c'est un document ayant une vignette, supprimer aussi la vignette // Si c'est un document ayant une vignette, supprimer aussi la vignette
if ($doc['id_vignette']) { if ($doc['id_vignette']) {
action_supprimer_document_dist($doc['id_vignette']); action_supprimer_document_dist($doc['id_vignette']);
objet_dissocier(array('document' => $doc['id_vignette']), '*'); objet_dissocier(['document' => $doc['id_vignette']], '*');
} }
// Si c'est un document ayant des documents annexes (sous-titre, ...) // Si c'est un document ayant des documents annexes (sous-titre, ...)
// les supprimer aussi // les supprimer aussi
...@@ -59,7 +59,7 @@ function action_supprimer_document_dist($id_document = 0) { ...@@ -59,7 +59,7 @@ function action_supprimer_document_dist($id_document = 0) {
} }
// dereferencer dans la base // dereferencer dans la base
objet_dissocier(array('document' => $id_document), '*', array('role' => '*')); objet_dissocier(['document' => $id_document], '*', ['role' => '*']);
sql_delete('spip_documents', 'id_document=' . intval($id_document)); sql_delete('spip_documents', 'id_document=' . intval($id_document));
...@@ -76,16 +76,16 @@ function action_supprimer_document_dist($id_document = 0) { ...@@ -76,16 +76,16 @@ function action_supprimer_document_dist($id_document = 0) {
pipeline( pipeline(
'post_edition', 'post_edition',
array( [
'args' => array( 'args' => [
'operation' => 'supprimer_document', // compat v<=2 'operation' => 'supprimer_document', // compat v<=2
'action' => 'supprimer_document', 'action' => 'supprimer_document',
'table' => 'spip_documents', 'table' => 'spip_documents',
'id_objet' => $id_document, 'id_objet' => $id_document,
'document' => $doc, 'document' => $doc,
), ],
'data' => null 'data' => null
) ]
); );
return true; return true;
......
action/supprimer_tous_orphelins.php
Voir le fichier @ c76b596a
...@@ -24,7 +24,7 @@ function action_supprimer_tous_orphelins() { ...@@ -24,7 +24,7 @@ function action_supprimer_tous_orphelins() {
//on recupere le contexte pour ne supprimer les orphelins que de ce dernier //on recupere le contexte pour ne supprimer les orphelins que de ce dernier
list($media, $distant, $statut, $sanstitre) = explode('/', $arg); list($media, $distant, $statut, $sanstitre) = explode('/', $arg);
$where = array(); $where = [];
//critere sur le media //critere sur le media
if ($media) { if ($media) {
$select = 'media=' . sql_quote($media); $select = 'media=' . sql_quote($media);
......
action/tourner.php
Voir le fichier @ c76b596a
...@@ -104,12 +104,12 @@ function action_tourner_post($id_document, $angle) { ...@@ -104,12 +104,12 @@ function action_tourner_post($id_document, $angle) {
// succes ! // succes !
if ($largeur > 0 and $hauteur > 0) { if ($largeur > 0 and $hauteur > 0) {
$set = array( $set = [
'fichier' => set_spip_doc($dest), 'fichier' => set_spip_doc($dest),
'largeur' => $largeur, 'largeur' => $largeur,
'hauteur' => $hauteur, 'hauteur' => $hauteur,
'distant' => 'non' // le document n'est plus distant apres une transformation 'distant' => 'non' // le document n'est plus distant apres une transformation
); ];
if ($taille = @filesize($dest)) { if ($taille = @filesize($dest)) {
$set['taille'] = $taille; $set['taille'] = $taille;
} }
...@@ -121,18 +121,18 @@ function action_tourner_post($id_document, $angle) { ...@@ -121,18 +121,18 @@ function action_tourner_post($id_document, $angle) {
// pipeline pour les plugins // pipeline pour les plugins
pipeline( pipeline(
'post_edition', 'post_edition',
array( [
'args' => array( 'args' => [
'table' => 'spip_documents', 'table' => 'spip_documents',
'table_objet' => 'documents', 'table_objet' => 'documents',
'spip_table_objet' => 'spip_documents', 'spip_table_objet' => 'spip_documents',
'type' => 'document', 'type' => 'document',
'id_objet' => $id_document, 'id_objet' => $id_document,
'champs' => array('rotation' => $angle, 'orientation' => $var_rot, 'fichier' => $row['fichier']), 'champs' => ['rotation' => $angle, 'orientation' => $var_rot, 'fichier' => $row['fichier']],
'action' => 'tourner', 'action' => 'tourner',
), ],
'data' => $set 'data' => $set
) ]
); );
} }
} }
...@@ -142,7 +142,8 @@ function action_tourner_post($id_document, $angle) { ...@@ -142,7 +142,8 @@ function action_tourner_post($id_document, $angle) {
// https://code.spip.net/@tourner_selon_exif_orientation // https://code.spip.net/@tourner_selon_exif_orientation
function tourner_selon_exif_orientation($id_document, $fichier) { function tourner_selon_exif_orientation($id_document, $fichier) {
if (function_exists('exif_read_data') if (
function_exists('exif_read_data')
and $exif = exif_read_data($fichier) and $exif = exif_read_data($fichier)
and ( and (
$ort = $exif['IFD0']['Orientation'] $ort = $exif['IFD0']['Orientation']
...@@ -161,7 +162,7 @@ function tourner_selon_exif_orientation($id_document, $fichier) { ...@@ -161,7 +162,7 @@ function tourner_selon_exif_orientation($id_document, $fichier) {
$rot = -90; $rot = -90;
} }
if ($rot) { if ($rot) {
action_tourner_post(array(null, $id_document, $rot)); action_tourner_post([null, $id_document, $rot]);
} }
} }
} }
action/verifier_documents_brises.php
Voir le fichier @ c76b596a
...@@ -29,7 +29,7 @@ function action_verifier_documents_brises_dist() { ...@@ -29,7 +29,7 @@ function action_verifier_documents_brises_dist() {
$res = sql_select('fichier,brise,id_document', 'spip_documents', "distant='non'"); $res = sql_select('fichier,brise,id_document', 'spip_documents', "distant='non'");
while ($row = sql_fetch($res)) { while ($row = sql_fetch($res)) {
if (($brise = !@file_exists(get_spip_doc($row['fichier']))) != $row['brise']) { if (($brise = !@file_exists(get_spip_doc($row['fichier']))) != $row['brise']) {
sql_updateq('spip_documents', array('brise' => $brise), 'id_document=' . intval($row['id_document'])); sql_updateq('spip_documents', ['brise' => $brise], 'id_document=' . intval($row['id_document']));
} }
} }
} }
......
action/verifier_documents_liens.php
Voir le fichier @ c76b596a
...@@ -36,5 +36,5 @@ function action_verifier_documents_liens_dist($id_document = null) { ...@@ -36,5 +36,5 @@ function action_verifier_documents_liens_dist($id_document = null) {
$id_document = ($id_document == '*') ? '*' : intval($id_document); $id_document = ($id_document == '*') ? '*' : intval($id_document);
include_spip('action/editer_liens'); include_spip('action/editer_liens');
objet_optimiser_liens(array('document' => $id_document), '*'); objet_optimiser_liens(['document' => $id_document], '*');
} }
base/medias.php
Voir le fichier @ c76b596a
...@@ -30,13 +30,13 @@ function medias_declarer_tables_interfaces($interfaces) { ...@@ -30,13 +30,13 @@ function medias_declarer_tables_interfaces($interfaces) {
$interfaces['table_des_tables']['documents'] = 'documents'; $interfaces['table_des_tables']['documents'] = 'documents';
$interfaces['table_des_tables']['types_documents'] = 'types_documents'; $interfaces['table_des_tables']['types_documents'] = 'types_documents';
$interfaces['exceptions_des_tables']['documents']['type_document'] = array('types_documents', 'titre'); $interfaces['exceptions_des_tables']['documents']['type_document'] = ['types_documents', 'titre'];
$interfaces['exceptions_des_tables']['documents']['extension_document'] = array('types_documents', 'extension'); $interfaces['exceptions_des_tables']['documents']['extension_document'] = ['types_documents', 'extension'];
$interfaces['exceptions_des_tables']['documents']['mime_type'] = array('types_documents', 'mime_type'); $interfaces['exceptions_des_tables']['documents']['mime_type'] = ['types_documents', 'mime_type'];
$interfaces['exceptions_des_tables']['documents']['media_document'] = array('types_documents', 'media'); $interfaces['exceptions_des_tables']['documents']['media_document'] = ['types_documents', 'media'];
$interfaces['exceptions_des_jointures']['spip_documents']['id_forum'] = array('spip_documents_liens', 'id_forum'); $interfaces['exceptions_des_jointures']['spip_documents']['id_forum'] = ['spip_documents_liens', 'id_forum'];
$interfaces['exceptions_des_jointures']['spip_documents']['vu'] = array('spip_documents_liens', 'vu'); $interfaces['exceptions_des_jointures']['spip_documents']['vu'] = ['spip_documents_liens', 'vu'];
$interfaces['table_date']['types_documents'] = 'date'; $interfaces['table_date']['types_documents'] = 'date';
$interfaces['table_des_traitements']['FICHIER'][] = 'get_spip_doc(%s)'; $interfaces['table_des_traitements']['FICHIER'][] = 'get_spip_doc(%s)';
...@@ -53,7 +53,7 @@ function medias_declarer_tables_interfaces($interfaces) { ...@@ -53,7 +53,7 @@ function medias_declarer_tables_interfaces($interfaces) {
*/ */
function medias_declarer_tables_principales($tables_principales) { function medias_declarer_tables_principales($tables_principales) {
$spip_types_documents = array( $spip_types_documents = [
'extension' => "varchar(10) DEFAULT '' NOT NULL", 'extension' => "varchar(10) DEFAULT '' NOT NULL",
'titre' => "text DEFAULT '' NOT NULL", 'titre' => "text DEFAULT '' NOT NULL",
'descriptif' => "text DEFAULT '' NOT NULL", 'descriptif' => "text DEFAULT '' NOT NULL",
...@@ -62,15 +62,15 @@ function medias_declarer_tables_principales($tables_principales) { ...@@ -62,15 +62,15 @@ function medias_declarer_tables_principales($tables_principales) {
'upload' => "ENUM('oui', 'non') DEFAULT 'oui' NOT NULL", 'upload' => "ENUM('oui', 'non') DEFAULT 'oui' NOT NULL",
'media_defaut' => "varchar(10) DEFAULT 'file' NOT NULL", 'media_defaut' => "varchar(10) DEFAULT 'file' NOT NULL",
'maj' => 'TIMESTAMP' 'maj' => 'TIMESTAMP'
); ];
$spip_types_documents_key = array( $spip_types_documents_key = [
'PRIMARY KEY' => 'extension', 'PRIMARY KEY' => 'extension',
'KEY inclus' => 'inclus' 'KEY inclus' => 'inclus'
); ];
$tables_principales['spip_types_documents'] = $tables_principales['spip_types_documents'] =
array('field' => &$spip_types_documents, 'key' => &$spip_types_documents_key); ['field' => &$spip_types_documents, 'key' => &$spip_types_documents_key];
return $tables_principales; return $tables_principales;
} }
...@@ -83,25 +83,25 @@ function medias_declarer_tables_principales($tables_principales) { ...@@ -83,25 +83,25 @@ function medias_declarer_tables_principales($tables_principales) {
*/ */
function medias_declarer_tables_auxiliaires($tables_auxiliaires) { function medias_declarer_tables_auxiliaires($tables_auxiliaires) {
$spip_documents_liens = array( $spip_documents_liens = [
'id_document' => "bigint(21) DEFAULT '0' NOT NULL", 'id_document' => "bigint(21) DEFAULT '0' NOT NULL",
'id_objet' => "bigint(21) DEFAULT '0' NOT NULL", 'id_objet' => "bigint(21) DEFAULT '0' NOT NULL",
'objet' => "VARCHAR (25) DEFAULT '' NOT NULL", 'objet' => "VARCHAR (25) DEFAULT '' NOT NULL",
'vu' => "ENUM('non', 'oui') DEFAULT 'non' NOT NULL", 'vu' => "ENUM('non', 'oui') DEFAULT 'non' NOT NULL",
'rang_lien' => "int(4) DEFAULT '0' NOT NULL" 'rang_lien' => "int(4) DEFAULT '0' NOT NULL"
); ];
$spip_documents_liens_key = array( $spip_documents_liens_key = [
'PRIMARY KEY' => 'id_document,id_objet,objet', 'PRIMARY KEY' => 'id_document,id_objet,objet',
'KEY id_document' => 'id_document', 'KEY id_document' => 'id_document',
'KEY id_objet' => 'id_objet', 'KEY id_objet' => 'id_objet',
'KEY objet' => 'objet', 'KEY objet' => 'objet',
); ];
$tables_auxiliaires['spip_documents_liens'] = array( $tables_auxiliaires['spip_documents_liens'] = [
'field' => &$spip_documents_liens, 'field' => &$spip_documents_liens,
'key' => &$spip_documents_liens_key 'key' => &$spip_documents_liens_key
); ];
return $tables_auxiliaires; return $tables_auxiliaires;
} }
...@@ -121,9 +121,9 @@ function medias_declarer_tables_objets_surnoms($surnoms) { ...@@ -121,9 +121,9 @@ function medias_declarer_tables_objets_surnoms($surnoms) {
function medias_declarer_tables_objets_sql($tables) { function medias_declarer_tables_objets_sql($tables) {
$tables['spip_articles']['champs_versionnes'][] = 'jointure_documents'; $tables['spip_articles']['champs_versionnes'][] = 'jointure_documents';
$tables['spip_documents'] = array( $tables['spip_documents'] = [
'table_objet_surnoms' => array('doc', 'img', 'emb'), 'table_objet_surnoms' => ['doc', 'img', 'emb'],
'type_surnoms' => array(), 'type_surnoms' => [],
'url_voir' => 'document_edit', 'url_voir' => 'document_edit',
'url_edit' => 'document_edit', 'url_edit' => 'document_edit',
'page' => '', 'page' => '',
...@@ -137,7 +137,7 @@ function medias_declarer_tables_objets_sql($tables) { ...@@ -137,7 +137,7 @@ function medias_declarer_tables_objets_sql($tables) {
'titre' => "CASE WHEN length(titre)>0 THEN titre ELSE fichier END as titre, '' AS lang", 'titre' => "CASE WHEN length(titre)>0 THEN titre ELSE fichier END as titre, '' AS lang",
'date' => 'date', 'date' => 'date',
'principale' => 'oui', 'principale' => 'oui',
'field' => array( 'field' => [
'id_document' => 'bigint(21) NOT NULL', 'id_document' => 'bigint(21) NOT NULL',
'id_vignette' => "bigint(21) DEFAULT '0' NOT NULL", 'id_vignette' => "bigint(21) DEFAULT '0' NOT NULL",
'extension' => "VARCHAR(10) DEFAULT '' NOT NULL", 'extension' => "VARCHAR(10) DEFAULT '' NOT NULL",
...@@ -158,34 +158,34 @@ function medias_declarer_tables_objets_sql($tables) { ...@@ -158,34 +158,34 @@ function medias_declarer_tables_objets_sql($tables) {
'date_publication' => "datetime DEFAULT '0000-00-00 00:00:00' NOT NULL", 'date_publication' => "datetime DEFAULT '0000-00-00 00:00:00' NOT NULL",
'brise' => 'tinyint DEFAULT 0', 'brise' => 'tinyint DEFAULT 0',
'maj' => 'TIMESTAMP' 'maj' => 'TIMESTAMP'
), ],
'key' => array( 'key' => [
'PRIMARY KEY' => 'id_document', 'PRIMARY KEY' => 'id_document',
'KEY id_vignette' => 'id_vignette', 'KEY id_vignette' => 'id_vignette',
'KEY mode' => 'mode', 'KEY mode' => 'mode',
'KEY extension' => 'extension' 'KEY extension' => 'extension'
), ],
'join' => array( 'join' => [
'id_document' => 'id_document', 'id_document' => 'id_document',
'extension' => 'extension' 'extension' => 'extension'
), ],
'statut' => array( 'statut' => [
array( [
'champ' => 'statut', 'champ' => 'statut',
'publie' => 'publie', 'publie' => 'publie',
'previsu' => 'publie,prop,prepa', 'previsu' => 'publie,prop,prepa',
'post_date' => 'date_publication', 'post_date' => 'date_publication',
'exception' => array('statut', 'tout') 'exception' => ['statut', 'tout']
) ]
), ],
'tables_jointures' => array('types_documents'), 'tables_jointures' => ['types_documents'],
'rechercher_champs' => array( 'rechercher_champs' => [
'titre' => 3, 'titre' => 3,
'descriptif' => 1, 'descriptif' => 1,
'fichier' => 1, 'fichier' => 1,
'credits' => 1, 'credits' => 1,
), ],
'champs_editables' => array( 'champs_editables' => [
'titre', 'titre',
'descriptif', 'descriptif',
'date', 'date',
...@@ -201,8 +201,8 @@ function medias_declarer_tables_objets_sql($tables) { ...@@ -201,8 +201,8 @@ function medias_declarer_tables_objets_sql($tables) {
'extension', 'extension',
'id_vignette', 'id_vignette',
'media' 'media'
), ],
'champs_versionnes' => array( 'champs_versionnes' => [
'id_vignette', 'id_vignette',
'titre', 'titre',
'descriptif', 'descriptif',
...@@ -214,17 +214,17 @@ function medias_declarer_tables_objets_sql($tables) { ...@@ -214,17 +214,17 @@ function medias_declarer_tables_objets_sql($tables) {
'credits', 'credits',
'fichier', 'fichier',
'distant' 'distant'
), ],
'modeles' => array('document', 'doc', 'img', 'emb', 'image', 'video', 'audio', 'file'), 'modeles' => ['document', 'doc', 'img', 'emb', 'image', 'video', 'audio', 'file'],
'modeles_styliser' => 'medias_modeles_styliser', 'modeles_styliser' => 'medias_modeles_styliser',
); ];
// jointures sur les forum pour tous les objets // jointures sur les forum pour tous les objets
$tables[]['tables_jointures'][] = 'documents_liens'; $tables[]['tables_jointures'][] = 'documents_liens';
// recherche jointe sur les documents pour les articles et rubriques // recherche jointe sur les documents pour les articles et rubriques
$tables['spip_articles']['rechercher_jointures']['document'] = array('titre' => 2, 'descriptif' => 1); $tables['spip_articles']['rechercher_jointures']['document'] = ['titre' => 2, 'descriptif' => 1];
$tables['spip_rubriques']['rechercher_jointures']['document'] = array('titre' => 2, 'descriptif' => 1); $tables['spip_rubriques']['rechercher_jointures']['document'] = ['titre' => 2, 'descriptif' => 1];
return $tables; return $tables;
} }
...@@ -243,13 +243,13 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') { ...@@ -243,13 +243,13 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') {
// charger en memoire tous les types deja definis pour limiter les requettes // charger en memoire tous les types deja definis pour limiter les requettes
$rows = sql_allfetsel('mime_type,titre,inclus,extension,' . $champ_media . ',upload,descriptif', 'spip_types_documents', '', '', '', '', '', $serveur); $rows = sql_allfetsel('mime_type,titre,inclus,extension,' . $champ_media . ',upload,descriptif', 'spip_types_documents', '', '', '', '', '', $serveur);
$deja = array(); $deja = [];
foreach ($rows as $k => $row) { foreach ($rows as $k => $row) {
$deja[$row['extension']] = &$rows[$k]; $deja[$row['extension']] = &$rows[$k];
} }
$insertions = array(); $insertions = [];
$updates = array(); $updates = [];
foreach ($tables_mime as $extension => $type_mime) { foreach ($tables_mime as $extension => $type_mime) {
if (isset($tables_images[$extension])) { if (isset($tables_images[$extension])) {
...@@ -271,18 +271,20 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') { ...@@ -271,18 +271,20 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') {
// type de media // type de media
$media = 'file'; $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, ['application/illustrator'])) {
$media = 'image'; $media = 'image';
} elseif (preg_match(',^audio/,', $type_mime)) { } elseif (preg_match(',^audio/,', $type_mime)) {
$media = 'audio'; $media = 'audio';
} elseif (preg_match(',^video/,', $type_mime) or in_array( } elseif (
$type_mime, preg_match(',^video/,', $type_mime) or in_array(
array('application/ogg', 'application/x-shockwave-flash', 'application/mp4') $type_mime,
)) { ['application/ogg', 'application/x-shockwave-flash', 'application/mp4']
)
) {
$media = 'video'; $media = 'video';
} }
$set = array( $set = [
'mime_type' => $type_mime, 'mime_type' => $type_mime,
'titre' => $titre, 'titre' => $titre,
'inclus' => $inclus, 'inclus' => $inclus,
...@@ -290,7 +292,7 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') { ...@@ -290,7 +292,7 @@ function creer_base_types_doc($serveur = '', $champ_media = 'media_defaut') {
$champ_media => $media, $champ_media => $media,
'upload' => 'oui', 'upload' => 'oui',
'descriptif' => '', 'descriptif' => '',
); ];
if (!isset($deja[$extension])) { if (!isset($deja[$extension])) {
$insertions[] = $set; $insertions[] = $set;
} elseif (array_diff($deja[$extension], $set)) { } elseif (array_diff($deja[$extension], $set)) {
...@@ -322,15 +324,17 @@ function medias_optimiser_base_disparus($flux) { ...@@ -322,15 +324,17 @@ function medias_optimiser_base_disparus($flux) {
// optimiser les liens morts : // optimiser les liens morts :
// entre documents vers des objets effaces // entre documents vers des objets effaces
// depuis des documents effaces // depuis des documents effaces
$flux['data'] += objet_optimiser_liens(array('document' => '*'), '*'); $flux['data'] += objet_optimiser_liens(['document' => '*'], '*');
// on ne nettoie volontairement pas automatiquement les documents orphelins // on ne nettoie volontairement pas automatiquement les documents orphelins
// mais il faut nettoyer les logos qui ne sont plus liés à rien // mais il faut nettoyer les logos qui ne sont plus liés à rien
$res = sql_select("D.id_document", $res = sql_select(
"spip_documents AS D 'D.id_document',
'spip_documents AS D
LEFT JOIN spip_documents_liens AS L LEFT JOIN spip_documents_liens AS L
ON (L.id_document=D.id_document)", ON (L.id_document=D.id_document)',
sql_in('D.mode', ['logoon', 'logooff']) . " AND L.id_document IS NULL"); sql_in('D.mode', ['logoon', 'logooff']) . ' AND L.id_document IS NULL'
);
$supprimer_document = charger_fonction('supprimer_document', 'action'); $supprimer_document = charger_fonction('supprimer_document', 'action');
while ($row = sql_fetch($res)) { while ($row = sql_fetch($res)) {
......
base/typedoc.php
Voir le fichier @ c76b596a
...@@ -16,7 +16,7 @@ if (!defined('_ECRIRE_INC_VERSION')) { ...@@ -16,7 +16,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
global $tables_images, $tables_sequences, $tables_documents, $tables_mime, $mime_alias; global $tables_images, $tables_sequences, $tables_documents, $tables_mime, $mime_alias;
$tables_images = array( $tables_images = [
// Images reconnues par PHP // Images reconnues par PHP
'jpg' => 'JPEG', 'jpg' => 'JPEG',
'png' => 'PNG', 'png' => 'PNG',
...@@ -26,11 +26,11 @@ $tables_images = array( ...@@ -26,11 +26,11 @@ $tables_images = array(
// Autres images (peuvent utiliser le tag <img>) // Autres images (peuvent utiliser le tag <img>)
'bmp' => 'BMP', 'bmp' => 'BMP',
'svg' => 'SVG', 'svg' => 'SVG',
); ];
// Multimedia (peuvent utiliser le tag <embed>) // Multimedia (peuvent utiliser le tag <embed>)
$tables_sequences = array( $tables_sequences = [
'aac' => 'Advanced Audio Coding', 'aac' => 'Advanced Audio Coding',
'ac3' => 'AC-3 Compressed Audio', 'ac3' => 'AC-3 Compressed Audio',
'aifc' => 'Compressed AIFF Audio', 'aifc' => 'Compressed AIFF Audio',
...@@ -86,10 +86,10 @@ $tables_sequences = array( ...@@ -86,10 +86,10 @@ $tables_sequences = array(
'y4m' => 'YUV4MPEG2', 'y4m' => 'YUV4MPEG2',
'3ga' => '3GP Audio File', '3ga' => '3GP Audio File',
'3gp' => '3rd Generation Partnership Project' '3gp' => '3rd Generation Partnership Project'
); ];
// Documents varies // Documents varies
$tables_documents = array( $tables_documents = [
'7z' => '7 Zip', '7z' => '7 Zip',
'abw' => 'Abiword', 'abw' => 'Abiword',
'ai' => 'Adobe Illustrator', 'ai' => 'Adobe Illustrator',
...@@ -210,9 +210,9 @@ $tables_documents = array( ...@@ -210,9 +210,9 @@ $tables_documents = array(
'xlsx' => 'Excel', 'xlsx' => 'Excel',
'xltm' => 'Excel template', 'xltm' => 'Excel template',
'xltx' => 'Excel template' 'xltx' => 'Excel template'
); ];
$tables_mime = array( $tables_mime = [
// Images reconnues par PHP // Images reconnues par PHP
'jpg' => 'image/jpeg', 'jpg' => 'image/jpeg',
'png' => 'image/png', 'png' => 'image/png',
...@@ -399,10 +399,10 @@ $tables_mime = array( ...@@ -399,10 +399,10 @@ $tables_mime = array(
'xlsx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet', 'xlsx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
'xltm' => 'application/vnd.ms-excel.template.macroEnabled.12', 'xltm' => 'application/vnd.ms-excel.template.macroEnabled.12',
'xltx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.template' 'xltx' => 'application/vnd.openxmlformats-officedocument.spreadsheetml.template'
); ];
$mime_alias = array( $mime_alias = [
'application/x-ogg' => 'application/ogg', 'application/x-ogg' => 'application/ogg',
'audio/3gpp' => 'video/3gpp', 'audio/3gpp' => 'video/3gpp',
'audio/x-mpeg' => 'audio/mpeg', 'audio/x-mpeg' => 'audio/mpeg',
...@@ -411,4 +411,4 @@ $mime_alias = array( ...@@ -411,4 +411,4 @@ $mime_alias = array(
'video/flv' => 'video/x-flv', 'video/flv' => 'video/x-flv',
'video/mp4' => 'application/mp4', 'video/mp4' => 'application/mp4',
'image/jpg' => 'image/jpeg' 'image/jpg' => 'image/jpeg'
); ];
composer.json 0 → 100644
Voir le fichier @ c76b596a
{
"require-dev": {
"dealerdirect/phpcodesniffer-composer-installer": "^0.7.1",
"spip/coding-standards": "^1.2",
"phpstan/phpstan": "^0.12.98"
}
}
formulaires/changer_fichier_document.php
Voir le fichier @ c76b596a
...@@ -17,7 +17,7 @@ if (!defined('_ECRIRE_INC_VERSION')) { ...@@ -17,7 +17,7 @@ if (!defined('_ECRIRE_INC_VERSION')) {
function formulaires_changer_fichier_document_charger_dist($id_document) { function formulaires_changer_fichier_document_charger_dist($id_document) {
$valeurs = sql_fetsel('id_document,fichier,distant', 'spip_documents', 'id_document=' . intval($id_document)); $valeurs = sql_fetsel('id_document,fichier,distant', 'spip_documents', 'id_document=' . intval($id_document));
if (!$valeurs) { if (!$valeurs) {
return array('editable' => false); return ['editable' => false];
} }
$charger = charger_fonction('charger', 'formulaires/joindre_document'); $charger = charger_fonction('charger', 'formulaires/joindre_document');
...@@ -29,7 +29,7 @@ function formulaires_changer_fichier_document_charger_dist($id_document) { ...@@ -29,7 +29,7 @@ function formulaires_changer_fichier_document_charger_dist($id_document) {
} }
function formulaires_changer_fichier_document_verifier_dist($id_document) { function formulaires_changer_fichier_document_verifier_dist($id_document) {
$erreurs = array(); $erreurs = [];
if (_request('copier_local')) { if (_request('copier_local')) {
} else { } else {
$verifier = charger_fonction('verifier', 'formulaires/joindre_document'); $verifier = charger_fonction('verifier', 'formulaires/joindre_document');
...@@ -42,7 +42,7 @@ function formulaires_changer_fichier_document_verifier_dist($id_document) { ...@@ -42,7 +42,7 @@ function formulaires_changer_fichier_document_verifier_dist($id_document) {
function formulaires_changer_fichier_document_traiter_dist($id_document) { function formulaires_changer_fichier_document_traiter_dist($id_document) {
if (_request('copier_local')) { if (_request('copier_local')) {
$copier_local = charger_fonction('copier_local', 'action'); $copier_local = charger_fonction('copier_local', 'action');
$res = array('editable' => true); $res = ['editable' => true];
if (($err = $copier_local($id_document)) === true) { if (($err = $copier_local($id_document)) === true) {
$res['message_ok'] = _T('medias:document_copie_locale_succes'); $res['message_ok'] = _T('medias:document_copie_locale_succes');
} else { } else {
...@@ -50,7 +50,8 @@ function formulaires_changer_fichier_document_traiter_dist($id_document) { ...@@ -50,7 +50,8 @@ function formulaires_changer_fichier_document_traiter_dist($id_document) {
} }
} else { } else {
// liberer le nom de l'ancien fichier pour permettre le remplacement par un fichier du meme nom // 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)) 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); spip_unlink($f);
......
formulaires/configurer_documents.php
Voir le fichier @ c76b596a
...@@ -27,11 +27,13 @@ if (!defined('_ECRIRE_INC_VERSION')) { ...@@ -27,11 +27,13 @@ if (!defined('_ECRIRE_INC_VERSION')) {
* Environnement du formulaire * Environnement du formulaire
**/ **/
function formulaires_configurer_documents_charger_dist() { function formulaires_configurer_documents_charger_dist() {
$valeurs = array(); $valeurs = [];
foreach (array( foreach (
[
'documents_objets', 'documents_objets',
'documents_date', 'documents_date',
) as $m) { ] as $m
) {
$valeurs[$m] = isset($GLOBALS['meta'][$m]) ? $GLOBALS['meta'][$m] : ''; $valeurs[$m] = isset($GLOBALS['meta'][$m]) ? $GLOBALS['meta'][$m] : '';
} }
$valeurs['documents_objets'] = explode(',', $valeurs['documents_objets']); $valeurs['documents_objets'] = explode(',', $valeurs['documents_objets']);
...@@ -46,7 +48,7 @@ function formulaires_configurer_documents_charger_dist() { ...@@ -46,7 +48,7 @@ function formulaires_configurer_documents_charger_dist() {
* Retours du traitement * Retours du traitement
**/ **/
function formulaires_configurer_documents_traiter_dist() { function formulaires_configurer_documents_traiter_dist() {
$res = array('editable' => true); $res = ['editable' => true];
if (!is_null($v = _request($m = 'documents_date'))) { if (!is_null($v = _request($m = 'documents_date'))) {
ecrire_meta($m, $v == 'oui' ? 'oui' : 'non'); ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
} }
......
formulaires/editer_document.php
Voir le fichier @ c76b596a
...@@ -25,7 +25,7 @@ function formulaires_editer_document_charger_dist( ...@@ -25,7 +25,7 @@ function formulaires_editer_document_charger_dist(
$retour = '', $retour = '',
$lier_trad = 0, $lier_trad = 0,
$config_fonc = 'documents_edit_config', $config_fonc = 'documents_edit_config',
$row = array(), $row = [],
$hidden = '' $hidden = ''
) { ) {
$valeurs = formulaires_editer_objet_charger('document', $id_document, $id_parent, $lier_trad, $retour, $config_fonc, $row, $hidden); $valeurs = formulaires_editer_objet_charger('document', $id_document, $id_parent, $lier_trad, $retour, $config_fonc, $row, $hidden);
...@@ -36,11 +36,11 @@ function formulaires_editer_document_charger_dist( ...@@ -36,11 +36,11 @@ function formulaires_editer_document_charger_dist(
} }
// relier les parents // relier les parents
$valeurs['parents'] = array(); $valeurs['parents'] = [];
$valeurs['_hidden'] = ''; $valeurs['_hidden'] = '';
$parents = sql_allfetsel('objet,id_objet', 'spip_documents_liens', 'id_document=' . intval($id_document)); $parents = sql_allfetsel('objet,id_objet', 'spip_documents_liens', 'id_document=' . intval($id_document));
foreach ($parents as $p) { foreach ($parents as $p) {
if (in_array($p['objet'], array('article', 'rubrique')) and $p['id_objet'] > 0) { if (in_array($p['objet'], ['article', 'rubrique']) and $p['id_objet'] > 0) {
$valeurs['parents'][] = $p['objet'] . '|' . $p['id_objet']; $valeurs['parents'][] = $p['objet'] . '|' . $p['id_objet'];
} else { } else {
$valeurs['_hidden'] .= "<input type='hidden' name='parents[]' value='" . $p['objet'] . '|' . $p['id_objet'] . "' />"; $valeurs['_hidden'] .= "<input type='hidden' name='parents[]' value='" . $p['objet'] . '|' . $p['id_objet'] . "' />";
...@@ -66,7 +66,7 @@ function formulaires_editer_document_charger_dist( ...@@ -66,7 +66,7 @@ function formulaires_editer_document_charger_dist(
); );
$valeurs['type_document'] = $row['type_document']; $valeurs['type_document'] = $row['type_document'];
$valeurs['_inclus'] = $row['inclus']; $valeurs['_inclus'] = $row['inclus'];
if (in_array($valeurs['extension'], array('jpg', 'gif', 'png', 'svg'))) { if (in_array($valeurs['extension'], ['jpg', 'gif', 'png', 'svg'])) {
$valeurs['apercu'] = get_spip_doc($valeurs['fichier']); $valeurs['apercu'] = get_spip_doc($valeurs['fichier']);
} }
...@@ -76,7 +76,8 @@ function formulaires_editer_document_charger_dist( ...@@ -76,7 +76,8 @@ function formulaires_editer_document_charger_dist(
include_spip('inc/renseigner_document'); include_spip('inc/renseigner_document');
$infos = renseigner_taille_dimension_image(get_spip_doc($valeurs['fichier']), $valeurs['extension']); $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'] if (
$infos['taille'] != $valeurs['taille']
or ($infos['type_image'] && ($infos['largeur'] != $valeurs['largeur'])) or ($infos['type_image'] && ($infos['largeur'] != $valeurs['largeur']))
or ($infos['type_image'] && ($infos['hauteur'] != $valeurs['hauteur'])) or ($infos['type_image'] && ($infos['hauteur'] != $valeurs['hauteur']))
) { ) {
...@@ -106,7 +107,7 @@ function formulaires_editer_document_charger_dist( ...@@ -106,7 +107,7 @@ function formulaires_editer_document_charger_dist(
function documents_edit_config($row) { function documents_edit_config($row) {
global $spip_lang; global $spip_lang;
$config = array();//$GLOBALS['meta']; $config = [];//$GLOBALS['meta'];
$config['lignes'] = 8; $config['lignes'] = 8;
$config['langue'] = $spip_lang; $config['langue'] = $spip_lang;
...@@ -121,10 +122,10 @@ function formulaires_editer_document_verifier_dist( ...@@ -121,10 +122,10 @@ function formulaires_editer_document_verifier_dist(
$retour = '', $retour = '',
$lier_trad = 0, $lier_trad = 0,
$config_fonc = 'documents_edit_config', $config_fonc = 'documents_edit_config',
$row = array(), $row = [],
$hidden = '' $hidden = ''
) { ) {
$erreurs = formulaires_editer_objet_verifier('document', $id_document, is_numeric($id_document) ? array() : array('titre')); $erreurs = formulaires_editer_objet_verifier('document', $id_document, is_numeric($id_document) ? [] : ['titre']);
// verifier l'upload si on a demande a changer le document // 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')) {
...@@ -137,7 +138,8 @@ function formulaires_editer_document_verifier_dist( ...@@ -137,7 +138,8 @@ function formulaires_editer_document_verifier_dist(
// On ne vérifie la date que si on avait le droit de la modifier // On ne vérifie la date que si on avait le droit de la modifier
if (lire_config('documents_date') == 'oui') { if (lire_config('documents_date') == 'oui') {
if (!$date = recup_date(_request('saisie_date') . ' ' . _request('saisie_heure') . ':00') 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'); $erreurs['saisie_date'] = _T('medias:format_date_incorrect');
...@@ -158,16 +160,16 @@ function formulaires_editer_document_traiter_dist( ...@@ -158,16 +160,16 @@ function formulaires_editer_document_traiter_dist(
$retour = '', $retour = '',
$lier_trad = 0, $lier_trad = 0,
$config_fonc = 'documents_edit_config', $config_fonc = 'documents_edit_config',
$row = array(), $row = [],
$hidden = '' $hidden = ''
) { ) {
if (is_null(_request('parents'))) { if (is_null(_request('parents'))) {
set_request('parents', array()); set_request('parents', []);
} }
// verifier les infos de taille et dimensions sur les fichiers locaux // verifier les infos de taille et dimensions sur les fichiers locaux
// cas des maj de fichier directes par ftp // cas des maj de fichier directes par ftp
foreach (array('taille', 'largeur', 'hauteur') as $c) { foreach (['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); set_request($c, $v);
} }
...@@ -176,7 +178,8 @@ function formulaires_editer_document_traiter_dist( ...@@ -176,7 +178,8 @@ function formulaires_editer_document_traiter_dist(
$res = formulaires_editer_objet_traiter('document', $id_document, $id_parent, $lier_trad, $retour, $config_fonc, $row, $hidden); $res = formulaires_editer_objet_traiter('document', $id_document, $id_parent, $lier_trad, $retour, $config_fonc, $row, $hidden);
set_request('parents'); set_request('parents');
$autoclose = "<script type='text/javascript'>if (window.jQuery) jQuery.modalboxclose();</script>"; $autoclose = "<script type='text/javascript'>if (window.jQuery) jQuery.modalboxclose();</script>";
if (_request('copier_local') if (
_request('copier_local')
or _request('joindre_upload') or _request('joindre_upload')
or _request('joindre_ftp') or _request('joindre_ftp')
or _request('joindre_distant') or _request('joindre_distant')
...@@ -185,7 +188,7 @@ function formulaires_editer_document_traiter_dist( ...@@ -185,7 +188,7 @@ function formulaires_editer_document_traiter_dist(
$autoclose = ''; $autoclose = '';
if (_request('copier_local')) { if (_request('copier_local')) {
$copier_local = charger_fonction('copier_local', 'action'); $copier_local = charger_fonction('copier_local', 'action');
$res = array('editable' => true); $res = ['editable' => true];
if (($err = $copier_local($id_document)) === true) { if (($err = $copier_local($id_document)) === true) {
$res['message_ok'] = (isset($res['message_ok']) ? $res['message_ok'] . '<br />' : '') . _T('medias:document_copie_locale_succes'); $res['message_ok'] = (isset($res['message_ok']) ? $res['message_ok'] . '<br />' : '') . _T('medias:document_copie_locale_succes');
} else { } else {
...@@ -194,7 +197,8 @@ function formulaires_editer_document_traiter_dist( ...@@ -194,7 +197,8 @@ function formulaires_editer_document_traiter_dist(
set_request('credits'); // modifie par la copie locale set_request('credits'); // modifie par la copie locale
} else { } else {
// liberer le nom de l'ancien fichier pour permettre le remplacement par un fichier du meme nom // 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)) if (
$ancien_fichier = sql_getfetsel('fichier', 'spip_documents', 'id_document=' . intval($id_document))
and !tester_url_absolue($ancien_fichier) and !tester_url_absolue($ancien_fichier)
and @file_exists($rename = get_spip_doc($ancien_fichier)) and @file_exists($rename = get_spip_doc($ancien_fichier))
) { ) {
......

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