[REF][PHP8.1] Fix a couple of deprecations in php8.1 by specifying th…

…at return type may change in BaseParamQuery and by ensuring that we don't pass null into function_exists in I18n class

Replace some ReturnTypeWillChange with later return types as they should be backward compatible

Dave D Fix

Completely switch to just not empty on the customTranslationFunction check

CRM/Core/I18n.php

@@ -801,7 +801,7 @@ function ts($text, $params = []) {
     if ($bootstrapReady) {
       // just got ready: determine whether there is a working custom translation function
       $config = CRM_Core_Config::singleton();
-      if (isset($config->customTranslateFunction) and function_exists($config->customTranslateFunction)) {
+      if (!empty($config->customTranslateFunction) && function_exists($config->customTranslateFunction)) {
         $function = $config->customTranslateFunction;
       }
     }

CRM/Utils/SQL/BaseParamQuery.php

@@ -184,7 +184,7 @@ public function param($keys, $value = NULL) {
    *
    * @return bool
    */
-  public function offsetExists($offset) {
+  public function offsetExists($offset): bool {
     return isset($this->params[$offset]);
   }
 
@@ -202,6 +202,7 @@ public function offsetExists($offset) {
    * @see param()
    * @see ArrayAccess::offsetGet
    */
+  #[ReturnTypeWillChange]
   public function offsetGet($offset) {
     return $this->params[$offset];
   }
@@ -223,7 +224,7 @@ public function offsetGet($offset) {
    * @see param()
    * @see ArrayAccess::offsetSet
    */
-  public function offsetSet($offset, $value) {
+  public function offsetSet($offset, $value): void {
     $this->param($offset, $value);
   }
 
@@ -234,7 +235,7 @@ public function offsetSet($offset, $value) {
    * @see param()
    * @see ArrayAccess::offsetUnset
    */
-  public function offsetUnset($offset) {
+  public function offsetUnset($offset): void {
     unset($this->params[$offset]);
   }