AnantLabs / codesmith

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

CSLA Nightly Builds CSLA-r2952 DynamicRoot.Generated Friend scope instead of Public #681

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.Visual basic after generating...

What is the expected output? What do you see instead?

on DynamicRoot.Generated.cst in #Region "Synchronous Factory Methods"
the scope of the method is "Friend" instead of "Public", this issue is not 
encountered on #Region "Asynchronous Factory Methods", Is this the desired 
behaviour?

I did not checked the C# version...

Please use labels and text to provide additional information.

Original issue reported on code.google.com by yoselsua...@gmail.com on 5 Mar 2013 at 9:24

GoogleCodeExporter commented 9 years ago
Can you paste the comment block at the top of your DynamicRoot.Generated vb 
file. So I can see what version you are using as well as what template was 
being used. Also can you go into Manage Outputs (right click csp) and then 
double click the entities.cst output and then just click ok and save. Next, 
when you regenerate do you see this issue?

Original comment by bniemyjski on 5 Mar 2013 at 9:37

GoogleCodeExporter commented 9 years ago
'------------------------------------------------------------------------------
' <autogenerated>
'     This code was generated using CodeSmith: v6.0.3, CSLA Templates: 
v4.0.0.0, CSLA Framework: v4.3.x.
'     Changes to this file will be lost after each regeneration.
'     To extend the functionality of this class, please modify the partial 
class 'AS2MessageMDN.vb.
'
'     Template: DynamicRoot.Generated.cst
'     Template website: http://code.google.com/p/codesmith/
' </autogenerated>
'------------------------------------------------------------------------------

...well, I solved my problem, I just changed the template on 
"...\Templates\Frameworks\Csla\VisualBasic\BusinessLayer\Internal\DynamicRoot.Ge
nerated.cst".

If it is a bug, I just want you to know about it...and when I download future 
versions of it, I do not have to remember to redo the change on the Template on 
my side.

Thank you for your great support!!!...It could not be faster!!!.

Original comment by yoselsua...@gmail.com on 5 Mar 2013 at 10:00

GoogleCodeExporter commented 9 years ago
This issue was updated by revision r2956.

Fixed a bug where DynamicRoot's Async Factory methods were public and not 
internal

Original comment by bniemyjski on 6 Mar 2013 at 6:22

GoogleCodeExporter commented 9 years ago
Hello,

Thanks for pointing this out. They are supposed to be internal and I've updated 
the templates to fix this bug. The DynamicRoot business object is not meant to 
be retrieved like a root object. You should pick a root business object type 
for this functionality.

Original comment by bniemyjski on 6 Mar 2013 at 6:24