Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete message doesn't show slide or slider title #1

Open
Franciscof-Serfe opened this issue May 13, 2023 · 3 comments
Open

Delete message doesn't show slide or slider title #1

Franciscof-Serfe opened this issue May 13, 2023 · 3 comments

Comments

@Franciscof-Serfe
Copy link

Franciscof-Serfe commented May 13, 2023

When you delete a slide from the backend, the message does not show the title of the slide that is being deleted.

image

FIX:
Slide:

diff --git a/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidesActions.php b/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidesActions.php
index 9da2bbfc1..8b533f8bf 100755
--- a/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidesActions.php
+++ b/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidesActions.php
@@ -85,8 +85,8 @@ class SlidesActions extends \Magento\Ui\Component\Listing\Columns\Column
                             ),
                             'label' => __('Delete'),
                             'confirm' => [
-                                'title' => __('Delete "${ $.$data.title }"'),
-                                'message' => __('Are you sure you wan\'t to delete the Slide "${ $.$data.title }" ?')
+                                'title' => __('Delete "' . $item['title'] . '"'),
+                                'message' => __('Are you sure you want to delete the slide "' . $item['title'] . '"?')
                             ]
                         ]
                     ];
                     

Slider:

`diff --git a/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidersActions.php b/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidersActions.php
index d84d769b5..14b60ef43 100755
--- a/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidersActions.php
+++ b/app/code/Mageants/BannerSlider/Ui/Component/Listing/Column/SlidersActions.php
@@ -85,8 +85,8 @@ class SlidersActions extends \Magento\Ui\Component\Listing\Columns\Column
                             ),
                             'label' => __('Delete'),
                             'confirm' => [
-                                'title' => __('Delete "${ $.$data.name }"'),
-                                'message' => __('Are you sure you wan\'t to delete the Slider "${ $.$data.label }" ?')
+                               'title' => __('Delete "' . $item['slider_name'] . '"'),
+                               'message' => __('Are you sure you want to delete the slide "' . $item['slider_name'] . '"?')
                             ]
                         ]
                     ];

`
Mageants BannerSlider v2.1.0 in Magento 2.4.4-p2

@Franciscof-Serfe Franciscof-Serfe changed the title Delete message doesn't show slide title Delete message doesn't show slide or slider title May 14, 2023
@MageAnts
Copy link
Owner

MageAnts commented May 15, 2023 via email

@Franciscof-Serfe
Copy link
Author

Hi, nice to meet you

I have access to the module because we are the development team of customers who purchased your module.

So in the first message, I included the diff between the files and the version that we are working with that presented us with this issue.
** Mageants BannerSlider v2.1.0 in Magento 2.4.4-p2

In your example backend it doesn't play because you stopped displaying the name of the slide or slider.
image
image

With the fix:

image
image

I hope you find it useful

Regards,
Francisco

@MageAnts
Copy link
Owner

MageAnts commented May 19, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants