Feeling Lucky
?
bug
PR-4.4-dev
[#42336] - Declare the context variable correctly
- Fixed in Code Base
- 18 Nov 2023
- Medium
- Build: 4.4-dev
- # 42336
- Diff
- Digital-Peak:wf/context
User tests: Successful: Unsuccessful:
Summary of Changes
The context variable is not correctly declared in the workflow publishing plugin. Additionally it checks for concrete event class as getArguments is only available there and not the interface.
Error is also not reported anymore in drone phan step.
Testing Instructions
- Enable workflow for Articles
- Change the state of an article
Actual result BEFORE applying this Pull Request
A warning is printed to the error log that the variable context is not declared.
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Labels |
Added:
PR-4.4-dev
|
||
chmst
- comment
- 17 Nov 2023
I have tested this item ✅ successfully on 4baa0ec
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42336.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 17 Nov 2023
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42336.
| Labels |
Added:
?
bug
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-11-18 14:05:55 |
| Closed_By | ⇒ | MacJoom | |
| Labels |
Added:
?
Removed: ? |
||
I have tested this item ✅ successfully on 6481ec1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42336.