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.


_______________________________________________
MediaWiki-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-l

Reply via email to