Update NHibernate setup, see #61 #123
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cd21h
commented
Jul 24, 2015
cd21h
commented
- Replace NHibernateSession with NHibernateSessionFactoryBuilder, fixes NHibernate event listeners registered through xml gets removed #54
- ValidatableObject now requires ValidationContext;
- HasUniqueDomainSignatureAttribute now requires ValidationContext;
- DataAnnotationsEventListener how provides IServiceProvider with ISession;
- TardisBank - use CastleWindsor to register ISession and ISessionFactory;
- TardisBank - update CastleWindsor installers.
- Use DependencyResolver in ASP.NET MVC
- Inherit ILinqRepository from IRepository
* Add NHibernateSessionFactoryBuilder, also fixes sharparchitecture#54 * Replace NHibernateSession with NHibernateSessionFactoryBuilder, see sharparchitecture#61 * ValidatableObject now requires ValidationContext; * HasUniqueDomainSignatureAttribute now requires ValidationContext; * DataAnnotationsEventListener how provides IServiceProvider with ISession; * TardisBank - use CastleWindsor to register ISession and ISessionFactory; * TardisBank - update CastleWindsor installers. * Use DependencyResolver in ASP.NET MVC * Inherit ILinqRepository from IRepository
| @@ -75,6 +62,18 @@ protected void Application_Start() | |||
| BundleConfig.RegisterBundles(BundleTable.Bundles); | |||
| } | |||
|
|
|||
| ISessionFactory CreateSessionFactory(IKernel kernel) | |||
There was a problem hiding this comment.
@Shatl would be better to move the NHibernate registration stuff into it's own installer
There was a problem hiding this comment.
@seif Yes, but I'd do this after we review multi-tenancy.
There was a problem hiding this comment.
@Shatl not sure how multi tenancy would affect the location of this code (I think multi tenancy is way far down the line, unless you meant multi database which would be sooner), but we can still add whatever functionality we want to the Installer instead of having this code here until then.
|
Nice one @Shatl :) |
* Move NHibernate initialization to Windsor installer; * Move Windsor FilterAttribute installation to SharpArch.Web.Mvc.Castle
|
@seif Updated |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.