? ? PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
24 Apr 2022

Summary of Changes

The module ordering controller task which is used in the ordering form field should use the model to fetch the modules for the same client and position.

Testing Instructions

Edit a module and click on the order form field.

Actual result BEFORE applying this Pull Request

The modules for the same client and position are listed.

Expected result AFTER applying this Pull Request

The modules for the same client and position are listed.

avatar laoneo laoneo - open - 24 Apr 2022
avatar laoneo laoneo - change - 24 Apr 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Apr 2022
Category Administration com_modules
avatar toivo
toivo - comment - 30 Apr 2022

I have tested this item successfully on 9d8af1d

Tested successfully in Joomla 4.2-alpha3-dev of 29 April in Wampserver 3.2.8 using PHP 8.0.15.

Had to change Error Reporting in Global Configuration from 'Maximum' to 'Simple' to get the Ordering info displayed, because of the PHP notice:
Notice: Only variable references should be returned by reference in C:\www\joomla42\libraries\src\MVC\Model\BaseDatabaseModel.php on line 418


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

avatar toivo toivo - test_item - 30 Apr 2022 - Tested successfully
avatar laoneo laoneo - change - 2 May 2022
Title
Get ordering from model in module controller
[4.2] Get ordering from model in module controller
avatar laoneo laoneo - edited - 2 May 2022
avatar laoneo laoneo - change - 2 May 2022
Labels Added: ?
avatar laoneo
laoneo - comment - 15 May 2022

Can you check if your branch is up to date with 4.2?

avatar roland-d
roland-d - comment - 18 Jun 2022

@laoneo This looks like something is broken in the PR, the list looks like this for me
image

Notice that the list restarts at 1 and this happens 3 times but the list is some other data every time.

avatar laoneo laoneo - change - 24 Jun 2022
Title
[4.2] Get ordering from model in module controller
[4.3] Get ordering from model in module controller
avatar laoneo laoneo - edited - 24 Jun 2022
avatar joomla-bot
joomla-bot - comment - 27 Jun 2022

A new pull request has been created automatically to convert this PR to the PSR-12 coding standard. The pr can be found at Digital-Peak#25

avatar laoneo laoneo - change - 28 Jun 2022
Labels Added: PR-4.3-dev ?
Removed: ?
avatar laoneo laoneo - change - 19 Jan 2023
Labels Removed: ?
avatar laoneo laoneo - change - 4 Apr 2023
Title
[4.3] Get ordering from model in module controller
Get ordering from model in module controller
avatar laoneo laoneo - edited - 4 Apr 2023
avatar laoneo laoneo - change - 4 Apr 2023
Labels Added: ? PR-4.4-dev
Removed: PR-4.3-dev
avatar laoneo
laoneo - comment - 4 Apr 2023

@roland-d can you test this again as I can't reproduce your issue at all?

avatar richard67 richard67 - test_item - 4 Apr 2023 - Tested successfully
avatar richard67
richard67 - comment - 4 Apr 2023

I have tested this item successfully on 0993076


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

avatar chmst chmst - test_item - 4 Apr 2023 - Tested successfully
avatar chmst
chmst - comment - 4 Apr 2023

I have tested this item successfully on 0993076


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

avatar chmst chmst - change - 4 Apr 2023
Status Pending Ready to Commit
avatar chmst
chmst - comment - 4 Apr 2023

RTC


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

avatar MacJoom MacJoom - change - 4 Apr 2023
Labels Added: ?
avatar MacJoom MacJoom - change - 4 Apr 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-04-04 13:22:52
Closed_By MacJoom
avatar MacJoom MacJoom - close - 4 Apr 2023
avatar MacJoom MacJoom - merge - 4 Apr 2023

Add a Comment

Login with GitHub to post a comment