From: Tim Otten <totten@civicrm.org>
Date: Mon, 14 Jan 2019 23:58:53 +0000 (-0800)
Subject: Follow-up security/core#25 - Consistently change interface
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=569b8347d8cfb54a24e018e4db72fd709cd9cfa1;p=civicrm-core.git

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 `[]`).
---

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.