CRM_Core_I18n - Fix usage of `getPath('[foo]/.')`
authorTim Otten <totten@civicrm.org>
Tue, 10 Mar 2020 10:44:32 +0000 (03:44 -0700)
committerTim Otten <totten@civicrm.org>
Tue, 10 Mar 2020 10:47:42 +0000 (03:47 -0700)
The interpretation of `/.` is evolving per #16735:

* When this code was first written, it was unspecified/variable whether the value `[foo]/.` would end in `/`
* During most of the testing of 5.23.beta1, this was defined to always return `/`
* During a regression fix in 5.23.1, we're flipping it back the other way so that `[foo]/.` never ends in `/`.

CRM/Core/I18n.php

index 1da5d28005514edabb6ab24f3bb277b4f6b82dc6..aff19c9f80d1f72a5805af9ef0dccdad3a698c9a 100644 (file)
@@ -282,7 +282,7 @@ class CRM_Core_I18n {
    * @return string
    */
   public static function getResourceDir() {
-    return \Civi::paths()->getPath('[civicrm.l10n]/.');
+    return CRM_Utils_File::addTrailingSlash(\Civi::paths()->getPath('[civicrm.l10n]/.'));
   }
 
   /**