[dev] Re: [cvs] commit: horde signup.php horde/admin user.php horde/lib Signup.php horde/templates/admin/user approve.inc removequeued.inc

[Chuck Hagenbuch]

Quoting John Morrissey <jwm at horde.net>:

> >   * if there are no queued signups, say so in
> templates/admin/user/approve.inc
>
> I'd appreciate it if someone could check this for style; I wasn't sure what
> to do when mixing blocks of PHP and HTML like that.

When you have a set of PHP blocks in the same set of <?php ?> tags, I think it's
preferable to use the brace syntax rather than colons. Otherwise, looks fine.

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
They're just looking at a wall of meat.