Context Navigation

← Previous Ticket
Next Ticket →

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

Attachment patch.simpleurlengine.diff added

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.