MaxSem has uploaded a new change for review.

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

Change subject: Fix a couple warnings in beta
......................................................................

Fix a couple warnings in beta

* Notice: Undefined index: wpSave in 
/srv/common-local/wmf-config/CommonSettings.php on line 950
* Warning: Creating default object from empty value in 
/srv/common-local/wmf-config/filebackend-labs.php on line 13

Change-Id: I00bfea1530ef01009474161a7e1da5053278f149
---
M wmf-config/CommonSettings.php
M wmf-config/filebackend-labs.php
2 files changed, 5 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config 
refs/changes/52/148552/1

diff --git a/wmf-config/CommonSettings.php b/wmf-config/CommonSettings.php
index 31efc58..6bac858 100644
--- a/wmf-config/CommonSettings.php
+++ b/wmf-config/CommonSettings.php
@@ -947,7 +947,8 @@
                        gmdate( 'r' ) . "\t" .
                        "$uri\t" .
                        "{$_SERVER['HTTP_X_FORWARDED_FOR']}, 
{$_SERVER['REMOTE_ADDR']}\t" .
-                       ( $_REQUEST['wpSave'] ? 'save' : '' ) . "\n",
+                       ( ( isset( $_REQUEST['wpSave'] ) && $_REQUEST['wpSave'] 
) ? 'save' : '' )
+                               . "\n",
                        "udp://$wmfUdp2logDest/xff"
                );
                if ( $wgRequest->getIP() === '127.0.0.1' ) {
diff --git a/wmf-config/filebackend-labs.php b/wmf-config/filebackend-labs.php
index 614b1ac..9d08274 100644
--- a/wmf-config/filebackend-labs.php
+++ b/wmf-config/filebackend-labs.php
@@ -10,8 +10,9 @@
 # Whenever the timeline extension is provided an empty string as a file
 # backend, it will gracefully fallback to a FSFileBackend pointing to
 # "{$wgUploadDirectory}/timeline"
-$wgTimelineSettings->fileBackend = '';
-
+if ( isset( $wgTimelineSettings ) ) {
+       $wgTimelineSettings->fileBackend = '';
+}
 
 // labs does not have swift yet as of 20120525 -- hashar
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I00bfea1530ef01009474161a7e1da5053278f149
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to