DevForce Community Forum : Entity.OnPropertyChanged not called

Entity.OnPropertyChanged not called : If you can't upgrade to the...

Thu, 07 Jun 2012 14:21:24 -700

Author: stephenmcd1
Subject: 3307
Posted: 07-Jun-2012 at 2:21pm

If you can't upgrade to the latest version, you can use a hack similar to the one we needed to have until we were able to upgrade. I made a helper method that will attach a dummy event handler which will cause the OnPropertyChanged method to be called like normal. Here is some sample code:

        #region Property Changed Helper
        /// 
        /// Whether we've attached the handler to this instance.
        /// 
        private bool _propertyChangeEventHandlerAdded;

        /// 
        /// An instance of  that we can use for
        /// all instances as a dummy handler.
        /// 
        private static readonly PropertyChangedEventHandler _cachedHandler = delegate { };

        /// 
        /// Ensures that  will be fired when properties change.
        /// 
        /// 
        /// To ensure the method gets called, we need to attach a dummy event handler to 
        ///  - otherwise, DevForce won't even call the
        /// method.
        /// 
        private void EnsureOnPropertyChangedFires()
        {
            if (_propertyChangeEventHandlerAdded)
                return;

            PropertyChanged += _cachedHandler;
            _propertyChangeEventHandlerAdded = true;
        }

        #endregion

I put that code into the base class that all our entities derive from. Then when I wanted OnPropertyChanged to get fired, I called EnsureOnPropertyChangedFires() (and it's safe to call that method multiple times). We had the method call in a couple of different places because there are times when we actually didn't want OnPropertyChanged to be called. But to keep things simple, you could probably put that call in the entity's constructor maybe.

Entity.OnPropertyChanged not called : Hi Von,We don't normally...

Thu, 07 Jun 2012 13:01:47 -700

Author: DenisK
Subject: 3307
Posted: 07-Jun-2012 at 1:01pm

Hi Von,

We don't normally provide a hot fix patch to an older version. You can either upgrade to our latest 6.1.7.1 which can be found in our website, or to 6.1.5. I will PM you the link to download this.

Entity.OnPropertyChanged not called : Hi,We're still using Devforce...

Wed, 06 Jun 2012 20:19:31 -700

Author: Vonzkie
Subject: 3307
Posted: 06-Jun-2012 at 8:19pm

Hi,

We're still using Devforce 6.1.4 and we experienced the same problem, can you provide us a Hotfix Patch?

Thanks,

Von

Entity.OnPropertyChanged not called : We are still on 6.1.3.1 so that's...

Wed, 29 Feb 2012 16:51:54 -700

Author: stephenmcd1
Subject: 3307
Posted: 29-Feb-2012 at 4:51pm

We are still on 6.1.3.1 so that's probably why we aren't seeing this behavior. The test you provide is exactly the behavior I was expecting (but not seeing). Thanks for looking into it!

Entity.OnPropertyChanged not called : Hi stephenmcd1,What version are...

Wed, 29 Feb 2012 12:22:24 -700

Author: DenisK
Subject: 3307
Posted: 29-Feb-2012 at 12:22pm

Hi stephenmcd1,

What version are you using?

I believe this issue has been fixed as of 6.1.5. Can you confirm that we're talking about the same behavior here? Below is my test code.

 public partial class Employee {

    protected override void OnPropertyChanged(PropertyChangedEventArgs e) {
      if (e.PropertyName == "Country" && this.Country == "Country") {
        this.Country = "Narnia";
      }
    }
}

var emp = mgr.CreateEntity();
emp.Country = "Country";
Assert.IsTrue(emp.Country == "Narnia");

Entity.OnPropertyChanged not called : In our entities, we often override...

Tue, 28 Feb 2012 18:26:58 -700

Author: stephenmcd1
Subject: 3307
Posted: 28-Feb-2012 at 6:26pm

In our entities, we often override Entity.OnPropertyChanged to execute logic when a property changes. After a lot of debugging, I just found out that OnPropertyChanged does not get called if there are not any subscribers to the Entity.PropertyChanged event. I can understand the rational behind this - some kind of optimization I suppose. But in our case, we always want OnPropertyChanged to be fired - even if nobody external is watching the entity for changes, the entity itself might be waiting for property changes.

As a hack, I've updated our entities to attach a dummy handler to PropertyChanged when they are first created and that seems to fix the problem.....but it doesn't feel like a very nice hack. Is there a reason that EntityAspect.FirePropertyChanged short-circuits if there are no subscribers? It doesn't seem like it'd be that much of an overhead and it seems like non-obvious behavior to me.