[ https://issues.apache.org/jira/browse/SLING-2920?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Felix Meschberger updated SLING-2920: ------------------------------------- Attachment: SLING-2920.patch Proposed patch fixing the service.ranking problem. > Wrong handling of Sling Filter ordering > --------------------------------------- > > Key: SLING-2920 > URL: https://issues.apache.org/jira/browse/SLING-2920 > Project: Sling > Issue Type: Bug > Components: Engine > Affects Versions: Engine 2.2.8 > Reporter: Felix Meschberger > Attachments: SLING-2920.patch > > > It looks like the ordering of Sling Filters is not implemented as it is > documented on [1]. > The documented intent is: > * service.ranking ordering with higher numbers being higher preference over > lower numbers > * filter.order ordering with lower numbers being higher preference over > higher numbers > * filter.order is ignored if service.ranking is defined > * higher preferenced filters called before lower preferenced filters > Actual implementation: > * service.ranking ordering with lower numbers being higher preference over > higher numbers > * filter.order ordering with lower numbers being higher preference over > higher numbers > * filter.order is ignored if service.ranking is defined > * higher preferenced filters called before lower preferenced filters > As one can see, the service.ranking ordering is not properly implemented. It > looks like this has been wrong since the first implementation as of > SLING-1735 (Jan 2011). > We can either live with this actual implementation and fix the documentation > or fix the implementation at the cost of having to also fix any down-stream > filter providers using service.ranking values as implemented (and not as > documented). > Needs to be discussed what to do ... > [1] http://sling.apache.org/documentation/the-sling-engine/filters.html -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira