Skip to content

Add prepared statements for plg_fields-sql#21

Closed
HLeithner wants to merge 3 commits into4.0-devfrom
prepared-plg-fields-sql
Closed

Add prepared statements for plg_fields-sql#21
HLeithner wants to merge 3 commits into4.0-devfrom
prepared-plg-fields-sql

Conversation

@HLeithner
Copy link
Copy Markdown
Owner

@HLeithner HLeithner commented May 29, 2019

This PR tries to move the SQL custom field into a prepared statement, at least for the values part.

Summary of Changes

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

Testing Instructions

  • Create a SQL custom field
  • Use several SQL queries in the custom field
  • check the frontend if the expected result could be found.

Expected result

Nothing changed.

@HLeithner HLeithner closed this Jul 2, 2019
HLeithner pushed a commit that referenced this pull request Dec 3, 2020
HLeithner pushed a commit that referenced this pull request Oct 22, 2022
* Use the new option names

* Update KnownButtons.php

* Update ToolbarPresets.php
HLeithner pushed a commit that referenced this pull request Jan 16, 2023
HLeithner pushed a commit that referenced this pull request Oct 29, 2025
… tag name instead of the tag id (joomla#46174)

* update sql
* [CMS PR 46174] Update the update SQL scripts and for new installations supports.sql (#21)
* Names quotes for column names
* Remove some tabs
* Use separate lines for closing brackets
* Fix SQL error "operator does not exist"
* Append only once to array
* Update supports.sql for new installations
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