Remove pass-by-reference
authoreileen <emcnaughton@wikimedia.org>
Tue, 2 Jan 2024 01:36:54 +0000 (14:36 +1300)
committereileen <emcnaughton@wikimedia.org>
Tue, 2 Jan 2024 01:48:57 +0000 (14:48 +1300)
We have removed a bunch of these (added for php4.x support) with no
issues - it's only where there is inheritance we have
hit issues

CRM/Utils/Date.php

index e4d89c0d332ce8ae6e6510cdf0752c4ffa5cb6b5..fcb0ec2337821e54a7b4212f3cadc68db8a4a4ad 100644 (file)
@@ -167,7 +167,7 @@ class CRM_Utils_Date {
    *   0-based array with abbreviated weekday names
    *
    */
-  public static function getAbbrWeekdayNames() {
+  public static function getAbbrWeekdayNames(): array {
     $key = 'abbrDays_' . \CRM_Core_I18n::getLocale();
     if (empty(\Civi::$statics[__CLASS__][$key])) {
       $intl_formatter = IntlDateFormatter::create(CRM_Core_I18n::getLocale(), IntlDateFormatter::MEDIUM, IntlDateFormatter::MEDIUM, NULL, IntlDateFormatter::GREGORIAN, 'E');
@@ -204,7 +204,7 @@ class CRM_Utils_Date {
    *   0-based array with full weekday names
    *
    */
-  public static function getFullWeekdayNames() {
+  public static function getFullWeekdayNames(): array {
     $key = 'fullDays_' . \CRM_Core_I18n::getLocale();
     if (empty(\Civi::$statics[__CLASS__][$key])) {
       $intl_formatter = IntlDateFormatter::create(CRM_Core_I18n::getLocale(), IntlDateFormatter::MEDIUM, IntlDateFormatter::MEDIUM, NULL, IntlDateFormatter::GREGORIAN, 'EEEE');
@@ -258,13 +258,12 @@ class CRM_Utils_Date {
   /**
    * Return abbreviated month names according to the locale.
    *
-   * @param bool $month
+   * @param bool|string $month
    *
-   * @return array
+   * @return array|string
    *   1-based array with abbreviated month names
-   *
    */
-  public static function &getAbbrMonthNames($month = FALSE) {
+  public static function getAbbrMonthNames($month = FALSE) {
     $key = 'abbrMonthNames_' . \CRM_Core_I18n::getLocale();
     if (empty(\Civi::$statics[__CLASS__][$key])) {
       // Note: IntlDateFormatter provides even more strings than `strftime()` or `l10n/*/civicrm.mo`.
@@ -291,7 +290,7 @@ class CRM_Utils_Date {
    *   1-based array with full month names
    *
    */
-  public static function &getFullMonthNames() {
+  public static function getFullMonthNames(): array {
     $key = 'fullMonthNames_' . \CRM_Core_I18n::getLocale();
     if (empty(\Civi::$statics[__CLASS__][$key])) {
       // Note: IntlDateFormatter provides even more strings than `strftime()` or `l10n/*/civicrm.mo`.