Revert "Add type hint against LinkTarget"
authorBrian Wolff <bawolff+wn@gmail.com>
Mon, 29 Feb 2016 23:31:48 +0000 (23:31 +0000)
committerBrian Wolff <bawolff+wn@gmail.com>
Mon, 29 Feb 2016 23:31:48 +0000 (23:31 +0000)
Can cause fatal if your database has invalid titles
stored in it (Title::makeTitleSafe is fed to $lb->addObj()
without checking that the result is non-null)

This reverts commit 0e5a1f8879b211b44416e3762bf2d76c28f3f569.

Change-Id: I4174b2fdcc3d8e24ecf0bcc39cd0f0f2e8bfd6b2

includes/cache/LinkBatch.php

index 987aa71..8f334cc 100644 (file)
@@ -58,8 +58,12 @@ class LinkBatch {
        /**
         * @param LinkTarget $linkTarget
         */
-       public function addObj( LinkTarget $linkTarget ) {
-               $this->add( $linkTarget->getNamespace(), $linkTarget->getDBkey() );
+       public function addObj( $linkTarget ) {
+               if ( is_object( $linkTarget ) ) {
+                       $this->add( $linkTarget->getNamespace(), $linkTarget->getDBkey() );
+               } else {
+                       wfDebug( "Warning: LinkBatch::addObj got invalid LinkTarget object\n" );
+               }
        }
 
        /**