Changes the code around the permissions to comply with the new format.

This should mean that warnings are not thrown if and when the database is updated

Closes #5 (closed)

booking.php

@@ -148,9 +148,20 @@ function booking_civicrm_merge ($type, &$data, $mainId = NULL, $otherId = NULL,
 
 function booking_civicrm_permission(&$permissions) {
   $prefix = E::ts('CiviBooking') . ': ';
-  $permissions['administer CiviBooking'] = $prefix . E::ts('administer CiviBooking');
-  $permissions['create and update bookings'] = $prefix . E::ts('create and update bookings');
-  $permissions['view all bookings'] = $prefix . E::ts('view all bookings');
+  // Rewritten labels and descriptions following new docs: https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_permission/
+  // See issue: https://lab.civicrm.org/extensions/booking/-/issues/5
+  $permissions['administer CiviBooking'] = [
+    'label' => E::ts('Administer CiviBooking'), // Required
+    'description' => E::ts('Permission to administer the CiviBooking module'), // Optional but encouraged
+  ];
+  $permissions['create and update bookings'] = [
+    'label' => E::ts('Create and Update Bookings'), // Required
+    'description' => E::ts('Give a user permission to create and update bookings'), // Optional but encouraged
+  ];
+  $permissions['view all bookings'] = [
+    'label' => E::ts('View All Bookings'), // Required
+    'description' => E::ts('Give a user permission to view all bookings'), // Optional but encouraged
+  ];
 }
 
 /*