David Crossley wrote:
Thorsten Scherler wrote:
David Crossley escribi??:
...
BTW, i see that the dispatcher copy of resources.xmap does not
use the locationmap at all, whereas the main one does.
We (I think Ross mainly did it) updated the xmap and extracted the
matches to lm recently. The dispatcher resources.xmap did not got
adopted after this changes.
I meant that if someone is investigating the issue,
then please work from the main copy.
I have noticed a trend lately that when people are
helping to investigate core issues, they are considering
dispatcher only. We still have skins as default.
It is worrying that a plugin (i.e. the dispatcher) has a complete copy
of resources.xmap (and others) rather than overriding any matches that
are necessary to make the plugin work.
I raised this issue a very long time ago, way back in views 1. At that
time the justification was that the dispatcher (then views) would become
part of core, so I let it drop. However, now that it seems the
dispatcher is to remain a plugin this has become an important issues.
There should be zero code duplication between sitemaps in plugins and
sitemaps in core. The plugin framework has been built in such a way that
this should never be necessary. We have he ability to override selected
matches and that is all that we should be doing.
Ross