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

com_contact. Rm unused order_up, order_dn #42581

Merged
merged 4 commits into from Jan 24, 2024
Merged

com_contact. Rm unused order_up, order_dn #42581

merged 4 commits into from Jan 24, 2024

Conversation

ReLater
Copy link
Contributor

@ReLater ReLater commented Dec 31, 2023

order_up and order_dn are not used in Joomla core. Garbage from Joomla 1.6 or so.

Summary of Changes

Removed lines from com_contact list view.

// Preprocess the list of items to find ordering divisions.
// @todo: Complete the ordering stuff with nested sets
foreach ($this->items as &$item) {
 $item->order_up = true;
 $item->order_dn = true;
}

Testing Instructions

Actual result BEFORE applying this Pull Request

  • Unused garbage in file.

Expected result AFTER applying this Pull Request

  • No functional changes. Garbage removed.

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

`order_up` and `order_dn` are not used in Joomla core. Garbage from Joomla 1.6 or so.
@Quy
Copy link
Contributor

Quy commented Dec 31, 2023

I have tested this item ✅ successfully on f1ac63e


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

@ChristineWk
Copy link

I have tested this item ✅ successfully on f1ac63e

Joomla 5.1.0-alpha2


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

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 1, 2024
@richard67
Copy link
Member

RTC as it has 2 successful tests ... but it needs to be clarified with maintainers or release managers if this should be rebased to 5.1-dev.

@richard67 richard67 added the RMDQ ReleaseManagerDecisionQueue label Jan 1, 2024
@bembelimen bembelimen changed the base branch from 5.0-dev to 5.1-dev January 2, 2024 23:33
@bembelimen
Copy link
Contributor

Yes, 5.1 please

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.1-dev Unit/System Tests labels Jan 2, 2024
@Quy Quy removed the PR-5.0-dev label Jan 3, 2024
@LadySolveig LadySolveig added this to the Joomla! 5.1.0 milestone Jan 11, 2024
@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators Unit/System Tests NPM Resource Changed This Pull Request can't be tested by Patchtester labels Jan 11, 2024
@LadySolveig LadySolveig merged commit de23670 into joomla:5.1-dev Jan 24, 2024
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 24, 2024
@LadySolveig
Copy link
Contributor

Thank you @ReLater and also for testing @ChristineWk and @Quy !

@ReLater ReLater deleted the patch-1 branch January 24, 2024 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-5.1-dev RMDQ ReleaseManagerDecisionQueue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants