Ayende @ Rahien

Chris Holmes commented on NHibernate & INotifyPropertyChanged

Thu, 10 Sep 2009 16:57:38 GMT

Okay, that makes sense. I figured it had to be something like that; I am just not very familiar yet with how NHibernate does things yet. Thanks Oren!

Ayende Rahien commented on NHibernate & INotifyPropertyChanged

Thu, 10 Sep 2009 16:50:46 GMT

Chris, This is because we aren't creating an instance yet. If you want Load to support it as well you need to provide a ProxyFactoryFactory implementation that will support it. In other words, there are two places that you need it. In the interceptor (when NH create the actual instance) and in the proxy factory (when NH creates the proxy)

Chris Holmes commented on NHibernate & INotifyPropertyChanged

Thu, 10 Sep 2009 16:45:40 GMT

Oren, I ran into a problem with this that is really strange (but maybe makes sense). When I call Get

Chris Holmes commented on NHibernate & INotifyPropertyChanged

Thu, 10 Sep 2009 16:43:17 GMT

Jon, All you need to do is add one line of code to your Fluent Buildup. Here's what mine looks like: var intercepter = new DataBindingIntercepter(); SessionFactory = Fluently.Configure() .Database(MsSqlConfiguration.MsSql2005 .ConnectionString(x => x.FromConnectionStringWithKey("Movies")) .DefaultSchema("dbo") .ShowSql ) .Mappings(m => m.FluentMappings.AddFromAssemblyOf x.SetInterceptor(intercepter)) .BuildSessionFactory(); intercepter.SessionFactory = SessionFactory; That last line sets the SessionFactory on the interceptor. >

Jon Masters commented on NHibernate & INotifyPropertyChanged

Tue, 18 Aug 2009 14:04:47 GMT

Turns out the issue was that SessionFactory on the DataBindingInterceptor was null. I had been setting the interceptor in the fluent buildup of the factory .ExposeConfiguration(config=>config.SetInterceptor(new DataBindingInterceptor())) Instead I modified my OpenSession method to: IInterceptor dataBinding = new DataBindingInterceptor {SessionFactory = factory}; return factory.OpenSession(dataBinding) Works great now.

Ayende Rahien commented on NHibernate & INotifyPropertyChanged

Tue, 18 Aug 2009 02:02:03 GMT

Jon, Not off the top of my head. Try creating a small test case that shows this.

Jon Masters commented on NHibernate & INotifyPropertyChanged

Tue, 18 Aug 2009 01:54:41 GMT

I'm really excited about getting this to work, but i've run into a problem implementing. My domain objects are in a different project from my repositories, which I believe causes all the Type.GetType(clazz) to return null. I was able to get around it by using public Type FindType(string typeName) { foreach (Assembly assembly in AppDomain.CurrentDomain.GetAssemblies()) { Type foundType = assembly.GetType(typeName); if (foundType != null) return foundType; } return null; } but SessionFactory.GetClassMetadata(clazz).SetIdentifier(instance, id, entityMode); errors with a null reference exception. I tried to switch from clazz, to type that was returned from FindType, but same result. Any ideas?

Ayende Rahien commented on NHibernate & INotifyPropertyChanged

Sat, 15 Aug 2009 20:08:28 GMT

Glenn, Virtuals are not an issue, NH already has that requirement. Type wrapping is actually something we are considering for DP 3.0 The problem is with instance management and leaking this at that point.

Glenn Block commented on NHibernate & INotifyPropertyChanged

Sat, 15 Aug 2009 19:23:20 GMT

Nice post Oren! I like the way you went further and added nested subscriptions. One thing though, this requires all the props to be virtual. Another idea I had been toying with to some success was to use mixins to create a proxy that delegates to a poco rather than deriving from it.. This way the poco does not need to have virtual props. You could take the approach further to then allow specifying exactly which members are exposed in the ViewModel rather than automatically adding all of the members to the VM surface. Any thoughts on this? Glenn

Ayende Rahien commented on NHibernate & INotifyPropertyChanged

Mon, 10 Aug 2009 23:45:42 GMT

Jernej, I have no idea, ISTR that DDD mentions factories, but regardless, I don't care about DDD in this context.

Jernej Logar commented on NHibernate & INotifyPropertyChanged

Mon, 10 Aug 2009 22:39:10 GMT

One question. How does this way of creating entities go together with aggregates (as in DDD)? You can't have the agg POCO create a proxied POCO this way.

José Romaniello commented on NHibernate & INotifyPropertyChanged

Sat, 08 Aug 2009 14:30:08 GMT

Besides of what Tuna said, I implement the code in unhaddins.wpf to work with INotifyPropertyChanged, INotifyCollectionChanged, IEditableObject (two implementatios). For InotifyPropertyChanged my IInterceptor is very similar (although I have another interceptor for the entity name thing)... and I use another extension point of nhibernate to inject the proxy...

Ayende Rahien commented on NHibernate & INotifyPropertyChanged

Sat, 08 Aug 2009 13:18:16 GMT

Dmitry, Yes, sort of. You would need to write a accessor, take a look at how NHibernate Generics was implemented.

Tuna Toksoz commented on NHibernate & INotifyPropertyChanged

Fri, 07 Aug 2009 22:26:39 GMT

Here is a post on the same issue, he also has INotifyCollectionChanged etc. very cool posts. [jfromaniello.blogspot.com/.../...anged-as-aop.html](http://jfromaniello.blogspot.com/2009/07/inotifypropertychanged-as-aop.html)

Frank Quednau commented on NHibernate & INotifyPropertyChanged

Fri, 07 Aug 2009 21:28:32 GMT

I've used a similar approach on a project last year. It was more of a mapping between a ViewModel described just as an interface and some POCO. The interface would implement INPC and its nature allowed us to do funny things like declaratively adding Undo, Commit & Rollback functionality towards an associated POCO.

Dmitry commented on NHibernate & INotifyPropertyChanged

Fri, 07 Aug 2009 21:18:02 GMT

I really like this implementation. As you said, it is simple and elegant. Is there a way for NHibernate to instantiate BindingList for POCO properties of type IList ?

Krzysztof Koźmic commented on NHibernate & INotifyPropertyChanged

Fri, 07 Aug 2009 20:50:25 GMT

This is like 5th impl of INPC via DynamicProxy I've seen in last month :) Seems everyone is doing it. Anyway, this is a great example, although I would split all the logic you put into the interceptor, between InterceptorSelector, the interceptor and use mixin for the actual INPC implementation. I understand however that you probably did it in one place to keep the example short.