Feature RTC PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
23 May 2025

Pull Request for Issue # .

Summary of Changes

This PR enhances the ListView class to:

  • have the modern toolbar respecting all the permissions
  • handle errors in a way, that the view does not crash
  • handle empty list behaviour

It updates the following core views:

  • ActionLogs view
  • Banners view
  • Contacts view
  • Modules view

as they're very straight forward

Testing Instructions

For testing this you have to check the 4 views mentioned above. It's important to play around with the component permission, e.g. remove "create" permissions and see if in old and in new the same buttons are visible.

Also test with super user account and without if the buttons of the components are still the same with the same behaviour.

Also delete all the items in the component to see, that the empty state behaviour is the same.

Expected result AFTER applying this Pull Request

Toolbars behaves the same

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

avatar bembelimen bembelimen - open - 23 May 2025
avatar bembelimen bembelimen - change - 23 May 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 May 2025
Category Administration com_banners com_contact com_modules Libraries
avatar bembelimen bembelimen - change - 23 May 2025
Labels Added: Feature PR-6.0-dev
avatar exlemor
exlemor - comment - 9 Aug 2025

@bembelimen, I wish I had good news...

When the PR is applied (6.0 nightly from this afternoon), the New and Actions button are NON functional on Banners, Contacts, Modules, ... yet if remove the Patch, they work perfectly fine... ;(

(for the ActionsLog, the Delete buttons are clickable and the Warning window appears but it doesn't actually delete entries :/

Console: (could be cumulative errors from the Banners, Contacts, Modules screens to be clear)

ScreenFloat Shot of Safari at Aug 9, 2025 at 6_28_43 PM

)

(sorry)

avatar bembelimen
bembelimen - comment - 11 Aug 2025

@exlemor the problems were the conflicts, I fixed them and you can test again please. Thank you, really appreciate it.

avatar exlemor
exlemor - comment - 11 Aug 2025

@exlemor the problems were the conflicts, I fixed them and you can test again please. Thank you, really appreciate it.

Hi @bembelimen, Benjamin,

When I try to turn on the PR via Patch Tester, I get this error message in Red:
The file marked for modification does not exist: phpstan-baseline.neon

Trying to apply it to a nightly of "‎6.0.0-alpha4-dev", I installed yesterday.

:( (sorry)

avatar brianteeman
brianteeman - comment - 11 Aug 2025

@exlemor that is expected as that file is not part of the distributed Joomla. You really need to learn how to use git and then you wont have these problems

avatar exlemor exlemor - test_item - 16 Aug 2025 - Tested successfully
avatar exlemor
exlemor - comment - 16 Aug 2025

I have tested this item ✅ successfully on 9673cb8

I have tested this successfully - thanks for the hard word and the fixes @bembelimen!


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

avatar Bodge-IT Bodge-IT - test_item - 16 Aug 2025 - Tested successfully
avatar Bodge-IT
Bodge-IT - comment - 16 Aug 2025

I have tested this item ✅ successfully on 9673cb8

Tested 4 views across 3 users with diff perms. I couldn't break it but learned a ton about permissions.


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

avatar richard67 richard67 - change - 16 Aug 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 16 Aug 2025

RTC


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

avatar softforge softforge - change - 16 Aug 2025
Labels Added: RTC
avatar softforge softforge - change - 16 Aug 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-08-16 13:51:22
Closed_By softforge
avatar softforge softforge - close - 16 Aug 2025
avatar softforge softforge - merge - 16 Aug 2025
avatar softforge
softforge - comment - 16 Aug 2025

Thank you for your work on this @bembelimen and the testers

Add a Comment

Login with GitHub to post a comment