Skip to content

Commit 9bea7da

Browse files
Merge remote-tracking branch 'owls/MC-3430-remove-newsletter' into cms-team-2-sprint-15
2 parents 9d40f18 + de6dcb1 commit 9bea7da

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+87
-1378
lines changed

app/code/Magento/PageBuilder/Model/Template/Filter.php

Lines changed: 0 additions & 136 deletions
This file was deleted.

app/code/Magento/PageBuilder/Plugin/Cms/Block/Block.php

Lines changed: 0 additions & 41 deletions
This file was deleted.

app/code/Magento/PageBuilder/Plugin/Cms/Block/Page.php

Lines changed: 0 additions & 41 deletions
This file was deleted.

app/code/Magento/PageBuilder/Setup/DataConverter/Renderer/Newsletter.php

Lines changed: 54 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -44,31 +44,67 @@ public function render(array $itemData, array $additionalData = []) : string
4444
}
4545
$eavData = $this->eavAttributeLoader->load($itemData['entityId']);
4646

47-
$rootElementAttributes = [
48-
'data-element' => 'main',
49-
'data-role' => 'newsletter',
50-
'data-appearance' => 'default',
51-
'class' => $itemData['formData']['css_classes'] ?? '',
52-
'data-button-text' => $eavData['button_text'] ?? '',
53-
'data-label-text' => $eavData['label'] ?? '',
54-
'data-title' => $eavData['title'] ?? '',
55-
'data-placeholder' => $eavData['placeholder'] ?? '',
56-
];
47+
$rootElementHtml = '<div data-element="main" data-role="html" data-appearance="default">';
48+
$rootElementHtml .= $this->getNewsletterHtml($itemData, $eavData) . '</div>';
5749

50+
return $rootElementHtml;
51+
}
52+
53+
/**
54+
* Get the html for the newsletter
55+
*
56+
* @param array $itemData
57+
* @param array $eavData
58+
* @return string
59+
*/
60+
private function getNewsletterHtml(array $itemData, array $eavData): string
61+
{
62+
$newsletterButtonText = $eavData['button_text'] ?? __('Subscribe');
63+
$newsletterLabel = $eavData['label'] ?? __('Sign Up for Our Newsletter:');
64+
$newsletterTitle = $eavData['title'] ?? __('Newsletter');
65+
$newsletterPlaceHolder = $eavData['placeholder'] ?? __('Enter your email address');
66+
$newsletterClasses = $eavData['css_classes'] ?? '';
67+
$newsletterStyles = '';
5868
if (isset($itemData['formData'])) {
69+
$justifyContent = '';
70+
if (isset($itemData['formData']['align'])) {
71+
$convertAlignContent = [
72+
'left' => 'flex-start',
73+
'right' => 'flex-end',
74+
'center' => 'center'
75+
];
76+
$itemData['formData']['display'] = 'flex';
77+
$justifyContent = ' justify-content: ' . $convertAlignContent[$itemData['formData']['align']] . ';';
78+
unset($itemData['formData']['align']);
79+
}
5980
$style = $this->styleExtractor->extractStyle($itemData['formData']);
6081
if ($style) {
61-
$rootElementAttributes['style'] = $style;
82+
$newsletterStyles = $style . $justifyContent;
6283
}
6384
}
6485

65-
$rootElementHtml = '<div';
66-
foreach ($rootElementAttributes as $attributeName => $attributeValue) {
67-
$rootElementHtml .= $attributeValue ? " $attributeName=\"$attributeValue\"" : '';
68-
}
86+
$newsletterHtml = '<div class="' . $newsletterClasses . '" style="' . $newsletterStyles . '">' .
87+
'<div class="block newsletter"><div class="title"><strong>' . $newsletterTitle . '</strong></div>' .
88+
'<div class="content">' .
89+
'<form class="form subscribe" novalidate method="post" ' .
90+
'action="{{store direct_url=\'newsletter/subscriber/new\' _secure=1}}" ' .
91+
'data-mage-init=\'{"validation": {"errorClass": "mage-error"}}\'>' .
92+
'<div class="field newsletter">' .
93+
'<label class="label" for="newsletter-' . $itemData['entityId'] . '">' .
94+
'<span>' . $newsletterLabel . '</span>'.
95+
'</label>' .
96+
'<div class="control">' .
97+
'<input name="email" type="email" id="newsletter-' . $itemData['entityId'] . '" '.
98+
'placeholder="' . $newsletterPlaceHolder . '" ' .
99+
'data-mage-init=\'{"mage/trim-input":{}}\' ' .
100+
'data-validate="{required:true, \'validate-email\':true}"/>' .
101+
'</div></div>' .
102+
'<div class="actions">' .
103+
'<button class="action subscribe primary" title="' . $newsletterButtonText . '" type="submit">' .
104+
'<span>' . $newsletterButtonText . '</span>' .
105+
'</button>' .
106+
'</div></form></div></div></div>';
69107

70-
$rootElementHtml .= '></div>';
71-
72-
return $rootElementHtml;
108+
return $newsletterHtml;
73109
}
74110
}

app/code/Magento/PageBuilder/Test/Mftf/ActionGroup/ContentTypeNewsletterActionGroup.xml

Lines changed: 0 additions & 27 deletions
This file was deleted.

app/code/Magento/PageBuilder/Test/Mftf/ActionGroup/PanelActionGroup.xml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,12 @@
2929
<see userInput="Column" selector="{{PageBuilderPanel.layoutGroupContentType}}" stepKey="seeLayoutGroupColumn"/>
3030
<see userInput="Tabs" selector="{{PageBuilderPanel.layoutGroupContentType}}" stepKey="seeLayoutGroupTabs"/>
3131
<click selector="{{PageBuilderPanel.elementsGroup}}" stepKey="clickElementsGroup"/>
32-
<seeNumberOfElements selector="{{PageBuilderPanel.allGroupContentTypes('elements')}}" userInput="6" stepKey="seeNumberOfElementsGroupContentTypes"/>
32+
<seeNumberOfElements selector="{{PageBuilderPanel.allGroupContentTypes('elements')}}" userInput="5" stepKey="seeNumberOfElementsGroupContentTypes"/>
3333
<see userInput="Elements" selector="{{PageBuilderPanel.elementsGroup}}" stepKey="seeElementsGroup"/>
3434
<see userInput="Text" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupText"/>
3535
<see userInput="Heading" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupHeading"/>
3636
<see userInput="Buttons" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupButtons"/>
3737
<see userInput="Divider" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupDivider"/>
38-
<see userInput="Newsletter" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupNewsletter"/>
3938
<see userInput="HTML Code" selector="{{PageBuilderPanel.elementsGroupContentType}}" stepKey="seeElementsGroupHtml"/>
4039
<click selector="{{PageBuilderPanel.mediaGroup}}" stepKey="clickMediaGroup"/>
4140
<seeNumberOfElements selector="{{PageBuilderPanel.allGroupContentTypes('media')}}" userInput="5" stepKey="seeNumberOfMediaGroupContentTypes"/>

app/code/Magento/PageBuilder/Test/Mftf/Data/ContentTypeData.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,6 @@
7171
<data key="role">divider</data>
7272
<data key="pageNamePrefix">divider</data>
7373
</entity>
74-
<entity name="PageBuilderNewsletterContentType" type="pagebuilder_content_type">
75-
<data key="name">Newsletter</data>
76-
<data key="section">Elements</data>
77-
<data key="role">newsletter</data>
78-
<data key="pageNamePrefix">newsletter</data>
79-
</entity>
8074
<entity name="PageBuilderHtmlContentType" type="pagebuilder_content_type">
8175
<data key="name">HTML Code</data>
8276
<data key="section">Elements</data>

0 commit comments

Comments
 (0)