Abstract static method replaced by a runtime exception in *Peer #436

Closed
wants to merge 1 commit into from

3 participants

@Martiusweb

According to strict PHP standards with PHP 5.2 and more, abstract static
functions are not allowed.

This post on Stack Overflow summarize main points about this topic:
http://stackoverflow.com/questions/999066/why-does-php-5-2-disallow-abstract-static-class-methods

@Martiusweb Martiusweb Abstract static method replaced by a runtime exception
According to strict PHP standards with PHP 5.2 and more, abstract static
functions are not allowed.

This post on Stack Overflow summarize main points about this topic:
http://stackoverflow.com/questions/999066/why-does-php-5-2-disallow-abstract-static-class-methods
58f51d3
@travisbot

This pull request passes (merged 58f51d3 into 6d31fa6).

@willdurand willdurand added a commit that closed this pull request Aug 7, 2012
@willdurand willdurand Fix #436 e443b61
@willdurand willdurand closed this in e443b61 Aug 7, 2012
@Martiusweb

Hi will, Thank you merging the PR, but php 5.4 (and I think it's the same for php 5.3) still triggers notices about abstract static methods. I don't believe that abstract static method are allowed at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment