[MGNLSTK-409] Expose STKUtil as EL functions Created: 30/Jun/09 Updated: 04/Nov/15 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Magnolia Standard Templating Kit (closed) |
| Component/s: | None |
| Affects Version/s: | 1.1 |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major |
| Reporter: | Thomas Duffey | Assignee: | Philipp Bärfuss |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Date of First Response: |
| Description |
|
STKUtil methods should be exposed as EL functions for people using JSP rather than Freemarker. This is a simple update and I'm happy to provide the patch but should all public static methods be exposed or just a selection? Perhaps Maven has a tool to generate the TLD automatically? |
| Comments |
| Comment by Magnolia International [ 30/Jun/09 ] |
|
This should be fairly simple to add as a feature to the xdoclet-plugin-web, with a tag along the lines of @jsp.function: See http://xdoclet.codehaus.org/TaglibPlugin for some details. Sources at https://xdoclet-plugins.svn.sourceforge.net/svnroot/xdoclet-plugins/trunk/xdoclet-plugins/plugin-web Since we already use the plugin for regular TLD, it should not pose too much problems to do this too. (although we do have functions in magnolia-taglib-cms — maybe they'd need to be split in a separate module) I'm concerned about the static nature of these functions, but afaik, STKUtil is currently all-static. MagnoliaTemplatingUtilities isn't. |
| Comment by Thomas Duffey [ 07/Aug/09 ] |
|
This patch only exposes two methods from STKUtil, the ones I currently need Will investigate a more complete patch in the future but would be great to get this in as a start. |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |