Ticket #108 (closed enhancement: fixed)

Opened 4 months ago

Last modified 2 months ago

Ajout d'un lien dans le mail envoyé aux modos depuis le topoguide

Reported by: fbunoz Assigned to: somebody
Priority: minor Milestone:
Component: component1 Version:
Keywords: Cc:
Triage Stage: Unreviewed State of Approval: Unnecessary
Attached Patches: None Complexity: Unknown
Compatibility: Unknown Specification: Unnecessary

Description

Lorsqu'on clique sur "Signaler un problème", ce serait cool qu'il y ait un lien vers document.
Pour cela, il suffirait d'ajouter au lien le type et numéro du document :
http://www.camptocamp.org/forums/misc.php?email=108544 &doc=routes/131046/fr
Dans le code du forum, on utiliserait l'info pour mettre du texte par défaut dans la zone de saisie du message à envoyer (modif facile).
Pour la modif de la page du topoguide, c'est facile ou non ?

Change History

07/23/08 09:20:47 changed by xbrrr

Pour le topoguide c'est assez facile :

Il faut modifier la fonction button_report() du helper ButtonHelper.php, ainsi que chaque partiel _nav.php qui utilise cette fonction.

A noter que la fonction est utilisée par d'autres partiels (_nav4list.php, _nav4home.php) qui eux ne peuvent pas fournir qqch comme &doc=routes/131046/fr (mais éventuellement d'autres infos) donc il faut prévoir des cas par défaut dans button_report()

07/23/08 22:29:08 changed by fbunoz

Pour les autres partiels, pas de problème on fait pareil, le but est d'avoir le suffixe après camptocamp.org/ pour pouvoir reconstruire l'url.

07/24/08 09:57:17 changed by xbrrr

S'il s'agit de récupérer systématiquement la fin de l'url, c'est encore plus simple, il suffit de modifier ButtonHelper.php (pas besoin de toucher aux autres fichiers) 

Index: apps/frontend/lib/helper/ButtonHelper.php
===================================================================
--- apps/frontend/lib/helper/ButtonHelper.php   (révision 257)
+++ apps/frontend/lib/helper/ButtonHelper.php   (copie de travail)
@@ -229,7 +229,7 @@
 {
     use_helper('Forum');
     return f_link_to(__('Report problem'),
-                     'misc.php?email=' . sfConfig::get('app_moderator_user_id'),
+                     'misc.php?email=' . sfConfig::get('app_moderator_user_id') . '&doc=' . $_SERVER['REQUEST_URI'],
                      array('title' => __('Report problem'),
                            'class' => 'action_report nav_edit'));
 }

Par contre il y a un / au début systèmatiquement, mais c'est mieux si tu veux récupérer le home

09/16/08 14:04:54 changed by xbrrr

plus simple peut-être : tu peux utiliser $_SERVER['HTTP_REFERER'] pour connaître la page en question (vu qu'on accède à la page 'signaler un problème' depuis cette page)

09/16/08 16:44:21 changed by fbunoz

Je vais utiliser ta solution du 24/7, car elle n'est pas dépendante des magouilles de l'utilisateur (du style utilisation d'une extension FF qui permet de choisir son REFERER).

(follow-up: ↓ 7 ) 09/16/08 18:22:51 changed by xbrrr

par contre faut voir peut-être au niveau de l'encodage de l'url, je ne sais pas trop ce qui se passe, si l'url passée contient des & et autres (genre faire un coup de htmlentities() ou autre dessus ?)

(in reply to: ↑ 6 ) 09/16/08 18:26:52 changed by xbrrr

Replying to xbrrr:

par contre faut voir peut-être au niveau de l'encodage de l'url, je ne sais pas trop ce qui se passe, si l'url passée contient des & et autres (genre faire un coup de htmlentities() ou autre dessus ?)

après recherche, urlencode() doit être le bon candidat

10/01/08 20:44:02 changed by xbrrr

r330 : côté topoguide

10/03/08 21:47:36 changed by fbunoz

  • status changed from new to closed.
  • resolution set to fixed.

r331 et r333 : côté forum

C'est donc tout bon :-))