Hi! > Only with short tags on ... 5.4 has detached it from that, but 5.3 still > switches it off by default, and convincing ISP to switch short tags on again > is > a lost cause :( Fixing it in PHP5.3 was rejected :(
Errm... So you are complaining we're not going back in time now and make different decision about PHP 5.3 that was made years ago? Or what is your point exactly? > Fine, so I just add 'static' rework all the code, and fix the error, but I > think > I should also be adding public as well? It's not needed to clear the warning, If your functions are public, I'd recommend adding public. However, as the same manual states, public is optional for BC reasons and if access is omitted, public is assumed. Same manual, same page, one paragraph above: For compatibility with PHP 4, if no visibility declaration is used, then the property or method will be treated as if it was declared as public. > mind so much some of the rework, but I've no confidence that I will not be > doing > this exercise again next year - on the same code :( If you want somebody to assure you that the code that nobody except you have seen would not contain any problems after you fix some of the problems you have now but nobody except you knows about - then I'm afraid it's not something that anybody here is able to promise you. If you have some specific proposal - let's hear it. Otherwise I'd recommend following current best practices for writing good PHP code - e.g. what described in the manual and many framework style guides, etc., don't use any "clever" tricks that abuse PHP constructs for what they weren't meant to do - and you have reasonable assurance it will keep working fine. -- Stanislav Malyshev, Software Architect SugarCRM: http://www.sugarcrm.com/ (408)454-6900 ext. 227 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
