search

AnantLabs / codesmith

Automatically exported from code.google.com/p/codesmith
1 stars 0 forks source link

CSLA vb.net Contructor.cst needs to revert to r2498 #638

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I think some core constructor options have been incorrectly deleted.

For instance editable root objects will need to add root items by 
SafeDataReader from editable root list DataPortal_Fetch statements (as per code 
generator for r1706).  See 
http://code.google.com/p/codesmith/source/browse/trunk/Templates/Frameworks/Csla
/VisualBasic/Common/DataPortalFetchList.StoredProcedures.cst
to confirm the need for constructors to accept DataSafeReader objects.

See 
http://code.google.com/p/codesmith/source/diff?spec=svn2591&r=2539&format=side&p
ath=/trunk/Templates/Frameworks/Csla/VisualBasic/Common/Constructor.cst&old_path
=/trunk/Templates/Frameworks/Csla/VisualBasic/Common/Constructor.cst&old=2498

Original issue reported on code.google.com by jamie.cl...@jenasysdesign.com.au on 29 Mar 2012 at 6:57

GoogleCodeExporter commented 9 years ago 
Hello,

This is the correct behavior. A Get factory method is now generated which calls 
DataPortal.FechChild which in return calls 

   Private Sub Child_Fetch(reader As SafeDataReader)
            Map(reader)
        End Sub

This change has fixed a few bugs in the template and is consistent with rocky's 
samples.

If you are running into a compile error, please let me know.

Original comment by bniemyjski on 5 Apr 2012 at 4:17