llAvatarOnSitTarget once again requires llSitTarget

Oohhhh kay!

So, PoseCubes is rolling out. I'm helping Phil Murdock and Snow Hare get the first PosAbility enabled furniture out.
"Admin commands aren't working!"
... checked everything, script settings, new copy of script, cut and pasted new script, sim, etc etc.
Did some debugging llSay commands... and it becaame clear... llAvatarOnSitTarget was retuning a key of all 0s! Problem!

Thanks to some quick thinking with Brent Linden, I have come to this explanation:

Pre-last patch, llAvatarOnSitTarget did NOT need llSitTarget to return the sitter's key. According to documentation for LSL, it needs it. Linden Lab likely ninja-changed it back to match the documentation, and either forgot to mention it in the release notes, or I outright missed the mention of it.

So once again, llAvatarOnSitTarget now REQUIRES llSitTarget to return anything but NULL_KEY.

Fortunately, this only affects newly compiled code, so it does not affect any PoseCubes already sold. And fortunately, the fix was a mere 1 line of code in my init routine. PHEW!

I wanted to give you folks the heads up, I'll be checking the release notes to check when this was fixed if it was mentioned.

just checked http://history.secondserver.net/index.php/Release_Notes

1.6 on ... did not see any mention of this *sighs*

err, I think this is always the way it has worked man. I've never gotten anything other than null keys without a sit target before.

Agreed, I've always had to set a sit target before it worked. On the other hand, there's been times that I set a sit target on a prim, then deleted the script to start over. The sit target, being a property of the prim itself, still existed. It would usually then be not noticed until I copied the prim or the script.


- Newfie

I'm pretty sure I managed to use it without a sittarget at one point. Perhaps we can get a comment from Kelly on whether anything was actually changed?

It's been this way for the past... five... ten patches? Pre-1.6 I think. Really this isn't too cumbersome in my opinion.

That's utterly fucking stupid. They should have changed the docs, not the code. SL should allow developers the full extent of whats technically achievable with it, without imposing arbitrary restrictions on our creativity.