Commit graph

132 commits

Author SHA1 Message Date
Mateusz Rybarski b9d9e9216e XW-2444: extracting duplicated logic to method 2016-12-29 15:51:46 +01:00
Mateusz Rybarski 81a6a74f30 XW-2444: fallback to default color instead of displaying error when wrong color is provided 2016-12-29 09:46:05 +01:00
Mateusz Rybarski 475d7c61b6 XW-2444: params validator as a instance field 2016-12-29 09:28:54 +01:00
Mateusz Rybarski 129d8bb49a cr remarks 2016-12-28 16:39:53 +01:00
Mateusz Rybarski e4b28b1691 XW-2444: adding # to color if not provided, formatting 2016-12-28 16:30:24 +01:00
Mateusz Rybarski 8f10a879ca XW-2444: 'use' statements for exceptions 2016-12-28 16:01:15 +01:00
Mateusz Rybarski 11b917d0f7 XW-2444: minor cr remarks 2016-12-28 15:51:33 +01:00
Mateusz Rybarski 2402a0210f XW-2444: cr remarks 2016-12-28 15:18:51 +01:00
Mateusz Rybarski 2498b0111a XW-2444: cr remarks 2016-12-28 15:14:54 +01:00
Mateusz Rybarski 34bea1521f XW-2444: notice fix 2016-12-28 13:55:24 +01:00
Mateusz Rybarski 39ccd95df9 XW-2444: render custom accent colors 2016-12-28 13:10:57 +01:00
Mateusz Rybarski 3ad591cec6 Merge branch 'pi-render-service-refactor' into XW-2444 2016-12-28 11:29:36 +01:00
Mateusz Rybarski c2fc1f2e76 XW-2444: handling case when no color is set 2016-12-28 09:38:27 +01:00
Mateusz Rybarski 9e89577acf XW-2444: extracting validation from ParserTagController 2016-12-28 09:27:25 +01:00
Mateusz Rybarski 194b8e4e03 XW-2444: error when color is in wrong format 2016-12-27 15:57:14 +01:00
Mateusz Rybarski d56e77855c XW-2444: trim params value 2016-12-27 15:22:45 +01:00
idradm 36b582a164 refactor render service to skin based services 2016-12-27 15:09:55 +01:00
Mateusz Rybarski 1ec85d8594 XW-2444: validating hex color, support for accent-color-text params 2016-12-27 13:50:53 +01:00
Mateusz Rybarski be71d1b0f4 XW-2444: accent-color param without validation 2016-12-27 12:10:11 +01:00
grunny 36f77e2a8d Add comment explaining where the Mercury PI replacements occur 2016-06-20 18:24:38 +10:00
grunny c006c42035 (CE-3844) Refactor fix for PI and parser tag fix
A security fix from MW core added quotes to the Parser's strip tag
markers in order to prevent an XSS vulnerability. When this was deployed
PIs and our Wikia tag extensions broke in Mercury because ArticlesAsJson
JSON encodes the parser output in the ParserAfterTidy, but then PI and
the tag extensions were manually replacing the Parser tag strip markers
on the already JSON encoded string within their own ParserAfterTidy methods
(which were always run after the ArticlesAsJson method). This meant that the
strip marker had the quote escaped with a backslash in the encoded string,
so the string replacements failed.

This change follows the previous quick and dirty fix by moving the replacements
in PI and the tag extensions to just before ArticlesAsJson encodes the output.
Which also removes the need from before the fix in #10701 to JSON encode the
parser output of the tags that will be replaced into the text.
2016-06-17 20:57:12 +10:00
grunny 61bdc6e652 (DAT-4494) Only decode when it's the main JSON array 2016-06-17 04:14:47 +10:00
grunny ddd6c61e79 (DAT-4494) We don't need to json encode here anymore 2016-06-17 03:59:52 +10:00
grunny 899b127b00 (DAT-4494) Don't perform replacements on JSON encoded content 2016-06-16 17:39:06 +00:00
grunny 8186618eb9 (DAT-4494) Fix galleries and tabber in PortableInfoboxes 2016-06-16 16:59:48 +00:00
Igor Rogatty 3ef5a8e72b XW-1429 | remove whitespace after original marker and put a single space after new marker position 2016-05-09 15:14:02 +02:00
Igor Rogatty bc7f3efb35 XW-1429 | add unit tests for moveFirstMarkerToTop method 2016-05-09 14:50:37 +02:00
Igor Rogatty 464bd9ccee XW-1429 | move whitespace together with marker 2016-05-09 14:49:56 +02:00
Igor Rogatty 81830d7024 XW-1429 | use ArticleAsJsonBeforeEncode hook to move the first infobox marker 2016-05-09 13:56:28 +02:00
Igor Rogatty c5ff6c70df XW-1429 | move the first infobox to the top of article content 2016-05-09 12:46:23 +02:00
Lukasz Konieczny 56e5697cfd CE-3554: Fix typo 2016-04-07 10:24:18 +00:00
Lukasz Konieczny 6cc68eaa19 CE-3554: Add doc and comment 2016-04-07 09:29:26 +00:00
Lukasz Konieczny 1a92b184bb CE-3554: Add fallback for existing infobox page props data 2016-04-07 09:27:59 +00:00
Lukasz Konieczny 4f11b96e48 CE-3554: Changes after CR 2016-04-01 11:07:30 +00:00
Lukasz Konieczny a773ee1a49 CE-3554: Extract labels from infobox markup and save them in page props 2016-03-31 09:58:34 +00:00
idradm 20d6a46421 DAT-3707 create infobox builder extension 2016-02-12 11:38:47 +01:00
idradm 6e60746d0f DAT-3707 refactor save 2016-02-12 10:53:22 +01:00
idradm 41ef2d5266 DAT-3707 refactor save 2016-02-12 10:52:46 +01:00
idradm 5bbad66ec0 DAT-3707 remove unused entry point code 2016-02-12 10:05:16 +01:00
idradm 45aaaa5ecd make infobox builder viewport bigger 2016-02-11 10:34:09 +01:00
idradm 490ae75d53 DAT-3735 load builder only 2016-02-10 16:20:36 +01:00
idradm dbbfabceda change mercury route 2016-02-09 12:20:20 +01:00
idradm d4953f7680 Merge branch 'dev' into infobox-builder-feature-branch
# Conflicts:
#	extensions/wikia/AdminDashboard/AdminDashboardLogic.class.php
#	extensions/wikia/AssetsManager/config.php
#	extensions/wikia/EditPageLayout/templates/EditPageLayout_EditPage.php
2016-02-09 11:46:56 +01:00
Nikodem Hynek 37f07b8f11 Merge pull request #9659 from Wikia/DAT-3624
DAT-3624 Fix for broken references in infobox image captions
2016-02-05 15:25:10 +01:00
idradm b9af596dbc DAT-3622 always pass array to custom frame on api call 2016-02-04 16:52:51 +01:00
Sebastian Marzjan c5d17f315e DAT-3624 Rename 'Infobo' to 'Infobox' 2016-02-03 12:06:44 +01:00
Sebastian Marzjan dc561a4157 DAT-3614 Always pass an *ARRAY* of params to the infobox NodeFactory 2016-01-08 07:44:30 +01:00
idradm 5f42ce9742 make infoboxes use params same way as mediawiki, enable 0 param 2016-01-04 14:34:10 +01:00
Inez Korczynski 7d1f542f6c MAIN-5838: Check if $arguments is an array before handling it in array way ;) 2015-11-14 00:05:41 +00:00
Inez Korczynski 8a2ddb554b VE-2041: Added comment regarding commented out code 2015-11-12 21:23:38 +00:00