zikula-modules / Pages

Simple HTML Pages
https://ziku.la/
18 stars 4 forks source link

'formutil_getpassedvalue' plugin seems not to work with shorturls #38

Closed Portugao closed 9 years ago

Portugao commented 10 years ago

This plugin is the follower of the plugin 'varcleanfrominput'. With the deprecated one I have not this issue. See also https://github.com/zikula/core/issues/1859

craigh commented 10 years ago

according to the core ticket, this works when shorturls are not enabled and doesn't work when shorturls is enabled. So the problem is with shorturls not, the plugin.

Portugao commented 10 years ago

Are you sure it is not an issue of pages because of its handling of routing?

craigh commented 10 years ago

sure? no. but I am making an educated guess based on other work I've been doing.

Because Pages is still a 1.3.x-type (non-namespaced) module, it is not using 'routing' in the 1.4 sense. It is using the same old code from 1.3.x so it should work. I have one fix pending that might address this.

craigh commented 9 years ago

@Portugao please test this again and report back. I believe the shorturls issues in the core have been solved.