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] Categories don't contain only articles #33186

Closed
wants to merge 1 commit into from

Conversation

Bakual
Copy link
Contributor

@Bakual Bakual commented Apr 18, 2021

Batch copying a category allows to reverse order all articles in the selected categories. But categories are not only for articles. It can be anything. So "items" is a better term.

Summary of Changes

Changing the string to order items, not articles.

Testing Instructions

Go to any categories manager (eg articles categories, banner categories, contact categories), select at least one category and then select "Batch" from the "Actions" dropdown.

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

None

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.0-dev labels Apr 18, 2021
@brianteeman
Copy link
Contributor

Are you sure that the bug is not the text but that it should only be shown when the context is com_content? In J3 it only displays in com_content

@Bakual
Copy link
Contributor Author

Bakual commented Apr 18, 2021

Oops, that could be the case. I just assumed it works, but didn't test it. I'll check that and close this PR in the meantime.

@Bakual Bakual closed this Apr 18, 2021
@Bakual Bakual deleted the 4_ItemsNotArticles branch April 18, 2021 16:24
@Bakual
Copy link
Contributor Author

Bakual commented Apr 18, 2021

@brianteeman You're right, it only affects articles. But the bug is also in J3, as it shows in all category types, not only com_content. It probably shouldn't have been implemented in the com_category code 😄

@Bakual
Copy link
Contributor Author

Bakual commented Apr 18, 2021

New PR: #33190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants