Project

General

Profile

Evolution #4300

Rendre facultatif les plugins-dist

Added by Franck D 8 months ago. Updated about 1 month ago.

Status:
En cours
Priority:
Bas
Assignee:
-
Category:
-
Target version:
Start date:
03/01/2019
Due date:
% Done:

0%

Resolution:

Description

Hello :-)
Le commit de toutati https://zone.spip.net/trac/spip-zone/changeset/114194/spip-zone m'a fait repenser à ce vieux sujet (2009) https://blog.spip.net/SPIP-Core-tout-change-rien-ne-change.html
Jluc avait fait un sujet à ce propos https://contrib.spip.net/plugins-dist-indispensables-ou-non
Je me doute que cela représente un boulot de malade... mais ne faudrait'il pas dire au moins dans la F.A.Q. https://www.spip.net/fr_rubrique170.html les plugins-dist qui sont encore obligatoire ?

J'ai fait le choix de "evolution" et non de "documentation" concernant le ticket, car sans ticket personne n'y pense :-D
Pour spip 3.3, c'est sans doute un peu tard, mais à chaque sortie majeure, ne serait'il pas possible de rendre au moins "un" plugins-dist optionnel ?

Pour le moment, en spip 3.3 [24240], il y a comme plugins-dist https://zone.spip.net/trac/spip-zone/browser/spip-zone/_core_/plugins
Mais tous ne sont pas obligatoirement installer avec la dist (grenier, ?)
Faudrait rayer les plugs qui sont en options

Aide
Archiviste
Breves
Compagnon
Compresseur
dev
dist
dump
filtre_images
forum
grenier
jquery_ui
mediabox
medias
mots
msie_compat
organiseur
petitions
plan
porte_plume
revisions
safehtml
sites
squelettes_par-rubrique
statistique
svp
textwhell
url_etendues
vertèbres

Franck

selection_conditionnelle.diff View (2.98 KB) marcimat 🌈, 03/03/2019 07:12 PM

selection_conditionnelle_2.diff View (3.5 KB) marcimat 🌈, 03/03/2019 08:37 PM

selection_conditionnelle_3.diff View (5.32 KB) marcimat 🌈, 03/04/2019 01:43 PM

champs_selectionnes_3.png View (686 KB) marcimat 🌈, 03/04/2019 01:44 PM

History

#2 Updated by marcimat 🌈 8 months ago

Suite à https://www.mail-archive.com/spip-dev@rezo.net/msg66757.html

Une proposition de critère {selection_conditionnelle} qui crée des {id_xxx} pour chaque clé primaire des tables déclarées objets éditables.
Très simple. Mais aucune option au critère pour le moment.

#3 Updated by marcimat 🌈 8 months ago

  • File selection_conditionnelle_2.diff added

Un autre patch pour `selection_conditionnelle` qui prend en compte les champs id_xx de la table en cours également en plus.

#5 Updated by marcimat 🌈 8 months ago

  • File deleted (selection_conditionnelle_2.diff)

#6 Updated by marcimat 🌈 8 months ago

Nouvelle proposition (patch 3) :

- On accepte tous les champs 'id_xxx' de la table, ainsi que le champ 'objet' pour créer des critères conditionnels
- On accepte les champs id_xxx, primaires d’autres tables si :
-- notre primary fait partie des champs de cette autre table,
-- ou une table de liaison existe pour notre table
-- ou une table de liaison existe pour l’autre table

La fonction lister_champs_selection_conditionnelle($table) retourne la liste des champs possibles pour cette table

Squelette pour vérifier les champs sélectionnés pour chaque table :

<h1>#SQUELETTE</h1>

<style>
    table {
        border-collapse: collapse;
    }
    th {
        background-color:#eee;
    }
    td, th {
        text-align:left;
        border-top: 1px solid #aaa;
        border-left: 1px solid #ccc;
        border-right: 1px solid #ccc;
        padding:4px 8px;
    }
</style>
<B_tables>
    <table>
        <thead>
            <tr>
                <th>Table</th>
                <th>Nb champs</th>
                <th>Champs</th>
            </tr>
        </thead>
        <tbody>
<BOUCLE_tables(DATA){source tableau, #NULL|lister_tables_spip}>
<tr>
    <th>#CLE</th>
<B_champs>
    <td>#TOTAL_BOUCLE</td>
    <td>
<BOUCLE_champs(DATA){source tableau, #CLE|lister_champs_selection_conditionnelle}{", "}>
    #VALEUR
</BOUCLE_champs>
    </td>
</tr>
</BOUCLE_tables>
        </tbody>
    </table>
</B_tables>

#7 Updated by cedric - 8 months ago

Amha juste un pipeline à mettre sur la fonction lister_champs_selection_conditionnelle() pour que les plugins puissent y enlever/ajouter certains champs au besoin (ex de SVP qui pourrait enlever ses id_depot, id_plugin, id_paquet sur les auteurs pour pas charger la mule) et ça me parait très bien !

#8 Updated by b b 8 months ago

Voir r24243 & r24244 pour faire le lien (merci redmine ^^).

#9 Updated by b b 8 months ago

  • Status changed from Nouveau to En cours

#10 Updated by b b 8 months ago

Suite avec r24246 & r24247

#11 Updated by marcimat 🌈 about 1 month ago

Ça a bien avancé de ce côté, mais il y a encore quelques soucis sur un SPIP dans plugins-dist, dont un nouveau, sur chaque recherche de logo :

- Filtre image_recadre non défini | ../prive/squelettes/inclure/favicon-head.html
- Filtre image_format non défini | ../prive/squelettes/inclure/favicon-head.html
- Erreur SQL HY000 / 1 | no such table: spip_documents | inc/chercher_logo.php : inc_chercher_logo_dist(){ sql_fetsel(); }

SELECT D.* FROM spip_documents AS D JOIN spip_documents_liens AS L ON L.id_document=D.id_document WHERE D.mode='logoon' AND L.objet='site' AND id_objet=0

- Notice: Undefined index: forums_publics in inc/filtres_ecrire.php on line 173 dans avoir_visiteurs()
- Notice: Undefined index: messagerie_agenda in prive/objets/liste/auteurs_fonctions.php on line 154 dans auteur_lien_messagerie()
- Notice: Undefined index: toujours_paragrapher dans filtre_introduction_dist( )

Also available in: Atom PDF