logique de la fonction lister_objets_lies #3176

Closed
opened 9 years ago by severo · 1 comments
severo commented 9 years ago

Actuellement la fonction lister_objets_lies (http://core.spip.org/projects/spip/repository/entry/spip/ecrire/inc/filtres_ecrire.php#L500) a un comportement différent selon qu'on recherche, disons, des articles liés à un ticket, ou des tickets liés à un ticket.

Dans le premier cas, s'il s'agit des tickets liés à l'article 10, on cherche (id_ticket=*, id_objet=10, objet=article) dans la table spip_tickets_liens.
Dans le deuxième cas, s'il s'agit des tickets liés au ticket 11, on cherche (id_ticket=11, id_objet=*, objet=ticket). Pourquoi ne pas faire simplement (id_ticket=*, id_objet=11, objet=ticket) qui est le comportement attendu.

Pour corriger (cerdic), il suffit de remplacer le test http://core.spip.org/projects/spip/repository/entry/spip/ecrire/inc/filtres_ecrire.php#L500


if ($objet_lien==$objet){
par

if ($objet_lien!==$objet_source){
dans le cas où les deux objets sont identiques.

Actuellement la fonction lister_objets_lies (http://core.spip.org/projects/spip/repository/entry/spip/ecrire/inc/filtres_ecrire.php#L500) a un comportement différent selon qu'on recherche, disons, des articles liés à un ticket, ou des tickets liés à un ticket. Dans le premier cas, s'il s'agit des tickets liés à l'article 10, on cherche `(id_ticket=*, id_objet=10, objet=article)` dans la table spip_tickets_liens. Dans le deuxième cas, s'il s'agit des tickets liés au ticket 11, on cherche `(id_ticket=11, id_objet=*, objet=ticket)`. Pourquoi ne pas faire simplement `(id_ticket=*, id_objet=11, objet=ticket)` qui est le comportement attendu. Pour corriger (cerdic), il suffit de remplacer le test http://core.spip.org/projects/spip/repository/entry/spip/ecrire/inc/filtres_ecrire.php#L500<pre> if ($objet_lien==$objet){ </pre>par<pre> if ($objet_lien!==$objet_source){ </pre>dans le cas où les deux objets sont identiques.
b_b commented 9 years ago
Owner

"Bonjour, je m'appelle Redmine, je ferme ce ticket automagiquement" ^^
Statut changé à Fermé

"Bonjour, je m'appelle Redmine, je ferme ce ticket automagiquement" ^^ **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
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.