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

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

Integrated in !http://hudson.magnolia-cms.com/images/16x16/blue.gif! 
[magnolia_main-4.3-branch 
#3|http://hudson.magnolia-cms.com/job/magnolia_main-4.3-branch/3/]
     Merged revisions 34930 via svnmerge from 
http://svn.magnolia-cms.com/svn/community/magnolia/trunk

........
  r34930 | lucaboati | 2010-06-08 16:59:26 +0200 (Tue, 08 Jun 2010) | 10 lines
  
  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