Skip to content

Commit 39ff541

Browse files
authored
Merge pull request #2575 from magento-borg/MAGETWO-89410-testValidComposer
Fixed issues: - MAGETWO-86709 Widget isn't updated on Storefront if related Products are updated - MAGETWO-89410 [2.3] Test testValidComposerJson fails on bundle extensions - MAGETWO-87492 Rename 2.3 upgrade script
2 parents 4a61ed6 + 5fb7d1c commit 39ff541

File tree

6 files changed

+38
-5
lines changed

6 files changed

+38
-5
lines changed

app/code/Magento/Catalog/etc/widget.xml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,11 @@
6464
</parameter>
6565
<parameter name="cache_lifetime" xsi:type="text" visible="true">
6666
<label translate="true">Cache Lifetime (Seconds)</label>
67-
<description translate="true">86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache.</description>
67+
<description translate="true">
68+
<![CDATA[Time in seconds between the widget updates.
69+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
70+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed.]]>
71+
</description>
6872
</parameter>
6973
</parameters>
7074
<containers>

app/code/Magento/Catalog/i18n/en_US.csv

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -705,7 +705,13 @@ Template,Template
705705
"New Products Names Only Template","New Products Names Only Template"
706706
"New Products Images Only Template","New Products Images Only Template"
707707
"Cache Lifetime (Seconds)","Cache Lifetime (Seconds)"
708-
"86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache.","86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache."
708+
"Time in seconds between the widget updates.
709+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
710+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed."
711+
,
712+
"Time in seconds between the widget updates.
713+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
714+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed."
709715
"Catalog Product Link","Catalog Product Link"
710716
"Link to a Specified Product","Link to a Specified Product"
711717
"Select Product...","Select Product..."

app/code/Magento/CatalogWidget/etc/widget.xml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,11 @@
4040
</parameter>
4141
<parameter name="cache_lifetime" xsi:type="text" visible="true">
4242
<label translate="true">Cache Lifetime (Seconds)</label>
43-
<description translate="true">86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache.</description>
43+
<description translate="true">
44+
<![CDATA[Time in seconds between the widget updates.
45+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
46+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed.]]>
47+
</description>
4448
</parameter>
4549
<parameter name="condition" xsi:type="conditions" visible="true" required="true" sort_order="10"
4650
class="Magento\CatalogWidget\Block\Product\Widget\Conditions">

app/code/Magento/CatalogWidget/i18n/en_US.csv

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,11 @@ Title,Title
1616
Template,Template
1717
"Products Grid Template","Products Grid Template"
1818
"Cache Lifetime (Seconds)","Cache Lifetime (Seconds)"
19-
"86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache.","86400 by default, if not set. To refresh instantly, clear the Blocks HTML Output cache."
19+
"Time in seconds between the widget updates.
20+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
21+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed."
22+
,
23+
"Time in seconds between the widget updates.
24+
<br/>If not set, equals to 86400 seconds (24 hours). To update widget instantly, go to Cache Management and clear Blocks HTML Output cache.
25+
<br/>Widget will not show products that begin to match the specified conditions until cache is refreshed."
2026
Conditions,Conditions

dev/tests/static/testsuite/Magento/Test/Integrity/ComposerTest.php

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,9 @@ private function assertMagentoConventions($dir, $packageType, \StdClass $json)
171171
switch ($packageType) {
172172
case 'magento2-module':
173173
$xml = simplexml_load_file("$dir/etc/module.xml");
174-
$this->assertConsistentModuleName($xml, $json->name);
174+
if ($this->isVendorMagento($json->name)) {
175+
$this->assertConsistentModuleName($xml, $json->name);
176+
}
175177
$this->assertDependsOnPhp($json->require);
176178
$this->assertPhpVersionInSync($json->name, $json->require->php);
177179
$this->assertDependsOnFramework($json->require);
@@ -210,6 +212,17 @@ private function assertMagentoConventions($dir, $packageType, \StdClass $json)
210212
}
211213
}
212214

215+
/**
216+
* Returns true if package vendor is Magento.
217+
*
218+
* @param string $packageName
219+
* @return bool
220+
*/
221+
private function isVendorMagento($packageName)
222+
{
223+
return strpos($packageName, 'magento') === 0;
224+
}
225+
213226
/**
214227
* Assert that component registrar is autoloaded in composer json
215228
*

0 commit comments

Comments
 (0)