jenkins-bot has submitted this change and it was merged.
Change subject: CSSMin: remove dot segments in relative local URLs
......................................................................
CSSMin: remove dot segments in relative local URLs
Call wfRemoveDotSegments() (if available) on relative local URLs, so that
relative dot segments are collapsed.
Change-Id: Icc1c23a2a7d96c869412965bd8c3730d3c3b3a53
---
M includes/libs/CSSMin.php
M tests/phpunit/includes/libs/CSSMinTest.php
2 files changed, 7 insertions(+), 4 deletions(-)
Approvals:
Krinkle: Looks good to me, approved
Bartosz Dziewoński: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/includes/libs/CSSMin.php b/includes/libs/CSSMin.php
index f415c9b..118463a 100644
--- a/includes/libs/CSSMin.php
+++ b/includes/libs/CSSMin.php
@@ -399,7 +399,7 @@
if ( $local === false ) {
// Assume that all paths are relative to $remote, and
make them absolute
- return $remote . '/' . $url;
+ $url = $remote . '/' . $url;
} else {
// We drop the query part here and instead make the
path relative to $remote
$url = "{$remote}/{$file}";
@@ -418,8 +418,11 @@
}
// If any of these conditions failed (file missing, we
don't want to embed it
// or it's not embeddable), return the URL (possibly
with ?timestamp part)
- return $url;
}
+ if ( function_exists( 'wfRemoveDotSegments' ) ) {
+ $url = wfRemoveDotSegments( $url );
+ }
+ return $url;
}
/**
diff --git a/tests/phpunit/includes/libs/CSSMinTest.php
b/tests/phpunit/includes/libs/CSSMinTest.php
index 6142f96..dc3809f 100644
--- a/tests/phpunit/includes/libs/CSSMinTest.php
+++ b/tests/phpunit/includes/libs/CSSMinTest.php
@@ -102,12 +102,12 @@
array(
'Without trailing slash',
array( 'foo { prop: url(../bar.png); }', false,
'http://example.org/quux', false ),
- 'foo { prop:
url(http://example.org/quux/../bar.png); }',
+ 'foo { prop: url(http://example.org/bar.png);
}',
),
array(
'With trailing slash on remote (bug 27052)',
array( 'foo { prop: url(../bar.png); }', false,
'http://example.org/quux/', false ),
- 'foo { prop:
url(http://example.org/quux/../bar.png); }',
+ 'foo { prop: url(http://example.org/bar.png);
}',
),
array(
'Guard against stripping double slashes from
query',
--
To view, visit https://gerrit.wikimedia.org/r/231897
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Icc1c23a2a7d96c869412965bd8c3730d3c3b3a53
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Bartosz Dziewoński <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
