Skip to content

MusicStore/src/MvcMusicStore/Models/MusicStoreEntities.cs Question #383

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

Closed
danofa opened this issue Mar 13, 2015 · 3 comments
Closed

MusicStore/src/MvcMusicStore/Models/MusicStoreEntities.cs Question #383

danofa opened this issue Mar 13, 2015 · 3 comments
Labels

Comments

@danofa
Copy link

danofa commented Mar 13, 2015

Just a small question pertaining to MusicStore/src/MvcMusicStore/Models/MusicStoreEntities.cs
on line 8 you have : base("name=MusicStoreEntities")

now as i understand it, this is not necessary as the derived context is already named "MusicStoreEntities", am i correct or have i misunderstood its use?
thanks.

@Eilon
Copy link
Contributor

Eilon commented Mar 25, 2015

@divega @rowanmiller thoughts on this?

@Eilon
Copy link
Contributor

Eilon commented Mar 25, 2015

Oh actually sorry that's the old MVC Music Store. The original MVC Music Store in that repo (MvcMusicStore) it pretty much an as-is copy of the original, so we deliberately didn't change almost anything in it. The new music store with ASP.NET 5, MVC 6, and EF 7 is called just MusicStore and doesn't have that code.

@rowanmiller
Copy link
Contributor

@danofa it's not required but it does force the context to get the connection from the config file. Without that line, if the context didn't load the string from config it would pick another database by convention. I personally also think including it makes it more explicit where the string is coming from.

ryanbrandenburg pushed a commit that referenced this issue Nov 19, 2018
natemcmaster pushed a commit that referenced this issue Nov 30, 2018
Reference `Entropy` localization samples
@ghost ghost locked as resolved and limited conversation to collaborators Dec 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants