Skip to content

Prepared plg content contact#17

Closed
HLeithner wants to merge 15 commits into4.0-devfrom
prepared-plg-content-contact
Closed

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

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 users
  • Create contacts linked to the users
  • Create articles linked to different users
  • Contact should be shown at the article
  • use different settings in the plugin

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 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