search

kswoll / ReactiveUI.Fody

C# Fody extension to generate RaisePropertyChange notifications for properties and ObservableAsPropertyHelper properties.
MIT License
154 stars 31 forks source link

One of the advertised dangers of AOP that bit me #30

Open buckleyGI opened 7 years ago

buckleyGI commented 7 years ago

Hi,

I have been looking the good last few hours of the day at a problem where one of the nested properties would not announce its change

The problem was not that it was nested but that the the type of the nested properties was not inheriting from ReactiveObject. The weaver gives not error and there is no run time warning in the output console.

Can we throw an error while weaving or let Fody inject the inheritance to ReactiveObject if its not already inheriting? I suspect others will run into this as well when using it in a bit more advanced scenarios

Kr, Tom

kswoll commented 7 years ago

Yeah, I've run into that myself. It's a good idea.