Follow-up security/core#25 - Consistently change interface
authorTim Otten <totten@civicrm.org>
Mon, 14 Jan 2019 23:58:53 +0000 (15:58 -0800)
committerSeamus Lee <seamuslee001@gmail.com>
Fri, 22 Feb 2019 00:09:07 +0000 (11:09 +1100)
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

index 6b72908318c0f6215ba78e926603a2e02bf4be2a..aa6772c99bdc5bf0cbed608df7ec89a42d7ac6bc 100644 (file)
@@ -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.