BindPropertyChanged doesn't Fire

[FunkMonkey33]

I'm submitting a bug report

• Library Version: "aurelia-framework": "npm:aurelia-framework@^1.0.0-beta.1.1.0",

Please tell us about your environment: Windows 10

• Node Version: 6.10.3

• NPM Version: 3.10.10

• Aurelia CLI OR JSPM OR Webpack AND Version JSPM version 0.16.26

• Browser: Chrome Version 63.0.3239.132 (Official Build) (64-bit)

• Language: ES6

Current behavior:

If you handle the bind event in your view model, then any bindableChanged events don't fire. This is the case even if the bind() handler is empty.

`@bindable myProperty

bind() { // even an empty method causes this behavior. } myPropertyChanged(newValue, oldValue) { debugger; // this method will never get called. } `

Expected/desired behavior: Not that. It's not really a big deal, because you probably won't ever need both bindableChanged and the bind event, but this tripped me up for several hours.

• What is the motivation / use case for changing the behavior? Save developers some time and some hair.

[StrahilKazlachev]

This is documented in 3. bind(bindingContext: Object, overrideContext: Object).

[AshleyGrant]

@StrahilKazlachev we probably could do a better job of calling this behavior out.

[bigopon]

Yes pls @StrahilKazlachev @AshleyGrant 😄

[FunkMonkey33]

I see it! Thanks.

And yes, it could be bolder.

Thanks, guys.

Aaron

On Wed, Jan 31, 2018 at 3:06 PM, bigopon notifications@github.com wrote:

Yes pls @StrahilKazlachev https://github.com/strahilkazlachev @AshleyGrant https://github.com/ashleygrant 😄

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/aurelia/framework/issues/856#issuecomment-362087029, or mute the thread https://github.com/notifications/unsubscribe-auth/ABjEcRYIvdIsJegRkIvcgkRNHgOnLmdpks5tQOPzgaJpZM4R0Mcc .

[AshleyGrant]

Done! PR Submitted

[Alexander-Taran]

PR looks legit (-: @EisenbergEffect can you please merge it? or decline.. that would require reasoning though..