Skip to content

Add prepared statements for plg_content_pagenavigation#16

Closed
HLeithner wants to merge 6 commits into4.0-devfrom
prepared-plg-content-pagenavigation
Closed

Add prepared statements for plg_content_pagenavigation#16
HLeithner wants to merge 6 commits into4.0-devfrom
prepared-plg-content-pagenavigation

Conversation

@HLeithner
Copy link
Copy Markdown
Owner

@HLeithner HLeithner commented May 6, 2019

Summary of Changes

Updated SQL queries to prepared statements and made some cleanups around the queries.

Testing Instructions

Use the plugin in all ways you can think of.

  • Activate the page navigation feature
  • create multiple articles in the same category
  • check the frontend if you can navigate to the next article
  • set the ordering in the menu and the global options, try different dates (create,published, modified)

Expected result

Nothing changed.

@HLeithner HLeithner marked this pull request as ready for review May 6, 2019 18:02
@HLeithner HLeithner closed this Jul 2, 2019
HLeithner pushed a commit that referenced this pull request Sep 29, 2021
* Class per event

* Argument validations

* global
HLeithner added a commit that referenced this pull request Jan 31, 2022
* Add some events when fetching media data

* Use dispatcher to trigger event

* Use an event instance

* Class per event (#15)

* Argument validations and getter (#16)

* Class per event

* Argument validations

* global

* Use result from events

* Copy year

* Use internal variable

* void

* adapt model

* Validate events (#17)

* Revert commit ec8b4c8 ccd02cb and 9a2a119

* Validate and immutable events

* Update administrator/components/com_media/src/Event/FetchMediaFileEvent.php

Co-authored-by: Harald Leithner <leithner@itronic.at>

* Update administrator/components/com_media/src/Event/FetchMediaFilesEvent.php

Co-authored-by: Harald Leithner <leithner@itronic.at>

* Rename the events

* Cleanup events

Co-authored-by: Fedir Zinchuk <getthesite@gmail.com>
Co-authored-by: Harald Leithner <leithner@itronic.at>
HLeithner pushed a commit that referenced this pull request Jan 16, 2023
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

Successfully merging this pull request may close these issues.

1 participant