[MediaWiki-commits] [Gerrit] Fix ForeignDBViaLBRepo favicon - change (mediawiki/core)

2014-02-25 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Fix ForeignDBViaLBRepo favicon
..


Fix ForeignDBViaLBRepo favicon

Due to the weird inheritance structure of the FileRepo tree,
ForeignDBViaLBRepo inherited the wrong getInfo behavior and
returned the favicon of the local wiki.

Change-Id: I4892247c0e4a2e05c9148efe089f0ff52459bbd0
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/253
---
M includes/filerepo/ForeignDBViaLBRepo.php
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Gilles: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/filerepo/ForeignDBViaLBRepo.php 
b/includes/filerepo/ForeignDBViaLBRepo.php
index 18f8d5d..8153ffb 100644
--- a/includes/filerepo/ForeignDBViaLBRepo.php
+++ b/includes/filerepo/ForeignDBViaLBRepo.php
@@ -90,4 +90,8 @@
protected function assertWritableRepo() {
throw new MWException( get_class( $this ) . ': write operations 
are not supported.' );
}
+
+   public function getInfo() {
+   return FileRepo::getInfo();
+   }
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4892247c0e4a2e05c9148efe089f0ff52459bbd0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Gergő Tisza gti...@wikimedia.org
Gerrit-Reviewer: Aarcos aarcos.w...@gmail.com
Gerrit-Reviewer: Gilles gdu...@wikimedia.org
Gerrit-Reviewer: jenkins-bot 

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


[MediaWiki-commits] [Gerrit] Fix ForeignDBViaLBRepo favicon - change (mediawiki/core)

2014-02-24 Thread Code Review
Gergő Tisza has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/115314

Change subject: Fix ForeignDBViaLBRepo favicon
..

Fix ForeignDBViaLBRepo favicon

Due to the weird inheritance structure of the FileRepo tree,
ForeignDBViaLBRepo inherited the wrong getInfo behavior and
returned the favicon of the local wiki.

Change-Id: I4892247c0e4a2e05c9148efe089f0ff52459bbd0
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/253
---
M includes/filerepo/ForeignDBViaLBRepo.php
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/14/115314/1

diff --git a/includes/filerepo/ForeignDBViaLBRepo.php 
b/includes/filerepo/ForeignDBViaLBRepo.php
index 18f8d5d..8153ffb 100644
--- a/includes/filerepo/ForeignDBViaLBRepo.php
+++ b/includes/filerepo/ForeignDBViaLBRepo.php
@@ -90,4 +90,8 @@
protected function assertWritableRepo() {
throw new MWException( get_class( $this ) . ': write operations 
are not supported.' );
}
+
+   public function getInfo() {
+   return FileRepo::getInfo();
+   }
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4892247c0e4a2e05c9148efe089f0ff52459bbd0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Gergő Tisza gti...@wikimedia.org

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