developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
#917 closed enhancement (fixed)
Amelioration de simple_urlengine_entrypoints
Reported by: | geekbay | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | Jelix 1.2 beta |
Component: | jelix:plugins:urls | Version: | 1.1 |
Severity: | trivial | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Documentation needed: | no | |
Hosting Provider: | Php version: |
Description
Dans [simple_urlengine_entrypoints] si j'ai un module foo et que je veux que toutes les actions de bar soit associe a un entrypoint, on est oblige de definir chaque action a chaque fois.
Le patch que je propose est de faire une syntax plus simple, au lieu d'avoir a faire: admin="foo~bar:index, foo~bar:test, foo~bar:precreate"
on peut ecrire simplement:
admin="foo~bar"
Attachments (2)
Change History (12)
Changed 12 years ago by geekbay
comment:1 Changed 12 years ago by foxmask
ca marche pas
foo~bar:*
?
comment:2 Changed 12 years ago by geekbay
non, regarde dans le code
if($action && isset($this->urlspe[$s1 = $module.'~'.$action.'@'.$requestType])){ $script = $this->urlspe[$s1]; }elseif($module && isset($this->urlspe[$s2 = $module.'~*@'.$requestType])){
comme tu vois on a $module~$action et $action est par exemple foo:bar, et il n'y a pas de check sur *
comment:3 Changed 12 years ago by foxmask
ok, dommage
comment:4 Changed 12 years ago by Julien
Hello,
pour la cohérence du truc, on devrait plutôt pouvoir indiquer (et c'est plus logique comme foxmask l'a indiqué intuitivement) :
foo~bar:*
car le sélecteur d'action
foo~bar
est en principe équivalent à
foo~default:bar
comment:5 Changed 12 years ago by laurentj
- Component changed from jelix to jelix:plugins:urls
- Milestone set to Jelix 1.2
- Severity changed from normal to trivial
d'accord avec julien.
Il faut mettre aussi à jour les commentaires sur la section dans defaultconfig.ini.php de lib/jelix/core et le defaultconfig.ini.php.tpl de jelix-scripts.
comment:6 Changed 12 years ago by bballizlife
Review needed ?
comment:7 Changed 11 years ago by laurentj
- review set to review?
comment:8 Changed 11 years ago by laurentj
- Resolution set to fixed
- review changed from review? to review+
- Status changed from new to closed
landed. http://bitbucket.org/jelix/jelix-trunk/changeset/7fc3af47bc4f/
I added tests.
comment:9 Changed 11 years ago by laurentj
- Documentation needed set
comment:10 Changed 11 years ago by laurentj
- Documentation needed unset
Proposition de patch pour simple url engine