[ 
http://jira.magnolia-cms.com/browse/MAGNOLIA-3204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=28600#action_28600
 ] 

Hudson CI server commented on MAGNOLIA-3204:
--------------------------------------------

Integrated in !http://hudson.magnolia-cms.com/images/16x16/blue.gif! 
[magnolia_main-trunk 
#1673|http://hudson.magnolia-cms.com/job/magnolia_main-trunk/1673/]
     MAGNOLIA-3224 With MAGNOLIA-3204  the method resetAggregationState has 
been modified and now it set these aggregationState properties to:
this.originalURI=null;
this.originalURL=null;
this.originalBrowserURI=null;
this.originalBrowserURL=null;
this.currentURI = null;

cannot be passed a null value to UnicodeNormalizer.normalizeNFC(null), this 
cause NPE.

The UnicodeNormaliazioneFilter has to normalize originalURI/URL, to overwrite 
these properties it needs to reset it, otherwise IllegalStateException could 
occurs.


> aggregation state looses all information on forwards
> ----------------------------------------------------
>
>                 Key: MAGNOLIA-3204
>                 URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-3204
>             Project: Magnolia
>          Issue Type: Bug
>    Affects Versions: 4.3.1
>            Reporter: Philipp Bärfuss
>            Assignee: Philipp Bärfuss
>            Priority: Critical
>             Fix For: 4.3.2
>
>         Attachments: MAGNOLIA-3204.patch
>
>
> h4. The Problem
> The aggregation state is set to null on each forward. This is problematic as 
> we also store the locale and site in the aggregation state. As a consequence 
> all this information is lost after a virtual uri mapping which forwards.
> Example:
> # request to /de/page/param.html
> #- locale is set to de
> #- current uri is /page/param.html
> # virtual uri mapping
> #- forward to /page.html?param
> # re-execution 
> #- ContentTypeFilter resets the aggregation state
> #- locale is now the default locale:en
> h4. Solution
> We could simply not reset the aggregation state but this conflicts with 
> MAGNOLIA-3203 (resetting original URI on forward). So we can either fix 
> MAGNOLIA-3203 or reset the aggregation state partially.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira



----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------

Reply via email to