Skip to content

Use new event trigger and more#582

Merged
joomdonation merged 2 commits intojoomla-extensions:5.x-devfrom
joomdonation:use_new_event_trigger
Mar 22, 2025
Merged

Use new event trigger and more#582
joomdonation merged 2 commits intojoomla-extensions:5.x-devfrom
joomdonation:use_new_event_trigger

Conversation

@joomdonation
Copy link
Copy Markdown
Contributor

@joomdonation joomdonation commented Mar 22, 2025

Pull Request for Issue # .

Summary of Changes

This PR does several things :

  • Replace the deprecated legacy $app->triggerEvent by the new event trigger
  • Replace the deprecated Toolbar::getInstance by $this->getDocument()->getToolbar()
  • Raise minimum Joomla requirement to 5.3. For 5.x-dev, we would only focus on the development for latest version of Joomla 5 and upcoming Joomla 6.

Testing Instructions

  • Code review
  • Tests pass

Expected result

Actual result

Documentation Changes Required

@joomdonation joomdonation merged commit 9258e8d into joomla-extensions:5.x-dev Mar 22, 2025
1 check passed
@joomdonation joomdonation deleted the use_new_event_trigger branch March 22, 2025 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@joomdonation