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

Joomla 5: Joomla Articles particle issue with Pagination #3239

Open
tonamphong opened this issue Mar 21, 2024 · 3 comments
Open

Joomla 5: Joomla Articles particle issue with Pagination #3239

tonamphong opened this issue Mar 21, 2024 · 3 comments

Comments

@tonamphong
Copy link

Hi there,
There's an issue with the Pagination (Joomla Articles particle) and Joomla 5 when using the Gantry 5 particle as a Module (No problem via Layout)
Steps:

  • [Back-end] Create a Joomla module with Gantry 5 Particles. Site Modules -> New -> Gantry 5 Particle -> Pick Joomla Articles -> select Show for Pagination option
  • [Font-end] Click Next button to see more articles
  • Result: Joomla\CMS\Event\Module\RenderModuleEvent::onSetSubject(): Argument Add a Gitter chat badge to README.md #1 ($value) must be of type object, null given, called in /Users/dev/Sites/j5_test/libraries/src/Event/AbstractEvent.php on line 225

Joomla 4 is working fine, whether you use the Joomla articles particle via module or layout.
Thank you

@pablop76
Copy link

pablop76 commented Mar 21, 2024

I don't know why, but $this doesn't return the class it's in. Changing $this to AbstractEvent::class or 'AbstractEvent' fixes the problem.
line 224 in libraries\src\Event\AbstractEvent.php

if (method_exists(AbstractEvent::class, $methodName1))
or
if (method_exists('AbstractEvent', $methodName1))

hexplor added a commit that referenced this issue Mar 29, 2024
…cle Joomla article - Joomla 5 - Error on previous/next button #3218
@hexplor
Copy link
Collaborator

hexplor commented Mar 29, 2024

@tonamphong I fixed that in Gantry CI builds, should be available in a few minutes for download at: https://gantry.org/downloads. Can you test if it's fixed?

@tonamphong
Copy link
Author

Thank you, it worked :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants