forrest-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thorsten Scherler <>
Subject Re: [jira] Created: (FOR-816) Modify pelt.fv from site-author to reflect our current design and values
Date Mon, 13 Feb 2006 08:27:46 GMT
El lun, 13-02-2006 a las 09:44 +1100, David Crossley escribió:
> > Modify pelt.fv from site-author to reflect our current design and values
> > ------------------------------------------------------------------------
> > 
> >          Key: FOR-816
> >          URL:
> >      Project: Forrest
> >         Type: Sub-task
> >   Components: Dispatcher (aka views)  
> >     Reporter: Thorsten Scherler
> > 
> > 
> > While fixing FOR-807 I added a default structurer for the site-author.
> > 
> > However we need to add our default values and modify it to reflect
> We have issue FOR-811
> to tidy up the themes.core/themes/pelt.fv
> and associated contracts.

Yeah, like you said FOR-811 is to tidy up the default pelt theme.

> Gav was working on that for ForrestFriday.
> However we have not yet got his patch.
> How are we going to keep these two copies
> of the pelt structure synchronised?

Let give me you an example why we need a structurer for site-author. The
skinconf.xml can be found in *all* projects, right? What does
skinconf.xml do? Customizing the skin for the current project (change
logos and project name, ...) Now the skinconf.xml is growing up to the

Trying to say if we use themes.core/themes/pelt.fv then we have MyGroup
and MyProject, ... but that is not reflecting the reality since we have
custom contract configuration in our site-author.

This can be overcome if we solve
because we then can enhance the structurer (pelt.fv) and corresponding
contracts to automatically take the properties of
(if defined) and use it instead the right now hard coded values (which
would become fallbacks in the contracts). 

Like (just brainstorming here):

<xsl:stylesheet version="1.1"
  <xsl:param name="defaultVariables" select="'test.html'"/>
  <xsl:param name="branding-logo-name" select="'MyGroup'"/>     
  <xsl:variable name="projectBranding-logo-name"
  <xsl:variable name="finalBranding-logo-name">
    <xsl:if test="$projectBranding-logo-name=''">  
      <xsl:value-of select="$branding-logo-name"/>
    <xsl:if test="not($projectBranding-logo-name='')">  
      <xsl:value-of select="$projectBranding-logo-name"/>

> I reckon that we should continue to work
> on the default core pelt.fv and regularly
> do 'svn merge' to this site-author version.
> So the only changes to the site-author one
> would be specific to our f.a.o site.

Well, that will end in a build failed for the site-author site since we
do not have e.g. the requested logos (see my comment at 12/Feb/06 01:14

> Otherwise maintenance will be a nightmare.
> We also need to be able to test the pelt.fv
> with the seed-sample site.

In the sample seed we do not use customized logos.


I am not sure. 

I agree we should enhance the dispatcher but IMO we should focus on
"The dispatcher needs an aggregation of"
which means we need to resolve
and then we can close FOR-816 by using FOR-811, but till then I am
afraid we need FOR-816.


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

View raw message