Aaron Schulz has uploaded a new change for review.
https://gerrit.wikimedia.org/r/63824
Change subject: Made DatabaseBase::getSoftwareLink() dynamic.
......................................................................
Made DatabaseBase::getSoftwareLink() dynamic.
* All callers are calling it this way and it breaks
hhvm when declared statically and called dynamically.
Change-Id: I894e615fd828615384aa8457a16a759c8aa416ef
---
M includes/db/Database.php
M includes/db/DatabaseMssql.php
M includes/db/DatabaseMysql.php
M includes/db/DatabaseOracle.php
M includes/db/DatabasePostgres.php
M includes/db/DatabaseSqlite.php
6 files changed, 6 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/24/63824/1
diff --git a/includes/db/Database.php b/includes/db/Database.php
index 85c459e..09866dc 100644
--- a/includes/db/Database.php
+++ b/includes/db/Database.php
@@ -184,7 +184,7 @@
*
* @return string: wikitext of a link to the server software's web site
*/
- static function getSoftwareLink();
+ function getSoftwareLink();
/**
* A string describing the current software version, like from
diff --git a/includes/db/DatabaseMssql.php b/includes/db/DatabaseMssql.php
index 130ac70..240a097 100644
--- a/includes/db/DatabaseMssql.php
+++ b/includes/db/DatabaseMssql.php
@@ -654,7 +654,7 @@
/**
* @return string wikitext of a link to the server software's web site
*/
- public static function getSoftwareLink() {
+ public function getSoftwareLink() {
return "[http://www.microsoft.com/sql/ MS SQL Server]";
}
diff --git a/includes/db/DatabaseMysql.php b/includes/db/DatabaseMysql.php
index d8a3723..ca5a2b4 100644
--- a/includes/db/DatabaseMysql.php
+++ b/includes/db/DatabaseMysql.php
@@ -671,7 +671,7 @@
/**
* @return string
*/
- public static function getSoftwareLink() {
+ public function getSoftwareLink() {
return '[http://www.mysql.com/ MySQL]';
}
diff --git a/includes/db/DatabaseOracle.php b/includes/db/DatabaseOracle.php
index c197d91..4fa2397 100644
--- a/includes/db/DatabaseOracle.php
+++ b/includes/db/DatabaseOracle.php
@@ -837,7 +837,7 @@
/**
* @return string wikitext of a link to the server software's web site
*/
- public static function getSoftwareLink() {
+ public function getSoftwareLink() {
return '[http://www.oracle.com/ Oracle]';
}
diff --git a/includes/db/DatabasePostgres.php b/includes/db/DatabasePostgres.php
index b5ac5cb..367335e 100644
--- a/includes/db/DatabasePostgres.php
+++ b/includes/db/DatabasePostgres.php
@@ -1059,7 +1059,7 @@
/**
* @return string wikitext of a link to the server software's web site
*/
- public static function getSoftwareLink() {
+ public function getSoftwareLink() {
return '[http://www.postgresql.org/ PostgreSQL]';
}
diff --git a/includes/db/DatabaseSqlite.php b/includes/db/DatabaseSqlite.php
index 53a4dcf..ed6c848 100644
--- a/includes/db/DatabaseSqlite.php
+++ b/includes/db/DatabaseSqlite.php
@@ -612,7 +612,7 @@
/**
* @return string wikitext of a link to the server software's web site
*/
- public static function getSoftwareLink() {
+ public function getSoftwareLink() {
return "[http://sqlite.org/ SQLite]";
}
--
To view, visit https://gerrit.wikimedia.org/r/63824
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I894e615fd828615384aa8457a16a759c8aa416ef
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
