Skip to content

Fix date() warning#1

Open
fritzmg wants to merge 1 commit intomagesuite:2.xfrom
fritzmg:patch-1
Open

Fix date() warning#1
fritzmg wants to merge 1 commit intomagesuite:2.xfrom
fritzmg:patch-1

Conversation

@fritzmg
Copy link

@fritzmg fritzmg commented Mar 10, 2022

This fixes the following warning:

Exception #0 (Exception): Warning: date() expects parameter 2 to be int, string given in vendor/creativestyle/magesuite-product-positive-indicators/view/frontend/templates/expecteddelivery/product.phtml on line 11

I am not sure what the original intention here was though. $block->getShipDayName() already returns a translated weekday name (like "Montag" in German). The date() function needs a UNIX timestamp for the second parameter. This PR simply removes the usage of date, so that the message says

You can get this product on Monday (12:00) if you buy it now!

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

Successfully merging this pull request may close these issues.

1 participant