Re: [wtp-incubator-dev] StylesheetModel question

[DOUG SATCHWELL <doug.satchwell@xxxxxxxxxxxxxx>]

Also, just to clarify the model a little bit, the Stylesheet class is a representation of a stylesheet, so its imports and includes are the ones declared in that stylesheet.
The StylesheetModel class, however, is a representation of the composed stylesheet i.e. its includes, imports, templates etc. are the sum of all such things in the stylesheet and all its included stylesheets. Its a bit like the distinction between .java files and .class files in the JDT.
I say this because one of the bugs mentions adding xsl:param to the model. To do this we would need to add:
public List<Param> getParams()
to the Stylesheet class (where Param is a new class). But the StylesheetModel interface would not change; its just that the Param's would get added to the list of global variables. Its a subtle distinction, but an important one.
We should probably make that distinction clearer by moving the StylesheetModel  class and any associated classes into a separate package.



----- Original Message ----
> From: DOUG SATCHWELL <doug.satchwell@xxxxxxxxxxxxxx>
> To: WTP Incubator Dev list <wtp-incubator-dev@xxxxxxxxxxx>
> Sent: Monday, 8 September, 2008 4:27:14 PM
> Subject: Re: [wtp-incubator-dev] StylesheetModel question
> 
> > I'm not sure having the validation framework keep it in synch is 
> > necessarily the right approach,
> We are doing it the same way as the XML project does it
> > and maybe we need 
> > buildStylesheet(IDOMDocument document) method to allow just passing in a 
> > IDOMDocument for building the model instead of relying on a file.  It 
> > can do what it needs under the cover, but from an editor standpoint it 
> > makes more sense to pass the current StructuredTextEditor DOM in instead 
> > of having to reload it.
> 
> If you take a look at StylesheetBuilder.build(IFile file), you will see that it 
> actually gets the StructuredModel associated with that file, so we are in fact 
> alredy using the StructuredTextEditor DOM under the covers.
> 
> _______________________________________________
> wtp-incubator-dev mailing list
> wtp-incubator-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/wtp-incubator-dev