forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thorsten Scherler <>
Subject [headsup] Merging locationmap_branch to head
Date Tue, 05 Jul 2005 21:18:24 GMT
Hello devs,

I merged the view related stuff from locationmap_branch to head. Now I
was up to do the same for the whole locationmap.

I tested like
22:39:54 thorsten@cartman>/guada/home/thorsten/apache/forrest-trunk $
svn merge --dry-run -r190293:head

Besides the following files everything went well:
C  main/webapp/forrest.xmap
C  main/webapp/tabs.xmap
C  main/webapp/linkmap.xmap
C  main/webapp/menu.xmap

Now I did a small diff [t]-trunk/[l]-locationmap in my eclipse and found
following differences. Some need IMO discussion. 

1) C  main/webapp/forrest.xmap
In the locationmap are the <map:call resource=""/> and the @type="i18n"
from the match missing. Is this intended?
<map:match type="i18n" pattern="{project:content.xdocs}{1}.*.xml">
          <map:generate src="{source}" />
          <map:call resource="transform-to-document">
            <map:parameter name="src" value="{source}" />
          <map:serialize type="xml-document"/>
<map:match pattern="**.xml">
         <map:select type="exists">
           <map:when test="{lm:{0}}">
             <map:generate src="{lm:{0}}"/>
             <map:generate src="{project:content.xdocs}{0}"/>
         <map:serialize type="xml"/>

2) C  main/webapp/tabs.xmap
Here we have again i18n differences.
     <map:transform type="i18n">
	        <map:parameter name="locale" value="{request:locale}"/>
     <map:transform type="i18n">
                <map:parameter name="locale"

3) C  main/webapp/linkmap.xmap
exchanges only the generator.

4) C  main/webapp/menu.xmap
Again i18n

   <map:parameter name="locale" value="{request-param:locale}"/>

   <map:parameter name="locale" value="{lang}{country}{variant}"/>


BTW all this new <map:select type="exists"> should be rethought. Nicola
can you explain how and maybe have a look in the commit mail 209151. Is
this what you meant?


"Together we stand, divided we fall!" 
Hey you (Pink Floyd)

View raw message