Uploaded image for project: 'Wicket'
  1. Wicket
  2. WICKET-5057

FilteringHeaderResponse requires a usage of FilteringHeaderResponse.IHeaderResponseFilter for no reason when FilteredHeaderItem is used

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 6.6.0
    • 6.7.0
    • wicket
    • None

    Description

      org.apache.wicket.markup.head.filter.FilteringHeaderResponse#render(HeaderItem) doesn't need a filter when a FilteredHeaderItem is processed. But later org.apache.wicket.markup.head.filter.FilteringHeaderResponse#render(HeaderItem item, String filterName) requires such filter.

      I think we can create a bundle (List<HeaderItem>) on the fly in this case for the first FHI.

      Otherwise right now the developer has to use:

      AbstractHeaderResponseFilter dummyFilter = new AbstractHeaderResponseFilter(coreFilterName) {
      @Override
      protected boolean acceptsWrapped(HeaderItem item)

      { return false; }

      };
      filters.add(dummyFilter);

      in his HeaderResponseDecorator.

      Attachments

        Activity

          People

            mgrigorov Martin Tzvetanov Grigorov
            mgrigorov Martin Tzvetanov Grigorov
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: