File tree Expand file tree Collapse file tree 3 files changed +32
-7
lines changed
app/code/Magento/PageBuilderAnalytics/Model
_files/Magento/TestModuleCmsPageBuilderAnalytics/etc
testsuite/Magento/CmsPageBuilderAnalytics/_files Expand file tree Collapse file tree 3 files changed +32
-7
lines changed Original file line number Diff line number Diff line change @@ -101,10 +101,13 @@ public function getReport($name) : \IteratorIterator
101
101
foreach ($ batchQuery ->fetchAll () as $ row ) {
102
102
foreach ($ contentTypes as $ type ) {
103
103
// Count the amount of content types within the content
104
- $ typeCounts [$ type ['name ' ]] += substr_count (
105
- $ row ['content ' ],
106
- 'data-content-type=" ' . $ type ['name ' ] . '" '
107
- );
104
+ $ rowContent = $ row ['content ' ] ?? '' ;
105
+ if (strlen ($ rowContent ) > 0 ) {
106
+ $ typeCounts [$ type ['name ' ]] += substr_count (
107
+ $ rowContent ,
108
+ 'data-content-type=" ' . $ type ['name ' ] . '" '
109
+ );
110
+ }
108
111
}
109
112
}
110
113
}
Original file line number Diff line number Diff line change 16
16
</using >
17
17
</link-source >
18
18
<filter glue =" and" >
19
- <condition attribute =" identifier" operator =" eq " >page-builder-analytics-test-page</condition >
19
+ <condition attribute =" identifier" operator =" like " >page-builder-analytics-test-page% </condition >
20
20
</filter >
21
21
</source >
22
22
</report >
30
30
</using >
31
31
</link-source >
32
32
<filter glue =" and" >
33
- <condition attribute =" identifier" operator =" eq " >page-builder-analytics-test-page</condition >
33
+ <condition attribute =" identifier" operator =" like " >page-builder-analytics-test-page% </condition >
34
34
</filter >
35
35
</source >
36
36
</report >
37
- </config >
37
+ </config >
Original file line number Diff line number Diff line change 390
390
->setContent ($ content )
391
391
->setPageLayout ('cms-full-width ' )
392
392
->save ();
393
+
394
+ /** @var $emptyContentPage \Magento\Cms\Model\Page */
395
+ $ emptyContentPage = \Magento \TestFramework \Helper \Bootstrap::getObjectManager ()->create (\Magento \Cms \Model \Page::class);
396
+ $ emptyContentPage ->setTitle ('Page Builder Analytics Test Page - Empty Content ' )
397
+ ->setIdentifier ('page-builder-analytics-test-page-empty ' )
398
+ ->setStores ([0 , 1 ])
399
+ ->setIsActive (1 )
400
+ ->setCreatedIn (1 )
401
+ ->setContent ('' )
402
+ ->setPageLayout ('cms-full-width ' )
403
+ ->save ();
404
+
405
+ /** @var $nullContentPage \Magento\Cms\Model\Page */
406
+ $ nullContentPage = \Magento \TestFramework \Helper \Bootstrap::getObjectManager ()->create (\Magento \Cms \Model \Page::class);
407
+ $ nullContentPage ->setTitle ('Page Builder Analytics Test Page - Null Content ' )
408
+ ->setIdentifier ('page-builder-analytics-test-page-null ' )
409
+ ->setStores ([0 , 1 ])
410
+ ->setIsActive (1 )
411
+ ->setCreatedIn (1 )
412
+ ->setContent (null )
413
+ ->setPageLayout ('cms-full-width ' )
414
+ ->save ();
You can’t perform that action at this time.
0 commit comments