Sven, do you expect your fix for this to get into the next point release of 
Wicket 7, or will it only be in Wicket 8?

Just trying to determine if I need to build a work-around or if I can wait for 
an official fix.

Thanks!

Boris


On 4/25/17, 9:07 PM, "Boris Goldowsky" <bgoldow...@cast.org> wrote:

    Thank you!
    
    https://issues.apache.org/jira/browse/WICKET-6362
    
    Boris
    
    
    On 4/25/17, 6:02 PM, "Sven Meier" <s...@meiers.net> wrote:
    
        Hi Boris,
        
        this is a bug in ResourceAggregator, please file a Jira issue - I think 
        I have a fix ready by tomorrow.
        
        Have fun
        Sven
        
        
        
        On 25.04.2017 15:32, Boris Goldowsky wrote:
        > I think Sven is right.
        > Debugging through, it appears to be here in ResourceAggregator:
        >
        >       private void recordHeaderItem(HeaderItem item, Set<HeaderItem> 
depsDone)
        >       {
        >               renderDependencies(item, depsDone);
        >               RecordedHeaderItem recordedItem = 
itemsToBeRendered.get(item);
        >
        >   “recordedItem” ends up as the old header item rather than the new 
one.
        >
        > Bng
        >
        >
        > On 4/24/17, 6:09 PM, "Sven Meier" <s...@meiers.net> wrote:
        >
        >      HI,
        >      
        >      this might be caused by ResourceAggregator - it marks items as 
being
        >      rendered, instead of render tokens.
        >      
        >      I'll check tomorrow.
        >      
        >      Sven
        >      
        >      
        >      On 24.04.2017 23:41, Martin Grigorov wrote:
        >      > On Mon, Apr 24, 2017 at 11:39 PM, Martin Grigorov 
<mgrigo...@apache.org>
        >      > wrote:
        >      >
        >      >> Hi,
        >      >>
        >      >> On Mon, Apr 24, 2017 at 10:20 PM, Boris Goldowsky 
<bgoldow...@cast.org>
        >      >> wrote:
        >      >>
        >      >>> I have a situation like this:
        >      >>>
        >      >>> public void renderHead(IHeaderResponse response) {
        >      >>>      …
        >      >>>      
response.render(JavaScriptHeaderItem.forReference(resRef,
        >      >>> pageParameters1, “id1”));
        >      >>>      
response.render(JavaScriptHeaderItem.forReference(resRef,
        >      >>> pageParameters2, “id2”));
        >      >>> }
        >      >>>
        >      >>> where same ResourceReference is used for both resources – 
the different
        >      >>> page parameters point it to different actual Resources.
        >      >>>
        >      >>> However, the check for uniqueness of header items seems to 
consider them
        >      >>> equal, despite the different PageParameters and different 
IDs, and only one
        >      >>> of them actually gets rendered in the page head.
        >      >>>
        >      >> Which check exactly do you refer ?
        >      >>
        >      >> 
org.apache.wicket.markup.head.internal.HeaderResponse#markItemRendered()
        >      >> does such check by calling 
org.apache.wicket.markup.head.HeaderItem#
        >      >> getRenderTokens().
        >      >> 
org.apache.wicket.markup.head.JavaScriptReferenceHeaderItem#getRenderTokens()
        >      >> uses the url and the id. The url contains the parameters.
        >      >> All looks good to me!
        >      >>
        >      > The other check is at jQuery.Head.containsElement() 
(wicket-ajax-jquery.js)
        >      > and it uses the value "src", i.e. it should not match because 
of the
        >      > different parameters.
        >      >
        >      >>
        >      >>> Is this a bug, or is there a way to force the two items to 
both be
        >      >>> included?
        >      >>>
        >      >>> I’m using Wicket 7.5.0.
        >      >>>
        >      >>> Boris
        >      >>>
        >      >>>
        >      >>>
        >      >>>
        >      >>>
        >      
        >      
        >      
---------------------------------------------------------------------
        >      To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
        >      For additional commands, e-mail: users-h...@wicket.apache.org
        >      
        >      
        >
        >
        > ---------------------------------------------------------------------
        > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
        > For additional commands, e-mail: users-h...@wicket.apache.org
        >
        
        
        ---------------------------------------------------------------------
        To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
        For additional commands, e-mail: users-h...@wicket.apache.org
        
        
    
    
?B�KKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKKCB�?�?[��X��ܚX�K??K[XZ[?�?\�\��][��X��ܚX�P?�X��]?�\?X�?K�ܙ�B��܈?Y??]?[ۘ[??��[X[�?�??K[XZ[?�?\�\��Z?[???�X��]?�\?X�?K�ܙ�B�

Reply via email to