Ticket #1084 (closed defect: fixed)
<tpl:LoopPosition> ne fonctionne pas dans <tpl:Attachments>
| Reported by: | Moe | Owned by: | akewea |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.5.2 |
| Component: | module:public | Version: | 2.2 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
La boucle <tpl:Attachments> utilise un ArrayObject? et non un "record" ce qui empêche la balise <tpl:LoopPosition> de fonctionner : http://dev.dotclear.org/2.0/browser/trunk/inc/public/class.dc.template.php?rev=3198#L746
Source : http://forum.dotclear.net/viewtopic.php?pid=288809#p288809
Change History
comment:2 Changed 12 years ago by kozlika
Bruno, le passage des pièces jointes en plugin a-t-il permis d'en profiter pour corriger ça ? http://dev.dotclear.org/2.0/ticket/1084
comment:3 Changed 11 years ago by franck
Non pas de résolution pour l'instant, le code généré est resté identique.
comment:8 Changed 10 years ago by franck
Je viens de voir le PR, c'est cool.
Je propose d'attendre que la 2.5.1 sorte (samedi 20/07 a priori) avant de le fusionner.
comment:9 Changed 10 years ago by franck
- Owner changed from team to akewea
- Status changed from onhold to new
comment:10 Changed 10 years ago by Franck Paul <carnet.franck.paul@…>
- Status changed from new to closed
- Resolution set to fixed
(In [6b02ef1bc544]) Merged in akewea/dotclear/ticket1084 (pull request #24)
fixes #1084
comment:11 Changed 10 years ago by franck
- Status changed from closed to reopened
- Resolution fixed deleted
La résolution est incomplète (il faudrait revoir le plugins attachment côté public) et voir ce qu'on peut faire pour les éventuels plugins/thèmes qui se serviraient de l'ancienne méthode.
comment:12 Changed 10 years ago by akewea
A la base, je n'avais pas mis le "staticRecord::newFromArray" dans le getPostMedia, pour éviter les effet de bord. Sauf que dans mon élan, j'ai tout de même modifié la méthode avec le ArrayObject?... :(
Dans l'état : il y a des corrections à faire sur l'admin d'un post et d'un page pour que cela fonctionne avec la nouvelle méthode.
Pour corriger et éviter les regressions, je vois plusieurs solutions :
- "getPostMedia" redevient comme avant mais deprecated, et on ajoute une méthode "getPostMediaRs" qui du coup retourne un resultset (intégre la convertion ArrayObject? et le staticRecord).
- "GetPostMedia?" prend un argument supplémentaire (boolean) pour indiquer le type de retour : comme avant (par défaut) ou ArrayObject? + resulset.
Dans les 2 cas, cela assure qu'il n'y aura pas de regression sur les plugins "hors distrib".
comment:13 Changed 10 years ago by franck
Personnellement je pencherai plutôt pour la seconde, de mon point de vue, ce n'est pas vraiment une "deprecated".
My two cents
comment:14 Changed 10 years ago by Franck Paul <carnet.franck.paul@…>
- Status changed from reopened to closed
- Resolution set to fixed
(In [383c341506b4]) Merged in akewea/dotclear/ticket1084 (pull request #45)
fixes #1084 (2)