Feeling Lucky
?
?
[#35597] - [4.1] Undefined property notice in image layout (Re-do of #29513)
- Fixed in Code Base
- 11 Mar 2022
- Medium
- Build: 4.0-dev
- # 35597
- Diff
- AndySDH:patch-19
User tests: Successful: Unsuccessful:
Summary of Changes
Fixes undefined property notice in some cases.
A fix made in #29513 was somehow un-done and lost in the echo, and the issue is now back. So fixing it again with the same solution as #29513
Actual result BEFORE applying this Pull Request
If image_fulltext_caption is missing in the database, you get Undefined property notice in image layout
Notice: Undefined property: stdClass::$image_intro_caption
Expected result AFTER applying this Pull Request
No PHP Notices
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Layout |
AndySDH
- comment
- 19 Sep 2021
Yup, can you test this @richard67?
| Labels |
Added:
?
Conflicting Files
|
||
Quy
- comment
- 9 Jan 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35597.
Quy
- comment
- 9 Jan 2022
Please fix conflicts.
richard67
- comment
- 9 Jan 2022
Done.
AndySDH
- comment
- 9 Jan 2022
Thank you!
richard67
- comment
- 9 Jan 2022
I have tested this item
By code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35597.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 9 Jan 2022
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35597.
| Labels |
Added:
?
Removed: Conflicting Files |
||
| Labels |
Added:
?
Removed: ? |
||
| Title |
|
||||||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-03-11 18:59:21 |
| Closed_By | ⇒ | bembelimen |
bembelimen
- comment
- 11 Mar 2022
Thx
It seems one of the 2 changes from PR #29513 got lost with PR #30784 .