Development #7400
Ajout d'un objet "Lien interne"
0%
Description
Il y a maintenant la possibilité de faire simplement un lien, un lien ver une catégorie wcs, mais pas possibilité de faireu n lien vers une page interne de combo. Donc voilà demande effectuée
Fichiers
Révisions associées
linkcell with link_page and anchor: add tests (#7400)
linkcell: link_page can be empty (#7400, cont.)
linkcell: additional label can be the internal page title (#7400, cont.)
Historique
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-linkcell-can-link-to-a-combo-page-7400.patch 0001-linkcell-can-link-to-a-combo-page-7400.patch ajouté
Proposition de patch qui ajoute une liste "link page" dans la cellule "lien" (#7322)
Si une page est indiquée, c'est son URL et son titre qui forment le lien, sauf si un titre est proposé.
Si ok sur le principe, j'ajoute des tests.
Mis à jour par Thomas Noël il y a presque 9 ans
- Assigné à mis à Thomas Noël
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-linkcell-can-link-to-a-page-can-have-an-anchor-7400.patch 0001-linkcell-can-link-to-a-page-can-have-an-anchor-7400.patch ajouté
- Statut changé de Nouveau à En cours
sur idée de Fred, ajout d'un champ "ancre" (ce qui permettra de faire un lien vers une page interne + ancre)
Mis à jour par Frédéric Péters il y a presque 9 ans
Ça me va (même si je me rends compte que le get_cell_extra_context n'a sans doute pas des tonnes d'utilité), avec quelques lignes de test ça sera ok.
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-linkcell-with-link_page-and-anchor-add-tests-7400.patch 0001-linkcell-with-link_page-and-anchor-add-tests-7400.patch ajouté
Poussé par erreur avec un message de commit mal espacé :/
Voici en patch séparé, les tests.
Mis à jour par Thomas Noël il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
Ack via jabber
commit e53d411fb20da4155c0b5d6bb324b5aeb2c90770 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Fri May 29 15:36:55 2015 +0200 linkcell with link_page and anchor: add tests (#7400)
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0002-linkcell-additional-label-can-be-the-internal-page-t.patch 0002-linkcell-additional-label-can-be-the-internal-page-t.patch ajouté
- Fichier 0001-linkcell-link_page-can-be-empty-7400-cont.patch ajouté
- Statut changé de Résolu (à déployer) à En cours
Le premier bogue (absence du blank=True sur le link_page) est passé au travers des tests (et ça m'étonne).
Le second est une histoire d'affichage.
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier 0001-linkcell-link_page-can-be-empty-7400-cont.patch 0001-linkcell-link_page-can-be-empty-7400-cont.patch ajouté
Manquait la migration dans le 1er
Mis à jour par Thomas Noël il y a presque 9 ans
- Fichier
0001-linkcell-link_page-can-be-empty-7400-cont.patchsupprimé
Mis à jour par Frédéric Péters il y a presque 9 ans
(ça passe les tests parce qu'ils ne passent pas par les formulaires).
ok.
Mis à jour par Thomas Noël il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
Modif poussés
commit b0e8071708a470826afad7d9765f57c0429ff051 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Fri May 29 17:02:30 2015 +0200 linkcell: additional label can be the internal page title (#7400, cont.) commit 6d6bd722e2d602b46a95fe52fd028d2b1bd4fd55 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Fri May 29 17:01:47 2015 +0200 linkcell: link_page can be empty (#7400, cont.)
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
linkcell: can link to a page; can have an anchor(#7400)