Valider bffba08a rédigé par marcimat's avatar marcimat
Parcourir les fichiers

Fix: Ne pas enregistrer d’identifiant si aucun identifiant n’est transmis (null)

Je suppute que ce code dans le pipeline `pre_edition` est devenu inutile,
car il me semble que tous les `collecter_requests()`, du moins du core
et des plugins-dist, utilisent bien la liste des champs éditables connus.

Refs: #4
parent cfadc06a
Chargement en cours
Chargement en cours
Chargement en cours
Chargement en cours
+4 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -138,10 +138,12 @@ function identifiants_formulaire_verifier($flux) {
 *
 * @note
 * On ne devrait pas avoir à utiliser ce pipeline,
 * mais certains objets mettent une whitelist en dur dans leur action de modification
 * mais certains objets mettent une liste d’inclusion en dur dans leur action de modification
 * au lieu de prendre la liste des champs éditables, du coup les champs ajoutés
 * dans `declarer_tables_objets_sql()` passent à la trappe.
 *
 * @see collecter_requests()
 *
 * @pipeline editer_contenu_objet
 * @param  array $flux Données du pipeline
 * @return array       Données du pipeline
@@ -155,6 +157,7 @@ function identifiants_pre_edition($flux) {
		and $table_objet = $flux['args']['spip_table_objet']
		and $tables_identifiables = identifiants_lister_tables_identifiables()
		and in_array($table_objet, $tables_identifiables)
		and _request('identifiant') !== null
		and include_spip('inc/autoriser')
		and autoriser('modifier', '_identifiant')
	) {