From 569b8347d8cfb54a24e018e4db72fd709cd9cfa1 Mon Sep 17 00:00:00 2001 From: Tim Otten Date: Mon, 14 Jan 2019 15:58:53 -0800 Subject: [PATCH] Follow-up security/core#25 - Consistently change interface The previous commit 4c1e702f96403bdc84b6900027d1be61ea601321 expanded the signature of `fillWithSql()` to accept a third argument, but it wasn't consistent about whether the third argument was optional or required. This makes it consistently optional (default `[]`). --- CRM/Core/PrevNextCache/Interface.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CRM/Core/PrevNextCache/Interface.php b/CRM/Core/PrevNextCache/Interface.php index 6b72908318..aa6772c99b 100644 --- a/CRM/Core/PrevNextCache/Interface.php +++ b/CRM/Core/PrevNextCache/Interface.php @@ -44,7 +44,7 @@ interface CRM_Core_PrevNextCache_Interface { * An array of SQLParams to be used with the $sql * @return bool */ - public function fillWithSql($cacheKey, $sql, $sqlParams); + public function fillWithSql($cacheKey, $sql, $sqlParams = []); /** * Store the contents of an array in the cache. -- 2.25.1