ItSpiderman has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/327503 )

Change subject: Added checks that prevent buildIndex from crashing on invalid 
page
......................................................................

Added checks that prevent buildIndex from crashing on invalid page

Change-Id: I8d2f6bffa0a9d253c97c0c19871188cdf4657193
---
M ExtendedSearch/includes/BuildIndex/BuildIndexMainControl.class.php
M ExtendedSearch/includes/BuildIndex/BuildIndexMwArticles.class.php
2 files changed, 5 insertions(+), 2 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions 
refs/changes/03/327503/1

diff --git a/ExtendedSearch/includes/BuildIndex/BuildIndexMainControl.class.php 
b/ExtendedSearch/includes/BuildIndex/BuildIndexMainControl.class.php
index f5a69d4..593974f 100644
--- a/ExtendedSearch/includes/BuildIndex/BuildIndexMainControl.class.php
+++ b/ExtendedSearch/includes/BuildIndex/BuildIndexMainControl.class.php
@@ -140,6 +140,7 @@
                if ( is_null( $oTitle ) ) return;
 
                $iPageID = $oTitle->getArticleID();
+               if( $iPageID == 0 ) return;
                $iPageNamespace = $oTitle->getNamespace();
                $sPageTitle = $oTitle->getText();
                $iPageTimestamp = $oTitle->getTouched();
@@ -761,4 +762,4 @@
                } catch ( Exception $e ) {}
        }
 
-}
\ No newline at end of file
+}
diff --git a/ExtendedSearch/includes/BuildIndex/BuildIndexMwArticles.class.php 
b/ExtendedSearch/includes/BuildIndex/BuildIndexMwArticles.class.php
index 3294220..742f847 100644
--- a/ExtendedSearch/includes/BuildIndex/BuildIndexMwArticles.class.php
+++ b/ExtendedSearch/includes/BuildIndex/BuildIndexMwArticles.class.php
@@ -118,6 +118,8 @@
                                $this->writeLog( $oTitle->getText() );
 
                                $iPageID = $oTitle->getArticleID();
+                               if( $iPageID == 0 ) continue;
+                               
                                $sPageTitle = $oTitle->getText();
                                $sPageNamespace = $oTitle->getNamespace();
 
@@ -153,4 +155,4 @@
                }
        }
 
-}
\ No newline at end of file
+}

-- 
To view, visit https://gerrit.wikimedia.org/r/327503
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8d2f6bffa0a9d253c97c0c19871188cdf4657193
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: master
Gerrit-Owner: ItSpiderman <d.savulje...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to