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

        Ate Douma created issue -
        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.
        Ate Douma made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        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...
        Ate Douma made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        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
        Ate Douma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Martin Grigorov made changes -
        Component/s wicket-portlet [ 12311803 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        24m 57s 1 Ate Douma 15/Jun/07 11:14
        Resolved Resolved Reopened Reopened
        2h 12m 1 Ate Douma 15/Jun/07 13:26
        Reopened Reopened Resolved Resolved
        16m 8s 1 Ate Douma 15/Jun/07 13:42

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development