Why is llDetectedGroup ignored in this script?

This is a curiosity....... A shop I visit occasionally offers freebies to group members, but I have always been able to get the free gift, even if I forget to activate the group tag. (I'm not trying to cheat -- honest. I am always wearing an active tag; it's just for another group, and I have a mind like a seive. ) By chance, I happened to get a copy of the vendor script, which is very simple. I can't see why it would allow a non-group member to get a freebie. Can you?

default
{
    touch_start(integer total_number)
    {
        integer number = 0;
        do
        {
            if (llDetectedGroup(number)) 
                llGiveInventory(llDetectedKey(number), llGetInventoryName(INVENTORY_OBJECT,0));
            else
                llSay(0, "Wrong active group!");
        }while(total_number > ++number);
    }
}

Looks like the same code from http://wiki.secondlife.com/wiki/LlDetectedGroup.

I don't see any problem with it. How long has this been happening?

At least a couple of weeks, probably longer. I wasn't even paying attention until someone pointed out that I had to be wearing the group tag to get the freebie ..... and I had been getting freebies on several previous visits and had never thought about the tag before. I just went back this morning to check .... and got the freebie again without the tag.

It worries me because I use llDetectedGroup a fair amount and have never had a reason not to trust it before now.

Are you sure that the giving object still has its own group set correctly?
.

Are you SURE that is the script in the giver?

That one appears to work just fine.

If the object's own group isn't set properly, it should still refuse to give ME a freebie unless MY group is set to whatever it's group is. (Unless, of course, both of us are set to NO group. That's not the case, though, since I never set my own tag to NO group.)

No, I can't be confident that this is the script they are using, but I'm 90% sure. They accidentally included it along with the freebies for a week, so it was evidently in the vendor's contents. That's a good thing to check, though. I just didn't want to alarm the shop owner until I had exhausted other possibilities.

Still a puzzle.....

it's possible that the version they have in the giver has been modified. a simple modification might be to add a list of previous validated users, and allow them if they are found in the list. (or it could be that the script they are using is borked in some other way like the addition of a direct comparison that actually being done as an assignment)

/me guesses this is right. I've also recently done some group-only code for a in-sim tp system and it works as-advertised..

OK.... thanks, team. I'm glad to hear there isn't some odd feature of llDetectedGroup that I had been overlooking. If it's just their implimentation that's somehow borked, I can live with that. One less group tag to remember to wear....