[r129] Pinned Behavior UID Bug

Link to .capx file (required!):

https://dl.dropboxusercontent.com/u/76227787/HealthBarBug.capx

Steps to reproduce:

1. Click "Swordsman" on left to create a blue swordman.

2. Click "Swordsman" on right to create a red swordman.

3. Watch their health bars.

4. While they are fighting... wait when they get 50% damage, click any "Swordsman" to reinforce.

Observed result:

Pinned UID is broken because it's registered to each UID to sprites. It affects every instances, not unique instances.

Expected result:

Pinned UID worked in older versions.

Browsers affected:

Chrome: yes

Firefox: yes

Internet Explorer: yes

Operating system & service pack:

Windows 8 64 bit

Construct 2 version:

129

There are too many events in that .capx to be able to easily say it's a bug and not just a mistake in your events. BTW, we haven't changed how 'pinnedUID' works for many releases now. If there really is a bug with it, it should be easy to reproduce in a new empty project.

Ashley

There no mistake, please try r126 stable version you see it's working against r129 which is broken.

I'm sorry for not commenting about the events, there are 3 events which are related to Pinned UID are event number 7, 8 and 14.

Joannesalfa - can you reduce the project further then? It's very, very difficult to debug projects with more than a few events. Also I'd recommend checking through the release notes to see if any breaking changes have affected your project, in which case it's not a bug and you'd need to change your project. If you can also narrow it down to the exact release where it changed, that would be helpful too.

Ashley There no "breaking change" between 127 and 129 versions, maybe there a feature "Pick by UID" could cause Pinned UID to stop working.

I edited .capx

https://dl.dropboxusercontent.com/u/76227787/healthbugdebug.capx

Make sure I'm clear about the screenshots what im talking the differences.

This is working, this project is using r126

<img src="http://i.imgur.com/FMMFz1X.png" border="0">

Then, it broke in r129

<img src="http://i.imgur.com/nm9gpKC.png" border="0">

Gotcha?

Thanks for making a minimal .capx, it's much easier this way.

I'm not sure why this worked in r126. Your events look wrong.

The event "Blue: pick instance with UID BlueHP.Pin.PinnedUID" should never have worked, but I guess it depended on the implementation details of 'pick by UID' which we changed recently. Add 'for each BlueHP' above it and it works. Since I don't really want to change 'Pick by UID' back to how it used to work (it's much faster now) I think the best thing to do is change your project; hopefully just adding a few 'for each' conditions will fix it.

Ashley Ironic.

It worked like r126 does in r129

<img src="http://i.imgur.com/HokLsLt.png" border="0" />

It's so confused.

You should have published a breaking change if UID for behaviors has been changed.

Anyways, it's not a bug after all, thanks...

Joannesalfa - I didn't realise it was a breaking change - I'll changelog it for the next stable release.