Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inserts unused partial into definition of abstract base controller #79

Open
timabell opened this issue Jun 7, 2016 · 2 comments
Open

Comments

@timabell
Copy link

timabell commented Jun 7, 2016

Resharper was complaining at me because T4MVC had modified my abstract base controller by adding partial to the definition, even though T4MVC then doesn't generate another partial for it.

Cheers, great tool, very grateful!

@davidebbo
Copy link
Contributor

It does in some cases generate a partial class. e.g. see here and here. I think here it does it because it needs a default ctor. Maybe if there was one already it would not have needed the partial.

Feel free to send a PR if you find a way to only add partial when needed.

@timabell
Copy link
Author

timabell commented Jun 7, 2016

Oh I see, cheers for the info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants