Discussion :

[bricecol]

Bonjour,

Bon, jusqu'à présent, j'utilisais des URL du type :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

action?parm1=p1&param2=p2...

Typiquement : site?sid=5

J'aimerais transformer çà en : site/google.fr (en supposant ici que 5 soit l'ID du site google.fr, bref...)

Je me tourne donc sur le fichier monapp/config/routing.yml
Et j'ouvre la doc symfony 1.4.....

Et je n'y arrive toujours pas :p, j'ai besoin d'un petit coup de main SVP

Si je suis la doc, je crois que je dois écrire quelque chose comme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
site:
  url:   /site/:hostname
  param: { module: sites, action: one }

Vous comprenez je pense, que je cherche à ce que site/google.fr fasse appel à l'action one du module sites en passant en paramètre hostname=google.fr

Est-ce écrit correctement, sinon comment l'écrire ?
Et comment générer la bonne URL dans un contrôleur et dans un template avec url_for() ?

Merci d'avance pour votre aide.

[Vicrabb]

Est-ce que hostname est bien le nom de la colonne dans lequel ton google.fr est contenu?

Par ailleurs, ce qui se trouve dans param me semble correct.

url_for() se construit ainsi, du moins, dans sa forme simplifiée:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

url_for('nom_de_la_route', 'objet')

Donc par exemple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

url_for('site', $site)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
//dans du html:
<a href="<?php echo url_for('site',$site) ?>">Site</a>

[bricecol]

J'ai beau essayer de plein de manières différentes, je n'y arrive pas...

Voici mes différents essaie, rien n'a marché, il doit sûrement manquer quelque chose...

Mon objet Site a effectivement un attribut hostname

Le routing :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
site:
  url:   /site/:hostname
  param: { module: sites, action: onebyhost }

--------------------------------------------------------

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$this->redirect('site', $site)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

The "/site/:hostname" route has some missing mandatory parameters (:hostname).

--------------------------------------------------------

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$this->redirect('@site', $site)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

The route "@site" does not exist.

--------------------------------------------------------

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$this->redirect('site', array('hostname' => $site->hostname))

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Empty module and/or action after parsing the URL "/site/google.com" (/).

[Vicrabb]

Attention, ce n'est $site->hostname qui permet de récupérer le hostname mais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$site->getHostname();

Essaie un peu de le rajouter dans tes redirections.

[bricecol]

Non, ->hostname et getHostname(), c'est pareil je pense, en tout cas çà renvoie pareil.

J'ai réussi à faire une URL sous la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

http://.../sites/scorecard/google.com

Au lieu de se que je souhaiterais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

http://.../site/google.com

En fait, je ne peux pas supprimer l'action de l'URL, ni renommer le module, le matching ne se fait pas sinon.

Si quelqu'un sait comment faire...

[mathieu44800]

Bonjour Brice,

Selon la documentation officielle http://www.symfony-project.org/api/1...ethod_redirect, les 2eme paramètre est le code HTTP.

Pourquoi ne pas tenter ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$this->redirect('@site?hostname'.$site);

Pas eu le temps de tester cependant...

Mathieu

[bricecol]

Selon la documentation officielle http://www.symfony-project.org/api/1...ethod_redirect, les 2eme paramètre est le code HTTP.

Oui effectivement, c'était pour simplifier, en fait j'utilise (route, params) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$this->redirect($this->getController()->genUrl(..., ...))

Le redirect, par ailleurs, ne prend pas les noms de routes comme @uneroute. C'est url_for() ou link_to qui le font, dans les templates.

J'ai réussi à avoir une URL de la forme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

http://.../sites/scorecard/hostname/google.com

Qui corresponds à :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
scorecard:
  url:   /sites/scorecard/:hostname
  param: { module: sites, action: scorecard }

Voilà comment je construis l'URL :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$this->redirect($this->getController()->genUrl('sites/scorecard?hostname='.$result['found']->getVeryShortHostname()));

L'URL fonctionne, l'action scorecard du module sites est bien appelée et je récupère le paramètre hostname avec $request->getParameter('hostname').

Ce que j'aimerais c'est avoir le même fonctionnement mais avec une URL du style :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

http://.../site/google.com

Est-possible ?

[mathieu44800]

est ce que tu peux afficher ce que te renvoies un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 var_dump($this->getController()->genUrl(..., ...));

[bricecol]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

http://.../sites/scorecard/hostname/google.com

[mathieu44800]

Je pense que tes URLs dans ton routing se croisent.

Essayes de commenter ta route @scorecard dans ton routing et de faire une redirection simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$this->redirect('@site?hostname='.$site);

J'espère que j'aurais pu t'aider...

[Michel Rotta]

Puis-je me permettre de jeter mon grain, de sel, sur la route ?

L'idée est bonne et les conseils judicieux mais il n'y a pas une grosses erreurs ici, mais plusieurs petites, et il faudra toutes les corriger pour arriver à faire marcher la chose.

Pour la route, je ne suis pas très sur qu'envoyer une url ainsi, dans le chemin soit une très bonne idée, notamment à cause des "/" qui pourraient s'y trouver et perturber notablement le moteur de route de symfony. Je serais plutôt partisan de partir du nom du site slugifier. Doctrine introduit des outils permettant de générer facilement un slug unique pour chaque enregistrement. Il est aussi possible de slugifier l'url.

Pour la route ta déclaration :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
scorecard:
  url:   /sites/scorecard/:hostname
  param: { module: sites, action: scorecard }

est correcte, même si l'url pourrait être allégée, il n'est en rien indispensable qu'elle ressemble au module + action, vu qu'ils sont indiqué dans les paramètres.

Pour que cela marche il faut appeler ta route de la manière suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
url_for('@scorecard?hostname=' . $site->getHostname() );

Tu récupère alors dans le module sites par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
public function executeScorecard(sfWebRequest $request)
{
   ...
   $urm = $request->getParameter('hostname');
   ...
}

A noter que tu as aussi la possibilité d'utiliser des routes doctrine qui peuvent apporter beaucoup en terme de développement. Je t'invite à générer un module CRUD non admin et a en regarder le code, tu trouveras largement de quoi t'inspirer.

[Inarius]

Par contre, il y a une erreur dans la déclaration de ta route. Si tu veut utiliser la chose suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

url_for('site', $site)

Il faut que ta route utilise une sfDoctrineRoute ( si tu utilise l'ORM Doctrine )

Donc ta route doit être de la forme:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
site:
  url: /site/:hostname
  class: sfDoctrineRoute
  options: { model: ModelDeTonSite, type: object, method_for_query: retrieveRouteObject } // method_for_query est optionnel mais utile si tu veut réaliser des JOIN, des WHERE, ....
  param: { module: sites, action: onebyhost }

Par contre un des "désavantages" de cette méthode est que tu ne peut avoir que des requetes HTTP GET. Donc tu neut peut pas l'utiliser pour des formulaires, où la requete HTTP est POST.

[Michel Rotta]

Par contre un des "désavantages" de cette méthode est que tu ne peut avoir que des requetes HTTP GET. Donc tu neut peut pas l'utiliser pour des formulaires, où la requete HTTP est POST.

Ca marche aussi avec des requêtes de type POST. Heureusement, c'est ce qui est utilisé dans le CRUD généré par symfony et doctrine

En fait, cela marche même avec des requêtes PUT et autres.

[Taka-kun]

Court et assez bien fait :
http://www.symfony-project.org/jobee...Doctrine/fr/05