J4 Issue ? ?
avatar chmst
chmst
4 Oct 2018

libraries/cms/html/jgrid.php seems to be dammaged. A review should be made to be sure that parameters are correct and consistent.
Some PRS made missing buttons visible again, but the parameters still are not consitent - especially the parameter text.

avatar chmst chmst - open - 4 Oct 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Oct 2018
avatar joomla-cms-bot joomla-cms-bot - change - 4 Oct 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Oct 2018
avatar laoneo
laoneo - comment - 5 Oct 2018

Since when it is damaged?

avatar chmst
chmst - comment - 5 Oct 2018

I noticed missing icons the day before this issue https://issues.joomla.org/tracker/joomla-cms/22420

avatar SharkyKZ
SharkyKZ - comment - 5 Oct 2018

#22294. But some calls were missing arguments even before.

avatar chmst
chmst - comment - 5 Oct 2018

At the moment,the ordering column is not working. If I remove the parameter "text" here https://github.com/joomla/joomla-cms/blame/14bfb01821e4b60811f4a306c28f8e00940a6e1e/libraries/cms/html/jgrid.php#L423 and here https://github.com/joomla/joomla-cms/blame/14bfb01821e4b60811f4a306c28f8e00940a6e1e/libraries/cms/html/jgrid.php#L423 it works, but then the parameter text is not used at all.

avatar SharkyKZ
SharkyKZ - comment - 5 Oct 2018

Need to restore second $text parameter in orderUp() and orderDown() that was removed with #22424. The unneeded param was already removed with #22349.

avatar chmst
chmst - comment - 5 Oct 2018

Thank you very much. I have commited these changes. #22437. So this should do it. Just seems strange to have the same param two times.

avatar laoneo
laoneo - comment - 5 Oct 2018

In #22349 it was reduced from three times to only two times.

avatar brianteeman brianteeman - change - 8 Oct 2018
Labels Added: J4 Issue
avatar brianteeman brianteeman - labeled - 8 Oct 2018
avatar chmst
chmst - comment - 19 Nov 2018

Please close this, thank you.


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

avatar joomla-cms-bot joomla-cms-bot - edited - 19 Nov 2018
avatar joomla-cms-bot joomla-cms-bot - change - 19 Nov 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-11-19 21:39:26
Closed_By joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 19 Nov 2018
avatar Quy Quy - change - 19 Nov 2018
Closed_By joomla-cms-bot Quy
avatar joomla-cms-bot
joomla-cms-bot - comment - 19 Nov 2018

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/22501

Add a Comment

Login with GitHub to post a comment