Skip to content

Conversation

@bembelimen
Copy link
Contributor

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

@exlemor
Copy link

exlemor commented Aug 9, 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)

@bembelimen
Copy link
Contributor Author

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

@exlemor
Copy link

exlemor commented Aug 11, 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)

@brianteeman
Copy link
Contributor

@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

@exlemor
Copy link

exlemor commented Aug 16, 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.

@Bodge-IT
Copy link
Contributor

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.

@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 16, 2025
@softforge softforge added this to the Joomla! 6.0.0 milestone Aug 16, 2025
@softforge softforge merged commit 61539c5 into joomla:6.0-dev Aug 16, 2025
30 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 16, 2025
@softforge
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants