Re: AttributeForm question

Chuck Hagenbuch <[email protected]>
Newsgroups gmane.comp.horde.whups
Message-ID <[email protected]>
Quoting Mike Baptiste <[email protected]>:

> In other words, doesn't this:
> 
> ~        if (isset($grouptype)) {
> ~            $this->addVariable(_("Viewable only by members of"),
> 'group', $grouptype, true);
> ~        }
> 
> require the following to be called earlier in the method:
> 
> ~        $groups = &Group::singleton();
> ~        $grouplist = $groups->getGroupMemberships(Auth::getAuth());
> ~        if (count($grouplist)) {
> ~            $grouptype = _getEnumType(array(0 => _("Any Group")) +
> $grouplist);
> ~        }

Yes; my guess is that Robert didn't get to implementing it completely, and
so left it out that way. I'll look at it when I get a chance, or I'm sure
he'll get to it at some point too.

> That's how the other group restrictable classes seem to function.  Just
> thought I'd mention it.  I'd spin a patch, but my Search.php has a bunch
> of other stuff added to it so it would be ugly :)  With any luck I'll be
> adding attachments via VFS to tickets by the weekend.

Cool!

-chuck

--
Charles Hagenbuch, <[email protected]>
"People ask me all the time what it will be like living without otters."
 - Google, thanks to Harpers

-- 
Whups mailing list
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.