FAQ
Edit report at https://pear.php.net/bugs/bug.php?id=17980&edit=1

ID: 17980
Updated by: daniel.oconnor@gmail.com
Reported By: jaromir dot dolecek at skype dot net
Summary: Static calls to Validate::uri() trigger E_STRICT
Status: Open
Type: Bug
Package: Services_Facebook
Operating System: Irrelevant
Package Version: 0.2.14
PHP Version: 5.3.3
Roadmap Versions:
New Comment:

TODO: Ensure Validate package is non static


Previous Comments:
------------------------------------------------------------------------

[2010-10-20 13:36:43] jdolecek

-Type: Feature/Change Request
+Type: Bug


------------------------------------------------------------------------

[2010-10-20 13:32:32] jdolecek

Added #patch
bug:17980;patch:Services_Facebook_Validate.diff;revision:1287581552;.

------------------------------------------------------------------------

[2010-10-20 13:31:17] jdolecek

Description:
------------
Methods in Validate package are not marked as static, so
calling them statically triggers an E_STRICT warning:

Non-static method Validate::uri() should not be called
statically, assuming $this from incompatible context

I reported bug to Validate guys regarding this as #17975, but
they generally appear to not prefer the static approach and
prefer if the functions wouldn't be called statically.

So I propose this change to the Services_Facebook codebase,
which also fixes the problem.

------------------------------------------------------------------------

Search Discussions

  • Daniel Oconnor at Jan 24, 2012 at 12:15 pm
    Edit report at https://pear.php.net/bugs/bug.php?id=17980&edit=1

    ID: 17980
    Updated by: daniel.oconnor@gmail.com
    Reported By: jaromir dot dolecek at skype dot net
    Summary: Static calls to Validate::uri() trigger E_STRICT
    -Status: Open
    +Status: Closed
    Type: Bug
    Package: Services_Facebook
    Operating System: Irrelevant
    Package Version: 0.2.14
    PHP Version: 5.3.3
    -Assigned To:
    +Assigned To: doconnor
    Roadmap Versions:
    New Comment:

    -Status: Open
    +Status: Closed
    -Assigned To:
    +Assigned To: doconnor
    This bug has been fixed in SVN.

    If this was a documentation problem, the fix will appear on pear.php.net
    by the end of next Sunday (CET).

    If this was a problem with the pear.php.net website, the change should
    be live shortly.

    Otherwise, the fix will appear in the package's next release.

    Thank you for the report and for helping us make PEAR better.

    Forked/applied


    Previous Comments:
    ------------------------------------------------------------------------

    [2012-01-24 13:12:56] doconnor

    TODO: Ensure Validate package is non static

    ------------------------------------------------------------------------

    [2010-10-20 13:36:43] jdolecek

    -Type: Feature/Change Request
    +Type: Bug


    ------------------------------------------------------------------------

    [2010-10-20 13:32:32] jdolecek

    Added #patch
    bug:17980;patch:Services_Facebook_Validate.diff;revision:1287581552;.

    ------------------------------------------------------------------------

    [2010-10-20 13:31:17] jdolecek

    Description:
    ------------
    Methods in Validate package are not marked as static, so
    calling them statically triggers an E_STRICT warning:

    Non-static method Validate::uri() should not be called
    statically, assuming $this from incompatible context

    I reported bug to Validate guys regarding this as #17975, but
    they generally appear to not prefer the static approach and
    prefer if the functions wouldn't be called statically.

    So I propose this change to the Services_Facebook codebase,
    which also fixes the problem.

    ------------------------------------------------------------------------

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedJan 24, '12 at 12:11p
activeJan 24, '12 at 12:15p
posts2
users1
websitepear.php.net

1 user in discussion

Daniel Oconnor: 2 posts

People

Translate

site design / logo © 2021 Grokbase