Feeling Lucky
?
[#36404] - [4] Ensure message is defined
- Fixed in Code Base
- 29 Dec 2021
- Medium
- Build: 4.0-dev
- # 36404
- Diff
- PhilETaylor:patch-319
User tests: Successful: Unsuccessful:
Summary of Changes
There is an edge case, where this method can be called with a set of ids, that gets pruned, leaving no ids to feature, which in turn rules a code path that leaves the redirect $message undefined.
Testing Instructions
Code review. Best viewed with no whitespace https://github.com/joomla/joomla-cms/pull/36404/files?diff=unified&w=1
Actual result BEFORE applying this Pull Request
$message is undefined.
Expected result AFTER applying this Pull Request
return early after identifying that we have no $ids to process.
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_content |
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-12-29 23:11:34 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 29 Dec 2021
Thanks!
I have tested this item✅ successfully on ae6d1f1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36404.