flack / openpsa

src/midcom/grid/provider.php

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
      */
     private array $_search = [];
 
-    public function __construct(array|client $source, string $datatype = 'json')
+    public function __construct(array | client $source, string $datatype = 'json')
     {
         $this->_datatype = $datatype;
         if ($source instanceof client) {
@@ -202,7 +202,7 @@ 
 
     private function _convert_to_localdata() : string
     {
-        return "var " . $this->_grid->get_identifier() . '_entries = ' .  json_encode($this->get_rows()) . ";\n";
+        return "var " . $this->_grid->get_identifier() . '_entries = ' . json_encode($this->get_rows()) . ";\n";
     }
 
     private function _render_json()
@@ -233,7 +233,7 @@ 
             $this->_sort_field = $query['sidx'];
             $this->_sort_direction = strtoupper($query['sord'] ?? 'ASC');
         }
-        if (   !empty($query['_search'])
+        if (!empty($query['_search'])
             && $query['_search'] === 'true') {
             foreach ($query as $field => $value) {
                 if (in_array($field, ['_search', 'nd', 'page', 'rows', 'sidx', 'sord'])) {
@@ -256,7 +256,7 @@ 
 
         $this->_total_rows = $query->count();
 
-        if (   $this->_datatype == 'json'
+        if ($this->_datatype == 'json'
             && !empty($this->_results_per_page)) {
             $query->set_limit($this->_results_per_page);
             if (!empty($this->_offset)) {

lib/midgard/admin/asgard/schemadb/config.php

Spacing +3 added lines, -3 removed lines

@@ -58,11 +58,11 @@ 
                 $fields[$key] = $this->detect_schema($key, $value);
             }
 
-            if (   !isset($this->config->_local[$key])
+            if (!isset($this->config->_local[$key])
                 || $this->config->_local[$key] == $this->config->_global[$key]) {
                 // No local configuration setting, note to user that this is the global value
                 $fields[$key]['title'] = $schema->get_l10n()->get($fields[$key]['title']);
-                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') .")</span>";
+                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') . ")</span>";
             }
         }
 
@@ -118,7 +118,7 @@ 
     /**
      * Ensure the configuration is valid (form validation callback)
      */
-    public function check_config(array $values) : array|true
+    public function check_config(array $values) : array | true
     {
         $current = $this->config->get_all();
         $result = [];

lib/org/openpsa/calendar/validator.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      *
      * @return mixed Array with error message or true on success
      */
-    public function validate(array $input) : array|true
+    public function validate(array $input) : array | true
     {
         $this->event->busy = $input['busy'];
         $this->event->participants = array_flip($input['participants']);

lib/org/openpsa/user/validator.php

Spacing +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function validate_edit_form(array $fields) : array|true
+    public function validate_edit_form(array $fields) : array | true
     {
         $result = $this->is_username_available($fields);
 
@@ -54,11 +54,11 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function validate_create_form(array $fields) : array|true
+    public function validate_create_form(array $fields) : array | true
     {
         $result = $this->is_username_available($fields);
 
-        if (   $fields['password']['switch']
+        if ($fields['password']['switch']
             && !$this->get_accounthelper()->check_password_strength((string) $fields['password']['password'])) {
             $result = ['password' => $this->l10n->get('password weak')];
         }
@@ -75,7 +75,7 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function verify_existing_password(array $fields) : array|true
+    public function verify_existing_password(array $fields) : array | true
     {
         if (midcom::get()->auth->can_user_do('org.openpsa.user:manage', class: org_openpsa_user_interface::class)) {
             //User has the necessary rights, so we're good
@@ -99,7 +99,7 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function username_exists(array $fields) : array|true
+    public function username_exists(array $fields) : array | true
     {
         if ($this->is_username_available(['username' => $fields['username']]) === true) {
             return ["username" => $this->l10n->get("unknown username")];
@@ -113,7 +113,7 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function email_exists(array $fields) : array|true
+    public function email_exists(array $fields) : array | true
     {
         $result = [];
         $qb = new midgard_query_builder(midcom::get()->config->get('person_class'));
@@ -134,7 +134,7 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function email_and_username_exist(array $fields) : array|true
+    public function email_and_username_exist(array $fields) : array | true
     {
         $result = [];
         $user = midcom::get()->auth->get_user_by_name($fields["username"]);
@@ -157,15 +157,15 @@ 
      * @param array $fields The form's data
      * @return mixed True on success, array of error messages otherwise
      */
-    public function password_check(array $fields) : array|true
+    public function password_check(array $fields) : array | true
     {
         $result = [];
 
         $accounthelper = $this->get_accounthelper(new midcom_db_person($fields["person"]));
-        if (!$accounthelper->check_password_reuse($fields['new_password'])){
+        if (!$accounthelper->check_password_reuse($fields['new_password'])) {
             $result['password'] = $this->l10n->get('password was already used');
         }
-        if (!$accounthelper->check_password_strength($fields['new_password'])){
+        if (!$accounthelper->check_password_strength($fields['new_password'])) {
             $result['password'] = $this->l10n->get('password weak');
         }
         return $result ?: true;

lib/org/openpsa/user/accounthelper.php

Spacing +6 added lines, -6 removed lines

@@ -151,7 +151,7 @@ 
         $account = $this->get_account();
         if (!empty($new_password)) {
             //check if the new encrypted password was already used
-            if (   !$this->check_password_reuse($new_password, true)
+            if (!$this->check_password_reuse($new_password, true)
                 || !$this->check_password_strength($new_password, true)) {
                 $this->errstr = "password strength too low";
                 return false;
@@ -270,7 +270,7 @@ 
         $password_length = mb_strlen($password);
 
         if ($password_length < $this->_config->get('min_password_length')) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error');
             }
             return false;
@@ -288,7 +288,7 @@ 
         }
 
         if ($score < $this->_config->get('min_password_score')) {
-            if ($show_ui_message){
+            if ($show_ui_message) {
                 midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error');
             }
             return false;
@@ -432,7 +432,7 @@ 
 
     public static function get_person_by_formdata(array $data) : ?midcom_db_person
     {
-        if (   empty($data['username'])
+        if (empty($data['username'])
             || empty($data['password'])) {
             return null;
         }
@@ -463,7 +463,7 @@ 
         $max_attempts = midcom_baseclasses_components_configuration::get($component, 'config')->get('max_password_attempts');
         $timeframe = midcom_baseclasses_components_configuration::get($component, 'config')->get('password_block_timeframe_min');
 
-        if (   $max_attempts == 0
+        if ($max_attempts == 0
             || $timeframe == 0) {
             return $stat;
         }
@@ -485,7 +485,7 @@ 
          * If the maximum number of attempts is reached and the oldest attempt
          * on the stack is within our defined timeframe, we block the account
          */
-        if (   count($attempts) >= $max_attempts
+        if (count($attempts) >= $max_attempts
             && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) {
             $this->disable_account();
             $stat = false;

lib/org/openpsa/products/validation.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      *
      * @param array $fields The form's data
      */
-    public function is_code_available(array $fields) : array|true
+    public function is_code_available(array $fields) : array | true
     {
         $result = [];
         if (!empty($fields['id'])) {

lib/org/openpsa/sales/validator.php

Spacing +4 added lines, -4 removed lines

@@ -13,24 +13,24 @@
  */
 class org_openpsa_sales_validator
 {
-    public function validate_subscription(array $fields) : array|true
+    public function validate_subscription(array $fields) : array | true
     {
         $result = $this->validate_units($fields);
 
         if ($result === true) {
             $result = [];
         }
-        if (   empty($fields['end'])
+        if (empty($fields['end'])
             && empty($fields['continuous'])) {
             $result['end'] = midcom::get()->i18n->get_string('select either end date or continuous', 'org.openpsa.sales');
         }
         return $result ?: true;
     }
 
-    public function validate_units(array $fields) : array|true
+    public function validate_units(array $fields) : array | true
     {
         $result = [];
-        if (   empty($fields['invoiceByActualUnits'])
+        if (empty($fields['invoiceByActualUnits'])
             && empty($fields['plannedUnits'])) {
             $result['plannedUnits'] = midcom::get()->i18n->get_string('select either planned units or invoice by actual units', 'org.openpsa.sales');
             return $result;