Skip to content

Commit

Permalink
Merge pull request #314 from creative-commoners/pulls/5/placeholders
Browse files Browse the repository at this point in the history
DOC Update namespace for use_placeholders_for_integer_ids
  • Loading branch information
GuySartorelli authored Aug 8, 2023
2 parents 03fe64a + 916cb8e commit 500e36a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion en/02_Developer_Guides/09_Security/08_SQL_Placeholders.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,6 @@ For increased performance, placeholders are not used when filtering by an array
There is no chance of SQL injection because of the exclusive use of integers for values. However, if you still wish for placeholders to be used for this type of query then you can enable them with the following config:

```yml
SilverStripe\ORM\Filters\ExactMatchFilter:
SilverStripe\ORM\DataList:
use_placeholders_for_integer_ids: true
```
2 changes: 1 addition & 1 deletion en/04_Changelogs/5.1.0.md
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ In the CMS, this new permission is available for files and pages by setting "Who
If you want to disable this optimisation you can do so with this configuration:

```yml
SilverStripe\ORM\Filters\ExactMatchFilter:
SilverStripe\ORM\DataList:
use_placeholders_for_integer_ids: true
```

Expand Down

0 comments on commit 500e36a

Please sign in to comment.