Skip to content

Prepared plg content joomla#15

Closed
HLeithner wants to merge 7 commits into4.0-devfrom
prepared-plg-content-joomla
Closed

Prepared plg content joomla#15
HLeithner wants to merge 7 commits into4.0-devfrom
prepared-plg-content-joomla

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.

  • Create an article in a category
  • try to delete the category, you should get an error that the category is not empty
  • create an article in the frontend, you should get a e-mail notification

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 HLeithner deleted the prepared-plg-content-joomla branch July 2, 2019 22:06
HLeithner pushed a commit that referenced this pull request Sep 29, 2021
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