Quantcast
Channel: patterns & practices: Prism
Viewing all articles
Browse latest Browse all 1878

Created Unassigned: BindableBase.OnPropertyChanged should be virtual! [10711]

$
0
0
protected void OnPropertyChanged(string propertyName)
should be:
protected __virtual__ void OnPropertyChanged(string propertyName)

It makes perfect sense to inherit BindableBase, and it also makes perfect sense to extend the "OnPropertyChanged" behavior. E.g. the DependsOnPropertyAttribute is a sensible extension of "OnPropertyChanged":

http://msdn.microsoft.com/en-us/library/microsoft.teamfoundation.mvvm.dependsonpropertyattribute(v=vs.110).aspx

Speaking of which, I think this attribute should be available in Prism!

Viewing all articles
Browse latest Browse all 1878

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>