Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Fix filter for workflows #24864

Merged
merged 2 commits into from
May 23, 2019

Conversation

chmst
Copy link
Contributor

@chmst chmst commented May 12, 2019

Pull Request for Issue #24814

Summary of Changes

The #__workflows table uses column name published. This causes several issues. One of them is that the ordering of workflows via state is not possible.
Another isssue is that filters are not set, as described in the related issue.

Testing Instructions

Ordering:
You must have at least two workflows. Set one of the unpublished and try ordering of the column "state"
Active Filters:
Or see test instruction in the related issue.

Expected result

Columns are ordered new if possible
Active Filters are set

Actual result

No ordering possible, active Filters are not set.

Documentation Changes Required

no, it is a bug

@Quy
Copy link
Contributor

Quy commented May 12, 2019

Please update to the right issue for this PR.

@chmst
Copy link
Contributor Author

chmst commented May 12, 2019

done, sry

@Quy
Copy link
Contributor

Quy commented May 12, 2019

I have tested this item ✅ successfully on eafbd9c


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

@brianteeman
Copy link
Contributor

I dont see how this has anything to do with #24814

@chmst
Copy link
Contributor Author

chmst commented May 12, 2019

Could you test please? I think setting the filter right does the trick also for you.

@brianteeman
Copy link
Contributor

I did test

@chmst chmst changed the title [4.0] Fix ordering of workflows in column "status" [4.0] Fix filter for workflows May 13, 2019
@chmst
Copy link
Contributor Author

chmst commented May 13, 2019

Description changed.

@ReLater
Copy link
Contributor

ReLater commented May 20, 2019

I have tested this item ✅ successfully on 3e45bc5


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

@ghost
Copy link

ghost commented May 20, 2019

@Quy can you please retest?

@Quy
Copy link
Contributor

Quy commented May 20, 2019

I have tested this item ✅ successfully on 3e45bc5


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

@Quy
Copy link
Contributor

Quy commented May 20, 2019

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 20, 2019
@roland-d roland-d added this to the Joomla 4.0 milestone May 23, 2019
@roland-d roland-d merged commit ebef846 into joomla:4.0-dev May 23, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 23, 2019
@roland-d
Copy link
Contributor

Thank you.

As stated this solves a different issue than #24814

@chmst chmst deleted the fix-workflows-ordering-by-condition branch October 5, 2019 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants