Wicket
  1. Wicket
  2. WICKET-647 New Wicket Portlet support
  3. WICKET-660

New Wicket Portlet support: Merge WicketPortletFilter back in WicketFilter using a delagate class for handling and (class) loading the portlet specific functionality

    Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.0-beta2
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      I moved the usage of all portlet api specific classes in WicketPortletFilter to a separately class, WicketFilterPortletContext, which will only be created when running in a portlet container.
      As effect, the resulting WicketPortletFilter code doesn't depend on the portlet api anymore, so using it within a non-portlet container context (plain Tomcat or Jetty for instance) is now possible too.

      But, now the remaing code in WicketPortletFilter really does do much itself anymore and can easily be integrated back in WicketFilter without problems.

      I've decided to do so as it makes Wicket portlet support even less intrusive as it was before: no need to configure a different filter anymore in web.xml !

        Activity

        Hide
        Ate Douma added a comment -

        Done.

        So now you can deploy a portlet-supporting wicket application also in plain web containers!

        I've build a new version of the wicket-examples.war which you can download from my apache home page

        http://people.apache.org/~ate/wicket/wicket-examples/r547598/wicket-examples.war

        This example can be used a drop in replacement in the jetspeed-2.1.1-beta1-wicket-demo-installer I've created yesterday (see: WICKET-658): just copy it into $

        {installationPath}

        /webapps/jetspeed/WEB-INF/deploy.
        And of course you can test run it in a plain web container like Tomcat or Jetty too.

        Show
        Ate Douma added a comment - Done. So now you can deploy a portlet-supporting wicket application also in plain web containers! I've build a new version of the wicket-examples.war which you can download from my apache home page http://people.apache.org/~ate/wicket/wicket-examples/r547598/wicket-examples.war This example can be used a drop in replacement in the jetspeed-2.1.1-beta1-wicket-demo-installer I've created yesterday (see: WICKET-658 ): just copy it into $ {installationPath} /webapps/jetspeed/WEB-INF/deploy. And of course you can test run it in a plain web container like Tomcat or Jetty too.
        Hide
        Ate Douma added a comment -

        The merge of WicketFilter and WicketPortletFilter isn't correct yet.
        Super calls allows for easy override of parameters, which shouldn't be overlooked when merging two methods...

        Show
        Ate Douma added a comment - The merge of WicketFilter and WicketPortletFilter isn't correct yet. Super calls allows for easy override of parameters, which shouldn't be overlooked when merging two methods...
        Hide
        Ate Douma added a comment -

        Fixed again.

        I've updated the downloadable wicket-examples.war too:

        http://people.apache.org/~ate/wicket/wicket-examples/r547651/wicket-examples.war

        Show
        Ate Douma added a comment - Fixed again. I've updated the downloadable wicket-examples.war too: http://people.apache.org/~ate/wicket/wicket-examples/r547651/wicket-examples.war

          People

          • Assignee:
            Ate Douma
            Reporter:
            Ate Douma
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development