PaloT wrote:
> 
> I commited some new features to SVN:
> - @Name annotation is generated for all method parameters (entity
> constructors, service methods, ...) - necessary for smartclient
> (should we turn it on/off by some property?)
> 

Yes, it should be turned off by default. We usually have property flags for
turning on/off individual features, such as:
generate.parameterNameAnnotation=true

Then we have project.nature, which is a higher level setting. In
GeneratorProperties.initDerivedDefaults you should set the generate flags
when hasProjectNature("business-tier") && hasProjectNature("smartclient")

This means that end user normally only has to care of high level project
nature property, but it makes it possible to override defaults at a fine
grained level also.


PaloT wrote:
> 
> - You can generate for all auditable entities "full audit log" by
> appending "generate.fullAuditable=true" to
> sculptor-generator.properties, this mean you can see in entity also
> old values through method receiveAuditHandler() which is generated to
> entity
> 

Good


PaloT wrote:
> 
> - Drools support - you can inject drools support to service tier
> calls, just add "generate.injectDrools=true" to
> sculptor-generator.properties.
> 

Good


PaloT wrote:
> 
> When I was doing some changes in templates I found that they are using
> mixed <TAB> <SPACES> indentation. Is it intention? I prefer pure <TAB>
> for indentation. I can update this in VIM by !retab command. What's
> your opinion (Patrik or anybody else ;-) )?
> 

We use 4 spaces for indentation in java code (no tabs). In templates we have
not cared about this and I think the xpand editor use tab and therefore it
is mostly tabs. In the end it doesn't matter in the templates. Please don't
reformat all templates now. We are working in  branch also and we must be
able to easily merge between them.



PaloT wrote:
> 
> I also find some strange construct like (L:294 in Spring.xpt):
> «IF isWar() -»
> «EXPAND headerWithMoreNamespaces»
> «ELSE -»
> «EXPAND headerWithMoreNamespaces»
> «ENDIF -»
> 
> Should I directly remove them when I see this? When I do this should I
> ask for review (this is best way for me)?
> 

This obviously wrong :-). Please remove. Right now we are trying to wrap up
the 1.7.0 release and we should avoid doing too much changes. When that
release is done you are welcome to do refactoring. I will review, especially
now for the 1.7.0 release. You checkin and I review when I synchronize with
svn.

Thanks, Patrik
-- 
View this message in context: 
http://old.nabble.com/New-features-tp26165006s17564p26191945.html
Sent from the Fornax-Platform mailing list archive at Nabble.com.


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Fornax-developer mailing list
Fornax-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fornax-developer

Reply via email to