[#39601] - [4.2] PHP8.2 Use the params correctly in the article form layout
- Fixed in Code Base
- 12 Jan 2023
- Medium
- Build: 4.2-dev
- # 39601
- Diff
- Digital-Peak:php82/formlayout
User tests: Successful: Unsuccessful:
Summary of Changes
The front from layout file uses the params data as direct attributes and not from the internal data store. This pr fixes it.
Testing Instructions
Open the article form on the PHP 8.2 on the front end with debug enabled.
Actual result BEFORE applying this Pull Request
The following warning is shown:
Creation of dynamic property Joomla\Registry\Registry::$show_urls_images_frontend is deprecated in
Expected result AFTER applying this Pull Request
No warning.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
| Status | New | ⇒ | Pending |
I have tested this item
This solves the issue. But I even don't know why we need that block of code. Maybe for history reason.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39601.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39601.
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-01-12 15:51:49 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
?
|
||
thanks
I have tested this item✅ successfully on f1c78e0
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39601.