JamesRezo a rédigé 3 years ago et marcimat a validé 3 years ago

Co-authored-by: JamesRezo <james@rezo.net>
Reviewed-on: https://git.spip.net/spip/medias/pulls/4854


Co-authored-by: JamesRezo <jamesrezo@noreply.git.spip.net>
Co-committed-by: JamesRezo <jamesrezo@noreply.git.spip.net>
(cherry picked from commit a5073a0a)

cerdic a rédigé 3 years ago et marcimat a validé 3 years ago

Quand on ajoute un document distant ne pas perdre la trace de la copie_locale eventuellement faite au passage, l'utiliser pour avoir les infos du document (plus rapide) et la sanitizer si besoin

(cherry picked from commit 13c293fa + eedbe981)

On introduit un sanitizer explicite, definissable extension par extension, car sanitizer dans la fonction de lecture des metadata est tout sauf propre et fiable

Revert de r106288 par svn merge -r106288:106287 . car entites_html n'est pas toujours chargee, donc risque erreurs fatales, et le probleme a ete traite en amont dans la fonction _T de maniere plus generique

echapper les entites du nom de fichier/source quand affiche dans les messages d'erreur (Jarrod Farncomb)

Il arrive parfois que des documents images soient téléversés avec la mauvaise extension par rapport à leur contenu réel. Par exemple recevoir un fichier.jpg alors que son contenu est du png.
On corrige cette subtilité lorsqu’on ajoute un document. Dans ce cas le nom du fichier reçoit en complément la nouvelle extension (il devient donc ici fichier.jpg.png), et on le stocke
dans le bon répertoire (ie: IMG/png/fichier.jpg.png). Quelques logs au passage de ces cas.

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

Regles : 
- un espace après chaque virgule
- un espace avant et apres chaque '='

Quand on insere un document distant, essayer d'en lire les meta-info en en recuperant les premiers 512ko.
Permet de recuperer les dimensions d'une video distante (ca semble marcher sur mp4 et webm, a confirmer sur les autres types de fichier)

On ajoute un pipeline renseigner_document, pendant de renseigner_document_distant pour les fichiers locaux car :
- On peut avoir plusieurs plugins récupérant des infos pour la même extension de fichier et on peut alors les combiner;
- Evite pour des plugins qui récupèrent les infos de x extentions diverses (pas un type en particulier) de créer x fichiers pour chaque extension de fichier (exemple xmp_php => http://zone.spip.org/trac/spip-zone/browser/_plugins_/xmp_php/trunk)

Report de r73073 : une coquille empechait la prise en charge des svg à l'upload (lecture des dimensions, protection des js inclus) (Sebastien)

Le media d'un document (image, video, audio, file) utilise par convenance dans l'interface ne peut dependre en dur de l'extension du fichier :
avec oEmbed, on peut se retrouver a integrer toute sorte de media au format html
Du coup on revoit la structure de la base :
spip_types_documents porte un champ media_defaut (renommage de l'ancien media) utilise comme valeur par defaut du media si non renseigne quand un document est ajoute
spip_documents porte son propre champ media, qui est peuple en fonction de media_defaut, sauf si renseigne lors de l'ajout du document
(ie via fonction de description ou pipeline)
Revision du code partout ou on referencait media, en dispatchant sur l'un ou l'autre selon les cas
upgrade de la base (en deux fois)

En fait c'est un problème de base donc je reviens sur le commit précédent ... c'est le int(11) de la table spip_documents pour le champs taille qui n'est pas suffisant apparemment ...

On fera une mise à jour de base après tests

SPIP ne pose pas de problèmes avec des fichiers de plus de 2Go sauf qu'il ne calcule pas bien leur taille ... ce petit patch pour cela... comme dans le core de la 2.1 depuis 5 minutes

http://core.spip.org/issues/2304

erreur dans r46173 : array_merge se faisant dans le mauvais sens, et de plus il ne faut completer les informations que si on passe par recuperer_infos_distantes.
En contrepartie les plugins doivent fournir fichier et mode a minima

Un pipeline renseigner_document_distant permet a un plugin de découvrir un document se cachant derriere une url avec sa propre methode (ex :oembed, mais non limitatif).
Les plugins qui se branchent dessus cherchent a renseigner si si $flux['fichier'] n'est pas encore renseigné
En sortie du pipeline, si $flux['fichier'] n'est pas present, on se rabat sur la méthode traditionnelle.
Tout le contenu du pipeline alimente directement la base, tous les champs de la table spip_documents peuvent donc être renseignés.

extraire les methodes de determination des metas data (largeur, hauteur, duree, ...) par extension, afin de permettre d'ajouter facilement de nouvelles methodes.
Integrer la methode de lecture de la taille et duree des flv (lapin_malin)

[en cours] merge avec le plugin mediatheque  (les documents ne sont plus fonctionnels en l'etat, ne pas mettre a jour dans cette version hors contexte developpement)