[dev] [cvs] commit: wicked/docs CHANGES wicked/lib Driver.php Page.php wicked/lib/Driver sql.php wicked/lib/Page AddPage.php DeletePage.php EditPage.php MergeOrRename.php NewPage.php StandardPage.php wicked/lib/Page/StandardPage StdHistoryPage.php ...

Michael M Slusarz slusarz at mail.curecanti.org
Thu Feb 23 15:59:06 PST 2006


Quoting Chuck Hagenbuch <chuck at horde.org>:

> chuck       2006-02-23 13:51:26 PST
>
>   Modified files:
>     docs                 CHANGES
>     lib                  Driver.php Page.php
>     lib/Driver           sql.php
>     lib/Page             AddPage.php DeletePage.php EditPage.php
>                          MergeOrRename.php StandardPage.php
>     lib/Page/StandardPage StdHistoryPage.php
>     templates/display    standard.inc
>     templates/edit       create.inc standard.inc
>   Added files:
>     lib/Page             NewPage.php
>     templates/edit       new.inc
>   Log:
>   [cjh] Don't create pages until they have content (Bug #2534).

Noticed on some of the commits that you have flip-flopped on the  
reference assignment syntax.  According to docs/CODING_STANDARDS:

If assigning a reference to a variable, place the ampersand next to the
referenced object, not the equal sign::

   $reference = &$foo;
   $reference = &foo();

Have we changed that now?

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]


More information about the dev mailing list