• Resolved designdrumm

    (@designdrumm)


    Just fyi. $this::version should be $this->version I believe in class.bcn_admin.php.
    Not sure where else.

    Best,
    designdrumm

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author John Havlik

    (@mtekk)

    That switch was intentional and was made quite a while ago (5.2.0 made that change which was released almost two years ago). If you want to run on PHP5.2, the latest release you can run is 5.1.1.

    Thread Starter designdrumm

    (@designdrumm)

    I am on PHP version 5.5.35. Are you saying it doesn’t work with php versions newer than 5.1.1 or is that the version number for the plugin?

    • This reply was modified 9 years, 5 months ago by designdrumm.
    • This reply was modified 9 years, 5 months ago by designdrumm.
    Thread Starter designdrumm

    (@designdrumm)

    Dreamweaver says your code is broken. That is what brought it to my attention.

    Plugin Author John Havlik

    (@mtekk)

    Sorry if I was not clear in my first post, if you are using PHP5.2.x the latest version of Breadcrumb NavXT that you can run is 5.1.1. PHP5.3.x (while ancient) should work fine with Breadcrumb NavXT 5.6.0.

    As for the $this::version, since version is a constant, the correct way to access it is via the :: operator. Now, instead of using self::version I am using $this::version so that if the class is extended, it will grab that class’ version rather than bcn_admin::version. As for Dreamweaver flagging it, I assume it is a bug in Dreamweaver’s PHP support (both Aptana and Kdevelop do not flag it and PHP doesn’t seem to complain). More reading on constants within classes in PHP: http://php.net/manual/en/language.oop5.constants.php#120083

    • This reply was modified 9 years, 5 months ago by John Havlik.
    Thread Starter designdrumm

    (@designdrumm)

    Hmmm, that’s interesting. It doesn’t flag any of your other constants that I have seen so far, so my thoughts are that the support is there. This is your plugin to do as you wish, so as long as it works. This was just an fyi. Thanks for the responses!

    Best,
    designdrumm

Viewing 5 replies - 1 through 5 (of 5 total)

The topic ‘Recent Update’ is closed to new replies.