Ticket #1820 (closed defect: fixed)
Pages secondaires des actions sur commentaires
| Reported by: | franck | Owned by: | bruno |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.6 |
| Component: | module:core | Version: | 2.5 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Je viens de tester avec le plugin typo, la page secondaire affichée avant le traitement et j'obtiens un double affichage : la page secondaire attendue suivie de la page principale des commentaires.
Change History
comment:2 Changed 12 years ago by Dsls
- Status changed from new to closed
- Resolution set to fixed
(In [df5107ab920c]) Avoid comments page display if action has been processed, fixes #1820
comment:3 follow-up: ↓ 4 Changed 12 years ago by franck
- Status changed from closed to reopened
- Resolution fixed deleted
Alors la double page n'est plus affichée. C'est bien, sauf que si je valide l'action ->
Warning: array_merge(): Argument #1 is not an array in /Users/franck/Documents/EGO/Geekeries/Internet/htdocs/hg-dc2-2.6/inc/admin/actions/class.dcaction.php on line 244
T'as testé avec quoi ?
comment:4 in reply to: ↑ 3 Changed 12 years ago by bruno
Replying to franck:
Alors la double page n'est plus affichée. C'est bien, sauf que si je valide l'action ->
Warning: array_merge(): Argument #1 is not an array in /Users/franck/Documents/EGO/Geekeries/Internet/htdocs/hg-dc2-2.6/inc/admin/actions/class.dcaction.php on line 244
T'as testé avec quoi ?
cf mon mail du 30/09 :) Je profite que la 2.6 soit pas encore officielle pour anticiper les prochaines modifications. Je préviens donc ici pour ceusses qui auraient déjà commencé à utiliser dcActionsPages.
En substance, je vais "juste" intervertir l'ordre des paramètres de la méthode redirect : le booléen indiquant si oui ou non on inclut les entrées dans la redirection va passer en premier, les paramètres additionnels seront précisés après (et seront facultatifs, tout comme le booléen qui va passer à true par défaut au lieu de false, vu que la plupart du temps, on veut garder les entrées).
Motivation : actuellement, les paramètres additionnels étaient (sont) essentiellement utilisés pour ajouter &upd=1 à l'url de redirection. Or, ils vont disparaître avec la résolution du ticket #1710 à venir (pour après la 2.6). Ce serait bête d'avoir des array() partout en paramètre. Je préfère donc anticiper, et passer cet argument en dernier, le code sera plus simple après :)