(11-19-2011 09:42 PM)MattR Wrote: The changes are incredibly minor and only affect a select list of hooks. Some functions will require a & to be placed before the variable that's passed to the function, and some require you to add return $var; to the end of the function, where $var is the value that's passed to the function (in fact it's actually possible to use the & method for all the changes).
I generally declare functions with an argument reference when I need it, as it makes sense to have it there IMO.
Do you
have to return something in most cases? I would presume if nothing is returned (ie null), the system should just ignore it... I'll check when I can be bothered.
If the ref was converted into a required return, then that would be a problem.
Also would need to check whether the changes will still allow backwards compatibility as I still support MyBB 1.4. There's probably a workaround regardless.
(11-19-2011 09:42 PM)MattR Wrote: call time pass by reference was deprecated in PHP 5.3, and as more hosts are upgrading to it, the change needs to be made, because otherwise you'll get errors on your forum and people will still complain.
Not really, I believe it only errors if you enable deprecated warnings, but MyBB disables it. Calling with a reference works fine in 5.3, whether it will or not in 5.4 is another thing though.
(11-19-2011 09:42 PM)MattR Wrote: I have 15 plugins and it took me under an hour to compile a list of the lines that need changing and then to make the changes. It's not like you're going to have to rewrite all your plugins.
Yeah it's probably easy if you know what you're looking for. For those that aren't being maintained, it could be more difficult.
Then again, every new MyBB version has the potential to break plugins (though this one's a bit more fundamental) so meh.
Search
Member List
Calendar
Help
