[Mediawiki-l] problem with mw 1.17 and extensions - fixed

Platonides Platonides at gmail.com
Fri Sep 16 21:52:36 UTC 2011


Daniel Walz wrote:
> There is a little difference:
> In the link I posted, it is
>
> if ( !isset( $this->mAccessedOptions ) ) {
> return array(); // BUGFIX: return empty array !!
> return false;
>
> In the official link you posted it is
>
> if ( !isset( $this->mAccessedOptions ) ) {
> return false;
> return array();
>
> So the order of the returns is reversed.

In the link I posted it was a diff, so
-			return false;
+			return array();
means to remove the first return and add the second one.

Which is equivalent to put the added return before the old (which would 
then never be reached).

>
> Anyway, I forgot to "purge" the cache, now the bug is fixed.
> Thanks!
>
> Daniel

I'm glad it turned out to be just that.




More information about the MediaWiki-l mailing list