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

[mike]

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&…

[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.

[laurentj]

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 !).

[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.

[laurentj]

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