developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#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)

patch.simpleurlengine.diff (799 bytes) - added by geekbay 11 years ago.
Proposition de patch pour simple url engine
patch.simpleurlengine.2.diff (801 bytes) - added by geekbay 11 years ago.
Patch pour syntax foo~bar:*

Download all attachments as: .zip

Change History (12)

Changed 11 years ago by geekbay

Proposition de patch pour simple url engine

comment:1 Changed 11 years ago by foxmask

ca marche pas

foo~bar:*

?

comment:2 Changed 11 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 11 years ago by foxmask

ok, dommage

comment:4 Changed 11 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 11 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.

Changed 11 years ago by geekbay

Patch pour syntax foo~bar:*

comment:6 Changed 11 years ago by bballizlife

Review needed ?

comment:7 Changed 10 years ago by laurentj

  • review set to review?

comment:8 Changed 10 years ago by laurentj

  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from new to closed

comment:9 Changed 10 years ago by laurentj

  • Documentation needed set
Note: See TracTickets for help on using tickets.