? ? Pending

User tests: Successful: Unsuccessful:

avatar ditsuke
ditsuke
6 Aug 2021

Summary of Changes

Fixes the options on display by using the right optionsFilter attribute instead of filter.

Testing Instructions

Go to the workflow manager for some extension, for example yoursite.example/index.php?option=com_workflow&extension=com_content

Actual result BEFORE applying this Pull Request

The state filter lists states not supported by com_workflows

image

Expected result AFTER applying this Pull Request

The state filter only lists states supported by com_workflows (the labels are still not right)

image

Documentation Changes Required

None

avatar ditsuke ditsuke - open - 6 Aug 2021
avatar ditsuke ditsuke - change - 6 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2021
Category Administration com_workflow
avatar richard67
richard67 - comment - 9 Aug 2021

@ditsuke Shouldn't that be done too for the transitions list and the stages list of the workflow?

avatar ditsuke
ditsuke - comment - 11 Aug 2021

@richard67 I didn't catch those! Getting them in soon.

avatar ditsuke ditsuke - change - 11 Aug 2021
Labels Added: ?
avatar ditsuke
ditsuke - comment - 11 Aug 2021

As an aside, would you say we can use a StatusField attribute to use JENABLED/JDISABLED instead?
I see com_plugins uses its own field because of this and here in Workflows we have the wrong labels (the item view uses a ListField instead).
On the other end over in Cronjobs I'm also using a custom field at the moment to get it right.

avatar brianteeman
brianteeman - comment - 4 Oct 2021

I have tested this item successfully on 52a8c0d


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

avatar brianteeman brianteeman - test_item - 4 Oct 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 4 Oct 2021

Good spot.


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

avatar RickR2H RickR2H - test_item - 28 Oct 2021 - Tested successfully
avatar RickR2H
RickR2H - comment - 28 Oct 2021

I have tested this item successfully on 52a8c0d


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

avatar RickR2H RickR2H - change - 28 Oct 2021
Status Pending Ready to Commit
avatar RickR2H
RickR2H - comment - 28 Oct 2021

RTC


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

avatar richard67 richard67 - change - 1 Nov 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-11-01 13:13:40
Closed_By richard67
Labels Added: ? ?
Removed: ?
avatar richard67 richard67 - close - 1 Nov 2021
avatar richard67 richard67 - merge - 1 Nov 2021
avatar richard67
richard67 - comment - 1 Nov 2021

Thanks

avatar ditsuke
ditsuke - comment - 1 Nov 2021

Thanks ?

Add a Comment

Login with GitHub to post a comment