r72133 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r72132‎ | r72133 | r72134 >
Date:18:10, 1 September 2010
Author:demon
Status:ok
Tags:
Comment:
Reduce some duplication, just implement stub in parent
Modified paths:
  • /trunk/phase3/includes/installer/DatabaseInstaller.php (modified) (history)
  • /trunk/phase3/includes/installer/OracleInstaller.php (modified) (history)
  • /trunk/phase3/includes/installer/PostgresInstaller.php (modified) (history)
  • /trunk/phase3/includes/installer/SqliteInstaller.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/installer/DatabaseInstaller.php
@@ -80,7 +80,9 @@
8181 * If the DB type has no settings beyond those already configured with
8282 * getConnectForm(), this should return false.
8383 */
84 - public abstract function getSettingsForm();
 84+ public function getSettingsForm() {
 85+ return false;
 86+ }
8587
8688 /**
8789 * Set variables based on the request array, assuming it was submitted via
@@ -88,7 +90,9 @@
8991 *
9092 * @return Status
9193 */
92 - public abstract function submitSettingsForm();
 94+ public function submitSettingsForm() {
 95+ return Status::newGood();
 96+ }
9397
9498 /**
9599 * Connect to the database using the administrative user/password currently
Index: trunk/phase3/includes/installer/OracleInstaller.php
@@ -88,15 +88,6 @@
8989 return $status;
9090 }
9191
92 -
93 - public function getSettingsForm() {
94 - // TODO
95 - }
96 -
97 - public function submitSettingsForm() {
98 - // TODO
99 - }
100 -
10192 public function getConnection() {
10293 // TODO
10394 }
Index: trunk/phase3/includes/installer/PostgresInstaller.php
@@ -112,14 +112,6 @@
113113 return $status;
114114 }
115115
116 - function getSettingsForm() {
117 - return false;
118 - }
119 -
120 - function submitSettingsForm() {
121 - return Status::newGood();
122 - }
123 -
124116 function setupDatabase() {
125117 }
126118
Index: trunk/phase3/includes/installer/SqliteInstaller.php
@@ -122,14 +122,6 @@
123123 return parent::needsUpgrade();
124124 }
125125
126 - public function getSettingsForm() {
127 - return false;
128 - }
129 -
130 - public function submitSettingsForm() {
131 - return Status::newGood();
132 - }
133 -
134126 public function setupDatabase() {
135127 $dir = $this->getVar( 'wgSQLiteDataDir' );
136128

Status & tagging log