Merge pull request #760 from tractorcow-farm/bugfix/legacy-permissions

BUG: Legacy permissions migration fix.
tractorcow-farm · Mar 27, 2022 · 4ba59a5 · 4ba59a5
2 parents f630298 + 5dcfe80
commit 4ba59a5
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Model/Locale.php b/src/Model/Locale.php
@@ -137,16 +137,18 @@ public function populateDefaults()
      */
     protected static $locales_by_title;
 
-    public function requireDefaultRecords()
+    public function requireDefaultRecords(): void
     {
         parent::requireDefaultRecords();
 
         // Migrate legacy permission codes to new codes
         $permissions = Permission::get()->filter('Code:StartsWith', 'CMS_ACCESS_Fluent_');
         $count = $permissions->count();
+
         if ($count) {
-            DB::alteration_message("Migrating ${$count} old fluent permissions", 'changed');
+            DB::alteration_message(sprintf('Migrating %d old fluent permissions', $count), 'changed');
         }
+
         foreach ($permissions as $permission) {
             $permission->Code = str_replace('CMS_ACCESS_Fluent_', 'Fluent_', $permission->Code);
             $permission->write();