plugin corrections

needs to be amended to remove the ‘ul’ markup otherwise mal-formed markup is generated. top level items on the adminbar already have a parent ul described.

In bp-cubepoint-core.php I noticed the use of ‘::’ which is 5.2.3 specific syntax any one with a slightly older version of PHP (which won’t be uncommon) will run into issues.

This issue surrounding syntax use has arisen on another thread. If possible alternatives should be used that are more backward compatible friendly.

I note an odd disconnect you have run a check on get_option(‘cp_donation’) and if it returns false you do not show the points value and link to donate on the bp-adminbar – all good! however not so great is the cubepoints widgets which still show the link to donate points despite it being disabled in admin options, could you pass that on to the cubepoint lads, they may want to amend that .

The :: is debatable as discussed on the other thread WP is stopping support for anything under PHP 5.2.3 so many might say it’s fine to use this syntax yet it will break in anything under 5.2.3, not sure it can be replaced without instantiation of the class, but I admit my understanding of classes is limited, I’m more a procedural guy, and all in all I’m probably being overly alarmist.

(I also understand that the double colon will be changed soon to double backslash ??!!)“People love PHP so much because it was written by a rodeo clown on acid.”