Redefining already defined constructor

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Chris Luke

    (@chrisy)

    8 years, 6 months ago

    Okay, I guess you have PHP 5.4 or newer set to be super picky. It’s complaining about defining both a classic constructor “TFO_Graphviz_Graphviz()” and a new-style one “__construct()” even though PHP has a well-defined heuristic for how to deal with this. The code needs the classic definition to be backward compatible.

    So really the question is: Have we arrived at a point where we can remove the old-style constructor? I’ll noodle that.

    Plugin Author Chris Luke

    (@chrisy)

    8 years, 6 months ago

    Okay, minimum requirement for WP is PHP 5.6 now, so yes, it’s safe to remove this backwards-compatibility thingy.

    I’ll push that out a bit later.

    Plugin Author Chris Luke

    (@chrisy)

    8 years, 6 months ago

    Done, let me know if it resolves your issue, or if it breaks something else 🙂

    Thread Starter samwilson

    (@samwilson)

    8 years, 6 months ago

    Yep, seems good. Now I’ve a different error though… 🙂

    I’ll start a new thread.

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Redefining already defined constructor’ is closed to new replies.