Lokal Profil has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370770 )
Change subject: Make api handle sparql harvested source fields
..
Make api handle sparql harvested source fields
Uppdated url handlers for:
* Wikitable
* HTML
* HTMLlist
Bug: T172841
Change-Id: I20e25de8c391ba9e06cc6a4374d094af401bdf44
---
M api/includes/CommonFunctions.php
M api/includes/FormatHtml.php
M api/includes/FormatHtmllist.php
M api/includes/FormatWikitable.php
M api/tests/CommonFunctionsTest.php
5 files changed, 122 insertions(+), 20 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/heritage
refs/changes/70/370770/1
diff --git a/api/includes/CommonFunctions.php b/api/includes/CommonFunctions.php
index 485b5c8..d8df10e 100644
--- a/api/includes/CommonFunctions.php
+++ b/api/includes/CommonFunctions.php
@@ -53,7 +53,8 @@
$projects = array("wikipedia", "wikivoyage", "wikidata", "wikimedia");
$projectRegex = "(" . join("|", $projects) . ")";
$var = NULL;
- if (!preg_match( '/(https?:)?\/\/(([a-z\-]+)\.' . $projectRegex .
'\.org\/w\/index\.php\?title=(.*)=(.*))/', $text, $var )) {
+ $pattern = '/(https?:)?\/\/(([a-z\-]+)\.' . $projectRegex .
'\.org\/w\/index\.php\?title=(.*)=(.*))/';
+ if ( !preg_match( $pattern, $text, $var )) {
throw new Exception('No project link in text.');
}
return $var;
@@ -63,6 +64,15 @@
return str_replace(' ', '_', $in_string);
}
+function matchWikidataQid($url) {
+ /* Extract the Qid from a Wikidata page or entity link */
+ $var = NULL;
+ $pattern = '/(https?:)?\/\/(www\.wikidata\.org\/(wiki|entity)\/(.*))/';
+ if ( !preg_match( $pattern, $text, $var )) {
+ throw new Exception('The provided url was not a wikidata
link.');
+ }
+ return $var;
+
function makeWikidataUrl($qid) {
/* Creates a URL to Wikidata from a Qid */
if ( $qid ) {
diff --git a/api/includes/FormatHtml.php b/api/includes/FormatHtml.php
index 7042dff..61a0fb4 100644
--- a/api/includes/FormatHtml.php
+++ b/api/includes/FormatHtml.php
@@ -4,7 +4,7 @@
* HTML output type, based on XML. This output is for users (and not automated
tools) so internationalization will be used.
* @author Joancreus (jcreus), based on Platonides work
*/
-// functions: processWikitext, matchWikiprojectLink, getImageFromCommons,
makeWikidataUrl
+// functions: processWikitext, matchWikiprojectLink, getImageFromCommons,
makeWikidataUrl, matchWikidataQid
require_once ( 'CommonFunctions.php' );
class FormatHtml extends FormatBase {
@@ -29,13 +29,13 @@
function outputBegin( $selectedItems ) {
echo '';
-$this->linebreak();
+ $this->linebreak();
echo '';
-$this->linebreak();
+ $this->linebreak();
echo '';
-$this->linebreak();
+ $this->linebreak();
echo '';
-$this->linebreak();
+ $this->linebreak();
echo '';
echo "\n\n\n";
@@ -64,7 +64,7 @@
if ( in_array( $name, $selectedItems ) ) {
// $label = $name.'http://commons.wikimedia.org/skins-1.17/common/images/sort_none.gif;
alt="↑">';
echo '' . _i18n(
'db-field-' . $name ) . '';
-$this->linebreak();
+ $this->linebreak();
}
}
echo '';
@@ -109,11 +109,11 @@
}
echo '' . $cellData . '';
-$this->linebreak();
+ $this->linebreak();
}
}
echo '';
-$this->linebreak();
+ $this->linebreak();
}
function outputEnd() {
@@ -134,8 +134,14 @@
$linkText = urldecode( $encodedLinkText );
return self::makeHTMLlink( 'https://' . $m[2],
$linkText );
} catch ( Exception $e ) {
- // Normal text
- return htmlspecialchars( $text );
+ // Possibly a wikidata entity/wiki link
+ try {
+ $m = matchWikidataQid( $text );
+ return self::makeHTMLlink( 'https://' . $m[2],
$m[4] );
+ } catch ( Exception $e ) {
+ // Normal text
+ return htmlspecialchars( $text );
+ }
}
}
diff --git a/api/includes/FormatHtmllist.php b/api/includes/FormatHtmllist.php
index ace5c27..8bbfc28 100644
--- a/api/includes/FormatHtmllist.php
+++ b/api/includes/FormatHtmllist.php
@@ -5,7 +5,7 @@
MaxSem has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370769 )
Change subject: test.php: add doctype and charset
..
test.php: add doctype and charset
Change-Id: I2863d7fee812c1303fa3f54c337fd220908cf9bb
---
M DiffTest/test.php
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/php/wikidiff2
refs/changes/69/370769/1
diff --git a/DiffTest/test.php b/DiffTest/test.php
index a405811..dc4736e 100644
--- a/DiffTest/test.php
+++ b/DiffTest/test.php
@@ -19,8 +19,10 @@
} );
echo <<
Diff changes
+
MaxSem has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370768 )
Change subject: Document build requirements
..
Document build requirements
Change-Id: If4cfba8abc8a37c1cb0d65f148c4500851dbd620
---
M README
1 file changed, 9 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/php/wikidiff2
refs/changes/68/370768/1
diff --git a/README b/README
index 67067fe..bf72bdd 100644
--- a/README
+++ b/README
@@ -15,12 +15,21 @@
Wikidiff2 is a PHP extension.
+== Dependencies ==
It requires the following library:
* libthai, a Thai language support library
http://linux.thai.net/plone/TLWG/libthai/
On Debian-based systems, you need libthai0 and libthai-dev packages
+* To build wikidiff2 as a HHVM extension on Debian systems, you need the
following packages:
+hhvm-dev libtbb-dev libtbb2 libboost-all-dev libdouble-conversion-dev \
+libdouble-conversion1 libgoogle-glog-dev libgoogle-glog-doc
libgoogle-glog0 \
+libjemalloc-dev libjemalloc1 libjemalloc1-dbg
+
+* To build wikidiff2 as a PHP extension, you need the following packages:
+php5-dev pkg-config
+
== Compilation and installation with Zend PHP ==
$ phpize
--
To view, visit https://gerrit.wikimedia.org/r/370768
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If4cfba8abc8a37c1cb0d65f148c4500851dbd620
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/php/wikidiff2
Gerrit-Branch: master
Gerrit-Owner: MaxSem
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
MarkAHershberger has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370767 )
Change subject: Emacs settings
..
Emacs settings
Change-Id: I6f00a10eede26256974008afba9344d3fbb29892
---
A .dir-locals.el
M .gitignore
2 files changed, 102 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CategoryWatch
refs/changes/67/370767/1
diff --git a/.dir-locals.el b/.dir-locals.el
new file mode 100644
index 000..75ce2ef
--- /dev/null
+++ b/.dir-locals.el
@@ -0,0 +1,89 @@
+((nil . ((mode . flycheck)
+(mode . company)
+(mode . edep)
+(mode . subword)
+(tab-width . 4)
+(c-basic-offset . 4)
+(indent-tabs-mode . t)
+(lice:default-license . "gpl-3.0")
+(eval . (progn (when (fboundp 'delete-trailing-whitespace)
+ (delete-trailing-whitespace))
+ (tabify (point-min)
(point-max
+(c-hanging-braces-alist
+ (defun-open after)
+ (block-open after)
+ (defun-close))
+(c-offsets-alist . (
+(access-label . -)
+(annotation-top-cont .
0)
+(annotation-var-cont .
+)
+(arglist-close .
php-lineup-arglist-close)
+(arglist-cont-nonempty
first
+
php-lineup-cascaded-calls
+
c-lineup-arglist)
+(arglist-intro .
php-lineup-arglist-intro)
+(block-close . 0)
+(block-open . 0)
+(brace-entry-open . 0)
+(brace-list-close . 0)
+(brace-list-entry . 0)
+(brace-list-intro . +)
+(brace-list-open . 0)
+(c .
c-lineup-C-comments)
+(case-label . 0)
+(catch-clause . 0)
+(class-close . 0)
+(comment-intro . 0)
+(composition-close . 0)
+(composition-open . 0)
+(cpp-define-intro
c-lineup-cpp-define +)
+(cpp-macro . [0])
+(cpp-macro-cont . +)
+(defun-block-intro . +)
+(defun-close . 0)
+(defun-open . 0)
+(do-while-closure . 0)
+(else-clause . 0)
+(extern-lang-close . 0)
+(extern-lang-open . 0)
+(friend . 0)
+(func-decl-cont . +)
+(inclass . +)
+(incomposition . +)
+(inexpr-class . +)
+(inexpr-statement . +)
+(inextern-lang . +)
+(inher-cont .
c-lineup-multi-inher)
+(inher-intro . +)
+(inlambda . 0)
+(inline-close . 0)
+(inline-open . 0)
+(inmodule . +)
+
MarkAHershberger has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370766 )
Change subject: i18n the modern, js way
..
i18n the modern, js way
Change-Id: I1fcf62124fd828a13b932571272967593b4d838a
---
D CategoryWatch.i18n.php
M extension.json
A i18n/ar.json
A i18n/arz.json
A i18n/ba.json
A i18n/be-tarask.json
A i18n/bg.json
A i18n/br.json
A i18n/bs.json
A i18n/ca.json
A i18n/cs.json
A i18n/cy.json
A i18n/da.json
A i18n/de.json
A i18n/dsb.json
A i18n/el.json
A i18n/en.json
A i18n/eo.json
A i18n/es.json
A i18n/eu.json
A i18n/fi.json
A i18n/fr.json
A i18n/frp.json
A i18n/gl.json
A i18n/gsw.json
A i18n/gu.json
A i18n/he.json
A i18n/hsb.json
A i18n/hu.json
A i18n/ia.json
A i18n/id.json
A i18n/it.json
A i18n/ja.json
A i18n/km.json
A i18n/ko.json
A i18n/ksh.json
A i18n/lb.json
A i18n/li.json
A i18n/lt.json
A i18n/mk.json
A i18n/ms.json
A i18n/nl.json
A i18n/nn.json
A i18n/no.json
A i18n/oc.json
A i18n/pdc.json
A i18n/pl.json
A i18n/pms.json
A i18n/pt-br.json
A i18n/pt.json
A i18n/qqq.json
A i18n/ro.json
A i18n/roa-tara.json
A i18n/ru.json
A i18n/sah.json
A i18n/sk.json
A i18n/sr-ec.json
A i18n/sr-el.json
A i18n/stq.json
A i18n/sv.json
A i18n/te.json
A i18n/tl.json
A i18n/tr.json
A i18n/uk.json
A i18n/vi.json
A i18n/vo.json
A i18n/yi.json
A i18n/zh-hans.json
A i18n/zh-hant.json
69 files changed, 949 insertions(+), 901 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CategoryWatch
refs/changes/66/370766/1
diff --git a/CategoryWatch.i18n.php b/CategoryWatch.i18n.php
deleted file mode 100644
index a70e3f2..000
--- a/CategoryWatch.i18n.php
+++ /dev/null
@@ -1,899 +0,0 @@
- 'Extends watchlist functionality to include
notification about membership changes of watched categories',
- 'categorywatch-emailsubject' => "Activity involving watched category
\"$1\"",
- 'categorywatch-catmovein' => "$1 has moved into $2 from $3",
- 'categorywatch-catmoveout' => "$1 has moved out of $2 into $3",
- 'categorywatch-catadd' => "$1 has been added to $2",
- 'categorywatch-catsub' => "$1 has been removed from $2",
- 'categorywatch-autocat' => "Automatically watched by $1",
-);
-
-/** Message documentation (Message documentation)
- * @author Fryed-peach
- * @author Purodha
- * @author Siebrand
- * @author The Evil IP address
- */
-$messages['qqq'] = array(
- 'categorywatch-desc' => '{{desc}}',
- 'categorywatch-catmovein' => 'Substituted as $5 in
{{msg-mw|categorywatch-emailbody}}.
-* $1 is a page name
-* $2 is the target category name
-* $3 is the source category name',
- 'categorywatch-catmoveout' => 'Substituted as $5 in
{{msg-mw|categorywatch-emailbody}}.
-* $1 is a page name
-* $2 is the source category name
-* $3 is the target category name',
- 'categorywatch-catadd' => 'Substituted as $5 in
{{msg-mw|categorywatch-emailbody}}.
-* $1 is a page name
-* $2 is a category name',
- 'categorywatch-catsub' => 'Substituted as $5 in
{{msg-mw|categorywatch-emailbody}}.
-* $1 is a page name
-* $2 is a category name',
-);
-
-/** Arabic (العربية)
- * @author Meno25
- * @author OsamaK
- */
-$messages['ar'] = array(
- 'categorywatch-desc' => 'يمدد وظيفة قائمة المراقبة لتشمل الإخطارات حول
تغييرات العضوية للتصنيفات المراقبة',
- 'categorywatch-emailsubject' => 'النشاط الذي يشمل التصنيف المراقب "$1"',
- 'categorywatch-catmovein' => 'نقل $1 إلى التصنيف $2 من $3',
- 'categorywatch-catmoveout' => 'نقل $1 من التصنيف $2 إلى $3',
- 'categorywatch-catadd' => 'أضاف $1 إلى التصنيف $2',
- 'categorywatch-catsub' => 'أزال $1 من التصنيف $2',
- 'categorywatch-autocat' => 'مراقبة تلقائيا بواسطة $1',
-);
-
-/** Egyptian Spoken Arabic (مصرى)
- * @author Ghaly
- * @author Meno25
- */
-$messages['arz'] = array(
- 'categorywatch-desc' => 'يمدد وظيفة قائمة المراقبة لتشمل الإخطارات حول
تغييرات العضوية للتصنيفات المراقبة',
- 'categorywatch-emailsubject' => 'النشاط الذى يشمل التصنيف المراقب "$1"',
- 'categorywatch-catmovein' => 'نقل $1 إلى التصنيف $2 من $3',
- 'categorywatch-catmoveout' => 'نقل $1 من التصنيف $2 إلى $3',
- 'categorywatch-catadd' => 'أضاف $1 إلى التصنيف $2',
- 'categorywatch-catsub' => 'ازال $1 من التصنيف $2',
-);
-
-/** Bashkir (Башҡортса)
- * @author Assele
- */
-$messages['ba'] = array(
- 'categorywatch-desc' => 'Күҙәтелгән категорияларға кергән биттәр
исемлегенең үҙгәреүе тураһында белгертеү мөмкинлеге менән күҙәтеү исемлеген
киңәйтә',
- 'categorywatch-emailsubject' => '"$1" категорияһына ҡағылған
үҙгәртеүҙәр',
- 'categorywatch-catmovein' => '$1 битен $3 категорияһынан $2
категорияһына күсергән',
- 'categorywatch-catmoveout' => '$1 битен $3 категорияһынан $2
категорияһына күсергән',
- 'categorywatch-catadd' => '$2 категорияһына $1 битен өҫтәгән',
- 'categorywatch-catsub' => '$2 категорияһынан $1 битен юйған',
- 'categorywatch-autocat'
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370763 )
Change subject: Add stats parameter for search query, for ES tracking
..
Add stats parameter for search query, for ES tracking
Bug: T172422
Change-Id: I8964981c1e576a3b8732fbb6eb67218db8ce40dc
---
M repo/includes/Search/Elastic/WikibasePrefixSearcher.php
M repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
M repo/tests/phpunit/data/entitySearch/search_de-ch.expected
M repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
M repo/tests/phpunit/data/entitySearch/search_en.expected
M repo/tests/phpunit/data/entitySearch/search_en_strict.expected
M repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
M repo/tests/phpunit/data/entitySearch/search_zh.expected
8 files changed, 23 insertions(+), 0 deletions(-)
Approvals:
EBernhardson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
b/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
index 71331bd..879a0af 100644
--- a/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
+++ b/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
@@ -61,6 +61,8 @@
$searchQuery->setSize( $this->limit );
}
$searchQuery->setParam( 'rescore',
$this->searchContext->getRescore() );
+ // Mark wikibase prefix searches for statistics
+ $searchQuery->addParam( 'stats', 'wikibase-prefix' );
if ( $this->returnExplain ) {
$searchQuery->setExplain( true );
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
index 297534c..17473e8 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
@@ -143,6 +143,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
index 7c22e89..a34d3b8 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
@@ -147,6 +147,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
index ee5d863..c35cd63 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
@@ -140,6 +140,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_en.expected
b/repo/tests/phpunit/data/entitySearch/search_en.expected
index 5948bbe..75f7a64 100644
--- a/repo/tests/phpunit/data/entitySearch/search_en.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_en.expected
@@ -121,6 +121,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
b/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
index 42f0864..9b0076a 100644
--- a/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
@@ -120,6 +120,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
b/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
index c98d337..bfc0ae0 100644
--- a/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
@@ -224,6 +224,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_zh.expected
b/repo/tests/phpunit/data/entitySearch/search_zh.expected
index 092680e..56a0756 100644
--- a/repo/tests/phpunit/data/entitySearch/search_zh.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_zh.expected
@@ -238,6 +238,9 @@
}
}
}
+],
+"stats": [
+
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/313625 )
Change subject: jobqueue: Add job_type to PSR logging context
..
jobqueue: Add job_type to PSR logging context
The mediawiki.runJobs errors are collected in Logstash but the whole job
description and errors are a single field message. That is challenging
to split logs per job type, get the longest running jobs ...
That can be worked around on the log receiving side by parsing
MediaWiki messages eg https://gerrit.wikimedia.org/r/#/c/312504/
Bryan Davis suggested a better long term solution is to use the PSR3
logger with structured log messages.
Culprit: 'type' is a reverved word. Hence prefix all context variables
with 'job_'.
Bug: T146469
Change-Id: Ib6a771c7d3f83bd75b2994bfab9bbebfd1f5aa6c
---
M includes/jobqueue/JobRunner.php
1 file changed, 29 insertions(+), 5 deletions(-)
Approvals:
Aaron Schulz: Looks good to me, approved
Krinkle: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/includes/jobqueue/JobRunner.php b/includes/jobqueue/JobRunner.php
index 49b7a45..db881d5e 100644
--- a/includes/jobqueue/JobRunner.php
+++ b/includes/jobqueue/JobRunner.php
@@ -281,7 +281,9 @@
private function executeJob( Job $job, LBFactory $lbFactory, $stats,
$popTime ) {
$jType = $job->getType();
$msg = $job->toString() . " STARTING";
- $this->logger->debug( $msg );
+ $this->logger->debug( $msg, [
+ 'job_type' => $job->getType(),
+ ] );
$this->debugCallback( $msg );
// Run the job...
@@ -339,12 +341,23 @@
}
if ( $status === false ) {
+ $msg = $job->toString() . " t={job_duration}
error={job_error}";
+ $this->logger->error( $msg, [
+ 'job_type' => $job->getType(),
+ 'job_duration' => $timeMs,
+ 'job_error' => $error,
+ ] );
+
$msg = $job->toString() . " t=$timeMs error={$error}";
- $this->logger->error( $msg );
$this->debugCallback( $msg );
} else {
+ $msg = $job->toString() . " t={job_duration} good";
+ $this->logger->info( $msg, [
+ 'job_type' => $job->getType(),
+ 'job_duration' => $timeMs,
+ ] );
+
$msg = $job->toString() . " t=$timeMs good";
- $this->logger->info( $msg );
$this->debugCallback( $msg );
}
@@ -488,9 +501,14 @@
}
$usedBytes = memory_get_usage();
if ( $maxBytes && $usedBytes >= 0.95 * $maxBytes ) {
+ $msg = "Detected excessive memory usage
({used_bytes}/{max_bytes}).";
+ $this->logger->error( $msg, [
+ 'used_bytes' => $usedBytes,
+ 'max_bytes' => $maxBytes,
+ ] );
+
$msg = "Detected excessive memory usage
($usedBytes/$maxBytes).";
$this->debugCallback( $msg );
- $this->logger->error( $msg );
return false;
}
@@ -552,8 +570,14 @@
}
$ms = intval( 1000 * $time );
+
+ $msg = $job->toString() . " COMMIT ENQUEUED
[{job_commit_write_ms}ms of writes]";
+ $this->logger->info( $msg, [
+ 'job_type' => $job->getType(),
+ 'job_commit_write_ms' => $ms,
+ ] );
+
$msg = $job->toString() . " COMMIT ENQUEUED [{$ms}ms of
writes]";
- $this->logger->info( $msg );
$this->debugCallback( $msg );
// Wait for an exclusive lock to commit
--
To view, visit https://gerrit.wikimedia.org/r/313625
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib6a771c7d3f83bd75b2994bfab9bbebfd1f5aa6c
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Hashar
Gerrit-Reviewer: Aaron Schulz
Gerrit-Reviewer: BryanDavis
Gerrit-Reviewer: Hashar
Gerrit-Reviewer: Krinkle
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Smalyshev has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370765 )
Change subject: Merging from b1f6f8748530b082e748baf51df1909ae6f1d7a5:
..
Merging from b1f6f8748530b082e748baf51df1909ae6f1d7a5:
Localisation updates from https://translatewiki.net.
Change-Id: I62674740a9b75c35b5294bb7f959f1f79113f16d
---
M i18n/arab.json
M i18n/de.json
M i18n/ec.json
M i18n/es.json
M i18n/fr.json
M i18n/hans.json
M i18n/hant.json
M i18n/hi.json
M i18n/kab.json
M i18n/ko.json
M i18n/lb.json
M i18n/mk.json
M i18n/ne.json
M i18n/sl.json
M i18n/ta.json
M i18n/tarask.json
M index.html
R js/wdqs.min.1fa8d96c3c8cc56cebc4.js
18 files changed, 18 insertions(+), 18 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/wikidata/query/gui-deploy
refs/changes/65/370765/1
--
To view, visit https://gerrit.wikimedia.org/r/370765
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I62674740a9b75c35b5294bb7f959f1f79113f16d
Gerrit-PatchSet: 1
Gerrit-Project: wikidata/query/gui-deploy
Gerrit-Branch: production
Gerrit-Owner: Smalyshev
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Smalyshev has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370765 )
Change subject: Merging from b1f6f8748530b082e748baf51df1909ae6f1d7a5:
..
Merging from b1f6f8748530b082e748baf51df1909ae6f1d7a5:
Localisation updates from https://translatewiki.net.
Change-Id: I62674740a9b75c35b5294bb7f959f1f79113f16d
---
M i18n/arab.json
M i18n/de.json
M i18n/ec.json
M i18n/es.json
M i18n/fr.json
M i18n/hans.json
M i18n/hant.json
M i18n/hi.json
M i18n/kab.json
M i18n/ko.json
M i18n/lb.json
M i18n/mk.json
M i18n/ne.json
M i18n/sl.json
M i18n/ta.json
M i18n/tarask.json
M index.html
R js/wdqs.min.1fa8d96c3c8cc56cebc4.js
18 files changed, 18 insertions(+), 18 deletions(-)
Approvals:
Smalyshev: Verified; Looks good to me, approved
--
To view, visit https://gerrit.wikimedia.org/r/370765
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I62674740a9b75c35b5294bb7f959f1f79113f16d
Gerrit-PatchSet: 1
Gerrit-Project: wikidata/query/gui-deploy
Gerrit-Branch: production
Gerrit-Owner: Smalyshev
Gerrit-Reviewer: Siebrand
Gerrit-Reviewer: Smalyshev
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Hoo man has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370764 )
Change subject: Avoid fetching terms more than once in
BufferingTermLookup::prefetchTerms
..
Avoid fetching terms more than once in BufferingTermLookup::prefetchTerms
Currently we often call out to BufferingTermLookup::prefetchTerms
via DispatchingTermBuffer::getTermsOfType with the same arguments
during client page parses (of pages with a lot of Wikibase content
on). This causes a lot of SELECTs on the wb_terms table to happen
(set a break point in TermSqlIndex::fetchTerms to see all of these).
This should reduce the amount of terms table queries in production
by a fair amount (I don't have any numbers, though).
Change-Id: I71687d7f929d0dd99e76f4d667637c1234dec336
---
M lib/includes/Store/BufferingTermLookup.php
M lib/tests/phpunit/Store/BufferingTermLookupTest.php
2 files changed, 80 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/64/370764/1
diff --git a/lib/includes/Store/BufferingTermLookup.php
b/lib/includes/Store/BufferingTermLookup.php
index ba16986..86ced04 100644
--- a/lib/includes/Store/BufferingTermLookup.php
+++ b/lib/includes/Store/BufferingTermLookup.php
@@ -114,12 +114,22 @@
return;
}
- // We could first check what's already in the buffer, but it's
hard to determine which
- // entities are actually "fully covered" by the cached terms.
Also, our current use case
- // (the ChangesListInitRows hook) would generally, trigger only
one call to prefetchTerms,
- // before any call to getTermsOfType().
+ // Try to detect whether the entities in question have already
been prefetched, in case we
+ // know the needed term types and language codes.
+ // If they are not/ only partially prefetched or we don't know
whether our prefetched data on
+ // them is complete, we just resort to fetching them (again).
+ $entityIdsToFetch = [];
+ if ( $termTypes !== null && $languageCodes !== null ) {
+ $entityIdsToFetch =
$this->getIncompletelyPrefetchedEntityIds( $entityIds, $termTypes,
$languageCodes );
+ } else {
+ $entityIdsToFetch = $entityIds;
+ }
- $entityIdsByType = $this->groupEntityIds( $entityIds );
+ if ( empty( $entityIdsToFetch ) ) {
+ return;
+ }
+
+ $entityIdsByType = $this->groupEntityIds( $entityIdsToFetch );
$terms = [];
foreach ( $entityIdsByType as $entityIdGroup ) {
@@ -131,11 +141,53 @@
$bufferedKeys = $this->setBufferedTermObjects( $terms );
if ( !empty( $languageCodes ) ) {
- $this->setUndefinedTerms( $entityIds, $termTypes,
$languageCodes, $bufferedKeys );
+ $this->setUndefinedTerms( $entityIdsToFetch,
$termTypes, $languageCodes, $bufferedKeys );
}
}
/**
+* Get a list of EntityIds for which we don't have all the needed data
prefetched for.
+*
+* @param EntityId[] $entityIds
+* @param string[] $termTypes
+* @param string[] $languageCodes
+*
+* @return EntityId[]
+*/
+ private function getIncompletelyPrefetchedEntityIds( array $entityIds,
array $termTypes, array $languageCodes ) {
+ $entityIdsToFetch = [];
+
+ foreach ( $entityIds as $entityId ) {
+ if ( $this->isIncompletelyPrefetched( $entityId,
$termTypes, $languageCodes ) ) {
+ $entityIdsToFetch[] = $entityId;
+ }
+ }
+
+ return $entityIdsToFetch;
+ }
+
+ /**
+* Has the term type and language code combination from the given
entity already been prefeteched?
+*
+* @param EntityId $entityId
+* @param string[] $termTypes
+* @param string[] $languageCodes
+*
+* @return bool
+*/
+ private function isIncompletelyPrefetched( EntityId $entityId, array
$termTypes, array $languageCodes ) {
+ foreach ( $termTypes as $termType ) {
+ foreach ( $languageCodes as $lang ) {
+ if ( $this->getPrefetchedTerm( $entityId,
$termType, $lang ) === null ) {
+ return true;
+ }
+ }
+ }
+
+ return false;
+ }
+
+ /**
* Returns a term that was previously loaded by prefetchTerms.
*
* @param EntityId $entityId
diff --git
Smalyshev has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370763 )
Change subject: Add stats parameter for search query, for ES tracking
..
Add stats parameter for search query, for ES tracking
Bug: T172422
Change-Id: I8964981c1e576a3b8732fbb6eb67218db8ce40dc
---
M repo/includes/Search/Elastic/WikibasePrefixSearcher.php
M repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
M repo/tests/phpunit/data/entitySearch/search_de-ch.expected
M repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
M repo/tests/phpunit/data/entitySearch/search_en.expected
M repo/tests/phpunit/data/entitySearch/search_en_strict.expected
M repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
M repo/tests/phpunit/data/entitySearch/search_zh.expected
8 files changed, 23 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase
refs/changes/63/370763/1
diff --git a/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
b/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
index 71331bd..879a0af 100644
--- a/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
+++ b/repo/includes/Search/Elastic/WikibasePrefixSearcher.php
@@ -61,6 +61,8 @@
$searchQuery->setSize( $this->limit );
}
$searchQuery->setParam( 'rescore',
$this->searchContext->getRescore() );
+ // Mark wikibase prefix searches for statistics
+ $searchQuery->addParam( 'stats', 'wikibase-prefix' );
if ( $this->returnExplain ) {
$searchQuery->setExplain( true );
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
index 297534c..17473e8 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch-en.expected
@@ -143,6 +143,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
index 7c22e89..a34d3b8 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch.expected
@@ -147,6 +147,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
b/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
index ee5d863..c35cd63 100644
--- a/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_de-ch_strict.expected
@@ -140,6 +140,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_en.expected
b/repo/tests/phpunit/data/entitySearch/search_en.expected
index 5948bbe..75f7a64 100644
--- a/repo/tests/phpunit/data/entitySearch/search_en.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_en.expected
@@ -121,6 +121,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
b/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
index 42f0864..9b0076a 100644
--- a/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_en_strict.expected
@@ -120,6 +120,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
b/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
index c98d337..bfc0ae0 100644
--- a/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_zh-de-ch.expected
@@ -224,6 +224,9 @@
}
}
}
+],
+"stats": [
+"wikibase-prefix"
]
},
"options": {
diff --git a/repo/tests/phpunit/data/entitySearch/search_zh.expected
b/repo/tests/phpunit/data/entitySearch/search_zh.expected
index 092680e..56a0756 100644
--- a/repo/tests/phpunit/data/entitySearch/search_zh.expected
+++ b/repo/tests/phpunit/data/entitySearch/search_zh.expected
@@ -238,6 +238,9 @@
}
}
}
+],
+"stats": [
+
Paladox has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370762 )
Change subject: Insert the description of the change.
..
Insert the description of the change.
Change-Id: I066ac7b1f08e142685aa1489c5af94301294b1a4
---
0 files changed, 0 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/62/370762/1
--
To view, visit https://gerrit.wikimedia.org/r/370762
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I066ac7b1f08e142685aa1489c5af94301294b1a4
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370427 )
Change subject: Update scroll type detection
..
Update scroll type detection
Port of a fix in the original source:
https://github.com/othree/jquery.rtl-scroll-type/commit/1c6d6845032c3cd91b2b387634642471d711380d
Change-Id: I2d53e467b79d586efc079a15ebc0a52382352f9e
---
M modules/ext.RevisionSlider.util.js
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
WMDE-leszek: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ext.RevisionSlider.util.js
b/modules/ext.RevisionSlider.util.js
index 6d9450d..66e51d6 100644
--- a/modules/ext.RevisionSlider.util.js
+++ b/modules/ext.RevisionSlider.util.js
@@ -95,13 +95,13 @@
$dummy = $( '' )
.css( {
dir: 'rtl',
- width: '1px',
+ width: '4px',
height: '1px',
position: 'absolute',
top: '-1000px',
overflow: 'scroll'
} )
- .text( 'A' )
+ .text( 'ABCD' )
.appendTo( 'body' )[ 0 ];
if ( $dummy.scrollLeft > 0 ) {
return 'default';
--
To view, visit https://gerrit.wikimedia.org/r/370427
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2d53e467b79d586efc079a15ebc0a52382352f9e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/RevisionSlider
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch
Gerrit-Reviewer: Addshore
Gerrit-Reviewer: Andrew-WMDE
Gerrit-Reviewer: Tobias Gritschacher
Gerrit-Reviewer: WMDE-leszek
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Nuria has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370761 )
Change subject: Adding License to analytics.wikimedia.org
..
Adding License to analytics.wikimedia.org
License should be CC0
Fixing broken logo
Bug: T172814
Change-Id: I182145018adff15ea2965ce8a7213f87f7949e4e
---
M index.html
1 file changed, 22 insertions(+), 12 deletions(-)
Approvals:
Nuria: Verified; Looks good to me, approved
diff --git a/index.html b/index.html
index 5422d1e..6b5cb1e 100644
--- a/index.html
+++ b/index.html
@@ -28,12 +28,17 @@
-
-https://wikimediafoundation.org;>
-
+
+
+
+
+ https://wikimediafoundation.org;>
+https://upload.wikimedia.org/wikipedia/commons/thumb/8/8b/Wikimedia-logo_black.svg/96px-Wikimedia-logo_black.svg.png
1.5x,
https://upload.wikimedia.org/wikipedia/commons/thumb/8/8b/Wikimedia-logo_black.svg/128px-Wikimedia-logo_black.svg.png
2x" data-file-width="1024" data-file-height="1024">
-
-
+
+
+
+
Dashboards
Browser
Statistics
@@ -49,10 +54,11 @@
mailto:analyt...@lists.wikimedia.org;
target="_blank">
-Collaborate
+Contact Us
-
+
+
@@ -60,15 +66,19 @@
Dashboards and Data Downloads for Wikimedia Projects
-Data compiled by community and staff, from
projects hosted by the Wikimedia Foundation.
+
+Data compiled by community and staff, from
projects hosted by the Wikimedia Foundation.
+All data, charts, and other content is available under the Creative Commons CC0
Dedication .
-
- Code for this page can be seen here:https://github.com/wikimedia/analytics-analytics.wikimedia.org;>
-
-
+
+
+ Code for this page can be seen here:https://github.com/wikimedia/analytics-analytics.wikimedia.org;>
+
+
+
--
To view, visit https://gerrit.wikimedia.org/r/370761
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I182145018adff15ea2965ce8a7213f87f7949e4e
Gerrit-PatchSet: 1
Gerrit-Project: analytics/analytics.wikimedia.org
Gerrit-Branch: master
Gerrit-Owner: Nuria
Gerrit-Reviewer: Fdans
Gerrit-Reviewer: Mforns
Gerrit-Reviewer: Nuria
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Nuria has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370761 )
Change subject: Adding License to analytics.wikimedia.org
..
Adding License to analytics.wikimedia.org
License should be CC0
Fixing broken logo
Bug: T172814
Change-Id: I182145018adff15ea2965ce8a7213f87f7949e4e
---
M index.html
1 file changed, 22 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/analytics/analytics.wikimedia.org
refs/changes/61/370761/1
diff --git a/index.html b/index.html
index 5422d1e..6b5cb1e 100644
--- a/index.html
+++ b/index.html
@@ -28,12 +28,17 @@
-
-https://wikimediafoundation.org;>
-
+
+
+
+
+ https://wikimediafoundation.org;>
+https://upload.wikimedia.org/wikipedia/commons/thumb/8/8b/Wikimedia-logo_black.svg/96px-Wikimedia-logo_black.svg.png
1.5x,
https://upload.wikimedia.org/wikipedia/commons/thumb/8/8b/Wikimedia-logo_black.svg/128px-Wikimedia-logo_black.svg.png
2x" data-file-width="1024" data-file-height="1024">
-
-
+
+
+
+
Dashboards
Browser
Statistics
@@ -49,10 +54,11 @@
mailto:analyt...@lists.wikimedia.org;
target="_blank">
-Collaborate
+Contact Us
-
+
+
@@ -60,15 +66,19 @@
Dashboards and Data Downloads for Wikimedia Projects
-Data compiled by community and staff, from
projects hosted by the Wikimedia Foundation.
+
+Data compiled by community and staff, from
projects hosted by the Wikimedia Foundation.
+All data, charts, and other content is available under the Creative Commons CC0
Dedication .
-
- Code for this page can be seen here:https://github.com/wikimedia/analytics-analytics.wikimedia.org;>
-
-
+
+
+ Code for this page can be seen here:https://github.com/wikimedia/analytics-analytics.wikimedia.org;>
+
+
+
--
To view, visit https://gerrit.wikimedia.org/r/370761
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I182145018adff15ea2965ce8a7213f87f7949e4e
Gerrit-PatchSet: 1
Gerrit-Project: analytics/analytics.wikimedia.org
Gerrit-Branch: master
Gerrit-Owner: Nuria
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/368939 )
Change subject: Allow bureaucrats on WMF wikis to grant and remove 'confirmed'
..
Allow bureaucrats on WMF wikis to grant and remove 'confirmed'
There was some consensus to make this avalaible for administrators but
there also was some concerns as well as if this was really needed for small
projects who do not care about user rights management. As such, and
given the current Meta-Wiki policy on not granting 'bureaucrat'
permissions but to well stablished communities that demonstrates a need
of one, plus other series of securities to avoid wiki takeover episodes
as we've seen in the past; I think granting this to bureaucrats will
satisfy both parties. This is without prejudice of individual wikis
willing their administrators to be able to grant and remove this
permission as well and without prejudice of leaving the current
configuration untouched as well: that is, wikis that have already
granted this to administrators or bureaucrats won't have their
configuration changed now unless it becomes redundant in which case a
proposal for cleanup would have to be filed and actioned.
Bug: T101983
Change-Id: Ia62ad0f40e8b71b7fa424b94da48feebaa5ebfa0
---
M wmf-config/InitialiseSettings.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Urbanecm: Looks good to me, but someone else must approve
Luke081515: Looks good to me, but someone else must approve
jenkins-bot: Verified
Dereckson: Looks good to me, approved
diff --git a/wmf-config/InitialiseSettings.php
b/wmf-config/InitialiseSettings.php
index 9a1425e..e1f5c01 100644
--- a/wmf-config/InitialiseSettings.php
+++ b/wmf-config/InitialiseSettings.php
@@ -9672,7 +9672,7 @@
// The '+' in front of the DB name means 'add to the default'. It saves
us duplicating
// changes to the default across all overrides --Andrew 2009-04-28
'default' => [
- 'bureaucrat' => [ 'sysop', 'bureaucrat', 'bot' ],
+ 'bureaucrat' => [ 'sysop', 'bureaucrat', 'bot', 'confirmed' ],
'sysop' => [ 'ipblock-exempt' ],
],
'+testwiki' => [
--
To view, visit https://gerrit.wikimedia.org/r/368939
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia62ad0f40e8b71b7fa424b94da48feebaa5ebfa0
Gerrit-PatchSet: 3
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: MarcoAurelio
Gerrit-Reviewer: Dereckson
Gerrit-Reviewer: Florianschmidtwelzow
Gerrit-Reviewer: Luke081515
Gerrit-Reviewer: MarcoAurelio
Gerrit-Reviewer: Urbanecm
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Umherirrender has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370760 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: I592a6c959257bb85a427ee2447febf4b0a6d0149
---
M ApiContributionTracking.php
M ContributionTracking.processor.php
M composer.json
M phpcs.xml
4 files changed, 9 insertions(+), 10 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ContributionTracking
refs/changes/60/370760/1
diff --git a/ApiContributionTracking.php b/ApiContributionTracking.php
index d4c0446..5fc44f2 100644
--- a/ApiContributionTracking.php
+++ b/ApiContributionTracking.php
@@ -37,7 +37,7 @@
/**
* Assembles the data for the API to return.
-* @param integer $id The Contribution Tracking ID.
+* @param int $id The Contribution Tracking ID.
* @param array $params Original (staged) request paramaters.
*/
function doReturn( $id, $params ) {
@@ -147,6 +147,7 @@
}
/**
+* @return array
* @deprecated since MediaWiki core 1.25
*/
public function getParamDescription() {
@@ -191,6 +192,7 @@
}
/**
+* @return array
* @deprecated since MediaWiki core 1.25
*/
public function getDescription() {
@@ -203,6 +205,7 @@
}
/**
+* @return array
* @deprecated since MediaWiki core 1.25
*/
public function getExamples() {
@@ -214,6 +217,7 @@
/**
* @see ApiBase::getExamplesMessages()
+* @return array
*/
protected function getExamplesMessages() {
return [
diff --git a/ContributionTracking.processor.php
b/ContributionTracking.processor.php
index 92050af..3f948fd 100644
--- a/ContributionTracking.processor.php
+++ b/ContributionTracking.processor.php
@@ -123,7 +123,7 @@
/**
* Effectively changes the name of a key in an array. If the key does
not
* exist, no change is made.
-* @param array $array The array to rekey (by reference)
+* @param array &$array The array to rekey
* @param string $oldkey The key to change
* @param string $newkey The new value for the key
*/
@@ -138,7 +138,7 @@
* Stages a value generated by a checkbox or similar control, for use
in our
* database. If the key exists and has not been set to exactly false,
it's
* "true".
-* @param array $array The array containing the value to stage, by
reference
+* @param array &$array The array containing the value to stage
* @param string $key The key of a checkbox-generated value
*/
static function stage_checkbox( &$array, $key ) {
@@ -213,7 +213,7 @@
* @param array $params Form / API data
* @param array $defaults A set of default values for a particular
* transaction type
-* @param boolean $nullify If true, keys with empty string values will
be
+* @param bool $nullify If true, keys with empty string values will be
* set to null in the return array.
* @return array
*/
diff --git a/composer.json b/composer.json
index bf202bf..fd295f4 100644
--- a/composer.json
+++ b/composer.json
@@ -7,7 +7,7 @@
"scripts": {
"fix": "phpcbf",
"test": [
- "parallel-lint . --exclude vendor",
+ "parallel-lint . --exclude vendor --exclude
node_modules",
"phpcs -p -s"
]
}
diff --git a/phpcs.xml b/phpcs.xml
index 5bc70ff..33ae494 100644
--- a/phpcs.xml
+++ b/phpcs.xml
@@ -2,15 +2,10 @@
-
-
-
.
- vendor
- node_modules
--
To view, visit https://gerrit.wikimedia.org/r/370760
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I592a6c959257bb85a427ee2447febf4b0a6d0149
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContributionTracking
Gerrit-Branch: master
Gerrit-Owner: Umherirrender
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Umherirrender has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370759 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: I62977ec4eb07f2b166cc5460172edfbc3612e75b
---
M CheckUser.hooks.php
M CheckUserEncryptedData.php
M api/ApiQueryCheckUser.php
M api/ApiQueryCheckUserLog.php
M phpcs.xml
M specials/SpecialCheckUser.php
6 files changed, 23 insertions(+), 23 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CheckUser
refs/changes/59/370759/1
diff --git a/CheckUser.hooks.php b/CheckUser.hooks.php
index 9b4efa3..a570d4f 100755
--- a/CheckUser.hooks.php
+++ b/CheckUser.hooks.php
@@ -204,7 +204,7 @@
* Saves user data into the cu_changes table
*
* @param User $user
-* @param boolean $autocreated
+* @param bool $autocreated
* @return true
*/
public static function onLocalUserCreated( User $user, $autocreated ) {
@@ -215,8 +215,8 @@
}
/**
-* @param $user User
-* @param $actiontext string
+* @param User $user
+* @param string $actiontext
* @return bool
*/
protected static function logUserAccountCreation( User $user,
$actiontext ) {
@@ -258,6 +258,7 @@
/**
* Hook function to prune data from the cu_changes table
+* @return true
*/
public static function maybePruneIPData() {
# Every 50th edit, prune the checkuser changes table.
@@ -406,7 +407,7 @@
/**
* Tell the parser test engine to create a stub cu_changes table,
* or temporary pages won't save correctly during the test run.
-* @param array $tables
+* @param array &$tables
* @return bool
*/
public static function checkUserParserTestTables( &$tables ) {
@@ -418,9 +419,9 @@
* Add a link to Special:CheckUser and Special:CheckUserLog
* on Special:Contributions/ for
* privileged users.
-* @param $id Integer: user ID
-* @param $nt Title: user page title
-* @param $links array: tool links
+* @param int $id User ID
+* @param Title $nt User page title
+* @param array &$links Tool links
* @return true
*/
public static function checkUserContributionsLinks( $id, $nt, &$links )
{
diff --git a/CheckUserEncryptedData.php b/CheckUserEncryptedData.php
index 04256ae..eede1d6 100644
--- a/CheckUserEncryptedData.php
+++ b/CheckUserEncryptedData.php
@@ -19,8 +19,9 @@
/**
* Create an EncryptedData object from
*
-* @param $data Mixed: data/object to be encryted
-* @param $publicKey: public key for encryption
+* @param mixed $data Data/object to be encryted
+* @param string $publicKey Public key for encryption
+* @param string $algorithmName
*/
public function __construct( $data, $publicKey, $algorithmName = 'rc4'
) {
$this->keyHash = crc32( $publicKey );
@@ -31,8 +32,9 @@
/**
* Decrypt the text in this object
*
-* @param $privateKey String with ascii-armored block, or the return of
openssl_get_privatekey
-* @return String plaintext
+* @param string $privateKey String with ascii-armored block,
+* or the return of openssl_get_privatekey
+* @return string plaintext
*/
public function getPlaintext( $privateKey ) {
$result = openssl_open(
@@ -53,8 +55,9 @@
/**
* Encrypt data with a public key
*
-* @param $data String
-* @param $publicKey String with ascii-armored block, or the return of
openssl_get_publickey
+* @param string $data
+* @param string $publicKey String with ascii-armored block,
+* or the return of openssl_get_publickey
*/
private function encryptData( $data, $publicKey ) {
openssl_seal( $data, $encryptedString, $envelopeKeys, [
$publicKey ], $this->algName );
diff --git a/api/ApiQueryCheckUser.php b/api/ApiQueryCheckUser.php
index 40815c3..de9d53f 100644
--- a/api/ApiQueryCheckUser.php
+++ b/api/ApiQueryCheckUser.php
@@ -278,6 +278,7 @@
/**
* @see ApiBase::getExamplesMessages()
+* @return array
*/
protected function getExamplesMessages() {
return [
diff --git a/api/ApiQueryCheckUserLog.php b/api/ApiQueryCheckUserLog.php
index 9ae7319..9b511d4 100644
--- a/api/ApiQueryCheckUserLog.php
+++ b/api/ApiQueryCheckUserLog.php
@@ -120,6 +120,7 @@
/**
* @see ApiBase::getExamplesMessages()
+* @return array
*/
protected function getExamplesMessages() {
return [
diff --git a/phpcs.xml b/phpcs.xml
index 26e39d0..a5ee126 100644
--- a/phpcs.xml
+++
Mobrovac has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370758 )
Change subject: Update restbase to 43aad0a
..
Update restbase to 43aad0a
List of changes:
43aad0a Temporary fallback to the new storage buckets in order to truncate the
tables
Change-Id: Ic4f887d237e093dbd143e83f10320d39427f6b24
---
M restbase
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Mobrovac: Verified; Looks good to me, approved
diff --git a/restbase b/restbase
index 26f4515..43aad0a 16
--- a/restbase
+++ b/restbase
@@ -1 +1 @@
-Subproject commit 26f4515c2661459cd37fd41f8861865b4b0de965
+Subproject commit 43aad0a4cf6c0db9c5f8b83961b2051055176a53
--
To view, visit https://gerrit.wikimedia.org/r/370758
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic4f887d237e093dbd143e83f10320d39427f6b24
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/restbase/deploy
Gerrit-Branch: master
Gerrit-Owner: Mobrovac
Gerrit-Reviewer: Mobrovac
Gerrit-Reviewer: Ppchelko
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Mobrovac has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370758 )
Change subject: Update restbase to 43aad0a
..
Update restbase to 43aad0a
List of changes:
43aad0a Temporary fallback to the new storage buckets in order to truncate the
tables
Change-Id: Ic4f887d237e093dbd143e83f10320d39427f6b24
---
M restbase
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/restbase/deploy
refs/changes/58/370758/1
diff --git a/restbase b/restbase
index 26f4515..43aad0a 16
--- a/restbase
+++ b/restbase
@@ -1 +1 @@
-Subproject commit 26f4515c2661459cd37fd41f8861865b4b0de965
+Subproject commit 43aad0a4cf6c0db9c5f8b83961b2051055176a53
--
To view, visit https://gerrit.wikimedia.org/r/370758
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic4f887d237e093dbd143e83f10320d39427f6b24
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/restbase/deploy
Gerrit-Branch: master
Gerrit-Owner: Mobrovac
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Bearloga has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370756 )
Change subject: Fix a bug in function 'compress' when number is between 0 and 1
..
Fix a bug in function 'compress' when number is between 0 and 1
Change-Id: I25ef7d1332d0bacafbd07d7ee64c6c22e3dd7bbd
---
M R/maths.R
M tests/testthat/test-maths.R
2 files changed, 3 insertions(+), 2 deletions(-)
Approvals:
Bearloga: Verified; Looks good to me, approved
diff --git a/R/maths.R b/R/maths.R
index aab5b7f..5cfd017 100644
--- a/R/maths.R
+++ b/R/maths.R
@@ -22,5 +22,6 @@
#' @export
compress <- function(x, round_by = 2) {
div <- findInterval(x, c(1, 1e3, 1e6, 1e9, 1e12))
- return(paste0(round( x / 10 ^ (3 * (div - 1)), round_by), c("", "", "K",
"M", "B", "T")[div + 1]))
+ return(paste0(round( x / 10 ^ (3 * ifelse(div - 1 < 0, 0, div - 1)),
round_by),
+c("", "", "K", "M", "B", "T")[div + 1]))
}
diff --git a/tests/testthat/test-maths.R b/tests/testthat/test-maths.R
index 9a84d57..2455e10 100644
--- a/tests/testthat/test-maths.R
+++ b/tests/testthat/test-maths.R
@@ -7,7 +7,7 @@
})
test_that("suffixes", {
- expect_equal(compress(c(0, 1, 10, 100)), c("0", "1", "10", "100"))
+ expect_equal(compress(c(0, 0.5, 1, 10, 100)), c("0", "0.5", "1", "10",
"100"))
expect_equal(compress(1.642e3, round_by = 1), "1.6K")
expect_equal(compress(c(10, 1e6, 1e12, 1e9)), c("100K", "1M", "1T",
"1B"))
expect_equal(compress(c(0, 1, 1e6, 1e3)), c("0", "1", "1M", "1K"))
--
To view, visit https://gerrit.wikimedia.org/r/370756
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I25ef7d1332d0bacafbd07d7ee64c6c22e3dd7bbd
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/discovery/polloi
Gerrit-Branch: master
Gerrit-Owner: Chelsyx
Gerrit-Reviewer: Bearloga
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Umherirrender has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370757 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: Ie2ea4ee2bb415e17f903a354170e55b49085414a
---
M CampaignsAuthenticationRequest.php
M composer.json
M phpcs.xml
3 files changed, 4 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Campaigns
refs/changes/57/370757/1
diff --git a/CampaignsAuthenticationRequest.php
b/CampaignsAuthenticationRequest.php
index 918e1a3..29e8d92 100644
--- a/CampaignsAuthenticationRequest.php
+++ b/CampaignsAuthenticationRequest.php
@@ -8,7 +8,7 @@
class CampaignsAuthenticationRequest extends AuthenticationRequest {
public $required = self::OPTIONAL;
- /** @var boolean Whether to use the campaign field from the request */
+ /** @var bool Whether to use the campaign field from the request */
private $useCampaignField;
/** @var string Campaign to log */
@@ -22,7 +22,7 @@
/**
* @param WebRequest $request Request to load data from
-* @param boolean $useCampaignField Whether to actually use the
'campaign' field
+* @param bool $useCampaignField Whether to actually use the 'campaign'
field
*/
public function __construct( WebRequest $request, $useCampaignField ) {
$this->useCampaignField = $useCampaignField;
diff --git a/composer.json b/composer.json
index bf202bf..fd295f4 100644
--- a/composer.json
+++ b/composer.json
@@ -7,7 +7,7 @@
"scripts": {
"fix": "phpcbf",
"test": [
- "parallel-lint . --exclude vendor",
+ "parallel-lint . --exclude vendor --exclude
node_modules",
"phpcs -p -s"
]
}
diff --git a/phpcs.xml b/phpcs.xml
index bf1d981..b15894e 100644
--- a/phpcs.xml
+++ b/phpcs.xml
@@ -7,7 +7,5 @@
.
-
- vendor
- node_modules
+
--
To view, visit https://gerrit.wikimedia.org/r/370757
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie2ea4ee2bb415e17f903a354170e55b49085414a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Campaigns
Gerrit-Branch: master
Gerrit-Owner: Umherirrender
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Chelsyx has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370756 )
Change subject: Fix a bug in function 'compress' when number is between 0 and 1
..
Fix a bug in function 'compress' when number is between 0 and 1
Change-Id: I25ef7d1332d0bacafbd07d7ee64c6c22e3dd7bbd
---
M R/maths.R
M tests/testthat/test-maths.R
2 files changed, 3 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/wikimedia/discovery/polloi
refs/changes/56/370756/1
diff --git a/R/maths.R b/R/maths.R
index aab5b7f..5cfd017 100644
--- a/R/maths.R
+++ b/R/maths.R
@@ -22,5 +22,6 @@
#' @export
compress <- function(x, round_by = 2) {
div <- findInterval(x, c(1, 1e3, 1e6, 1e9, 1e12))
- return(paste0(round( x / 10 ^ (3 * (div - 1)), round_by), c("", "", "K",
"M", "B", "T")[div + 1]))
+ return(paste0(round( x / 10 ^ (3 * ifelse(div - 1 < 0, 0, div - 1)),
round_by),
+c("", "", "K", "M", "B", "T")[div + 1]))
}
diff --git a/tests/testthat/test-maths.R b/tests/testthat/test-maths.R
index 9a84d57..2455e10 100644
--- a/tests/testthat/test-maths.R
+++ b/tests/testthat/test-maths.R
@@ -7,7 +7,7 @@
})
test_that("suffixes", {
- expect_equal(compress(c(0, 1, 10, 100)), c("0", "1", "10", "100"))
+ expect_equal(compress(c(0, 0.5, 1, 10, 100)), c("0", "0.5", "1", "10",
"100"))
expect_equal(compress(1.642e3, round_by = 1), "1.6K")
expect_equal(compress(c(10, 1e6, 1e12, 1e9)), c("100K", "1M", "1T",
"1B"))
expect_equal(compress(c(0, 1, 1e6, 1e3)), c("0", "1", "1M", "1K"))
--
To view, visit https://gerrit.wikimedia.org/r/370756
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I25ef7d1332d0bacafbd07d7ee64c6c22e3dd7bbd
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/discovery/polloi
Gerrit-Branch: master
Gerrit-Owner: Chelsyx
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Umherirrender has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370755 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: I90a518f784fb9f2a2f34032ae15afacefd1e1b66
---
M BounceHandlerHooks.php
M composer.json
M includes/ApiBounceHandler.php
M includes/BounceHandlerActions.php
M includes/EchoBounceHandlerPresentationModel.php
M includes/ProcessBounceEmails.php
M phpcs.xml
7 files changed, 12 insertions(+), 15 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BounceHandler
refs/changes/55/370755/1
diff --git a/BounceHandlerHooks.php b/BounceHandlerHooks.php
index 1ea01b6..225f373 100644
--- a/BounceHandlerHooks.php
+++ b/BounceHandlerHooks.php
@@ -23,7 +23,7 @@
* Generating VERP address for a batch of send emails is complex. This
feature is hence disabled
*
* @param MailAddress[] $recip Recipient's email array
-* @param string $returnPath return-path address
+* @param string &$returnPath return-path address
* @return bool
* @throws InvalidArgumentException
*/
@@ -40,7 +40,7 @@
* Process a given $to address and return its VERP return path
*
* @param MailAddress $to
-* @param string $returnPath return-path address
+* @param string &$returnPath return-path address
* @return bool true
*/
protected static function generateVerp( MailAddress $to, &$returnPath )
{
@@ -114,7 +114,7 @@
/**
* Add BounceHandler events to Echo
*
-* @param array $notifications Echo notifications
+* @param array &$notifications Echo notifications
* @return bool
*/
public static function onBeforeCreateEchoEvent( array &$notifications )
{
@@ -143,7 +143,7 @@
* Add user to be notified on echo event
*
* @param EchoEvent $event
-* @param User[] $users
+* @param User[] &$users
* @return bool
*/
public static function onEchoGetDefaultNotifiedUsers( EchoEvent $event,
array &$users ) {
diff --git a/composer.json b/composer.json
index bf202bf..fd295f4 100644
--- a/composer.json
+++ b/composer.json
@@ -7,7 +7,7 @@
"scripts": {
"fix": "phpcbf",
"test": [
- "parallel-lint . --exclude vendor",
+ "parallel-lint . --exclude vendor --exclude
node_modules",
"phpcs -p -s"
]
}
diff --git a/includes/ApiBounceHandler.php b/includes/ApiBounceHandler.php
index 2c6c323..96f254f 100644
--- a/includes/ApiBounceHandler.php
+++ b/includes/ApiBounceHandler.php
@@ -89,6 +89,7 @@
/**
* @see ApiBase::getExamplesMessages()
+* @return array
*/
public function getExamplesMessages() {
return [
diff --git a/includes/BounceHandlerActions.php
b/includes/BounceHandlerActions.php
index 26b8c2b..664a019 100644
--- a/includes/BounceHandlerActions.php
+++ b/includes/BounceHandlerActions.php
@@ -114,8 +114,8 @@
* Function to inject Echo notification to the last source of bounce
for an
* unsubscribed Global user
*
-* @param $bounceUserId
-* @param $originalEmail
+* @param int $bounceUserId
+* @param string $originalEmail
*/
public function notifyGlobalUser( $bounceUserId, $originalEmail ) {
$params = [
diff --git a/includes/EchoBounceHandlerPresentationModel.php
b/includes/EchoBounceHandlerPresentationModel.php
index fc94fbd..7ca22db 100644
--- a/includes/EchoBounceHandlerPresentationModel.php
+++ b/includes/EchoBounceHandlerPresentationModel.php
@@ -2,14 +2,14 @@
class EchoBounceHandlerPresentationModel extends EchoEventPresentationModel {
/**
-* {@inheritdoc}
+* @inheritDoc
*/
public function getIconType() {
return 'placeholder';
}
/**
-* {@inheritdoc}
+* @inheritDoc
*/
public function getPrimaryLink() {
return [
@@ -19,7 +19,7 @@
}
/**
-* {@inheritdoc}
+* @inheritDoc
*/
public function getHeaderMessage() {
$msg = parent::getHeaderMessage();
diff --git a/includes/ProcessBounceEmails.php b/includes/ProcessBounceEmails.php
index b9d87b7..ce06b55 100644
--- a/includes/ProcessBounceEmails.php
+++ b/includes/ProcessBounceEmails.php
@@ -228,7 +228,7 @@
/**
* Get a lazy connection to the bounce table
*
-* @param integer $index DB_MASTER/DB_SLAVE
+* @param int $index DB_MASTER/DB_SLAVE
* @param string $wiki The DB that the bounced email was sent from
* @return IDatabase
*/
diff --git a/phpcs.xml b/phpcs.xml
index
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370754 )
Change subject: POC: Send all the links for disambiguation pages in the summary
..
POC: Send all the links for disambiguation pages in the summary
Playing with async functions... :)
Bug: T171052
Change-Id: I8272e45b296e7c75adfc82dc0b33b2a9d3ca268b
---
M routes/mobile-sections.js
1 file changed, 54 insertions(+), 17 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps
refs/changes/54/370754/1
diff --git a/routes/mobile-sections.js b/routes/mobile-sections.js
index 9d6684d..625e3ca 100644
--- a/routes/mobile-sections.js
+++ b/routes/mobile-sections.js
@@ -352,6 +352,32 @@
});
}
+/*
+ * Get disambiguation links for a given page
+ * @param {!Request} req
+ * @return {!BBPromise}
+ */
+function getDisambiguationLinks(req) {
+const p = req.params;
+const query = {
+action: 'query',
+titles: p.title,
+prop: 'links',
+format: 'json',
+formatversion: '2'
+};
+
+return apiUtil.mwApiGet(app, p.domain, query).then((resp) => {
+const result = resp.body;
+const page = result && result.query && result.query.pages &&
result.query.pages[0];
+const links = page && page.links;
+if (links) {
+ return links.map((page)=>page.title);
+} else {
+ return [];
+}
+});
+}
/*
* Build a summary for the page given in req
@@ -360,25 +386,36 @@
* of deprecating.
* @return {!BBPromise}
*/
-function buildSummary(req) {
-return buildLeadObject(req, false).then((lead) => {
-let summary = {};
-const type = 'standard';
-let code = 200;
+async function buildSummary(req) {
+const lead = await buildLeadObject(req, false);
+let summary = {};
+let disambiguation_links;
+let type = 'standard';
+let code = 200;
+let d;
-if (!lead) {
-return false;
-} else if (lead.contentmodel || lead.ns !== 0) {
-code = 204;
-} else if (lead.intro) {
-summary = transforms.summarize(lead.intro);
+if (!lead) {
+return false;
+} else if (lead.contentmodel || lead.ns !== 0) {
+code = 204;
+} else if (lead.intro) {
+summary = transforms.summarize(lead.intro);
+}
+if (lead.disambiguation) {
+type = 'disambiguation';
+try {
+disambiguation_links = await getDisambiguationLinks(req);
+} catch (e) {
+console.log(e)
+disambiguation_links = [];
}
-return Object.assign({
-code,
-type,
-revision: lead.revision
-}, summary);
-});
+}
+return Object.assign({
+code,
+disambiguation_links,
+type,
+revision: lead.revision
+}, summary);
}
/**
--
To view, visit https://gerrit.wikimedia.org/r/370754
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8272e45b296e7c75adfc82dc0b33b2a9d3ca268b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Umherirrender has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370753 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: I6e2cda92da62074ab4440629235411a2046c6688
---
M DataCollector.php
M HookHandler.php
M TemplateParser.php
M composer.json
M phpcs.xml
M tests/phpunit/ParserTestHelper.php
6 files changed, 14 insertions(+), 19 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CommonsMetadata
refs/changes/53/370753/1
diff --git a/DataCollector.php b/DataCollector.php
index 865dff8..94416ac 100644
--- a/DataCollector.php
+++ b/DataCollector.php
@@ -55,7 +55,7 @@
}
/**
-* @param boolean $multiLang
+* @param bool $multiLang
*/
public function setMultiLang( $multiLang ) {
$this->multiLang = $multiLang;
@@ -87,7 +87,7 @@
* For fields with multiple values and/or in multiple languages the
format is more complex;
* see the documentation for the extmetadata API.
*
-* @param array $previousMetadata metadata collected so far;
+* @param array &$previousMetadata metadata collected so far;
* new metadata will be added to this array
* @param File $file
*/
@@ -322,7 +322,7 @@
/**
* Matches category names to a category => license mapping, removes the
matching categories
* and returns the corresponding licenses.
-* @param array $categories a list of human-readable category names.
+* @param array &$categories a list of human-readable category names.
* @return array
*/
protected function getLicensesAndRemoveFromCategories( &$categories ) {
@@ -341,7 +341,7 @@
/**
* Matches category names to a category => assessment mapping, removes
the matching categories
* and returns the corresponding assessments (valued image, picture of
the day etc).
-* @param array $categories a list of human-readable category names.
+* @param array &$categories a list of human-readable category names.
* @return array
*/
protected function getAssessmentsAndRemoveFromCategories( &$categories
) {
@@ -422,7 +422,7 @@
/**
* Normalizes the metadata to wfTimestamp()'s TS_DB format
-* @param array $metadata
+* @param array &$metadata
*/
protected function normalizeMetadataTimestamps( array &$metadata ) {
$fieldsToNormalize = [ 'DateTime', 'DateTimeOriginal' ];
diff --git a/HookHandler.php b/HookHandler.php
index 63156de..95ce26c 100644
--- a/HookHandler.php
+++ b/HookHandler.php
@@ -26,7 +26,7 @@
/**
* Hook handler for extended metadata
*
-* @param array $combinedMeta Metadata so far
+* @param array &$combinedMeta Metadata so far
* @param File $file The file object in question
* @param IContextSource $context Context. Used to select language
* @param bool $singleLang Get only target language, or all translations
diff --git a/TemplateParser.php b/TemplateParser.php
index 48ebebb..b92f5c4 100644
--- a/TemplateParser.php
+++ b/TemplateParser.php
@@ -123,8 +123,8 @@
*
* This is the main entry point to the class.
*
-* @param $html String The html to parse
-* @return Array The properties extracted from the page.
+* @param string $html The html to parse
+* @return array The properties extracted from the page.
*/
public function parsePage( $html ) {
if ( !$html ) { // DOMDocument does not like empty strings
@@ -207,7 +207,7 @@
* not required) to have one of the $informationFieldClasses.
* @param string $idOrClass id or class identifying the field, per
$informationFieldClasses Node
* is ignored if this is not a key of $informationFieldClasses. Also
ignored if this is null.
-* @param array $data
+* @param array &$data
*/
protected function parseInformationField(
DomNavigator $domNavigator, DOMElement $informationField,
$group, $idOrClass, array &$data
@@ -243,7 +243,7 @@
/**
* Sorts info template data groups according to
$informationFieldClasses, highest priority first
* Also removes the _type helper keys.
-* @param array $data info template data, as returned by
parseInformationFields()
+* @param array &$data info template data, as returned by
parseInformationFields()
*/
protected function sortInformationGroups( array &$data ) {
// PHP 5.3 does not like class references in closures
@@ -273,7 +273,7 @@
/**
* Prunes template data
* Removes blacklisted templates if they are not alone
-* @param array $data info template data
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370672 )
Change subject: Add better empty state to Offline Library screen.
..
Add better empty state to Offline Library screen.
(This is the first introduction of the new "library" and "packs" verbiage,
and can be independent of the Grand Renaming that will follow soon.)
Bug: T172553
Change-Id: I9bd2a83bd940bd713f5c91476643f98938937c21
---
M app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
M app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
A app/src/main/res/drawable/empty_offlinelibrary.png
M app/src/main/res/layout/fragment_local_compilations.xml
M app/src/main/res/values-qq/strings.xml
M app/src/main/res/values/strings.xml
6 files changed, 63 insertions(+), 3 deletions(-)
Approvals:
jenkins-bot: Verified
Mholloway: Looks good to me, approved
diff --git
a/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
b/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
index 8be08cd..b54c002 100644
--- a/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
+++ b/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
@@ -23,7 +23,7 @@
setWordmarkVisible(false);
if (getSupportActionBar() != null) {
getSupportActionBar().setDisplayHomeAsUpEnabled(true);
-
getSupportActionBar().setTitle(R.string.offline_compilations_title);
+getSupportActionBar().setTitle(R.string.offline_library_title);
}
}
diff --git
a/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
b/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
index d72dda4..a77a0a7 100644
--- a/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
+++ b/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
@@ -45,6 +45,7 @@
@BindView(R.id.compilations_count_text) TextView countText;
@BindView(R.id.disk_usage_view) DiskUsageView diskUsageView;
@BindView(R.id.compilation_search_error) WikiErrorView errorView;
+@BindView(R.id.compilation_empty_container) View emptyContainer;
private Unbinder unbinder;
private boolean updating;
@@ -131,7 +132,7 @@
}
}
-@OnClick(R.id.compilations_add_button) void onAddCompilationClick() {
+@OnClick({R.id.compilations_add_button,
R.id.compilation_empty_search_button}) void onAddCompilationClick() {
startActivity(RemoteCompilationsActivity.newIntent(getContext()));
}
@@ -204,16 +205,19 @@
progressBar.setVisibility(View.GONE);
searchEmptyView.setVisibility(View.GONE);
listContainer.setVisibility(View.GONE);
+emptyContainer.setVisibility(View.GONE);
return;
}
errorView.setVisibility(View.GONE);
progressBar.setVisibility(updating ? View.VISIBLE : View.GONE);
if (TextUtils.isEmpty(searchQuery)) {
searchEmptyView.setVisibility(View.GONE);
-listContainer.setVisibility(View.VISIBLE);
+listContainer.setVisibility(displayedItems.isEmpty() ? View.GONE :
View.VISIBLE);
+emptyContainer.setVisibility(displayedItems.isEmpty() ?
View.VISIBLE : View.GONE);
} else {
listContainer.setVisibility(displayedItems.isEmpty() ? View.GONE :
View.VISIBLE);
searchEmptyView.setVisibility(displayedItems.isEmpty() ?
View.VISIBLE : View.GONE);
+emptyContainer.setVisibility(View.GONE);
}
}
diff --git a/app/src/main/res/drawable/empty_offlinelibrary.png
b/app/src/main/res/drawable/empty_offlinelibrary.png
new file mode 100644
index 000..00745f4
--- /dev/null
+++ b/app/src/main/res/drawable/empty_offlinelibrary.png
Binary files differ
diff --git a/app/src/main/res/layout/fragment_local_compilations.xml
b/app/src/main/res/layout/fragment_local_compilations.xml
index 33bd03e..58e076e 100644
--- a/app/src/main/res/layout/fragment_local_compilations.xml
+++ b/app/src/main/res/layout/fragment_local_compilations.xml
@@ -83,6 +83,54 @@
+
+
+
+
+
+
+
+
+
+
+
+
Confirmation message
asking if the user really wants to cancel the download.
Affirmative answer to
cancel the download.\n{{Identical|Yes}}
Negative answer to
cancel the download.\n{{Identical|No}}
+ Title shown at the top of the activity
for managing the offline library.
+ Message shown when the offline
library is empty.
+ Explanation encouraging the
user to download an article pack for offline use later.
+ Button label for searching the
list of possible article packs to download.
Button label to skip the current onboarding
or tutorial screen.\n{{Identical|Skip}}
Button label to continue to the next
Ayounsi has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370661 )
Change subject: Remove pfw-codfw from DNS
..
Remove pfw-codfw from DNS
Bug: T171970
Change-Id: Id7b0564fe8f9ac52c50e6c8ca031c8a4df51f018
---
M templates/10.in-addr.arpa
M templates/153.80.208.in-addr.arpa
M templates/154.80.208.in-addr.arpa
M templates/wikimedia.org
M templates/wmnet
5 files changed, 1 insertion(+), 20 deletions(-)
Approvals:
jenkins-bot: Verified
Ayounsi: Looks good to me, approved
diff --git a/templates/10.in-addr.arpa b/templates/10.in-addr.arpa
index bc64966..1734a04 100644
--- a/templates/10.in-addr.arpa
+++ b/templates/10.in-addr.arpa
@@ -3536,8 +3536,6 @@
17 1H IN PTR asw-b-codfw.mgmt.codfw.wmnet.
18 1H IN PTR asw-c-codfw.mgmt.codfw.wmnet.
19 1H IN PTR asw-d-codfw.mgmt.codfw.wmnet.
-21 1H IN PTR pfw1-codfw.mgmt.codfw.wmnet.
-22 1H IN PTR pfw2-codfw.mgmt.codfw.wmnet.
23 1H IN PTR pfw3a-codfw.mgmt.codfw.wmnet.
24 1H IN PTR pfw3b-codfw.mgmt.codfw.wmnet.
25 1H IN PTR ps1-a1-codfw.mgmt.codfw.wmnet.
diff --git a/templates/153.80.208.in-addr.arpa
b/templates/153.80.208.in-addr.arpa
index c489910..542dad6 100644
--- a/templates/153.80.208.in-addr.arpa
+++ b/templates/153.80.208.in-addr.arpa
@@ -93,7 +93,6 @@
192 1H IN PTR cr1-codfw.wikimedia.org.
193 1H IN PTR cr2-codfw.wikimedia.org.
194 1H IN PTR pim-rp.wikimedia.org.
-195 1H IN PTR pfw-codfw.wikimedia.org.
196 1H IN PTR mr1-codfw.wikimedia.org.
197 1H IN PTR pfw3-codfw.wikimedia.org.
198 1H IN PTR cr1-eqdfw.wikimedia.org.
@@ -127,14 +126,6 @@
; 208.80.153.212/31 (cr2-codfw <--> cr1-eqdfw)
212 1H IN PTR xe-5-0-0.cr2-codfw.wikimedia.org.
213 1H IN PTR xe-1-0-0.cr1-eqdfw.wikimedia.org.
-
-; 208.80.153.214/31 (cr1-codfw <--> pfw-codfw)
-214 1H IN PTR xe-5-0-3.cr1-codfw.wikimedia.org.
-215 1H IN PTR xe-6-0-0.pfw-codfw.wikimedia.org.
-
-; 208.80.153.216/31 (cr2-codfw <--> pfw-codfw)
-216 1H IN PTR xe-5-0-3.cr2-codfw.wikimedia.org.
-217 1H IN PTR xe-15-0-0.pfw-codfw.wikimedia.org.
; 208.80.153.218/31 (cr1-codfw <--> cr2-codfw)
218 1H IN PTR ae0.cr1-codfw.wikimedia.org.
diff --git a/templates/154.80.208.in-addr.arpa
b/templates/154.80.208.in-addr.arpa
index 37d8096..d28bc0a 100644
--- a/templates/154.80.208.in-addr.arpa
+++ b/templates/154.80.208.in-addr.arpa
@@ -156,9 +156,7 @@
220 1H IN PTR xe-3-3-2.cr2-eqiad.wikimedia.org.
221 1H IN PTR xe-15-0-0.pfw-eqiad.wikimedia.org.
-; 208.80.154.222/31 (pfw-eqiad <--> pfw-codfw)
-222 1H IN PTR st0.pfw-eqiad.wikimedia.org.
-223 1H IN PTR st0.pfw-codfw.wikimedia.org.
+; 208.80.154.222/31 unused
; 208.80.154.224/27 (224-255) LVS Service IPs
diff --git a/templates/wikimedia.org b/templates/wikimedia.org
index 50e1b97..429e101 100644
--- a/templates/wikimedia.org
+++ b/templates/wikimedia.org
@@ -454,10 +454,6 @@
pfw1-eqiad 1H IN CNAME pfw-eqiad
pfw2-eqiad 1H IN CNAME pfw-eqiad
-pfw-codfw 1H IN A 208.80.153.195
-pfw1-codfw 1H IN CNAME pfw-codfw
-pfw2-codfw 1H IN CNAME pfw-codfw
-
pfw3-codfw 1H IN A208.80.153.197
; Cams
diff --git a/templates/wmnet b/templates/wmnet
index 507cfc9..63d3ff1 100644
--- a/templates/wmnet
+++ b/templates/wmnet
@@ -3241,8 +3241,6 @@
scs-a1-codfw1H IN A10.193.0.14
scs-c1-codfw1H IN A10.193.0.15
-pfw1-codfw 1H IN A10.193.0.21
-pfw2-codfw 1H IN A10.193.0.22
pfw3a-codfw 1H IN A10.193.0.23
pfw3b-codfw 1H IN A10.193.0.24
ps1-a1-codfw1H IN A10.193.0.25
--
To view, visit https://gerrit.wikimedia.org/r/370661
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id7b0564fe8f9ac52c50e6c8ca031c8a4df51f018
Gerrit-PatchSet: 1
Gerrit-Project: operations/dns
Gerrit-Branch: master
Gerrit-Owner: Ayounsi
Gerrit-Reviewer: Ayounsi
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370666 )
Change subject: Support loading Main Page from offline compilation.
..
Support loading Main Page from offline compilation.
Applies to loading the main page from the Feed card, as well as loading
the main page as the "default" page when opening a blank tab in
PageActivity.
Bug: T166312
Change-Id: Id18a1138b672e1d817e985f1592ab489123efec2
---
M app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
M app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java
M app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java
M app/src/main/java/org/wikipedia/page/PageActivity.java
4 files changed, 30 insertions(+), 12 deletions(-)
Approvals:
jenkins-bot: Verified
Mholloway: Looks good to me, approved
diff --git a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
index bc1f2d0..184d6c2 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
@@ -31,8 +31,7 @@
conditionallyAddPendingClient(new AggregatedFeedContentClient(),
online);
addPendingClient(new ContinueReadingClient());
-// TODO: enable this for offline when ready:
-conditionallyAddPendingClient(new MainPageClient(), age == 0 &&
online);
+conditionallyAddPendingClient(new MainPageClient(), age == 0);
conditionallyAddPendingClient(new BecauseYouReadClient(), online);
conditionallyAddPendingClient(new RandomClient(), age == 0);
diff --git
a/app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java
b/app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java
index b85735f..7bdf2e2 100644
--- a/app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java
+++ b/app/src/main/java/org/wikipedia/feed/mainpage/MainPageCardView.java
@@ -6,12 +6,9 @@
import android.view.View;
import org.wikipedia.R;
-import org.wikipedia.WikipediaApp;
import org.wikipedia.feed.view.FeedAdapter;
import org.wikipedia.feed.view.StaticCardView;
import org.wikipedia.history.HistoryEntry;
-import org.wikipedia.page.PageTitle;
-import org.wikipedia.staticdata.MainPageNameData;
import java.text.DateFormat;
import java.util.Date;
@@ -35,15 +32,12 @@
}
private class CallbackAdapter implements OnClickListener {
-@NonNull private WikipediaApp app = WikipediaApp.getInstance();
-
@Override
public void onClick(View view) {
if (getCallback() != null && getCard() != null) {
-PageTitle title = new PageTitle(MainPageNameData
-.valueFor(app.getAppOrSystemLanguageCode()),
app.getWikiSite());
getCallback().onSelectPage(getCard(),
-new HistoryEntry(title,
HistoryEntry.SOURCE_FEED_MAIN_PAGE));
+new HistoryEntry(MainPageClient.getMainPageTitle(),
+HistoryEntry.SOURCE_FEED_MAIN_PAGE));
}
}
}
diff --git a/app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java
b/app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java
index 9ecd74d..6784589 100644
--- a/app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java
+++ b/app/src/main/java/org/wikipedia/feed/mainpage/MainPageClient.java
@@ -1,11 +1,36 @@
package org.wikipedia.feed.mainpage;
+import android.support.annotation.NonNull;
+
+import org.wikipedia.WikipediaApp;
import org.wikipedia.dataclient.WikiSite;
import org.wikipedia.feed.dataclient.DummyClient;
import org.wikipedia.feed.model.Card;
+import org.wikipedia.offline.OfflineManager;
+import org.wikipedia.page.PageTitle;
+import org.wikipedia.staticdata.MainPageNameData;
+import org.wikipedia.util.DeviceUtil;
+import org.wikipedia.util.log.L;
+
+import java.io.IOException;
public class MainPageClient extends DummyClient {
@Override public Card getNewCard(WikiSite wiki) {
return new MainPageCard();
}
+
+@NonNull
+public static PageTitle getMainPageTitle() {
+WikipediaApp app = WikipediaApp.getInstance();
+PageTitle title = new PageTitle(MainPageNameData
+.valueFor(app.getAppOrSystemLanguageCode()),
app.getWikiSite());
+if (OfflineManager.hasCompilation() && !DeviceUtil.isOnline()) {
+try {
+title = new
PageTitle(OfflineManager.instance().getMainPageTitle(), app.getWikiSite());
+} catch (IOException e) {
+L.e(e);
+}
+}
+return title;
+}
}
diff --git a/app/src/main/java/org/wikipedia/page/PageActivity.java
b/app/src/main/java/org/wikipedia/page/PageActivity.java
index 4f37a45..7294385 100644
--- a/app/src/main/java/org/wikipedia/page/PageActivity.java
+++
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370748 )
Change subject: group0 wikis to 1.30.0-wmf.13 refs T170631
..
group0 wikis to 1.30.0-wmf.13 refs T170631
Change-Id: I8dfeefad70974d3b6559a7d283343fe9b3ea9b87
---
M php
M wikiversions.json
2 files changed, 7 insertions(+), 7 deletions(-)
Approvals:
20after4: Looks good to me, approved
jenkins-bot: Verified
diff --git a/php b/php
index 1a59817..a342232 12
--- a/php
+++ b/php
@@ -1 +1 @@
-php-1.30.0-wmf.12
\ No newline at end of file
+php-1.30.0-wmf.13
\ No newline at end of file
diff --git a/wikiversions.json b/wikiversions.json
index 0584b82..3a4ed1f 100644
--- a/wikiversions.json
+++ b/wikiversions.json
@@ -445,7 +445,7 @@
"kywikiquote": "php-1.30.0-wmf.12",
"kywiktionary": "php-1.30.0-wmf.12",
"labswiki": "php-1.30.0-wmf.12",
-"labtestwiki": "php-1.30.0-wmf.12",
+"labtestwiki": "php-1.30.0-wmf.13",
"ladwiki": "php-1.30.0-wmf.12",
"lawiki": "php-1.30.0-wmf.12",
"lawikibooks": "php-1.30.0-wmf.12",
@@ -487,7 +487,7 @@
"maiwikimedia": "php-1.30.0-wmf.12",
"map_bmswiki": "php-1.30.0-wmf.12",
"mdfwiki": "php-1.30.0-wmf.12",
-"mediawikiwiki": "php-1.30.0-wmf.12",
+"mediawikiwiki": "php-1.30.0-wmf.13",
"metawiki": "php-1.30.0-wmf.12",
"mgwiki": "php-1.30.0-wmf.12",
"mgwikibooks": "php-1.30.0-wmf.12",
@@ -760,9 +760,9 @@
"tcywiki": "php-1.30.0-wmf.12",
"techconductwiki": "php-1.30.0-wmf.12",
"tenwiki": "php-1.30.0-wmf.12",
-"test2wiki": "php-1.30.0-wmf.12",
-"testwiki": "php-1.30.0-wmf.12",
-"testwikidatawiki": "php-1.30.0-wmf.12",
+"test2wiki": "php-1.30.0-wmf.13",
+"testwiki": "php-1.30.0-wmf.13",
+"testwikidatawiki": "php-1.30.0-wmf.13",
"tetwiki": "php-1.30.0-wmf.12",
"tewiki": "php-1.30.0-wmf.12",
"tewikibooks": "php-1.30.0-wmf.12",
@@ -893,7 +893,7 @@
"zawikiquote": "php-1.30.0-wmf.12",
"zawiktionary": "php-1.30.0-wmf.12",
"zeawiki": "php-1.30.0-wmf.12",
-"zerowiki": "php-1.30.0-wmf.12",
+"zerowiki": "php-1.30.0-wmf.13",
"zh_classicalwiki": "php-1.30.0-wmf.12",
"zh_min_nanwiki": "php-1.30.0-wmf.12",
"zh_min_nanwikibooks": "php-1.30.0-wmf.12",
--
To view, visit https://gerrit.wikimedia.org/r/370748
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8dfeefad70974d3b6559a7d283343fe9b3ea9b87
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: 20after4
Gerrit-Reviewer: 20after4
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
20after4 has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370748 )
Change subject: group0 wikis to 1.30.0-wmf.13 refs T170631
..
group0 wikis to 1.30.0-wmf.13 refs T170631
Change-Id: I8dfeefad70974d3b6559a7d283343fe9b3ea9b87
---
M php
M wikiversions.json
2 files changed, 7 insertions(+), 7 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config
refs/changes/48/370748/1
diff --git a/php b/php
index 1a59817..a342232 12
--- a/php
+++ b/php
@@ -1 +1 @@
-php-1.30.0-wmf.12
\ No newline at end of file
+php-1.30.0-wmf.13
\ No newline at end of file
diff --git a/wikiversions.json b/wikiversions.json
index 0584b82..3a4ed1f 100644
--- a/wikiversions.json
+++ b/wikiversions.json
@@ -445,7 +445,7 @@
"kywikiquote": "php-1.30.0-wmf.12",
"kywiktionary": "php-1.30.0-wmf.12",
"labswiki": "php-1.30.0-wmf.12",
-"labtestwiki": "php-1.30.0-wmf.12",
+"labtestwiki": "php-1.30.0-wmf.13",
"ladwiki": "php-1.30.0-wmf.12",
"lawiki": "php-1.30.0-wmf.12",
"lawikibooks": "php-1.30.0-wmf.12",
@@ -487,7 +487,7 @@
"maiwikimedia": "php-1.30.0-wmf.12",
"map_bmswiki": "php-1.30.0-wmf.12",
"mdfwiki": "php-1.30.0-wmf.12",
-"mediawikiwiki": "php-1.30.0-wmf.12",
+"mediawikiwiki": "php-1.30.0-wmf.13",
"metawiki": "php-1.30.0-wmf.12",
"mgwiki": "php-1.30.0-wmf.12",
"mgwikibooks": "php-1.30.0-wmf.12",
@@ -760,9 +760,9 @@
"tcywiki": "php-1.30.0-wmf.12",
"techconductwiki": "php-1.30.0-wmf.12",
"tenwiki": "php-1.30.0-wmf.12",
-"test2wiki": "php-1.30.0-wmf.12",
-"testwiki": "php-1.30.0-wmf.12",
-"testwikidatawiki": "php-1.30.0-wmf.12",
+"test2wiki": "php-1.30.0-wmf.13",
+"testwiki": "php-1.30.0-wmf.13",
+"testwikidatawiki": "php-1.30.0-wmf.13",
"tetwiki": "php-1.30.0-wmf.12",
"tewiki": "php-1.30.0-wmf.12",
"tewikibooks": "php-1.30.0-wmf.12",
@@ -893,7 +893,7 @@
"zawikiquote": "php-1.30.0-wmf.12",
"zawiktionary": "php-1.30.0-wmf.12",
"zeawiki": "php-1.30.0-wmf.12",
-"zerowiki": "php-1.30.0-wmf.12",
+"zerowiki": "php-1.30.0-wmf.13",
"zh_classicalwiki": "php-1.30.0-wmf.12",
"zh_min_nanwiki": "php-1.30.0-wmf.12",
"zh_min_nanwikibooks": "php-1.30.0-wmf.12",
--
To view, visit https://gerrit.wikimedia.org/r/370748
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8dfeefad70974d3b6559a7d283343fe9b3ea9b87
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: 20after4
Gerrit-Reviewer: 20after4
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
20after4 has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370745 )
Change subject: scap prep: Fix comment in generated LocalSettings.php
..
scap prep: Fix comment in generated LocalSettings.php
The comment needs to actually be sorrounded by comments.
Change-Id: I540e3d8971a06885591a30192512f96c97d483c8
---
M scap/plugins/prep.py
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/mediawiki-config
refs/changes/45/370745/1
diff --git a/scap/plugins/prep.py b/scap/plugins/prep.py
index ea757ee..7b35beb 100644
--- a/scap/plugins/prep.py
+++ b/scap/plugins/prep.py
@@ -90,9 +90,9 @@
'wmf-config', 'CommonSettings.php')
ls_stub = (
'https://gerrit.wikimedia.org/r/370745
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I540e3d8971a06885591a30192512f96c97d483c8
Gerrit-PatchSet: 1
Gerrit-Project: operations/mediawiki-config
Gerrit-Branch: master
Gerrit-Owner: 20after4
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370736 )
Change subject: Always extract an infobox and paragraph
..
Always extract an infobox and paragraph
There is undocumented logic limiting lead paragraph and infobox
extraction to pages where there are more than 1 section.
For disambiguation pages e.g. https://en.m.wikipedia.org/wiki/Colombo_Airport
this is not helpful.
Change-Id: I393c7068cb9d1317530a9e010fe87b28d0275c81
---
M routes/mobile-sections.js
1 file changed, 2 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps
refs/changes/36/370736/1
diff --git a/routes/mobile-sections.js b/routes/mobile-sections.js
index 9073ffe..9d6684d 100644
--- a/routes/mobile-sections.js
+++ b/routes/mobile-sections.js
@@ -100,10 +100,8 @@
}
if (!legacy) {
-if (input.page.sections.length > 1) {
-infobox = transforms.extractInfobox(lead);
-intro = transforms.extractLeadIntroduction(lead);
-}
+infobox = transforms.extractInfobox(lead);
+intro = transforms.extractLeadIntroduction(lead);
text = lead.body.innerHTML;
} else {
// update text after extractions have taken place
--
To view, visit https://gerrit.wikimedia.org/r/370736
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I393c7068cb9d1317530a9e010fe87b28d0275c81
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Ppchelko has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370730 )
Change subject: Update restbase to 26f4515
..
Update restbase to 26f4515
List of changes:
26f4515 Update summary and licensing information; add pageview license
xxx Update node module dependencies
Change-Id: I641d47e2d2415f8e85098d31e4a0e95dc4817b64
---
M node_modules/content-type/package.json
M node_modules/core-js/Gruntfile.js
M node_modules/core-js/build/Gruntfile.ls
M node_modules/core-js/build/build.ls
M node_modules/core-js/build/config.js
M node_modules/core-js/build/index.js
M node_modules/core-js/client/core.js
M node_modules/core-js/client/core.min.js
M node_modules/core-js/client/core.min.js.map
M node_modules/core-js/client/library.js
M node_modules/core-js/client/library.min.js
M node_modules/core-js/client/library.min.js.map
M node_modules/core-js/client/shim.js
M node_modules/core-js/client/shim.min.js
M node_modules/core-js/client/shim.min.js.map
M node_modules/core-js/core/_.js
M node_modules/core-js/core/dict.js
M node_modules/core-js/core/number.js
M node_modules/core-js/core/regexp.js
M node_modules/core-js/core/string.js
M node_modules/core-js/es5/index.js
M node_modules/core-js/es6/array.js
M node_modules/core-js/es6/date.js
M node_modules/core-js/es6/function.js
M node_modules/core-js/es6/index.js
M node_modules/core-js/es6/map.js
M node_modules/core-js/es6/math.js
M node_modules/core-js/es6/number.js
M node_modules/core-js/es6/object.js
M node_modules/core-js/es6/parse-float.js
M node_modules/core-js/es6/parse-int.js
M node_modules/core-js/es6/promise.js
M node_modules/core-js/es6/reflect.js
M node_modules/core-js/es6/regexp.js
M node_modules/core-js/es6/set.js
M node_modules/core-js/es6/string.js
M node_modules/core-js/es6/symbol.js
M node_modules/core-js/es6/typed.js
M node_modules/core-js/es6/weak-map.js
M node_modules/core-js/es6/weak-set.js
M node_modules/core-js/es7/array.js
M node_modules/core-js/es7/error.js
A node_modules/core-js/es7/global.js
M node_modules/core-js/es7/index.js
M node_modules/core-js/es7/map.js
M node_modules/core-js/es7/math.js
M node_modules/core-js/es7/object.js
M node_modules/core-js/es7/observable.js
A node_modules/core-js/es7/promise.js
M node_modules/core-js/es7/set.js
M node_modules/core-js/es7/symbol.js
M node_modules/core-js/es7/system.js
A node_modules/core-js/es7/weak-map.js
A node_modules/core-js/es7/weak-set.js
M node_modules/core-js/fn/_.js
M node_modules/core-js/fn/array/concat.js
M node_modules/core-js/fn/array/copy-within.js
M node_modules/core-js/fn/array/entries.js
M node_modules/core-js/fn/array/every.js
M node_modules/core-js/fn/array/fill.js
M node_modules/core-js/fn/array/filter.js
M node_modules/core-js/fn/array/find-index.js
M node_modules/core-js/fn/array/find.js
A node_modules/core-js/fn/array/flat-map.js
A node_modules/core-js/fn/array/flatten.js
M node_modules/core-js/fn/array/for-each.js
M node_modules/core-js/fn/array/from.js
M node_modules/core-js/fn/array/includes.js
M node_modules/core-js/fn/array/index-of.js
M node_modules/core-js/fn/array/index.js
M node_modules/core-js/fn/array/is-array.js
M node_modules/core-js/fn/array/iterator.js
M node_modules/core-js/fn/array/join.js
M node_modules/core-js/fn/array/keys.js
M node_modules/core-js/fn/array/last-index-of.js
M node_modules/core-js/fn/array/map.js
M node_modules/core-js/fn/array/of.js
M node_modules/core-js/fn/array/pop.js
M node_modules/core-js/fn/array/push.js
M node_modules/core-js/fn/array/reduce-right.js
M node_modules/core-js/fn/array/reduce.js
M node_modules/core-js/fn/array/reverse.js
M node_modules/core-js/fn/array/shift.js
M node_modules/core-js/fn/array/slice.js
M node_modules/core-js/fn/array/some.js
M node_modules/core-js/fn/array/sort.js
M node_modules/core-js/fn/array/splice.js
M node_modules/core-js/fn/array/unshift.js
M node_modules/core-js/fn/array/values.js
M node_modules/core-js/fn/array/virtual/copy-within.js
M node_modules/core-js/fn/array/virtual/entries.js
M node_modules/core-js/fn/array/virtual/every.js
M node_modules/core-js/fn/array/virtual/fill.js
M node_modules/core-js/fn/array/virtual/filter.js
M node_modules/core-js/fn/array/virtual/find-index.js
M node_modules/core-js/fn/array/virtual/find.js
A node_modules/core-js/fn/array/virtual/flat-map.js
A node_modules/core-js/fn/array/virtual/flatten.js
M node_modules/core-js/fn/array/virtual/for-each.js
M node_modules/core-js/fn/array/virtual/includes.js
M node_modules/core-js/fn/array/virtual/index-of.js
M node_modules/core-js/fn/array/virtual/index.js
M node_modules/core-js/fn/array/virtual/iterator.js
M node_modules/core-js/fn/array/virtual/join.js
M node_modules/core-js/fn/array/virtual/keys.js
M node_modules/core-js/fn/array/virtual/last-index-of.js
M node_modules/core-js/fn/array/virtual/map.js
M node_modules/core-js/fn/array/virtual/reduce-right.js
M node_modules/core-js/fn/array/virtual/reduce.js
M
Ppchelko has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370730 )
Change subject: Update restbase to 26f4515
..
Update restbase to 26f4515
List of changes:
26f4515 Update summary and licensing information; add pageview license
xxx Update node module dependencies
Change-Id: I641d47e2d2415f8e85098d31e4a0e95dc4817b64
---
M node_modules/content-type/package.json
M node_modules/core-js/Gruntfile.js
M node_modules/core-js/build/Gruntfile.ls
M node_modules/core-js/build/build.ls
M node_modules/core-js/build/config.js
M node_modules/core-js/build/index.js
M node_modules/core-js/client/core.js
M node_modules/core-js/client/core.min.js
M node_modules/core-js/client/core.min.js.map
M node_modules/core-js/client/library.js
M node_modules/core-js/client/library.min.js
M node_modules/core-js/client/library.min.js.map
M node_modules/core-js/client/shim.js
M node_modules/core-js/client/shim.min.js
M node_modules/core-js/client/shim.min.js.map
M node_modules/core-js/core/_.js
M node_modules/core-js/core/dict.js
M node_modules/core-js/core/number.js
M node_modules/core-js/core/regexp.js
M node_modules/core-js/core/string.js
M node_modules/core-js/es5/index.js
M node_modules/core-js/es6/array.js
M node_modules/core-js/es6/date.js
M node_modules/core-js/es6/function.js
M node_modules/core-js/es6/index.js
M node_modules/core-js/es6/map.js
M node_modules/core-js/es6/math.js
M node_modules/core-js/es6/number.js
M node_modules/core-js/es6/object.js
M node_modules/core-js/es6/parse-float.js
M node_modules/core-js/es6/parse-int.js
M node_modules/core-js/es6/promise.js
M node_modules/core-js/es6/reflect.js
M node_modules/core-js/es6/regexp.js
M node_modules/core-js/es6/set.js
M node_modules/core-js/es6/string.js
M node_modules/core-js/es6/symbol.js
M node_modules/core-js/es6/typed.js
M node_modules/core-js/es6/weak-map.js
M node_modules/core-js/es6/weak-set.js
M node_modules/core-js/es7/array.js
M node_modules/core-js/es7/error.js
A node_modules/core-js/es7/global.js
M node_modules/core-js/es7/index.js
M node_modules/core-js/es7/map.js
M node_modules/core-js/es7/math.js
M node_modules/core-js/es7/object.js
M node_modules/core-js/es7/observable.js
A node_modules/core-js/es7/promise.js
M node_modules/core-js/es7/set.js
M node_modules/core-js/es7/symbol.js
M node_modules/core-js/es7/system.js
A node_modules/core-js/es7/weak-map.js
A node_modules/core-js/es7/weak-set.js
M node_modules/core-js/fn/_.js
M node_modules/core-js/fn/array/concat.js
M node_modules/core-js/fn/array/copy-within.js
M node_modules/core-js/fn/array/entries.js
M node_modules/core-js/fn/array/every.js
M node_modules/core-js/fn/array/fill.js
M node_modules/core-js/fn/array/filter.js
M node_modules/core-js/fn/array/find-index.js
M node_modules/core-js/fn/array/find.js
A node_modules/core-js/fn/array/flat-map.js
A node_modules/core-js/fn/array/flatten.js
M node_modules/core-js/fn/array/for-each.js
M node_modules/core-js/fn/array/from.js
M node_modules/core-js/fn/array/includes.js
M node_modules/core-js/fn/array/index-of.js
M node_modules/core-js/fn/array/index.js
M node_modules/core-js/fn/array/is-array.js
M node_modules/core-js/fn/array/iterator.js
M node_modules/core-js/fn/array/join.js
M node_modules/core-js/fn/array/keys.js
M node_modules/core-js/fn/array/last-index-of.js
M node_modules/core-js/fn/array/map.js
M node_modules/core-js/fn/array/of.js
M node_modules/core-js/fn/array/pop.js
M node_modules/core-js/fn/array/push.js
M node_modules/core-js/fn/array/reduce-right.js
M node_modules/core-js/fn/array/reduce.js
M node_modules/core-js/fn/array/reverse.js
M node_modules/core-js/fn/array/shift.js
M node_modules/core-js/fn/array/slice.js
M node_modules/core-js/fn/array/some.js
M node_modules/core-js/fn/array/sort.js
M node_modules/core-js/fn/array/splice.js
M node_modules/core-js/fn/array/unshift.js
M node_modules/core-js/fn/array/values.js
M node_modules/core-js/fn/array/virtual/copy-within.js
M node_modules/core-js/fn/array/virtual/entries.js
M node_modules/core-js/fn/array/virtual/every.js
M node_modules/core-js/fn/array/virtual/fill.js
M node_modules/core-js/fn/array/virtual/filter.js
M node_modules/core-js/fn/array/virtual/find-index.js
M node_modules/core-js/fn/array/virtual/find.js
A node_modules/core-js/fn/array/virtual/flat-map.js
A node_modules/core-js/fn/array/virtual/flatten.js
M node_modules/core-js/fn/array/virtual/for-each.js
M node_modules/core-js/fn/array/virtual/includes.js
M node_modules/core-js/fn/array/virtual/index-of.js
M node_modules/core-js/fn/array/virtual/index.js
M node_modules/core-js/fn/array/virtual/iterator.js
M node_modules/core-js/fn/array/virtual/join.js
M node_modules/core-js/fn/array/virtual/keys.js
M node_modules/core-js/fn/array/virtual/last-index-of.js
M node_modules/core-js/fn/array/virtual/map.js
M node_modules/core-js/fn/array/virtual/reduce-right.js
M node_modules/core-js/fn/array/virtual/reduce.js
M
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370674 )
Change subject: Ensure that wdqs-updater is not throttled.
..
Ensure that wdqs-updater is not throttled.
We trust the updater to not overload blazegraph, it should not be throttled.
The implementation relies on a HTTP header being set by nginx for all proxied
requests. The updater does not go through nginx but talks directly to
blazegraph, so the header set by nginx is not present.
Bug: T172782
Change-Id: Ic6ba59990d6e74f05b7ac51288fb1b21f85c9da0
---
M
blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
M
blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
M
blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlerTest.java
M war/src/main/webapp/WEB-INF/web.xml
4 files changed, 76 insertions(+), 11 deletions(-)
Approvals:
Smalyshev: Looks good to me, approved
jenkins-bot: Verified
diff --git
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
index da237d2..2b09df1 100644
---
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
+++
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
@@ -38,6 +38,17 @@
private final Callable createThrottlingState;
/**
+ * Throttling is only enabled if this header is set.
+ *
+ * This can be used to throttle only request coming through a revers proxy,
+ * which will set this specific header. Only the presence of the header is
+ * checked, not its value.
+ *
+ * If null, all requests will be throttled.
+ */
+private String enableThrottlingIfHeader;
+
+/**
* Constructor.
*
* Note that a bucket represent our approximation of a single client.
@@ -49,16 +60,19 @@
* when we start tracking a specific client
* @param stateStore the cache in which we store the per client state of
* throttling
+ * @param enableThrottlingIfHeader throttling is only enabled if this
header is present.
*/
public Throttler(
Duration requestTimeThreshold,
Bucketing bucketing,
Callable createThrottlingState,
-Cache stateStore) {
+Cache stateStore,
+String enableThrottlingIfHeader) {
this.requestTimeThreshold = requestTimeThreshold;
this.bucketing = bucketing;
this.state = stateStore;
this.createThrottlingState = createThrottlingState;
+this.enableThrottlingIfHeader = enableThrottlingIfHeader;
}
/**
@@ -68,10 +82,25 @@
* @return true if the request should be throttled
*/
public boolean isThrottled(HttpServletRequest request) {
+if (shouldBypassThrottling(request)) {
+return false;
+}
ThrottlingState throttlingState =
state.getIfPresent(bucketing.bucket(request));
if (throttlingState == null) return false;
return throttlingState.isThrottled();
+}
+
+/**
+ * Check whether this request should have throttling enabled.
+ * @param request
+ * @return if true then skip throttling
+ */
+private boolean shouldBypassThrottling(HttpServletRequest request) {
+if (enableThrottlingIfHeader == null) {
+return false;
+}
+return request.getHeader(enableThrottlingIfHeader) == null;
}
/**
@@ -81,6 +110,9 @@
* @param elapsed how long that request took
*/
public void success(HttpServletRequest request, Duration elapsed) {
+if (shouldBypassThrottling(request)) {
+return;
+}
try {
B bucket = bucketing.bucket(request);
ThrottlingState throttlingState;
@@ -105,6 +137,9 @@
* @param elapsed how long that request took
*/
public void failure(HttpServletRequest request, Duration elapsed) {
+if (shouldBypassThrottling(request)) {
+return;
+}
try {
ThrottlingState throttlingState =
state.get(bucketing.bucket(request), createThrottlingState);
diff --git
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
index 72a1198..9477ed4 100644
---
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
+++
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
@@ -96,14 +96,13 @@
/**
* Initialise the filter.
*
- * Configuration is loaded
MaxSem has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370698 )
Change subject: Move sources to includes/, remove b/c files
..
Move sources to includes/, remove b/c files
Change-Id: I2c0f0654f0a9e55f892b510aa4f66aa703b0df01
---
D PoolCounter.php
D PoolCounterClient.php
M extension.json
R includes/PoolCounter_Client.php
R includes/PoolCounter_ConnectionManager.php
5 files changed, 2 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PoolCounter
refs/changes/98/370698/1
diff --git a/PoolCounter.php b/PoolCounter.php
deleted file mode 12
index e390419..000
--- a/PoolCounter.php
+++ /dev/null
@@ -1 +0,0 @@
-PoolCounterClient.php
\ No newline at end of file
diff --git a/PoolCounterClient.php b/PoolCounterClient.php
deleted file mode 100644
index 597fe8e..000
--- a/PoolCounterClient.php
+++ /dev/null
@@ -1,3 +0,0 @@
-https://gerrit.wikimedia.org/r/370698
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2c0f0654f0a9e55f892b510aa4f66aa703b0df01
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PoolCounter
Gerrit-Branch: master
Gerrit-Owner: MaxSem
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370697 )
Change subject: Empty tags are omitted from summary output
..
Empty tags are omitted from summary output
Change-Id: I28e6c6518036daa331dd181f437e4997903a3f21
---
M lib/transformations/summarize.js
M test/lib/transformations/summarize.js
2 files changed, 7 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps
refs/changes/97/370697/1
diff --git a/lib/transformations/summarize.js b/lib/transformations/summarize.js
index 7668c07..8c78d2e 100644
--- a/lib/transformations/summarize.js
+++ b/lib/transformations/summarize.js
@@ -15,5 +15,6 @@
rmElementsWithSelector(doc, '.mw-ref');
rmElementsWithSelector(doc, '.noexcerpts');
rmElementsWithSelector(doc, 'math');
+rmElementsWithSelector(doc, 'span:empty,b:empty,i:empty,p:empty');
return doc.body.innerHTML;
};
diff --git a/test/lib/transformations/summarize.js
b/test/lib/transformations/summarize.js
index 43a2914..47f0745 100644
--- a/test/lib/transformations/summarize.js
+++ b/test/lib/transformations/summarize.js
@@ -8,6 +8,11 @@
describe('summarize', () => {
it('matches the spec', () => {
const testCases = [
+// Should flatten empty nodes
+[
+'f',
+'f'
+],
// Should flatten links
[
'This is some content with a link.',
@@ -31,7 +36,7 @@
// math tags are stripped but any math images are shown
[
'The Planck–Einstein relation connects the particulate
photon energy E with its associated wave
frequency f:\n\nhttp://www.w3.org/1998/Math/MathML\;>\n \n
\n \nE\n=\n
h\nf\n \n\n{\\displaystyle E=hf}\n
\nhttps://wikimedia.org/api/rest_v1/media/math/render/svg/f39fac3593bb1e2dec0282c112c4dff7a99007f6\;
class=\"mwe-math-fallback-image-inline\" aria-hidden=\"true\"
style=\"vertical-align: -0.671ex; width:7.533ex;
height:2.509ex;\">',
-'The Planck–Einstein relation connects the particulate
photon energy E with its associated wave
frequency f:\n\nhttps://wikimedia.org/api/rest_v1/media/math/render/svg/f39fac3593bb1e2dec0282c112c4dff7a99007f6\;
class=\"mwe-math-fallback-image-inline\" aria-hidden=\"true\"
style=\"vertical-align: -0.671ex; width:7.533ex;
height:2.509ex;\">'
+'The Planck–Einstein relation connects the particulate
photon energy E with its associated wave
frequency f:\n\nhttps://wikimedia.org/api/rest_v1/media/math/render/svg/f39fac3593bb1e2dec0282c112c4dff7a99007f6\;
class=\"mwe-math-fallback-image-inline\" aria-hidden=\"true\"
style=\"vertical-align: -0.671ex; width:7.533ex;
height:2.509ex;\">'
]
];
testCases.forEach((test) => {
--
To view, visit https://gerrit.wikimedia.org/r/370697
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I28e6c6518036daa331dd181f437e4997903a3f21
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
MaxSem has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370297 )
Change subject: Improve some parameter docs
..
Improve some parameter docs
Change-Id: I03fa9b58c72bcd28985c5a3467b82d8b98f3a0fc
---
M includes/AbuseFilter.class.php
M includes/AbuseFilter.hooks.php
M includes/Views/AbuseFilterViewDiff.php
M includes/parser/AbuseFilterTokenizer.php
4 files changed, 54 insertions(+), 54 deletions(-)
Approvals:
MaxSem: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/AbuseFilter.class.php b/includes/AbuseFilter.class.php
index f0af762..8a91828 100644
--- a/includes/AbuseFilter.class.php
+++ b/includes/AbuseFilter.class.php
@@ -1698,7 +1698,7 @@
}
/**
-* @param $user User
+* @param User $user
* @return String
*/
public static function autoPromoteBlockKey( $user ) {
@@ -1878,10 +1878,10 @@
}
/**
-* @param $rules String
-* @param $textName String
-* @param $addResultDiv Boolean
-* @param $canEdit Boolean
+* @param string $rules
+* @param string $textName
+* @param bool $addResultDiv
+* @param bool $canEdit
* @return string
*/
static function buildEditBox( $rules, $textName = 'wpFilterRules',
$addResultDiv = true,
@@ -1965,8 +1965,8 @@
* Each version is expected to be an array( $row, $actions )
* Returns an array of fields that are different.
*
-* @param $version_1
-* @param $version_2
+* @param array $version_1
+* @param array $version_2
*
* @return array
*/
@@ -2013,7 +2013,7 @@
}
/**
-* @param $row
+* @param stdClass $row
* @return array
*/
static function translateFromHistory( $row ) {
@@ -2053,7 +2053,7 @@
}
/**
-* @param $action string
+* @param string $action
* @return String
*/
static function getActionDisplay( $action ) {
@@ -2068,7 +2068,7 @@
}
/**
-* @param $row
+* @param stdClass $row
* @return AbuseFilterVariableHolder|null
*/
public static function getVarsFromRCRow( $row ) {
@@ -2091,7 +2091,7 @@
}
/**
-* @param $row
+* @param stdClass $row
* @return AbuseFilterVariableHolder
*/
public static function getCreateVarsFromRCRow( $row ) {
@@ -2114,7 +2114,7 @@
}
/**
-* @param $row
+* @param stdClass $row
* @return AbuseFilterVariableHolder
*/
public static function getEditVarsFromRCRow( $row ) {
@@ -2152,7 +2152,7 @@
}
/**
-* @param $row
+* @param stdClass $row
* @return AbuseFilterVariableHolder
*/
public static function getMoveVarsFromRCRow( $row ) {
@@ -2329,10 +2329,10 @@
/**
* @param $page
* @param $type
-* @param $title Title
-* @param $sk Skin
-* @param $args array
-* @param $filterWikilinks bool
+* @param Title $title
+* @param Skin $sk
+* @param array $args
+* @param bool $filterWikilinks
* @return String
*/
static function modifyActionText( $page, $type, $title, $sk, $args,
$filterWikilinks ) {
@@ -2445,7 +2445,7 @@
* the function returns with an empty string.
*
* @param Revision $revision a valid revision
-* @param $audience Integer: one of:
+* @param int $audience one of:
* Revision::FOR_PUBLIC to be displayed to all users
* Revision::FOR_THIS_USERto be displayed to the given user
* Revision::RAW get the text regardless of
permissions
@@ -2502,8 +2502,8 @@
/*
* Get the history ID of the first change to a given filter
*
-* @param $filterId integer: Filter id
-* @return integer|bool
+* @param int $filterId Filter id
+* @return int|bool
*/
public static function getFirstFilterChange( $filterID ) {
static $firstChanges = [];
diff --git a/includes/AbuseFilter.hooks.php b/includes/AbuseFilter.hooks.php
index b102341..fdba9a5 100644
--- a/includes/AbuseFilter.hooks.php
+++ b/includes/AbuseFilter.hooks.php
@@ -224,10 +224,10 @@
* @param bool $minoredit
* @param bool $watchthis
* @param string $sectionanchor
-* @param integer $flags
+* @param int $flags
* @param Revision $revision
-* @param $status Status
-* @param $baseRevId
+* @param Status $status
+* @param int $baseRevId
* @return bool
*/
public static function onPageContentSaveComplete(
@@ -288,8
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370696 )
Change subject: UEModulePDF: PDF Templates with new BlueSpice Logo
..
UEModulePDF: PDF Templates with new BlueSpice Logo
* modified page margin
* new header logo
Change-Id: I9e02c039a49f6dd2bf7323b46bfc4eea5a53e72f
---
M UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
7 files changed, 10 insertions(+), 12 deletions(-)
Approvals:
Robert Vogel: Looks good to me, approved
jenkins-bot: Verified
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
index ae0352d..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
Binary files differ
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
index fa576c1..4ed4eac 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
@@ -4,13 +4,12 @@
margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
background-size: 21cm;
- height: 3cm;
+ height: 3.5cm;
width: 21cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 2.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
@@ -33,7 +32,7 @@
}
.bs-runningheader table td{
- vertical-align: bottom;
+ vertical-align: bottom;
}
.bs-booktitle {
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
index f9d1bab..feb4545 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
@@ -11,7 +11,7 @@
'name' => 'BlueSpice Draft',
'author'=> 'Hallo Welt!',
'copyright' => 'Hallo Welt! GmbH',
- 'url' => 'http://www.hallowelt.biz',
+ 'url' => 'http://www.hallowelt.com',
'description' => 'This is the default BlueSpice Draft PDF
Template'
),
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
index 3eb0352..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
Binary files differ
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
index 5d9a0f2..99ee09b 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
@@ -15,7 +15,7 @@
*/
@page {
size: A4 landscape;
- margin: 3.5cm 1cm 2.5cm 2.5cm;
+ margin: 4.5cm 1cm 2.5cm 2.5cm;
background: url('../images/bs-page-background.png') repeat-x bottom;
background-size: 29.7cm 2.3cm;
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
index b7bd610..1aae1d4 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
@@ -1,16 +1,15 @@
/*** PAGE LAYOUT ELEMENTS ***/
#bs-runningheaderfix {
position: running( topleft );
- margin: 0 0 0 -0.5cm;
+ margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
- background-size: 21cm;
- height: 3cm;
+ background-size: 29.7cm;
+ height: 4cm;
width: 29.7cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 3.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
index 300a6f1..9035959 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/368128 )
Change subject: UEModulePDF: PDF Templates with new BlueSpice Logo
..
UEModulePDF: PDF Templates with new BlueSpice Logo
* modified page margin
* new header logo
Change-Id: I9e02c039a49f6dd2bf7323b46bfc4eea5a53e72f
---
M UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
7 files changed, 10 insertions(+), 12 deletions(-)
Approvals:
Robert Vogel: Looks good to me, approved
jenkins-bot: Verified
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
index ae0352d..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
Binary files differ
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
index fa576c1..4ed4eac 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
@@ -4,13 +4,12 @@
margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
background-size: 21cm;
- height: 3cm;
+ height: 3.5cm;
width: 21cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 2.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
@@ -33,7 +32,7 @@
}
.bs-runningheader table td{
- vertical-align: bottom;
+ vertical-align: bottom;
}
.bs-booktitle {
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
index f9d1bab..feb4545 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
@@ -11,7 +11,7 @@
'name' => 'BlueSpice Draft',
'author'=> 'Hallo Welt!',
'copyright' => 'Hallo Welt! GmbH',
- 'url' => 'http://www.hallowelt.biz',
+ 'url' => 'http://www.hallowelt.com',
'description' => 'This is the default BlueSpice Draft PDF
Template'
),
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
index 3eb0352..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
Binary files differ
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
index 5d9a0f2..99ee09b 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
@@ -15,7 +15,7 @@
*/
@page {
size: A4 landscape;
- margin: 3.5cm 1cm 2.5cm 2.5cm;
+ margin: 4.5cm 1cm 2.5cm 2.5cm;
background: url('../images/bs-page-background.png') repeat-x bottom;
background-size: 29.7cm 2.3cm;
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
index b7bd610..1aae1d4 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
@@ -1,16 +1,15 @@
/*** PAGE LAYOUT ELEMENTS ***/
#bs-runningheaderfix {
position: running( topleft );
- margin: 0 0 0 -0.5cm;
+ margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
- background-size: 21cm;
- height: 3cm;
+ background-size: 29.7cm;
+ height: 4cm;
width: 29.7cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 3.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
index 300a6f1..9035959 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
Robert Vogel has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370696 )
Change subject: UEModulePDF: PDF Templates with new BlueSpice Logo
..
UEModulePDF: PDF Templates with new BlueSpice Logo
* modified page margin
* new header logo
Change-Id: I9e02c039a49f6dd2bf7323b46bfc4eea5a53e72f
---
M UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
M UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
7 files changed, 10 insertions(+), 12 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions
refs/changes/96/370696/1
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
index ae0352d..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/images/bs-header.jpg
Binary files differ
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
index fa576c1..4ed4eac 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice/stylesheets/styles.css
@@ -4,13 +4,12 @@
margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
background-size: 21cm;
- height: 3cm;
+ height: 3.5cm;
width: 21cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 2.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
@@ -33,7 +32,7 @@
}
.bs-runningheader table td{
- vertical-align: bottom;
+ vertical-align: bottom;
}
.bs-booktitle {
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
index f9d1bab..feb4545 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Draft/template.php
@@ -11,7 +11,7 @@
'name' => 'BlueSpice Draft',
'author'=> 'Hallo Welt!',
'copyright' => 'Hallo Welt! GmbH',
- 'url' => 'http://www.hallowelt.biz',
+ 'url' => 'http://www.hallowelt.com',
'description' => 'This is the default BlueSpice Draft PDF
Template'
),
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
index 3eb0352..436a6f3 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/images/bs-header.jpg
Binary files differ
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
index 5d9a0f2..99ee09b 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/page.css
@@ -15,7 +15,7 @@
*/
@page {
size: A4 landscape;
- margin: 3.5cm 1cm 2.5cm 2.5cm;
+ margin: 4.5cm 1cm 2.5cm 2.5cm;
background: url('../images/bs-page-background.png') repeat-x bottom;
background-size: 29.7cm 2.3cm;
diff --git
a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
index b7bd610..1aae1d4 100644
--- a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
+++ b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/stylesheets/styles.css
@@ -1,16 +1,15 @@
/*** PAGE LAYOUT ELEMENTS ***/
#bs-runningheaderfix {
position: running( topleft );
- margin: 0 0 0 -0.5cm;
+ margin: 0 0 0 -2.5cm;
background: url('../images/bs-header.jpg') no-repeat 0 0;
- background-size: 21cm;
- height: 3cm;
+ background-size: 29.7cm;
+ height: 4cm;
width: 29.7cm;
- border-bottom: 1px solid;
}
#bs-runningheaderfix h1 {
- margin: 2.5cm 0 0 2.5cm;
+ margin: 3.3cm 0 0 2.5cm;
font-size: small;
color:#6c6d6f;
border: none;
diff --git a/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
b/UEModulePDF/data/PDFTemplates/BlueSpice_Landscape/template.php
index 300a6f1..9035959 100644
---
Legoktm has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370693 )
Change subject: Add manual page
..
Add manual page
Bug: T171396
Change-Id: I6d289acc3ef7cc0af13b8104ade4f9dcf64eb4e4
---
A uprightdiff.1
1 file changed, 59 insertions(+), 0 deletions(-)
Approvals:
Legoktm: Verified; Looks good to me, approved
diff --git a/uprightdiff.1 b/uprightdiff.1
new file mode 100644
index 000..f447325
--- /dev/null
+++ b/uprightdiff.1
@@ -0,0 +1,59 @@
+.TH UPRIGHTDIFF "1" "August 2017" "uprightdiff 1.0" "User Commands"
+.SH NAME
+uprightdiff \- examines the differences between two images
+.SH SYNOPSIS
+.B uprightdiff
+[\fI\,options\/\fR] \fI\, \/\fR
+.SH DESCRIPTION
+uprightdiff examines the differences between two images. It produces a visual
annotation
+and reports statistics.
+
+It is optimised for images which come from browser screenshots. It analyses
+the image for vertical motion, and annotates connected regions that have the
+same vertical displacement. Then it highlights any remaining ("residual")
+differences which are not explained by vertical motion on a pixel-by-pixel
+basis.
+.TP
+\fB\-\-help\fR
+Show help message and exit
+.TP
+\fB\-\-block\-size\fR arg
+Block size for initial search (default 16)
+.TP
+\fB\-\-window\-size\fR arg
+Initial range for vertical motion detection (default
+200)
+.TP
+\fB\-\-brush\-width\fR arg
+Brush width when heuristically expanding blocks. A
+higher value gives smoother motion regions. This
+should be an odd number. (default 9)
+.TP
+\fB\-\-outer\-hl\-window\fR arg
+The size of the outer square used for detecting
+isolated small features to highlight. This size
+defines what we mean by "isolated". It should be an
+odd number. (default 21)
+.TP
+\fB\-\-inner\-hl\-window\fR arg
+The size of the inner square used for detecting
+isolated small features to highlight. This size
+defines what we mean by "small". It should be an odd
+number. (default 5)
+.TP
+\fB\-\-intermediate\-dir\fR arg
+A directory where intermediate images should be
+placed. This is our equivalent of debug or trace
+output.
+.TP
+\fB\-v\fR [ \fB\-\-verbose\fR ]
+Write progress info to stderr.
+.TP
+\fB\-\-format\fR arg
+The output format for statistics, may be text (the
+default), json or none.
+.TP
+\fB\-t\fR [ \fB\-\-log\-timestamp\fR ]
+Annotate progress info with elapsed time.
+.SH AUTHOR
+Tim Starling
--
To view, visit https://gerrit.wikimedia.org/r/370693
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6d289acc3ef7cc0af13b8104ade4f9dcf64eb4e4
Gerrit-PatchSet: 1
Gerrit-Project: integration/uprightdiff
Gerrit-Branch: master
Gerrit-Owner: Legoktm
Gerrit-Reviewer: Hashar
Gerrit-Reviewer: Legoktm
Gerrit-Reviewer: Paladox
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Legoktm has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370695 )
Change subject: Fix uprightdiff debian-glue running on master
..
Fix uprightdiff debian-glue running on master
Change-Id: I877af37a0179af6f780317261eaf2243f17552ce
---
M zuul/layout.yaml
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/integration/config
refs/changes/95/370695/1
diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index 50dde13..539e8ff 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -725,7 +725,7 @@
#files: 'debian/.*'
skip-if:
- project: '^integration/uprightdiff$'
-branch: (?!debian$)
+branch: ^master$
skip-if:
- project: '^operations/software/cumin$'
branch: ^master$
--
To view, visit https://gerrit.wikimedia.org/r/370695
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I877af37a0179af6f780317261eaf2243f17552ce
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Legoktm
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370691 )
Change subject: Remove mediawiki-extension-* assert-no-mediawiki-errors
..
Remove mediawiki-extension-* assert-no-mediawiki-errors
This removes the assert-no-mediawiki-errors builder from:
mediawiki-extensions-hhvm-jessie
mediawiki-extensions-php55-trusty
mediawiki-extensions-php70-jessie
mediawiki-extensions-qunit-jessie
I have yet to see an extension pass this since deploying.
Partial revert of Ic8e06c4581da3441d4a5a7262583ed2b4fe7606a
Bug: T50002
Change-Id: Ib5535e0c7b544e333c0da7f440f81bff56e18d56
---
M jjb/mediawiki.yaml
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Thcipriani: Looks good to me, approved
jenkins-bot: Verified
diff --git a/jjb/mediawiki.yaml b/jjb/mediawiki.yaml
index 150706c..ae24e87 100644
--- a/jjb/mediawiki.yaml
+++ b/jjb/mediawiki.yaml
@@ -242,7 +242,6 @@
- composer-test-mwextension
- mw-fetch-composer-dev
- mw-run-phpunit-allexts
- - assert-no-mediawiki-errors
publishers:
- junit:
results: 'log/junit*.xml'
--
To view, visit https://gerrit.wikimedia.org/r/370691
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib5535e0c7b544e333c0da7f440f81bff56e18d56
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Thcipriani
Gerrit-Reviewer: Hashar
Gerrit-Reviewer: Paladox
Gerrit-Reviewer: Thcipriani
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370694 )
Change subject: Per spec return summary type
..
Per spec return summary type
Change-Id: I4e1c84cc490dc86551cae86bc4b96eafad42a89e
---
M routes/mobile-sections.js
1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps
refs/changes/94/370694/1
diff --git a/routes/mobile-sections.js b/routes/mobile-sections.js
index 6110b35..81a5f38 100644
--- a/routes/mobile-sections.js
+++ b/routes/mobile-sections.js
@@ -366,6 +366,7 @@
function buildSummary(req) {
return buildLeadObject(req, false).then((lead) => {
let summary = '';
+let type = 'standard';
let code = 200;
if (!lead) {
@@ -377,6 +378,7 @@
}
return {
code,
+type,
html: summary,
revision: lead.revision
};
--
To view, visit https://gerrit.wikimedia.org/r/370694
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4e1c84cc490dc86551cae86bc4b96eafad42a89e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Legoktm has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370693 )
Change subject: Add manual page
..
Add manual page
Bug: T171396
Change-Id: I6d289acc3ef7cc0af13b8104ade4f9dcf64eb4e4
---
A uprightdiff.1
1 file changed, 59 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/integration/uprightdiff
refs/changes/93/370693/1
diff --git a/uprightdiff.1 b/uprightdiff.1
new file mode 100644
index 000..f447325
--- /dev/null
+++ b/uprightdiff.1
@@ -0,0 +1,59 @@
+.TH UPRIGHTDIFF "1" "August 2017" "uprightdiff 1.0" "User Commands"
+.SH NAME
+uprightdiff \- examines the differences between two images
+.SH SYNOPSIS
+.B uprightdiff
+[\fI\,options\/\fR] \fI\, \/\fR
+.SH DESCRIPTION
+uprightdiff examines the differences between two images. It produces a visual
annotation
+and reports statistics.
+
+It is optimised for images which come from browser screenshots. It analyses
+the image for vertical motion, and annotates connected regions that have the
+same vertical displacement. Then it highlights any remaining ("residual")
+differences which are not explained by vertical motion on a pixel-by-pixel
+basis.
+.TP
+\fB\-\-help\fR
+Show help message and exit
+.TP
+\fB\-\-block\-size\fR arg
+Block size for initial search (default 16)
+.TP
+\fB\-\-window\-size\fR arg
+Initial range for vertical motion detection (default
+200)
+.TP
+\fB\-\-brush\-width\fR arg
+Brush width when heuristically expanding blocks. A
+higher value gives smoother motion regions. This
+should be an odd number. (default 9)
+.TP
+\fB\-\-outer\-hl\-window\fR arg
+The size of the outer square used for detecting
+isolated small features to highlight. This size
+defines what we mean by "isolated". It should be an
+odd number. (default 21)
+.TP
+\fB\-\-inner\-hl\-window\fR arg
+The size of the inner square used for detecting
+isolated small features to highlight. This size
+defines what we mean by "small". It should be an odd
+number. (default 5)
+.TP
+\fB\-\-intermediate\-dir\fR arg
+A directory where intermediate images should be
+placed. This is our equivalent of debug or trace
+output.
+.TP
+\fB\-v\fR [ \fB\-\-verbose\fR ]
+Write progress info to stderr.
+.TP
+\fB\-\-format\fR arg
+The output format for statistics, may be text (the
+default), json or none.
+.TP
+\fB\-t\fR [ \fB\-\-log\-timestamp\fR ]
+Annotate progress info with elapsed time.
+.SH AUTHOR
+Tim Starling
--
To view, visit https://gerrit.wikimedia.org/r/370693
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6d289acc3ef7cc0af13b8104ade4f9dcf64eb4e4
Gerrit-PatchSet: 1
Gerrit-Project: integration/uprightdiff
Gerrit-Branch: master
Gerrit-Owner: Legoktm
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370692 )
Change subject: Pages with empty lead sections should send a 204
..
Pages with empty lead sections should send a 204
If preview is an empty string we'll want to return a 204
per the spec.
Change-Id: Ib889905d647853e50afa2de656ee47882eee89d8
---
M routes/mobile-sections.js
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/mobileapps
refs/changes/92/370692/1
diff --git a/routes/mobile-sections.js b/routes/mobile-sections.js
index 6110b35..6a64dbd 100644
--- a/routes/mobile-sections.js
+++ b/routes/mobile-sections.js
@@ -370,7 +370,7 @@
if (!lead) {
return false;
-} else if (lead.contentmodel || lead.ns !== 0) {
+} else if (lead.contentmodel || lead.ns !== 0 || !lead.html) {
code = 204;
} else if (lead.intro) {
summary = transforms.summarize(lead.intro);
--
To view, visit https://gerrit.wikimedia.org/r/370692
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib889905d647853e50afa2de656ee47882eee89d8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/mobileapps
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Thcipriani has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370691 )
Change subject: Remove mediawiki-extension-* assert-no-mediawiki-errors
..
Remove mediawiki-extension-* assert-no-mediawiki-errors
This removes the assert-no-mediawiki-errors builder from:
mediawiki-extensions-hhvm-jessie
mediawiki-extensions-php55-trusty
mediawiki-extensions-php70-jessie
mediawiki-extensions-qunit-jessie
I have yet to see an extension pass this since deploying.
Partial revert of Ic8e06c4581da3441d4a5a7262583ed2b4fe7606a
Bug: T50002
Change-Id: Ib5535e0c7b544e333c0da7f440f81bff56e18d56
---
M jjb/mediawiki.yaml
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/integration/config
refs/changes/91/370691/1
diff --git a/jjb/mediawiki.yaml b/jjb/mediawiki.yaml
index 150706c..ae24e87 100644
--- a/jjb/mediawiki.yaml
+++ b/jjb/mediawiki.yaml
@@ -242,7 +242,6 @@
- composer-test-mwextension
- mw-fetch-composer-dev
- mw-run-phpunit-allexts
- - assert-no-mediawiki-errors
publishers:
- junit:
results: 'log/junit*.xml'
--
To view, visit https://gerrit.wikimedia.org/r/370691
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib5535e0c7b544e333c0da7f440f81bff56e18d56
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Thcipriani
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370689 )
Change subject: labservices: remove ferm rule that opens mysql to all internal
hosts
..
labservices: remove ferm rule that opens mysql to all internal hosts
This database is only used as a local cache, so there's no need
for access outside of the current host.
Bug: T169075
Change-Id: Ie10b9afa15cc9a78659df507e66614039c310f30
---
M modules/role/manifests/labs/dns.pp
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/89/370689/1
diff --git a/modules/role/manifests/labs/dns.pp
b/modules/role/manifests/labs/dns.pp
index e15edcb..0da4ff3 100644
--- a/modules/role/manifests/labs/dns.pp
+++ b/modules/role/manifests/labs/dns.pp
@@ -16,7 +16,6 @@
# install mysql locally on all dns servers
include role::mariadb::monitor::dba
-include role::mariadb::ferm
# Note: This will install mariadb but won't set up the
# pdns database. Manual steps are:
--
To view, visit https://gerrit.wikimedia.org/r/370689
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie10b9afa15cc9a78659df507e66614039c310f30
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Kji has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370688 )
Change subject: Modified self-links to determine display text identically to
non-self-links.
..
Modified self-links to determine display text identically to non-self-links.
Change-Id: I0b02457931e907450360d982d9b117288c23a807
---
M DisplayTitleHooks.php
M extension.json
2 files changed, 14 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/DisplayTitle
refs/changes/88/370688/1
diff --git a/DisplayTitleHooks.php b/DisplayTitleHooks.php
index e3bb63a..e21a39e 100644
--- a/DisplayTitleHooks.php
+++ b/DisplayTitleHooks.php
@@ -104,7 +104,19 @@
*/
public static function onSelfLinkBegin( Title $nt, &$html, &$trail,
&$prefix, &$ret ) {
- self::getDisplayTitle( $nt, $html );
+ //self::getDisplayTitle( $nt, $html );
+ //return true;
+ if ( isset( $html ) && is_string( $html ) ) {
+ $title = Title::newFromText( $html );
+ if ( !is_null( $title ) &&
+ $title->getText() === $nt->getText() &&
+ ( $title->getSubjectNsText() ===
$nt->getSubjectNsText() ||
+ $title->getSubjectNsText() === '' ) ) {
+ self::getDisplayTitle( $nt, $html );
+ }
+ } else {
+ self::getDisplayTitle( $nt, $html );
+ }
return true;
}
diff --git a/extension.json b/extension.json
index a4dd8df..1d85b6e 100644
--- a/extension.json
+++ b/extension.json
@@ -1,6 +1,6 @@
{
"name": "DisplayTitle",
- "version": "1.2",
+ "version": "1.3",
"author": [
"[https://www.mediawiki.org/wiki/User:Cindy.cicalese Cindy
Cicalese]",
"[https://www.semantic-mediawiki.org/wiki/User:Oetterer Tobias
Oetterer]"
--
To view, visit https://gerrit.wikimedia.org/r/370688
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0b02457931e907450360d982d9b117288c23a807
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/DisplayTitle
Gerrit-Branch: master
Gerrit-Owner: Kji
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370686 )
Change subject: Update toolinfo.json
..
Update toolinfo.json
Change-Id: I9f4d0139e2bc0352d49e25268d24f3fa3bad4e90
---
M public_html/toolinfo.json
1 file changed, 5 insertions(+), 5 deletions(-)
Approvals:
Zppix: Looks good to me, approved
jenkins-bot: Verified
diff --git a/public_html/toolinfo.json b/public_html/toolinfo.json
index 5c7ffd1..5216e6c 100644
--- a/public_html/toolinfo.json
+++ b/public_html/toolinfo.json
@@ -1,9 +1,9 @@
{
-"name" : "Zppixbot",
-"title" : "Zppixbot",
-"description" : "Experimental/dev version of PiagetBot",
+"name" : "ZppixBot",
+"title" : "ZppixBot",
+"description" : "IRC bot running Sopel",
"url" : "http://tools.wmflabs.org/zppixbot;,
"keywords" : "IRC, bot, piagetbot, piaget2, development, experimental,
Zppix",
-"author" : "tom29739 (bot operator), Zppix (Developer and experimental/dev
bot operator)",
-"repository" : "https://github.com/pix1234/sopel.git;
+"author" : "tom29739 (bot operator), Zppix (Developer and experimental/dev
bot operator)", "Reception123 (Developer)", "SwisterTwister",
+"repository" : "https://phabricator.wikimedia.org/diffusion/LTZB/;
}
--
To view, visit https://gerrit.wikimedia.org/r/370686
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9f4d0139e2bc0352d49e25268d24f3fa3bad4e90
Gerrit-PatchSet: 2
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Zppix
Gerrit-Reviewer: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Zppix has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370686 )
Change subject: Update toolinfo.json
..
Update toolinfo.json
Change-Id: I9f4d0139e2bc0352d49e25268d24f3fa3bad4e90
---
M public_html/toolinfo.json
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/ZppixBot
refs/changes/86/370686/2
diff --git a/public_html/toolinfo.json b/public_html/toolinfo.json
index 5c7ffd1..5216e6c 100644
--- a/public_html/toolinfo.json
+++ b/public_html/toolinfo.json
@@ -1,9 +1,9 @@
{
-"name" : "Zppixbot",
-"title" : "Zppixbot",
-"description" : "Experimental/dev version of PiagetBot",
+"name" : "ZppixBot",
+"title" : "ZppixBot",
+"description" : "IRC bot running Sopel",
"url" : "http://tools.wmflabs.org/zppixbot;,
"keywords" : "IRC, bot, piagetbot, piaget2, development, experimental,
Zppix",
-"author" : "tom29739 (bot operator), Zppix (Developer and experimental/dev
bot operator)",
-"repository" : "https://github.com/pix1234/sopel.git;
+"author" : "tom29739 (bot operator), Zppix (Developer and experimental/dev
bot operator)", "Reception123 (Developer)", "SwisterTwister",
+"repository" : "https://phabricator.wikimedia.org/diffusion/LTZB/;
}
--
To view, visit https://gerrit.wikimedia.org/r/370686
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9f4d0139e2bc0352d49e25268d24f3fa3bad4e90
Gerrit-PatchSet: 2
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Dbrant has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370685 )
Change subject: Add onboarding feed card for Offline Library.
..
Add onboarding feed card for Offline Library.
Also refine the logic of showing/hiding the offline onboarding card vs.
the offline notification card.
Bug: T172187
Bug: T172132
Change-Id: I1ea46e2867516aebcabfa15c1bfad2615545438b
---
M app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
M app/src/main/java/org/wikipedia/feed/FeedFragment.java
M app/src/main/java/org/wikipedia/feed/model/CardType.java
A app/src/main/java/org/wikipedia/feed/onboarding/OfflineOnboardingCard.java
M app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCard.java
M app/src/main/java/org/wikipedia/feed/onboarding/OnboardingClient.java
M app/src/main/java/org/wikipedia/settings/PrefsIoUtil.java
M app/src/main/res/values/preference_keys.xml
M app/src/main/res/values/strings.xml
9 files changed, 77 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia
refs/changes/85/370685/1
diff --git a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
index 5de6cb7..2fa54e8 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
@@ -12,6 +12,7 @@
import org.wikipedia.feed.onboarding.OnboardingClient;
import org.wikipedia.feed.random.RandomClient;
import org.wikipedia.feed.searchbar.SearchClient;
+import org.wikipedia.offline.OfflineManager;
import org.wikipedia.util.DeviceUtil;
import static org.wikipedia.util.ReleaseUtil.isPreBetaRelease;
@@ -27,7 +28,7 @@
boolean online = DeviceUtil.isOnline();
conditionallyAddPendingClient(new SearchClient(), age == 0);
-conditionallyAddPendingClient(new OfflineCompilationClient(), age == 0
&& !online && isPreBetaRelease());
+conditionallyAddPendingClient(new OfflineCompilationClient(), age == 0
&& !online && OfflineManager.hasCompilation() && isPreBetaRelease());
conditionallyAddPendingClient(new OnboardingClient(), age == 0);
conditionallyAddPendingClient(new AnnouncementClient(), age == 0 &&
online);
conditionallyAddPendingClient(new AggregatedFeedContentClient(),
online);
diff --git a/app/src/main/java/org/wikipedia/feed/FeedFragment.java
b/app/src/main/java/org/wikipedia/feed/FeedFragment.java
index 09d2701..59cdc96 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedFragment.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedFragment.java
@@ -425,7 +425,9 @@
@Override
public void onOnboardingPositiveAction(@NonNull Card card, @NonNull
OnboardingCard.OnboardingAction action) {
-
+if (action == OnboardingCard.OnboardingAction.OFFLINE_LIBRARY) {
+onViewCompilations();
+}
}
}
diff --git a/app/src/main/java/org/wikipedia/feed/model/CardType.java
b/app/src/main/java/org/wikipedia/feed/model/CardType.java
index f5e8c1e..1310e9a 100644
--- a/app/src/main/java/org/wikipedia/feed/model/CardType.java
+++ b/app/src/main/java/org/wikipedia/feed/model/CardType.java
@@ -13,6 +13,7 @@
import org.wikipedia.feed.news.NewsListCardView;
import org.wikipedia.feed.offline.OfflineCardView;
import org.wikipedia.feed.offline.OfflineCompilationCardView;
+import org.wikipedia.feed.onboarding.OnboardingCardView;
import org.wikipedia.feed.progress.ProgressCardView;
import org.wikipedia.feed.random.RandomCardView;
import org.wikipedia.feed.searchbar.SearchCardView;
@@ -90,6 +91,11 @@
return new OfflineCompilationCardView(ctx);
}
},
+ONBOARDING_OFFLINE(17) {
+@NonNull @Override public FeedCardView newView(@NonNull Context
ctx) {
+return new OnboardingCardView(ctx);
+}
+},
OFFLINE(98) {
@NonNull @Override public FeedCardView newView(@NonNull Context
ctx) {
return new OfflineCardView(ctx);
diff --git
a/app/src/main/java/org/wikipedia/feed/onboarding/OfflineOnboardingCard.java
b/app/src/main/java/org/wikipedia/feed/onboarding/OfflineOnboardingCard.java
new file mode 100644
index 000..475cea9
--- /dev/null
+++ b/app/src/main/java/org/wikipedia/feed/onboarding/OfflineOnboardingCard.java
@@ -0,0 +1,43 @@
+package org.wikipedia.feed.onboarding;
+
+import android.net.Uri;
+import android.support.annotation.NonNull;
+
+import org.wikipedia.R;
+import org.wikipedia.feed.model.CardType;
+import org.wikipedia.offline.OfflineManager;
+import org.wikipedia.util.DeviceUtil;
+
+public class OfflineOnboardingCard extends OnboardingCard {
+@NonNull @Override public CardType type() {
+return CardType.ONBOARDING_OFFLINE;
+}
+
+@Override public boolean shouldShow() {
+return super.shouldShow() && DeviceUtil.isOnline() &&
Dbrant has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370684 )
Change subject: Framework for onboarding cards in the Explore feed.
..
Framework for onboarding cards in the Explore feed.
Bug: T172187
Change-Id: Ic04249a6f655381574dcf2902b36bbd1067f10e1
---
M app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
M app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java
M app/src/main/java/org/wikipedia/feed/FeedFragment.java
M app/src/main/java/org/wikipedia/feed/model/Card.java
A app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCard.java
A app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCardView.java
A app/src/main/java/org/wikipedia/feed/onboarding/OnboardingClient.java
M app/src/main/java/org/wikipedia/feed/view/FeedAdapter.java
A app/src/main/res/layout/view_card_onboarding.xml
9 files changed, 183 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia
refs/changes/84/370684/1
diff --git a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
index bc1f2d0..5de6cb7 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedCoordinator.java
@@ -9,6 +9,7 @@
import org.wikipedia.feed.continuereading.ContinueReadingClient;
import org.wikipedia.feed.mainpage.MainPageClient;
import org.wikipedia.feed.offline.OfflineCompilationClient;
+import org.wikipedia.feed.onboarding.OnboardingClient;
import org.wikipedia.feed.random.RandomClient;
import org.wikipedia.feed.searchbar.SearchClient;
import org.wikipedia.util.DeviceUtil;
@@ -27,6 +28,7 @@
conditionallyAddPendingClient(new SearchClient(), age == 0);
conditionallyAddPendingClient(new OfflineCompilationClient(), age == 0
&& !online && isPreBetaRelease());
+conditionallyAddPendingClient(new OnboardingClient(), age == 0);
conditionallyAddPendingClient(new AnnouncementClient(), age == 0 &&
online);
conditionallyAddPendingClient(new AggregatedFeedContentClient(),
online);
addPendingClient(new ContinueReadingClient());
diff --git a/app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java
b/app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java
index cddf45a..f5fc669 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedCoordinatorBase.java
@@ -96,12 +96,14 @@
int position = cards.indexOf(card);
addHiddenCard(card);
removeCard(card, position);
+card.onDismiss();
return position;
}
public void undoDismissCard(@NonNull Card card, int position) {
unHideCard(card);
insertCard(card, position);
+card.onRestore();
}
void retryFromOffline(@NonNull WikiSite wiki) {
diff --git a/app/src/main/java/org/wikipedia/feed/FeedFragment.java
b/app/src/main/java/org/wikipedia/feed/FeedFragment.java
index 28b2841..09d2701 100644
--- a/app/src/main/java/org/wikipedia/feed/FeedFragment.java
+++ b/app/src/main/java/org/wikipedia/feed/FeedFragment.java
@@ -27,6 +27,7 @@
import org.wikipedia.feed.image.FeaturedImageCard;
import org.wikipedia.feed.model.Card;
import org.wikipedia.feed.news.NewsItemCard;
+import org.wikipedia.feed.onboarding.OnboardingCard;
import org.wikipedia.feed.random.RandomCardView;
import org.wikipedia.feed.view.FeedAdapter;
import org.wikipedia.feed.view.FeedView;
@@ -421,6 +422,11 @@
public void onViewCompilations() {
startActivity(LocalCompilationsActivity.newIntent(getContext()));
}
+
+@Override
+public void onOnboardingPositiveAction(@NonNull Card card, @NonNull
OnboardingCard.OnboardingAction action) {
+
+}
}
private class FeedScrollListener extends RecyclerView.OnScrollListener {
diff --git a/app/src/main/java/org/wikipedia/feed/model/Card.java
b/app/src/main/java/org/wikipedia/feed/model/Card.java
index 4e325b0..b0dfdc5 100644
--- a/app/src/main/java/org/wikipedia/feed/model/Card.java
+++ b/app/src/main/java/org/wikipedia/feed/model/Card.java
@@ -25,6 +25,12 @@
@NonNull public abstract CardType type();
+public void onDismiss() {
+}
+
+public void onRestore() {
+}
+
public String getHideKey() {
return Long.toString(type().code() + dismissHashCode());
}
diff --git
a/app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCard.java
b/app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCard.java
new file mode 100644
index 000..020fe31
--- /dev/null
+++ b/app/src/main/java/org/wikipedia/feed/onboarding/OnboardingCard.java
@@ -0,0 +1,24 @@
+package org.wikipedia.feed.onboarding;
+
+import android.net.Uri;
+import android.support.annotation.DrawableRes;
+import android.support.annotation.NonNull;
+import
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370679 )
Change subject: New release (v2) of ZppixBot
..
New release (v2) of ZppixBot
Change-Id: I4af24807ce0d09ea1114954f89dcb0f4b8df2deb
---
M public_html/index.php
1 file changed, 4 insertions(+), 2 deletions(-)
Approvals:
Zppix: Looks good to me, approved
jenkins-bot: Verified
diff --git a/public_html/index.php b/public_html/index.php
index 2aa56ed..5285058 100644
--- a/public_html/index.php
+++ b/public_html/index.php
@@ -3,6 +3,8 @@
If you need ZppixBot in your channel, please contact Zppix.
- Inital release of ZppixBot is here!
- For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>the release
tag !!!
+ The second release of ZppixBot is now here!
+ For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v2;>the release
tag !!!
+Previous releases:
+https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>v1 (30
July 2017)
--
To view, visit https://gerrit.wikimedia.org/r/370679
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4af24807ce0d09ea1114954f89dcb0f4b8df2deb
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
Gerrit-Reviewer: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Ayounsi has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370658 )
Change subject: Remove pfw-codfw from Smokeping, Rancid, Torrus, Icinga
..
Remove pfw-codfw from Smokeping, Rancid, Torrus, Icinga
Bug: T171970
Change-Id: If68ba22cbbbc765bdd9faab7fc3eb45f5dfaf89f
---
M modules/netops/manifests/monitoring.pp
M modules/rancid/files/core/router.db
M modules/role/manifests/torrus.pp
M modules/smokeping/files/config.d/Targets
4 files changed, 0 insertions(+), 8 deletions(-)
Approvals:
jenkins-bot: Verified
Ayounsi: Looks good to me, approved
diff --git a/modules/netops/manifests/monitoring.pp
b/modules/netops/manifests/monitoring.pp
index f493589..79f4f11 100644
--- a/modules/netops/manifests/monitoring.pp
+++ b/modules/netops/manifests/monitoring.pp
@@ -30,7 +30,6 @@
'cr1-codfw' => { ipv4 => '208.80.153.192', ipv6 =>
'2620:0:860:::1', bgp => true, },
'cr2-codfw' => { ipv4 => '208.80.153.193', ipv6 =>
'2620:0:860:::2', bgp => true, },
'mr1-codfw' => { ipv4 => '208.80.153.196', ipv6 =>
'2620:0:860:::6', },
-'pfw-codfw' => { ipv4 => '208.80.153.195', },
'pfw3-codfw' => { ipv4 => '208.80.153.197', parents => ['cr1-codfw',
'cr2-codfw'], bgp => true, alarms => false, },
# eqdfw
'cr1-eqdfw' => { ipv4 => '208.80.153.198', ipv6 =>
'2620:0:860:::5', bgp => true, },
diff --git a/modules/rancid/files/core/router.db
b/modules/rancid/files/core/router.db
index 49c73de..b93ec25 100644
--- a/modules/rancid/files/core/router.db
+++ b/modules/rancid/files/core/router.db
@@ -15,7 +15,6 @@
lab-ex4200-1.mgmt.eqiad.wmnet;juniper;down;
lab-ex4500-1.mgmt.eqiad.wmnet;juniper;down;
pfw1-eqiad.wikimedia.org;juniper;up;
-pfw1-codfw.wikimedia.org;juniper;up;
pfw3-codfw.wikimedia.org;juniper;up;
fasw-c-codfw.mgmt.codfw.wmnet;juniper;up;
cr1-ulsfo.wikimedia.org;juniper;up;
diff --git a/modules/role/manifests/torrus.pp b/modules/role/manifests/torrus.pp
index 0ac8b38..6df8763 100644
--- a/modules/role/manifests/torrus.pp
+++ b/modules/role/manifests/torrus.pp
@@ -17,7 +17,6 @@
'cr2-eqiad.wikimedia.org',
'cr2-knams.wikimedia.org',
'cr2-ulsfo.wikimedia.org',
-'pfw1-codfw.wikimedia.org',
'pfw1-eqiad.wikimedia.org',
]
diff --git a/modules/smokeping/files/config.d/Targets
b/modules/smokeping/files/config.d/Targets
index e0059da..4b152b5 100644
--- a/modules/smokeping/files/config.d/Targets
+++ b/modules/smokeping/files/config.d/Targets
@@ -214,11 +214,6 @@
menu = frack
title = Fundraising Infrastructure
-+++ pfw-codfw
-menu = pfw-codfw
-title = pfw-codfw.wikimedia.org
-host = pfw-codfw.wikimedia.org
-
+++ pfw3-codfw
menu = pfw3-codfw
title = pfw3-codfw.wikimedia.org
--
To view, visit https://gerrit.wikimedia.org/r/370658
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If68ba22cbbbc765bdd9faab7fc3eb45f5dfaf89f
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ayounsi
Gerrit-Reviewer: Ayounsi
Gerrit-Reviewer: Giuseppe Lavagetto
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
WMDE-Fisch has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370682 )
Change subject: Suppress more phan issues
..
Suppress more phan issues
I get this issues when run locally following instructions in
https://www.mediawiki.org/wiki/Continuous_integration/Phan#Manual_2
Change-Id: Ib5125166c2fc073c93d3116d16a4680edf20412e
---
M tests/phan/config.php
1 file changed, 14 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/82/370682/1
diff --git a/tests/phan/config.php b/tests/phan/config.php
index 8a82d74..970646b 100644
--- a/tests/phan/config.php
+++ b/tests/phan/config.php
@@ -297,6 +297,10 @@
* to this black-list to inhibit them from being reported.
*/
'suppress_issue_types' => [
+ // approximate error count: 16
+ "PhanCommentParamOnEmptyParamList",
+ // approximate error count: 35
+ "PhanCommentParamWithoutRealParam",
// approximate error count: 8
"PhanDeprecatedClass",
// approximate error count: 415
@@ -325,12 +329,18 @@
"PhanTypeMismatchArgument",
// approximate error count: 13
"PhanTypeMismatchArgumentInternal",
+ // approximate error count: 117
+ "PhanTypeMismatchDeclaredParamNullable",
// approximate error count: 14
"PhanTypeMismatchForeach",
// approximate error count: 56
"PhanTypeMismatchProperty",
+ // approximate error count: 7
+ "PhanTypeMismatchDeclaredParam",
// approximate error count: 74
"PhanTypeMismatchReturn",
+ // approximate error count: 2
+ "PhanMisspelledAnnotation",
// approximate error count: 11
"PhanTypeMissingReturn",
// approximate error count: 5
@@ -341,8 +351,12 @@
"PhanUndeclaredMethod",
// approximate error count: 1224
"PhanUndeclaredProperty",
+ // approximate error count: 13
+ "PhanUndeclaredVariable",
// approximate error count: 3
"PhanUndeclaredStaticMethod",
+ // approximate error count: 62
+ "PhanUndeclaredVariableDim",
],
/**
--
To view, visit https://gerrit.wikimedia.org/r/370682
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib5125166c2fc073c93d3116d16a4680edf20412e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: WMDE-Fisch
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Raimond Spekking has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370681 )
Change subject: [DebugTemplates] Remove key, message has content now
..
[DebugTemplates] Remove key, message has content now
Change-Id: I896311ad1351f2a6809265e34381af872c985099
---
M groups/MediaWiki/mediawiki-extensions.txt
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/translatewiki
refs/changes/81/370681/1
diff --git a/groups/MediaWiki/mediawiki-extensions.txt
b/groups/MediaWiki/mediawiki-extensions.txt
index f996f81..79ac1a4 100644
--- a/groups/MediaWiki/mediawiki-extensions.txt
+++ b/groups/MediaWiki/mediawiki-extensions.txt
@@ -822,7 +822,6 @@
Debug Templates
aliasfile = DebugTemplates/DebugTemplates.alias.php
-optional = apihelp-expandframe-example-1
Delete Batch
aliasfile = DeleteBatch/DeleteBatch.alias.php
--
To view, visit https://gerrit.wikimedia.org/r/370681
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I896311ad1351f2a6809265e34381af872c985099
Gerrit-PatchSet: 1
Gerrit-Project: translatewiki
Gerrit-Branch: master
Gerrit-Owner: Raimond Spekking
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Raimond Spekking has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370681 )
Change subject: [DebugTemplates] Remove key, message has content now
..
[DebugTemplates] Remove key, message has content now
Change-Id: I896311ad1351f2a6809265e34381af872c985099
---
M groups/MediaWiki/mediawiki-extensions.txt
1 file changed, 0 insertions(+), 1 deletion(-)
Approvals:
Raimond Spekking: Verified; Looks good to me, approved
diff --git a/groups/MediaWiki/mediawiki-extensions.txt
b/groups/MediaWiki/mediawiki-extensions.txt
index f996f81..79ac1a4 100644
--- a/groups/MediaWiki/mediawiki-extensions.txt
+++ b/groups/MediaWiki/mediawiki-extensions.txt
@@ -822,7 +822,6 @@
Debug Templates
aliasfile = DebugTemplates/DebugTemplates.alias.php
-optional = apihelp-expandframe-example-1
Delete Batch
aliasfile = DeleteBatch/DeleteBatch.alias.php
--
To view, visit https://gerrit.wikimedia.org/r/370681
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I896311ad1351f2a6809265e34381af872c985099
Gerrit-PatchSet: 1
Gerrit-Project: translatewiki
Gerrit-Branch: master
Gerrit-Owner: Raimond Spekking
Gerrit-Reviewer: Raimond Spekking
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370667 )
Change subject: QA: Relax toast notification QA steps
..
QA: Relax toast notification QA steps
It seems trying to test both the steps can cause
false positives. Relaxing these checks seems to make
our Jenkins job happy without breaking the tests themselves
Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
---
M tests/browser/features/step_definitions/common_article_steps.rb
1 file changed, 2 insertions(+), 3 deletions(-)
Approvals:
jenkins-bot: Verified
Jdlrobson: Looks good to me, approved
diff --git a/tests/browser/features/step_definitions/common_article_steps.rb
b/tests/browser/features/step_definitions/common_article_steps.rb
index df5d4fd..c08ee3a 100644
--- a/tests/browser/features/step_definitions/common_article_steps.rb
+++ b/tests/browser/features/step_definitions/common_article_steps.rb
@@ -32,14 +32,13 @@
# Minerva loads mediawiki.notify at startup which defers the loading of
this module
# We must wait until the lazy loading has happened before checking for the
toast (T170890)
page.wait_until_rl_module_ready('mediawiki.notification')
+expect(page.notification_area_element.when_visible).to be_visible
end
- expect(on(ArticlePage).notification_area_element.when_visible).to be_visible
- expect(on(ArticlePage).toast_element.when_visible).to be_visible
end
Then(/^I should see a toast with message "(.+)"$/) do |msg|
- step "I should see a toast notification"
on(ArticlePage) do |page|
+page.wait_until_rl_module_ready('mediawiki.notification')
page.wait_until do
page.toast_element.when_present.text.include? msg
end
--
To view, visit https://gerrit.wikimedia.org/r/370667
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/MinervaNeue
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
Gerrit-Reviewer: Jdlrobson
Gerrit-Reviewer: Pmiazga
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
MarkTraceur has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370680 )
Change subject: Skip extension check of input file
..
Skip extension check of input file
This is necessary for Thumbor support, since the temporary file for
STL input has no extension.
Change-Id: Iae903c78f8532f16d5d0e39131c0f007fd0577dd
---
M 3d2png.js
1 file changed, 4 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/3d2png refs/changes/80/370680/1
diff --git a/3d2png.js b/3d2png.js
index da29b88..d63a488 100755
--- a/3d2png.js
+++ b/3d2png.js
@@ -64,12 +64,10 @@
* @param {string} filePath Full path to the file
* @returns {THREE.Loader} File loader
*/
-ThreeDtoPNG.prototype.getLoader = function( filePath ) {
- if ( filePath.toLowerCase().endsWith( '.stl' ) ) {
- return new THREE.STLLoader();
- }
-
- throw 'Unexpected model file extension, only STL is supported';
+ThreeDtoPNG.prototype.getLoader = function() {
+ // TODO XXX if more file formats are supported later, need a command
line option
+ // to signify the expected format of the input file.
+ return new THREE.STLLoader();
};
/**
--
To view, visit https://gerrit.wikimedia.org/r/370680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iae903c78f8532f16d5d0e39131c0f007fd0577dd
Gerrit-PatchSet: 1
Gerrit-Project: 3d2png
Gerrit-Branch: master
Gerrit-Owner: MarkTraceur
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Reception123 has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370679 )
Change subject: New release (v2) of ZppixBot
..
New release (v2) of ZppixBot
Change-Id: I4af24807ce0d09ea1114954f89dcb0f4b8df2deb
---
M public_html/index.php
1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/ZppixBot
refs/changes/79/370679/1
diff --git a/public_html/index.php b/public_html/index.php
index 2aa56ed..5285058 100644
--- a/public_html/index.php
+++ b/public_html/index.php
@@ -3,6 +3,8 @@
If you need ZppixBot in your channel, please contact Zppix.
- Inital release of ZppixBot is here!
- For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>the release
tag !!!
+ The second release of ZppixBot is now here!
+ For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v2;>the release
tag !!!
+Previous releases:
+https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>v1 (30
July 2017)
--
To view, visit https://gerrit.wikimedia.org/r/370679
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4af24807ce0d09ea1114954f89dcb0f4b8df2deb
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Niedzielski has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370678 )
Change subject: Marvin: add NPM template
..
Marvin: add NPM template
Configure the new Marvin project with the stock NPM template.
https://phabricator.wikimedia.org/diffusion/MARV/
Change-Id: Icdad2bb49e0c5993dac8359f4c269225907a6617
---
M zuul/layout.yaml
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/integration/config
refs/changes/78/370678/1
diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index 50dde13..a25edde 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -1301,6 +1301,10 @@
test:
- debian-glue-non-voting
+ - name: marvin
+template:
+ - name: npm
+
- name: mapdata
test:
- mapdata-npm-node-6-jessie
--
To view, visit https://gerrit.wikimedia.org/r/370678
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icdad2bb49e0c5993dac8359f4c269225907a6617
Gerrit-PatchSet: 1
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Niedzielski
Gerrit-Reviewer: Sniedzielski
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370677 )
Change subject: nova fullstack: move to new jessie image
..
nova fullstack: move to new jessie image
Change-Id: If7bd0416afe78082f0c287a4913c2c052d88f754
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Andrew Bogott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index 44cc503..d38326c 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.8-jessie',
+default='debian-8.9-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index f5b787b..a5ba920 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.8-jessie \
+ --image debian-8.9-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
--
To view, visit https://gerrit.wikimedia.org/r/370677
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If7bd0416afe78082f0c287a4913c2c052d88f754
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
Gerrit-Reviewer: Andrew Bogott
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370677 )
Change subject: nova fullstack: move to new jessie image
..
nova fullstack: move to new jessie image
Change-Id: If7bd0416afe78082f0c287a4913c2c052d88f754
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/77/370677/1
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index 44cc503..d38326c 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.8-jessie',
+default='debian-8.9-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index f5b787b..a5ba920 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.8-jessie \
+ --image debian-8.9-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
--
To view, visit https://gerrit.wikimedia.org/r/370677
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If7bd0416afe78082f0c287a4913c2c052d88f754
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370676 )
Change subject: Revert "nova fullstack: use new jessie-8.9 image"
..
Revert "nova fullstack: use new jessie-8.9 image"
This reverts commit e9bab61a7d7ea77c7961466ca2522cfbb26bab29.
Change-Id: I148c491da18b4be3ef733ddd5e7f0db2554ea13f
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
M modules/varnishkafka
3 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Andrew Bogott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index d38326c..44cc503 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.9-jessie',
+default='debian-8.8-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index a5ba920..f5b787b 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.9-jessie \
+ --image debian-8.8-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
diff --git a/modules/varnishkafka b/modules/varnishkafka
index 8822159..a3651c8 16
--- a/modules/varnishkafka
+++ b/modules/varnishkafka
@@ -1 +1 @@
-Subproject commit 8822159469238c14aedfaf01d15e27658199a157
+Subproject commit a3651c852cfc00d6c3a81a0608b3328b5fb6159c
--
To view, visit https://gerrit.wikimedia.org/r/370676
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I148c491da18b4be3ef733ddd5e7f0db2554ea13f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
Gerrit-Reviewer: Alex Monk
Gerrit-Reviewer: Andrew Bogott
Gerrit-Reviewer: Volans
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Hello Alex Monk, jenkins-bot, Volans,
I'd like you to do a code review. Please visit
https://gerrit.wikimedia.org/r/370676
to review the following change.
Change subject: Revert "nova fullstack: use new jessie-8.9 image"
..
Revert "nova fullstack: use new jessie-8.9 image"
This reverts commit e9bab61a7d7ea77c7961466ca2522cfbb26bab29.
Change-Id: I148c491da18b4be3ef733ddd5e7f0db2554ea13f
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
M modules/varnishkafka
3 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/76/370676/1
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index d38326c..44cc503 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.9-jessie',
+default='debian-8.8-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index a5ba920..f5b787b 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.9-jessie \
+ --image debian-8.8-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
diff --git a/modules/varnishkafka b/modules/varnishkafka
index 8822159..a3651c8 16
--- a/modules/varnishkafka
+++ b/modules/varnishkafka
@@ -1 +1 @@
-Subproject commit 8822159469238c14aedfaf01d15e27658199a157
+Subproject commit a3651c852cfc00d6c3a81a0608b3328b5fb6159c
--
To view, visit https://gerrit.wikimedia.org/r/370676
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I148c491da18b4be3ef733ddd5e7f0db2554ea13f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
Gerrit-Reviewer: Alex Monk
Gerrit-Reviewer: Volans
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370675 )
Change subject: Add links for release tag and my user page.
..
Add links for release tag and my user page.
Change-Id: I5990650f4195e0cfe5473257ace40c03f7af1f9f
---
M public_html/contribs.php
M public_html/index.php
2 files changed, 5 insertions(+), 5 deletions(-)
Approvals:
Zppix: Looks good to me, approved
jenkins-bot: Verified
diff --git a/public_html/contribs.php b/public_html/contribs.php
index 068757a..359f449 100644
--- a/public_html/contribs.php
+++ b/public_html/contribs.php
@@ -2,4 +2,4 @@
Zppix- Operator
tom29739- Release version bot operator, developer
SwisterTwister- Zppixbot web design, and friend.
-Reception123 - Helps with new features
+https://meta.miraheze.org/wiki/User:Reception123;>Reception123 -
Helps with new features
diff --git a/public_html/index.php b/public_html/index.php
index fc37f65..2aa56ed 100644
--- a/public_html/index.php
+++ b/public_html/index.php
@@ -1,8 +1,8 @@
-For more information, click here.
-List of contributors, click here
+For more information, click here.
+List of contributors, click here
+If you need ZppixBot in your channel, please contact Zppix.
Inital release of ZppixBot is here!
- For more information please check out,
https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1 !!!
+ For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>the release
tag !!!
-If you need ZppixBot in your channel, please contact Zppix.
--
To view, visit https://gerrit.wikimedia.org/r/370675
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5990650f4195e0cfe5473257ace40c03f7af1f9f
Gerrit-PatchSet: 4
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
Gerrit-Reviewer: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Reception123 has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370675 )
Change subject: Add links for release tag and my talkpage.
..
Add links for release tag and my talkpage.
Change-Id: I5990650f4195e0cfe5473257ace40c03f7af1f9f
---
M public_html/contribs.php
M public_html/index.php
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/ZppixBot
refs/changes/75/370675/1
diff --git a/public_html/contribs.php b/public_html/contribs.php
index 068757a..a0e11ea 100644
--- a/public_html/contribs.php
+++ b/public_html/contribs.php
@@ -2,4 +2,4 @@
Zppix- Operator
tom29739- Release version bot operator, developer
SwisterTwister- Zppixbot web design, and friend.
-Reception123 - Helps with new features
+https://meta.miraheze.org/wiki/User:Reception123;>Reception123 - Helps
with new features
diff --git a/public_html/index.php b/public_html/index.php
index fc37f65..7f9f595 100644
--- a/public_html/index.php
+++ b/public_html/index.php
@@ -3,6 +3,6 @@
Inital release of ZppixBot is here!
- For more information please check out,
https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1 !!!
+ For more information please check out https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1;>the release
tag !!!
If you need ZppixBot in your channel, please contact Zppix.
--
To view, visit https://gerrit.wikimedia.org/r/370675
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I5990650f4195e0cfe5473257ace40c03f7af1f9f
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Gehel has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370674 )
Change subject: Ensure that wdqs-updater is not throttled.
..
Ensure that wdqs-updater is not throttled.
We trust the updater to not overload blazegraph, it should not be throttled.
The implementation relies on a HTTP header being set by nginx for all proxied
requests. The updater does not go through nginx but talks directly to
blazegraph, so the header set by nginx is not present.
Change-Id: Ic6ba59990d6e74f05b7ac51288fb1b21f85c9da0
---
M
blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
M
blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
M
blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlerTest.java
M war/src/main/webapp/WEB-INF/web.xml
4 files changed, 71 insertions(+), 11 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/wikidata/query/rdf
refs/changes/74/370674/1
diff --git
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
index da237d2..30c1186 100644
---
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
+++
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/Throttler.java
@@ -38,6 +38,17 @@
private final Callable createThrottlingState;
/**
+ * Throttling is only enabled if this header is set.
+ *
+ * This can be used to throttle only request coming through a revers proxy,
+ * which will set this specific header. Only the presence of the header is
+ * checked, not its value.
+ *
+ * If null, all requests will be throttled.
+ */
+private String enableThrottlingIfHeader;
+
+/**
* Constructor.
*
* Note that a bucket represent our approximation of a single client.
@@ -49,16 +60,19 @@
* when we start tracking a specific client
* @param stateStore the cache in which we store the per client state of
* throttling
+ * @param enableThrottlingIfHeader throttling is only enabled if this
header is present.
*/
public Throttler(
Duration requestTimeThreshold,
Bucketing bucketing,
Callable createThrottlingState,
-Cache stateStore) {
+Cache stateStore,
+String enableThrottlingIfHeader) {
this.requestTimeThreshold = requestTimeThreshold;
this.bucketing = bucketing;
this.state = stateStore;
this.createThrottlingState = createThrottlingState;
+this.enableThrottlingIfHeader = enableThrottlingIfHeader;
}
/**
@@ -68,10 +82,20 @@
* @return true if the request should be throttled
*/
public boolean isThrottled(HttpServletRequest request) {
+if (shouldBypassThrottling(request)) {
+return false;
+}
ThrottlingState throttlingState =
state.getIfPresent(bucketing.bucket(request));
if (throttlingState == null) return false;
return throttlingState.isThrottled();
+}
+
+private boolean shouldBypassThrottling(HttpServletRequest request) {
+if (enableThrottlingIfHeader == null) {
+return false;
+}
+return request.getHeader(enableThrottlingIfHeader) == null;
}
/**
@@ -81,6 +105,9 @@
* @param elapsed how long that request took
*/
public void success(HttpServletRequest request, Duration elapsed) {
+if (shouldBypassThrottling(request)) {
+return;
+}
try {
B bucket = bucketing.bucket(request);
ThrottlingState throttlingState;
@@ -105,6 +132,9 @@
* @param elapsed how long that request took
*/
public void failure(HttpServletRequest request, Duration elapsed) {
+if (shouldBypassThrottling(request)) {
+return;
+}
try {
ThrottlingState throttlingState =
state.get(bucketing.bucket(request), createThrottlingState);
diff --git
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
index 72a1198..9477ed4 100644
---
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
+++
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/throttling/ThrottlingFilter.java
@@ -96,14 +96,13 @@
/**
* Initialise the filter.
*
- * Configuration is loaded from TODO.
+ * See {@link ThrottlingFilter#loadStringParam(String, FilterConfig)} for
+ * the details of where the configuration is loaded from.
+ *
Andrew Bogott has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370673 )
Change subject: nova fullstack: use new jessie-8.9 image
..
nova fullstack: use new jessie-8.9 image
Change-Id: Ia850e58a0bbd685f2559487f9758e30eb321b635
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
M modules/varnishkafka
3 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Andrew Bogott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index 44cc503..d38326c 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.8-jessie',
+default='debian-8.9-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index f5b787b..a5ba920 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.8-jessie \
+ --image debian-8.9-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
diff --git a/modules/varnishkafka b/modules/varnishkafka
index a3651c8..8822159 16
--- a/modules/varnishkafka
+++ b/modules/varnishkafka
@@ -1 +1 @@
-Subproject commit a3651c852cfc00d6c3a81a0608b3328b5fb6159c
+Subproject commit 8822159469238c14aedfaf01d15e27658199a157
--
To view, visit https://gerrit.wikimedia.org/r/370673
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia850e58a0bbd685f2559487f9758e30eb321b635
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
Gerrit-Reviewer: Alex Monk
Gerrit-Reviewer: Andrew Bogott
Gerrit-Reviewer: Volans
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370673 )
Change subject: nova fullstack: use new jessie-8.9 image
..
nova fullstack: use new jessie-8.9 image
Change-Id: Ia850e58a0bbd685f2559487f9758e30eb321b635
---
M modules/openstack/files/nova_fullstack_test.py
M modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
M modules/varnishkafka
3 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/73/370673/1
diff --git a/modules/openstack/files/nova_fullstack_test.py
b/modules/openstack/files/nova_fullstack_test.py
index 44cc503..d38326c 100644
--- a/modules/openstack/files/nova_fullstack_test.py
+++ b/modules/openstack/files/nova_fullstack_test.py
@@ -357,7 +357,7 @@
argparser.add_argument(
'--image',
-default='debian-8.8-jessie',
+default='debian-8.9-jessie',
help='Image to use',
)
diff --git a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
index f5b787b..a5ba920 100644
--- a/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
+++ b/modules/openstack/templates/initscripts/nova-fullstack.upstart.erb
@@ -26,7 +26,7 @@
--max-pool <%= @max_pool %> \
--project admin-monitoring \
--prepend fullstackd \
- --image debian-8.8-jessie \
+ --image debian-8.9-jessie \
--flavor m1.small \
--keyfile <%= @keyfile %> \
--statsd statsd.eqiad.wmnet
diff --git a/modules/varnishkafka b/modules/varnishkafka
index a3651c8..8822159 16
--- a/modules/varnishkafka
+++ b/modules/varnishkafka
@@ -1 +1 @@
-Subproject commit a3651c852cfc00d6c3a81a0608b3328b5fb6159c
+Subproject commit 8822159469238c14aedfaf01d15e27658199a157
--
To view, visit https://gerrit.wikimedia.org/r/370673
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia850e58a0bbd685f2559487f9758e30eb321b635
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/369676 )
Change subject: Assert MediaWiki does not generate error logs
..
Assert MediaWiki does not generate error logs
After the test command has run, assert that MediaWiki did not emit any
logs.
Update:
mediawiki-core-qunit-selenium-jessie
mediawiki-extensions-hhvm-jessie
mediawiki-extensions-php55-trusty
mediawiki-extensions-php70-jessie
mediawiki-extensions-qunit-jessie
mediawiki-phpunit-hhvm-composer-jessie
mediawiki-phpunit-hhvm-jessie
mediawiki-phpunit-php55-trusty
mediawiki-phpunit-php70-jessie
Bug: T50002
Change-Id: Ic8e06c4581da3441d4a5a7262583ed2b4fe7606a
---
A jjb/assert-no-mediawiki-errors.bash
M jjb/mediawiki.yaml
2 files changed, 114 insertions(+), 0 deletions(-)
Approvals:
Thcipriani: Looks good to me, approved
Krinkle: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/jjb/assert-no-mediawiki-errors.bash
b/jjb/assert-no-mediawiki-errors.bash
new file mode 100644
index 000..8afe04a
--- /dev/null
+++ b/jjb/assert-no-mediawiki-errors.bash
@@ -0,0 +1,102 @@
+set -eu
+
+# The set of log files checked come from integration/jenkins
+# mediawiki/conf.d/00_set_debug_log.php
+#
+# Namely:
+# $wgDBerrorLog = "$wmgMwLogDir/mw-dberror.log";
+# $wgDebugLogGroups['exception'] = "$wmgMwLogDir/mw-exception.log";
+# $wgDebugLogGroups['error'] = "$wmgMwLogDir/mw-error.log";
+#
+ERROR_FILES=( mw-dberror.log mw-exception.log mw-error.log )
+echo "Asserting empty files: ${ERROR_FILES[*]}"
+
+# The minimum RELx_xxx version for which we want to run this test
+REL_MAJOR=1
+REL_MINOR=30
+
+# The minimum wmf/1.x.0-wmf.xx version for which we want to run this test
+WMF_MAJOR=30
+WMF_MINOR=12
+
+# A valid version is a version > wmf/1.WMF_MAJOR.0-wmf.WMF_MINOR
+valid_wmf_version() {
+ local version major minor
+ version="$1"
+
+ major=${version%.*.*}
+ major=${major##*.}
+
+ minor=${version##*.}
+
+ if (( major == WMF_MAJOR )) && (( minor >= WMF_MINOR )); then
+ return 0
+ elif (( major > WMF_MAJOR )); then
+ return 0
+ else
+ return 1
+ fi
+}
+
+# A valid version is a version > REL{REL_MAJOR}_{REL_MINOR}
+valid_rel_version() {
+ local version major minor
+ version="$1"
+ major=${version%%_*}
+ minor=${version##*_}
+
+ if (( major == REL_MAJOR )) && (( minor >= REL_MINOR )); then
+ return 0
+ elif (( major > REL_MAJOR )); then
+ return 0
+ else
+ return 1
+ fi
+}
+
+# Ensures that this is a ZUUL_BRANCH for which we want this to be a valid test
+valid_version() {
+ local version wmf_version rel_version
+
+ version="$1"
+
+ # Master is valid
+ if [[ "$version" == "master" ]]; then
+ return 0
+ fi
+
+ # Is this is a wmf/something
+ wmf_version="${version#wmf\/}"
+ if (( ${#version} != ${#wmf_version} )); then
+ valid_wmf_version "$wmf_version" && return 0
+ fi
+
+ # Is this is a RELsomething
+ rel_version="${version#REL}"
+ if (( ${#version} != ${#rel_version} )); then
+ valid_rel_version "$rel_version" && return 0
+ fi
+
+ # Unidentified version: invalid
+ return 1
+}
+
+if ! valid_version "$ZUUL_BRANCH"; then
+ printf 'Test not for branch: %s\n' "$ZUUL_BRANCH"
+ exit 0
+fi
+
+log_files="$( cd "$WORKSPACE/log"; ls "${ERROR_FILES[@]}" 2> /dev/null || :)"
+if [ ! "$log_files" ]; then
+ echo "No error files. GOOD"
+ exit 0
+fi
+
+echo "Dumping file(s) $log_files"
+set +e
+ # Use `grep --color . file list` to ensure that file names appear next
+ # to log messages
+ (cd "$WORKSPACE/log"; grep --color . "${ERROR_FILES[@]}" 2> /dev/null)
+set -e
+echo -e "MediaWiki emitted some errors. Check output above."
+exit 1
diff --git a/jjb/mediawiki.yaml b/jjb/mediawiki.yaml
index 00a4eee..150706c 100644
--- a/jjb/mediawiki.yaml
+++ b/jjb/mediawiki.yaml
@@ -60,6 +60,7 @@
./node_modules/.bin/grunt webdriver:test
fi
+ - assert-no-mediawiki-errors
publishers:
- castor-save
- archive-log-dir
@@ -153,6 +154,7 @@
- mw-apply-settings
- mw-fetch-composer-dev
- shell: "/srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh"
+ - assert-no-mediawiki-errors
publishers:
- phpunit-junit-2
- archive-log-dir
@@ -185,6 +187,7 @@
- mw-install-mysql
- mw-apply-settings
- shell: "/srv/deployment/integration/slave-scripts/bin/mw-phpunit.sh"
+ - assert-no-mediawiki-errors
publishers:
- phpunit-junit-2
- archive-log-dir
@@ -207,6 +210,13 @@
https://gerrit.wikimedia.org/r/p \
"$ZUUL_PROJECT"
fi
+
+- builder:
+name:
Dbrant has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370672 )
Change subject: Add better empty state to Offline Library screen.
..
Add better empty state to Offline Library screen.
(This is the first introduction of the new "library" and "packs" verbiage,
and can be independent of the Grand Renaming that will follow soon.)
Bug: T172553
Change-Id: I9bd2a83bd940bd713f5c91476643f98938937c21
---
M app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
M app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
A app/src/main/res/drawable/empty_offlinelibrary.png
M app/src/main/res/layout/fragment_local_compilations.xml
M app/src/main/res/values-qq/strings.xml
M app/src/main/res/values/strings.xml
6 files changed, 63 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/apps/android/wikipedia
refs/changes/72/370672/1
diff --git
a/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
b/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
index 8be08cd..b54c002 100644
--- a/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
+++ b/app/src/main/java/org/wikipedia/offline/LocalCompilationsActivity.java
@@ -23,7 +23,7 @@
setWordmarkVisible(false);
if (getSupportActionBar() != null) {
getSupportActionBar().setDisplayHomeAsUpEnabled(true);
-
getSupportActionBar().setTitle(R.string.offline_compilations_title);
+getSupportActionBar().setTitle(R.string.offline_library_title);
}
}
diff --git
a/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
b/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
index d72dda4..a77a0a7 100644
--- a/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
+++ b/app/src/main/java/org/wikipedia/offline/LocalCompilationsFragment.java
@@ -45,6 +45,7 @@
@BindView(R.id.compilations_count_text) TextView countText;
@BindView(R.id.disk_usage_view) DiskUsageView diskUsageView;
@BindView(R.id.compilation_search_error) WikiErrorView errorView;
+@BindView(R.id.compilation_empty_container) View emptyContainer;
private Unbinder unbinder;
private boolean updating;
@@ -131,7 +132,7 @@
}
}
-@OnClick(R.id.compilations_add_button) void onAddCompilationClick() {
+@OnClick({R.id.compilations_add_button,
R.id.compilation_empty_search_button}) void onAddCompilationClick() {
startActivity(RemoteCompilationsActivity.newIntent(getContext()));
}
@@ -204,16 +205,19 @@
progressBar.setVisibility(View.GONE);
searchEmptyView.setVisibility(View.GONE);
listContainer.setVisibility(View.GONE);
+emptyContainer.setVisibility(View.GONE);
return;
}
errorView.setVisibility(View.GONE);
progressBar.setVisibility(updating ? View.VISIBLE : View.GONE);
if (TextUtils.isEmpty(searchQuery)) {
searchEmptyView.setVisibility(View.GONE);
-listContainer.setVisibility(View.VISIBLE);
+listContainer.setVisibility(displayedItems.isEmpty() ? View.GONE :
View.VISIBLE);
+emptyContainer.setVisibility(displayedItems.isEmpty() ?
View.VISIBLE : View.GONE);
} else {
listContainer.setVisibility(displayedItems.isEmpty() ? View.GONE :
View.VISIBLE);
searchEmptyView.setVisibility(displayedItems.isEmpty() ?
View.VISIBLE : View.GONE);
+emptyContainer.setVisibility(View.GONE);
}
}
diff --git a/app/src/main/res/drawable/empty_offlinelibrary.png
b/app/src/main/res/drawable/empty_offlinelibrary.png
new file mode 100644
index 000..00745f4
--- /dev/null
+++ b/app/src/main/res/drawable/empty_offlinelibrary.png
Binary files differ
diff --git a/app/src/main/res/layout/fragment_local_compilations.xml
b/app/src/main/res/layout/fragment_local_compilations.xml
index 33bd03e..58e076e 100644
--- a/app/src/main/res/layout/fragment_local_compilations.xml
+++ b/app/src/main/res/layout/fragment_local_compilations.xml
@@ -83,6 +83,54 @@
+
+
+
+
+
+
+
+
+
+
+
+
Confirmation message
asking if the user really wants to cancel the download.
Affirmative answer to
cancel the download.\n{{Identical|Yes}}
Negative answer to
cancel the download.\n{{Identical|No}}
+ Title shown at the top of the activity
for managing the offline library.
+ Message shown when the offline
library is empty.
+ Explanation encouraging the
user to download an article pack for offline use later.
+ Button label for searching the
list of possible article packs to download.
Button label to skip the current onboarding
or tutorial screen.\n{{Identical|Skip}}
Button label to continue to the next
Jhernandez has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370671 )
Change subject: Doc: Use not-null markers on server/templates/page jsdoc
..
Doc: Use not-null markers on server/templates/page jsdoc
Change-Id: Ifb534d78fc212f9e8ddc8a57a1154f0da864d507
---
M src/server/templates/page.js
1 file changed, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/marvin refs/changes/71/370671/1
diff --git a/src/server/templates/page.js b/src/server/templates/page.js
index dbc26d1..19d3f6b 100644
--- a/src/server/templates/page.js
+++ b/src/server/templates/page.js
@@ -1,10 +1,10 @@
/**
* Render a full HTML page
- * @param {Object} params
+ * @param {!Object} params
* @param {?string} params.title Title of the page
- * @param {string} [params.body = ""] HTML to render in the body of the page
- * @param {Object.>} params.assets Manifest of
filename entry points to bundled assets.
- * @return {string} HTML
+ * @param {!string} [params.body = ""] HTML to render in the body of the page
+ * @param {!Object.>} params.assets Manifest
of filename entry points to bundled assets.
+ * @return {!string} HTML
*/
module.exports = function page({ body = "", assets, title }) {
return `
--
To view, visit https://gerrit.wikimedia.org/r/370671
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifb534d78fc212f9e8ddc8a57a1154f0da864d507
Gerrit-PatchSet: 1
Gerrit-Project: marvin
Gerrit-Branch: master
Gerrit-Owner: Jhernandez
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jhernandez has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370670 )
Change subject: Hygiene: Format package.json like npm does
..
Hygiene: Format package.json like npm does
npm init generated the package.json in a format and then npm install
changes it to something different.
This patch formats package.json like npm install does, so that when you
run npm install it doesn't modify your local copy.
Change-Id: I04ddf111f9d21f0f74576eba95507cad5f711bc1
---
M package.json
1 file changed, 10 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/marvin refs/changes/70/370670/1
diff --git a/package.json b/package.json
index f76b976..956d5d0 100644
--- a/package.json
+++ b/package.json
@@ -16,13 +16,21 @@
"precommit": "lint-staged"
},
"lint-staged": {
-"*.js": ["npm test"]
+"*.js": [
+ "npm test"
+]
},
"repository": {
"type": "git",
"url": "git+https://github.com/niedzielski/marvin.git;
},
- "keywords": ["mediawiki", "wikimedia", "web", "node", "pwa"],
+ "keywords": [
+"mediawiki",
+"wikimedia",
+"web",
+"node",
+"pwa"
+ ],
"author": "Wikimedia Foundation",
"license": "GPL-2.0",
"bugs": {
--
To view, visit https://gerrit.wikimedia.org/r/370670
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I04ddf111f9d21f0f74576eba95507cad5f711bc1
Gerrit-PatchSet: 1
Gerrit-Project: marvin
Gerrit-Branch: master
Gerrit-Owner: Jhernandez
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370610 )
Change subject: Add so that people know who to ask if they need ZppixBot in
their channel
..
Add so that people know who to ask if they need ZppixBot in their
channel
Change-Id: I56390c06fc42a07b919ae848a80bf2d5b81f3d31
---
M public_html/index.php
1 file changed, 3 insertions(+), 1 deletion(-)
Approvals:
Zppix: Looks good to me, approved
jenkins-bot: Verified
diff --git a/public_html/index.php b/public_html/index.php
index a8ac953..fc37f65 100644
--- a/public_html/index.php
+++ b/public_html/index.php
@@ -3,4 +3,6 @@
Inital release of ZppixBot is here!
- For more information please check out,
https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1 !!!
\ No newline at end of file
+ For more information please check out,
https://github.com/Pix1234/ZppixBot-Source/releases/tag/v1 !!!
+
+If you need ZppixBot in your channel, please contact Zppix.
--
To view, visit https://gerrit.wikimedia.org/r/370610
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I56390c06fc42a07b919ae848a80bf2d5b81f3d31
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
Gerrit-Reviewer: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370669 )
Change subject: Added breaks as now everything displayed on one line
..
Added breaks as now everything displayed on one line
Change-Id: Id6e87c3971603e06acb6be86243f12c263e91ab1
---
M public_html/contribs.php
1 file changed, 5 insertions(+), 5 deletions(-)
Approvals:
Zppix: Looks good to me, approved
jenkins-bot: Verified
diff --git a/public_html/contribs.php b/public_html/contribs.php
index e3c0fcf..068757a 100644
--- a/public_html/contribs.php
+++ b/public_html/contribs.php
@@ -1,5 +1,5 @@
-People whom have contributed to Zppixbot
-Zppix- Operator
-tom29739- Release version bot operator, developer
-SwisterTwister- Zppixbot web design, and friend.
-Reception123 - Helps with new features
+People whom have contributed to Zppixbot
+Zppix- Operator
+tom29739- Release version bot operator, developer
+SwisterTwister- Zppixbot web design, and friend.
+Reception123 - Helps with new features
--
To view, visit https://gerrit.wikimedia.org/r/370669
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id6e87c3971603e06acb6be86243f12c263e91ab1
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
Gerrit-Reviewer: Zppix
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Reception123 has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370669 )
Change subject: Added breaks as now everything displayed on one line
..
Added breaks as now everything displayed on one line
Change-Id: Id6e87c3971603e06acb6be86243f12c263e91ab1
---
M public_html/contribs.php
1 file changed, 5 insertions(+), 5 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/tools/ZppixBot
refs/changes/69/370669/1
diff --git a/public_html/contribs.php b/public_html/contribs.php
index e3c0fcf..068757a 100644
--- a/public_html/contribs.php
+++ b/public_html/contribs.php
@@ -1,5 +1,5 @@
-People whom have contributed to Zppixbot
-Zppix- Operator
-tom29739- Release version bot operator, developer
-SwisterTwister- Zppixbot web design, and friend.
-Reception123 - Helps with new features
+People whom have contributed to Zppixbot
+Zppix- Operator
+tom29739- Release version bot operator, developer
+SwisterTwister- Zppixbot web design, and friend.
+Reception123 - Helps with new features
--
To view, visit https://gerrit.wikimedia.org/r/370669
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id6e87c3971603e06acb6be86243f12c263e91ab1
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ZppixBot
Gerrit-Branch: master
Gerrit-Owner: Reception123
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370593 )
Change subject: Uppercase item ids received from services.
..
Uppercase item ids received from services.
Bug: T172642
Change-Id: I017d88425f734ffcdc45b565867ca70b142e88d2
---
M
blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplate.java
M
blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplateUnitTest.java
2 files changed, 8 insertions(+), 2 deletions(-)
Approvals:
Smalyshev: Looks good to me, approved
jenkins-bot: Verified
Gehel: Looks good to me, but someone else must approve
diff --git
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplate.java
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplate.java
index 89711d5..147247d 100644
---
a/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplate.java
+++
b/blazegraph/src/main/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplate.java
@@ -4,6 +4,7 @@
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
+import java.util.Locale;
import java.util.Map;
import java.util.Set;
@@ -372,7 +373,7 @@
case URI:
return new URIImpl(value);
case ITEM:
-return new URIImpl(WikibaseUris.getURISystem().entity() +
value);
+return new URIImpl(WikibaseUris.getURISystem().entity() +
value.toUpperCase(Locale.ROOT));
default:
throw new IllegalArgumentException("Can not produce URI for
non-URI type " + type);
}
diff --git
a/blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplateUnitTest.java
b/blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplateUnitTest.java
index 9abc865..8898ada 100644
---
a/blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplateUnitTest.java
+++
b/blazegraph/src/test/java/org/wikidata/query/rdf/blazegraph/mwapi/ApiTemplateUnitTest.java
@@ -22,6 +22,7 @@
import static org.hamcrest.Matchers.equalTo;
import static org.hamcrest.Matchers.not;
import static org.hamcrest.Matchers.instanceOf;
+import static org.hamcrest.Matchers.endsWith;
import static
org.wikidata.query.rdf.blazegraph.mwapi.MWApiServiceFactory.paramNameToURI;
public class ApiTemplateUnitTest extends AbstractRandomizedBlazegraphTestBase {
@@ -144,12 +145,16 @@
assertThat(var.getPath(), equalTo("@somedata"));
assertTrue(var.isURI());
assertThat(var.getURI("http://test.com/;), instanceOf(URI.class));
-// User-defined variable
+// URI keeps the case
+assertThat(var.getURI("http://test.com/test;).toString(),
endsWith("test"));
+// User-defined variable which is an item
var = outputs.get(2);
assertThat(var.getName(), equalTo("var3"));
assertThat(var.getPath(), equalTo("item/@wikibase_id"));
assertTrue(var.isURI());
assertThat(var.getURI("test"), instanceOf(URI.class));
+// T172642: Item URIs will be uppercased
+assertThat(var.getURI("test").toString(), endsWith("TEST"));
}
@Test(expected = NullPointerException.class)
--
To view, visit https://gerrit.wikimedia.org/r/370593
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I017d88425f734ffcdc45b565867ca70b142e88d2
Gerrit-PatchSet: 1
Gerrit-Project: wikidata/query/rdf
Gerrit-Branch: master
Gerrit-Owner: Smalyshev
Gerrit-Reviewer: Gehel
Gerrit-Reviewer: Smalyshev
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370663 )
Change subject: [bugfix] Maybe mainpage image does not exist
..
[bugfix] Maybe mainpage image does not exist
Ignore the test when the image is None
Change-Id: I9c1af39b1c1e82da58f78b1b4331411bc059a790
---
M tests/page_tests.py
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
Strainu: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tests/page_tests.py b/tests/page_tests.py
index b4d9a5e..17c69ef 100644
--- a/tests/page_tests.py
+++ b/tests/page_tests.py
@@ -563,7 +563,10 @@
if MediaWikiVersion(site.version()) < MediaWikiVersion('1.20'):
self.assertRaises(NotImplementedError, mainpage.page_image)
elif site.has_extension('PageImages'):
-self.assertIsInstance(mainpage.page_image(), pywikibot.FilePage)
+image = mainpage.page_image()
+if image is not None:
+self.assertIsInstance(mainpage.page_image(),
+ pywikibot.FilePage)
# for file pages, the API should return the file itself
self.assertEqual(image.page_image(), image)
else:
--
To view, visit https://gerrit.wikimedia.org/r/370663
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9c1af39b1c1e82da58f78b1b4331411bc059a790
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Xqt
Gerrit-Reviewer: Dalba
Gerrit-Reviewer: John Vandenberg
Gerrit-Reviewer: Strainu
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/370668 )
Change subject: bootstrap_vz firstboot: specify eth0 when refreshing dhcp
..
bootstrap_vz firstboot: specify eth0 when refreshing dhcp
I'm seeing an instance failure with messages like this:
localhost rc.local[406]: + /sbin/dhclient -1^M
localhost dhclient: No broadcast interfaces found - exiting.^M
There seems to be a bug in some releases where dhclient doesn't
work if the interface isn't specified. On OpenStack we can
be pretty sure that eth0 is the right call here.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=852173
Bug: T16
Change-Id: I05faceb8c6f88c0b79650eb440bc8571eda1ec19
---
M modules/labs_bootstrapvz/files/firstboot.sh
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Andrew Bogott: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/labs_bootstrapvz/files/firstboot.sh
b/modules/labs_bootstrapvz/files/firstboot.sh
index 8161478..b3c85f7 100644
--- a/modules/labs_bootstrapvz/files/firstboot.sh
+++ b/modules/labs_bootstrapvz/files/firstboot.sh
@@ -75,7 +75,7 @@
echo `date`
echo "Waiting for hostname to return the actual hostname."
sleep 10
-/sbin/dhclient -1
+/sbin/dhclient -1 eth0
hostname=`hostname`
done
--
To view, visit https://gerrit.wikimedia.org/r/370668
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I05faceb8c6f88c0b79650eb440bc8571eda1ec19
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
Gerrit-Reviewer: Andrew Bogott
Gerrit-Reviewer: jenkins-bot <>
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Andrew Bogott has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370668 )
Change subject: bootstrap_vz firstboot: specify eth0 when refreshing dhcp
..
bootstrap_vz firstboot: specify eth0 when refreshing dhcp
I'm seeing an instance failure with messages like this:
localhost rc.local[406]: + /sbin/dhclient -1^M
localhost dhclient: No broadcast interfaces found - exiting.^M
There seems to be a bug in some releases where dhclient doesn't
work if the interface isn't specified. On OpenStack we can
be pretty sure that eth0 is the right call here.
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=852173
Bug: T16
Change-Id: I05faceb8c6f88c0b79650eb440bc8571eda1ec19
---
M modules/labs_bootstrapvz/files/firstboot.sh
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/68/370668/1
diff --git a/modules/labs_bootstrapvz/files/firstboot.sh
b/modules/labs_bootstrapvz/files/firstboot.sh
index 8161478..b3c85f7 100644
--- a/modules/labs_bootstrapvz/files/firstboot.sh
+++ b/modules/labs_bootstrapvz/files/firstboot.sh
@@ -75,7 +75,7 @@
echo `date`
echo "Waiting for hostname to return the actual hostname."
sleep 10
-/sbin/dhclient -1
+/sbin/dhclient -1 eth0
hostname=`hostname`
done
--
To view, visit https://gerrit.wikimedia.org/r/370668
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I05faceb8c6f88c0b79650eb440bc8571eda1ec19
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
Jdlrobson has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/370667 )
Change subject: QA: Relax toast notification QA steps
..
QA: Relax toast notification QA steps
It seems trying to test both the steps can cause
false positives. Relaxing these checks seems to make
our Jenkins job happy without breaking the tests themselves
Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
---
M tests/browser/features/step_definitions/common_article_steps.rb
1 file changed, 2 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/skins/MinervaNeue
refs/changes/67/370667/1
diff --git a/tests/browser/features/step_definitions/common_article_steps.rb
b/tests/browser/features/step_definitions/common_article_steps.rb
index df5d4fd..c08ee3a 100644
--- a/tests/browser/features/step_definitions/common_article_steps.rb
+++ b/tests/browser/features/step_definitions/common_article_steps.rb
@@ -32,14 +32,13 @@
# Minerva loads mediawiki.notify at startup which defers the loading of
this module
# We must wait until the lazy loading has happened before checking for the
toast (T170890)
page.wait_until_rl_module_ready('mediawiki.notification')
+expect(page.notification_area_element.when_visible).to be_visible
end
- expect(on(ArticlePage).notification_area_element.when_visible).to be_visible
- expect(on(ArticlePage).toast_element.when_visible).to be_visible
end
Then(/^I should see a toast with message "(.+)"$/) do |msg|
- step "I should see a toast notification"
on(ArticlePage) do |page|
+page.wait_until_rl_module_ready('mediawiki.notification')
page.wait_until do
page.toast_element.when_present.text.include? msg
end
--
To view, visit https://gerrit.wikimedia.org/r/370667
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I119111e97f23d2f0dac7cbb0e5b86c1df0562598
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/MinervaNeue
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson
___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
1 - 100 of 190 matches
Mail list logo
|