id_table_objet manquant #1

Closed
opened 11 months ago by tcharlss · 3 comments
Owner

En jetant un coup d'oeil au dernier commit je note qu'il y a un problème visible un peu plus loin dans le fichier de fonctions, L.206 :

$id=$GLOBALS['page']['contexte']['id_'.$type];

devrait être remplacé par :

#include_spip('base/objets'); // si nécessaire
$cle_objet = id_table_objet($type);
$id = $GLOBALS['page']['contexte'][$cle_objet];

la clé primaire d'un objet éditorial n'est pas nécessairement basée sur le nom de la table.

Il y a peut être la même erreur ailleurs, à vérifier.

En jetant un coup d'oeil [au dernier commit](https://git.spip.net/spip-contrib-extensions/ckeditor-spip-plugin/commit/48009a1ce67294250230b2744c69c376f204cd51) je note qu'il y a un problème visible un peu plus loin [dans le fichier de fonctions, L.206](https://git.spip.net/spip-contrib-extensions/ckeditor-spip-plugin/src/branch/master/ckeditor_fonctions.php#L206) : ```php $id=$GLOBALS['page']['contexte']['id_'.$type]; ``` devrait être remplacé par : ```php #include_spip('base/objets'); // si nécessaire $cle_objet = id_table_objet($type); $id = $GLOBALS['page']['contexte'][$cle_objet]; ``` la clé primaire d'un objet éditorial n'est pas nécessairement basée sur le nom de la table. Il y a peut être la même erreur ailleurs, à vérifier.
Collaborator

Merci !

Est-ce que tu le fais ou est-ce que je le fais ?

Merci ! Est-ce que tu le fais ou est-ce que je le fais ?
Poster
Owner

J'ai vu ça en passant mais je n'utilise pas ce plugin.
Je te laisse corriger :)

J'ai vu ça en passant mais je n'utilise pas ce plugin. Je te laisse corriger :)
Collaborator

Intégré, merci

Intégré, merci
RealET closed this issue 11 months ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.