`_image_extensions_acceptees_en_entree()` et pas
`_image_extension_logo()`.

Ref: spip/spip#5999

b_b a rédigé Il y a 8 mois et JamesRezo a validé Il y a 6 mois

fix: remplacer les usages de la globale `$formats_logos` par la fonction `_image_extensions_logos()`

à intégrer après spip/spip!5987

Ref: #4958

cerdic a rédigé 1 year ago et marcimat a validé 1 year ago

fix: normaliser la fonction autoriser_document_voir() pour lui faire retourner uniquement true ou false

Refs: #5369

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>

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 '='

Retour sur r89155 : il est autorise d'associer des documents a d'autres documents, certains plugins le font deja, evitons la casse pas trop justifiee (b_b, kent1, RealET)

Ferme #3443 et suite de #3380 , qui complète r89155 (et 89157) : il faut que l'autorisation d'associer des documents, tout comme
le fait déjà la nouvelle autorisation de dissocier des documents, prenne en compte les identifiants négatifs. 

Ils correspondent à l'ajout d'un document sur un nouvel article encore en édition et non encore enregistré (pas d'id_article connu),
ce qui affecte le document avec 0-$id_auteur qui l'a envoyé.

Report de r89155 : Fix https://core.spip.net/issues/3380 : il faut verifier les droits d'associer/dissocier les documents a un objet dans le formulaire editer objet.
Pour cela on cree 2 autorisations associerdocuments et dissocierdocuments qui par defaut reposent sur autoriser(modifier).
+ n'afficher le lien 'modifier' sur un document que si on a bien le droit de le modifier

un document non publie peut etre modifie par tout le monde mais il faut quand même être logué et avoir un id_auteur quand même?

Joindre des documents a la creation : eviter d'appeler une cascade d'autorisation si l'objet ne recoit pas de documents dans la config, et n'appeler autoriser(modifier) que si l'id de l'objet est bien >0 (sinon c'est une creation d'objet avec -)

quand l'argument est entier, preferer intval() que sql_quote() qui produit un '' erronne sur une valeur vide

generalisation du branchement sur objet, fusion des deux meta documents_article et documents_rubrique en une seule meta documents_objets

prevoir que l'on peut rattacher un document a un autre. C'est une pre-figuration de la reorganisation des documents.
Dans spip_documents_liens id_document designe toujours l'enfant (dependant de) et (id_objet,objet) le parent.
Donc ici un document annexe (sous-titrage, licence, ..) est designe par id_document, et (id_objet,objet='document') pointe vers le document maitre.
Des variantes du mode 'vignette' sont donc a prevoir, pour les documents annexes