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

[4.1] Initialize the contact text attribute as string #37836

Merged
merged 1 commit into from
May 20, 2022

Conversation

laoneo
Copy link
Member

@laoneo laoneo commented May 20, 2022

Summary of Changes

When triggering the onContentPrepare event in the contact details page, the text attribute is expected to be a string. On PHP 8.1 a couple of deprecated messages are displayed as it is currently null.

Testing Instructions

  • Create a contact
  • Create a single contact menu item
  • Open the new menu item

Actual result BEFORE applying this Pull Request

The following deprecated message is displayed:
Deprecated: strpos(): Passing null to parameter 1 ($haystack) of type string is deprecated in /plugins/content/loadmodule/loadmodule.php on line 49

Expected result AFTER applying this Pull Request

No warning.

@laoneo laoneo added the PHP 8.x PHP 8.x deprecated issues label May 20, 2022
@chmst
Copy link
Contributor

chmst commented May 20, 2022

I have tested this item ✅ successfully on 275ad19


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37836.

1 similar comment
@alikon
Copy link
Contributor

alikon commented May 20, 2022

I have tested this item ✅ successfully on 275ad19


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37836.

@joomla-cms-bot joomla-cms-bot removed PR-4.1-dev PHP 8.x PHP 8.x deprecated issues labels May 20, 2022
@alikon
Copy link
Contributor

alikon commented May 20, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37836.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 20, 2022
@richard67 richard67 added the PHP 8.x PHP 8.x deprecated issues label May 20, 2022
@bembelimen bembelimen merged commit cc7e26d into joomla:4.1-dev May 20, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 20, 2022
@bembelimen
Copy link
Contributor

Thx

@bembelimen bembelimen added this to the Joomla 4.1.4 milestone May 20, 2022
@laoneo laoneo deleted the j4/contact/details-81 branch May 20, 2022 22:12
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request May 21, 2022
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request May 21, 2022
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request May 21, 2022
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PHP 8.x PHP 8.x deprecated issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants