JamesRezo a rédigé il y a 3 ans et marcimat a validé il y a 3 ans

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)

- une action "desordonner_liens_documents" est ajoutée (un peu sur le même fonctionnement que dissocier_document)
- une action "Réinitialiser l’ordre" est ajouté en fin de liste (visible si un rang_lien au moins est défini)
- les actions affectant l'ensemble des documents en fin de liste sont placées dans une div ".actions-liste"
- les documents déplaçables sont encadrés aussi d'une div (.sortable) essentiellement pour éviter de pouvoir les déplacer au dessus ou au dessous de la pagination ou des actions globales.

report de r96344 :  Lorsque des roles sont présents, faire en sorte que les boutons "Détacher" et "Supprimer" enlèvent effectivement tous les rôles.

Le comportement n'est pas modifié s'il n'y a pas de colonne de rôle, juste que la spec des fonctions objet_dissocier() indique
bien que sans précision, seul le lien avec le rôle par défaut est dissocié, ce qui ne semble pas le comportement souhaité ici.

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

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

Notices et erreur php en moins :  n'est pas défini dans l'une des fonctions de suppression de document. Dans le squelette de formulaire d'ajout de document, la variable fichier_upload revient parfois avec un tableau array(0) => ''. Je ne sais pas du tout quand elle sert, parce qu'en dehors de cela, elle semble tout le temps valoir ''. 

Mise a jour des annees pour les Copyrights (je ne connais pas l'utilite de cette info mais autant s'aligner sur les fichiers du core)

Ferme http://core.spip.org/issues/2632 : permettre de dissocier un document associe a la creation d'un article

on passe action=instituer dans post_edition, et non operation=instituer
de meme, homogeneisons en passant action partout

utilisation de la fonction objet_test_si_publie() pour publier un document en fonction du statut des objets liés

c'est un cas particulier car une rubrique est publiée des qu'elle contient un document qui l'est donc aussi.
il faut depublier la rubrique et mettre a jour le statut du document quand on supprime un lien document/objet

Lorsqu'on dissocie un document ajoute lors de la creation d'un article (ou autre), on appelle l'action/dissocier_document avec un id_objet negatif. Ne pas se prendre les pieds dans le tapis lors de la decomposition de $arg
(résoud accessoirement http://core.spip.org/issues/1990, mais sans aucun rapport avec la cause initiale du ticket)

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

[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)