Good catch and thanks for the fix. I’ll see if I can get it into the download soon.

–Ken

Fix:

void FindResourceFilesRecursivlyAndRecordEntries(ProjectItems items, string path)
{
foreach(ProjectItem item in items)
{
// WriteLine( item.Name + ” ” + item.ProjectItems.Count );

if(Path.GetExtension(item.Name) == “.resx”)
RecordEntriesInResourceFile(item, path);
if(item.Kind == Kind_PhysicalFolder)
FindResourceFilesRecursivlyAndRecordEntries(item.ProjectItems, path + “.” + item.Name);
else if (item.ProjectItems.Count > 0)
FindResourceFilesRecursivlyAndRecordEntries(item.ProjectItems, path );
}
}

Glad you found it useful. Thanks for pointing out the language attribute tweak for VS2008. I had forgotten about it.. It was actually in the T4MVC template that I started out with, but I removed it because it throws a warning in VS2010.

Regards,
–Ken

I had to add the language attribute to the template for VS2008 usage:

language=”C#v3.5″

Glad you found this helpful. I’ll take a closer look at this bug later. Thanks for the workaround.

Regards,
Ken

I did find an issue.
Because you sort all resources entries:
AllEntries.Sort( new Comparison( (e1, e2) => (e1.Path + e1.File + e1.Name).CompareTo(e2.Path + e2.File + e2.Name)));

the genereted code is getting into troubles determining when a new class should starts when resources file names start with the same name.

For example:
ABC.Resx
ABCD.Resx

The moment you have entries in the ABC.Resx that start with the name ABCD, the T4 will generate 2 ABC classes.

Commenting out the AllEntries.Sort solved the problem for me.

Hope this helps,
Linefeed

Good catch. I have corrected the code.

Thanks,
Ken

The project that I wrote this for has not gone to production yet but the approach has been working in development and testing for several months now with no issues.

Please let me know if you encounter any.

–Ken

I copy-pasted your code and cross my fingers it doesn’t introduce subtle errors. I am not comfortable with that “force model to null” statement.