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

Ticket #779 (delayed enhancement)

Opened 5 years ago

Last modified 2 years ago

Add an option to hide controller in an url generated by the 'basic_significant' engine

Reported by: mike Owned by:
Priority: normal Milestone:
Component: jelix:plugins:urls Version: 1.1 beta 1
Severity: minor Keywords: jurl, engine, significant
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Currently, urls generated by the 'basic_significant' engine follows the pattern :

entrypoint.php/module/controller/method?param=1&…

In real situation in can be something like that if you use the default controller:

index.php/articles/default/news?id=1&…

We should add something to remove the "default" string from this urls, it's useless !

As we spoke on irc, a good solution is to add a new section in the defaultconfig.

[basic_significant_urlengine_controller]
;for each module set if the engine have to write the controller's name in url
;by default it's on for every modules
news = off
shop = on

And now the new url looks like :

index.php/articles/news?id=1&…

Change History

comment:1 Changed 5 years ago by bballizlife

I do not know what was said on IRC when talking about that. My first reaction about this proposal is if you want specific urls, you need to use the significant engine, not the basic_significant one.

Unless someone gives me a real need for this feature, I think this ticket is invalid.

comment:2 follow-up: ↓ 3 Changed 5 years ago by laurentj

  • Owner set to laurentj
  • Component changed from jelix to jelix:core
  • Milestone Jelix 1.2 deleted

Bballizlife: I think this behavior keeps "basic" this url engine. We should admit that configuring a urls.xml is not simple :-)

However, in the proposed solution, since by default the controller name is added, the value to indicate is modulename=on if we want to remove the controller name, not off. So we don't have to explicitely list all modules.

And during the generation of the url, the url engine should trigger an exception if we try to ask an url for a controller other than "default", for a module which is "on" on the list.

(removed milestone target since we don't have a patch yet. The list of ticket of jelix 1.2 is already huge !).

comment:3 in reply to: ↑ 2 Changed 5 years ago by mike

Replying to laurentj: I think as you do, we should enable this feature in order to have an easy but powerful basic significant url engine. And as you said, for the behaviour it's seems good to me to reverse the on/off. Finally this feature is very useful if we use a lot of createdacrud since it creates a controller per module. For a very basic backoffice it's a great feature.

comment:4 Changed 4 years ago by laurentj

  • Owner laurentj deleted
  • Component changed from jelix:core to jelix:plugins:urls

comment:5 Changed 2 years ago by laurentj

  • Status changed from new to delayed

As the core of jUrl will be changed and will be based only on the significant engine, this feature won't probably be fixed.

Note: See TracTickets for help on using tickets.