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

fix: str_replace issue when deploy #808 #816

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sonbui00
Copy link

str_replace(): Argument #3 ($subject) must be of type array|string, int giv

Description

Fixed Issues (if relevant)

  1. str_replace(): Argument #3 ($subject) must be of type array|string, int given #808: str_replace(): Argument MAGECLOUD-792 #3 ($subject) must be of type array|string, int given

Manual testing scenarios

  1. env.php data value have: null, int

Release notes

For user-facing changes, add a meaningful release note. For examples, see Magento Cloud ECE-tools release notes.

Associated documentation updates

Add link to Magento DevDocs PR or Issue, if needed.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • Pull request introduces user-facing changes and includes meaningful updates for any required release notes and documentation changes
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

str_replace(): Argument magento#3 ($subject) must be of type array|string, int giv

Signed-off-by: Son Bui <[email protected]>
@sonbui00 sonbui00 marked this pull request as ready for review October 30, 2023 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant