Skip to content
Extraits de code Groupes Projets
Valider d86d3cb9 rédigé par Maïeul's avatar Maïeul
Parcourir les fichiers

Code mort

hop @rastapopoulos

en faisant de l'archéologie dans la doc de de formidable, j'ai suis tombé sur https://contrib.spip.net/ecrire/?exec=article&id_article=3486

j'ai vu que prévoyais au départ une fonction `update` pour les traitements. Et de facto cette fonction existait pour le traitement enregistrement.

Mais par ailleurs il n'y a aucun appel à cette fonction dans le code. De plus il me semble que c'est piégeux, car ca risque de demultiplier le code. Comme par ailleurs j'avais moi même ajouté `$retours['modification_reponse'] = true` au retour du traitement enregistrer, permettant aux traitements ultérieurs d'avoir l'info, je ne vois pas la logique de cette fonction distincte.

Je supprime donc.
parent c3953123
Aucune branche associée trouvée
Aucune étiquette associée trouvée
Aucune requête de fusion associée trouvée
...@@ -183,39 +183,6 @@ function traiter_enregistrement_dist($args, $retours) { ...@@ -183,39 +183,6 @@ function traiter_enregistrement_dist($args, $retours) {
return $retours; return $retours;
} }
function traiter_enregistrement_update_dist($id_formulaire, $traitement, $saisies_anciennes, $saisies_nouvelles) {
include_spip('inc/saisies');
include_spip('base/abstract_sql');
$comparaison = saisies_comparer($saisies_anciennes, $saisies_nouvelles);
// Si des champs ont été supprimés, il faut supprimer les réponses à ces champs
if ($comparaison['supprimees']) {
// On récupère les réponses du formulaire
$reponses = sql_allfetsel(
'id_formulaires_reponse',
'spip_formulaires_reponses',
'id_formulaire = '.$id_formulaire
);
$reponses = array_map('reset', $reponses);
// Tous les noms de champs à supprimer
$noms = array_keys($comparaison['supprimees']);
// On supprime
sql_delete(
'spip_formulaires_reponses_champs',
array(
sql_in('id_formulaires_reponse', $reponses),
sql_in('nom', $noms)
)
);
// On efface les vieux fichiers
foreach ($noms as $nom) {
if ($comparaison['supprimees'][$nom]['saisie'] == 'fichiers') {
formidable_effacer_fichiers_champ($id_formulaire, $reponses, $nom);
}
}
}
}
function traiter_enregistrement_verifier_dist($args, $erreurs) { function traiter_enregistrement_verifier_dist($args, $erreurs) {
$id_formulaire = $args['id_formulaire']; $id_formulaire = $args['id_formulaire'];
......
0% Chargement en cours ou .
You are about to add 0 people to the discussion. Proceed with caution.
Terminez d'abord l'édition de ce message.
Veuillez vous inscrire ou vous pour commenter