OpencachingDeutschland / oc-server3

htdocs/lib2/search/ftsearch.inc.php

Spacing +2 added lines, -2 removed lines

@@ -199,7 +199,7 @@ 
     $str = '';
 
     ftsearch_load_ignores();
-    for ($i = count($astr) - 1; $i >= 0; $i --) {
+    for ($i = count($astr) - 1; $i >= 0; $i--) {
         // ignore?
         if (array_search(mb_strtolower($astr[$i]), $ftsearch_ignores) !== false) {
             unset($astr[$i]);
@@ -254,7 +254,7 @@ 
 
     // doppelte chars ersetzen
     $ordZ = ord('z');
-    for ($c = ord('a'); $c <= $ordZ; $c ++) {
+    for ($c = ord('a'); $c <= $ordZ; $c++) {
         $old_str = '';
         while ($old_str !== $str) {
             $old_str = $str;

Doc Comments +5 added lines, -5 removed lines

@@ -68,9 +68,9 @@ 
 /**
  * str = single word
  * @param $str
- * @param $simple
+ * @param boolean $simple
  *
- * @return array
+ * @return string
  */
 function ftsearch_split(&$str, $simple)
 {
@@ -141,7 +141,7 @@ 
  * str = single word
  * @param $str
  *
- * @return mixed|string
+ * @return string
  */
 function ftsearch_text2simple($str)
 {
@@ -171,7 +171,7 @@ 
  * str = single word
  * @param $str
  *
- * @return mixed|string
+ * @return string
  */
 function ftsearch_text2sort($str)
 {
@@ -314,7 +314,7 @@ 
 /**
  * @param $text
  *
- * @return mixed|string
+ * @return string
  */
 function ftsearch_strip_html($text)
 {

htdocs/src/Oc/Util/ProcessSync.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
             $pidDaemon = fgets($pidFile, 20);
             fclose($pidFile);
 
-            $pidDaemon = (int) $pidDaemon;
+            $pidDaemon = (int)$pidDaemon;
 
             // bad PID file, e.g. due to system malfunction while creating the file?
             if ($pidDaemon <= 0) {

htdocs/lib2/ss_zip.class.php

Indentation +1 added lines, -2 removed lines

@@ -34,8 +34,7 @@
     public $cdir = [];
 
     /**    constructor
-
-*
+     *
 *@param string $zipFile if not empty must contain path to valid zip file, ss_zip will try to open and parse it.
      *               If this parameter is empty, the new empty zip archive is created. This parameter has no meaning in
      *               LIGHT verion, please upgrade to PROfessional version.

Spacing +9 added lines, -9 removed lines

@@ -141,9 +141,9 @@ 
         }
 
         $fnl = strlen($filename);
-        $fh = "\x14\x00";    // ver needed to extract
-        $fh .= "\x00\x00";    // gen purpose bit flag
-        $fh .= "\x08\x00";    // compression method
+        $fh = "\x14\x00"; // ver needed to extract
+        $fh .= "\x00\x00"; // gen purpose bit flag
+        $fh .= "\x08\x00"; // compression method
         $fh .= "\x00\x00\x00\x00"; // last mod time and date
         $fh .=
             pack(
@@ -170,16 +170,16 @@ 
                 'va*v3V2',
                 0,
                 $fh,
-                0,        // file comment length
-                0,        // disk number start
-                0,        // internal file attributes
-                $attr,    // external file attributes - 'archive/directory' bit set
+                0, // file comment length
+                0, // disk number start
+                0, // internal file attributes
+                $attr, // external file attributes - 'archive/directory' bit set
                 $this->offset
             ) . $filename;
 
         $this->offset += 42 + $fnl + $gzsize;
         $this->cdir[] = $cdir;
-        $this->cnt ++;
+        $this->cnt++;
         $this->idx = $this->cnt - 1;
     }
 
@@ -227,7 +227,7 @@ 
                 $cdsl, // size of central dir
                 $zdsl, // offset to start of central dir
                 0
-            );              // .zip file comment length
+            ); // .zip file comment length
         return $this->zipfile;
     }
 

htdocs/lib2/Net/IDNA2.php

Spacing +12 added lines, -14 removed lines

@@ -2322,8 +2322,7 @@ 
         // Forcing conversion of input to UCS4 array
         // If one time encoding is given, use this, else the objects property
         switch (($one_time_encoding) ? $one_time_encoding : $this->_api_encoding) {
-            case 'utf8':
-                $decoded = $this->_utf8_to_ucs4($decoded);
+            case 'utf8' : $decoded = $this->_utf8_to_ucs4($decoded);
                 break;
             case 'ucs4_string':
                 $decoded = $this->_ucs4_string_to_ucs4($decoded);
@@ -2478,8 +2477,7 @@ 
         // The output is UTF-8 by default, other output formats need conversion here
         // If one time encoding is given, use this, else the objects property
         switch (($one_time_encoding) ? $one_time_encoding : $this->_api_encoding) {
-            case 'utf8':
-                return $return;
+            case 'utf8' : return $return;
                 break;
             case 'ucs4_string':
                 return $this->_ucs4_to_ucs4_string($this->_utf8_to_ucs4($return));
@@ -2714,12 +2712,12 @@ 
                     break;
                 }
 
-                $w = (int) ($w * ($this->_base - $t));
+                $w = (int)($w * ($this->_base - $t));
             }
 
             $bias = $this->_adapt($idx - $old_idx, $deco_len + 1, $is_first);
             $is_first = false;
-            $char += (int) ($idx / ($deco_len + 1));
+            $char += (int)($idx / ($deco_len + 1));
             $idx %= ($deco_len + 1);
 
             if ($deco_len > 0) {
@@ -2746,14 +2744,14 @@ 
      */
     private function _adapt($delta, $npoints, $is_first)
     {
-        $delta = (int) ($is_first ? ($delta / $this->_damp) : ($delta / 2));
-        $delta += (int) ($delta / $npoints);
+        $delta = (int)($is_first ? ($delta / $this->_damp) : ($delta / 2));
+        $delta += (int)($delta / $npoints);
 
         for ($k = 0; $delta > (($this->_base - $this->_tmin) * $this->_tmax) / 2; $k += $this->_base) {
-            $delta = (int) ($delta / ($this->_base - $this->_tmin));
+            $delta = (int)($delta / ($this->_base - $this->_tmin));
         }
 
-        return (int) ($k + ($this->_base - $this->_tmin + 1) * $delta / ($delta + $this->_skew));
+        return (int)($k + ($this->_base - $this->_tmin + 1) * $delta / ($delta + $this->_skew));
     }
 
     /**
@@ -2899,8 +2897,8 @@ 
 
         $result = [];
         $T = $this->_tbase + $sindex % $this->_tcount;
-        $result[] = (int) ($this->_lbase + $sindex / $this->_ncount);
-        $result[] = (int) ($this->_vbase + ($sindex % $this->_ncount) / $this->_tcount);
+        $result[] = (int)($this->_lbase + $sindex / $this->_ncount);
+        $result[] = (int)($this->_vbase + ($sindex % $this->_ncount) / $this->_tcount);
 
         if ($T != $this->_tbase) {
             $result[] = $T;
@@ -3141,7 +3139,7 @@ 
                     throw new UnexpectedValueException('This might be UTF-8, but I don\'t understand it at byte ' . $k);
                 }
                 if ('add' == $mode) {
-                    $output[$out_len] = (int) $v;
+                    $output[$out_len] = (int)$v;
                     ++$out_len;
                     continue;
                 }
@@ -3361,7 +3359,7 @@ 
             return mb_strlen($string, '8bit');
         }
 
-        return strlen((binary) $string);
+        return strlen((binary)$string);
     }
 
     // }}}}

htdocs/lib2/search/search.xml.inc.php

Doc Comments +1 added lines, -1 removed lines

@@ -205,7 +205,7 @@
 
 /**
  * @param string $str
- * @return mixed
+ * @return string
  */
 function lf2crlf($str)
 {

htdocs/translate.php

Switch Indentation +100 added lines, -100 removed lines

@@ -86,83 +86,83 @@ 
 }
 
 switch ($action) {
-    case 'selectlang':
-        break;
+        case 'selectlang':
+            break;
 
-    case 'verify':
-        verify();
-        break;
+        case 'verify':
+            verify();
+            break;
 
-    case 'resetids':
-        resetIds();
-        break;
+        case 'resetids':
+            resetIds();
+            break;
 
-    case 'clearcache':
-        clearCache();
-        break;
+        case 'clearcache':
+            clearCache();
+            break;
 
-    case 'export':
-        export();
-        break;
+        case 'export':
+            export();
+            break;
 
-    case 'xmlexport':
-        xmlexport();
-        break;
+        case 'xmlexport':
+            xmlexport();
+            break;
 
-    case 'xmlimport':
-        break;
+        case 'xmlimport':
+            break;
 
-    case 'xmlimport2':
-        xmlimport2();
-        break;
+        case 'xmlimport2':
+            xmlimport2();
+            break;
 
-    case 'xmlimport3':
-        xmlimport3();
-        break;
+        case 'xmlimport3':
+            xmlimport3();
+            break;
 
-    case 'textexportnew':
-        textexport($translang, false);
-        break;
+        case 'textexportnew':
+            textexport($translang, false);
+            break;
 
-    case 'textexportall':
-        textexport($translang, true);
-        break;
+        case 'textexportall':
+            textexport($translang, true);
+            break;
 
-    case 'textimport':
-        break;
+        case 'textimport':
+            break;
 
-    case 'textimport2':
-        textimport($translang);
-        break;
+        case 'textimport2':
+            textimport($translang);
+            break;
 
-    case 'edit':
-        if (!$access->mayTranslate($translang)) {
-            $tpl->error(ERROR_NO_ACCESS);
-        }
-        edit();
-        break;
+        case 'edit':
+            if (!$access->mayTranslate($translang)) {
+                $tpl->error(ERROR_NO_ACCESS);
+            }
+            edit();
+            break;
 
-    case 'copy_en':
-        copy_english_texts();
-        break;
+        case 'copy_en':
+            copy_english_texts();
+            break;
 
-    case 'listfaults':
-        $trans = sql(
-            'SELECT `sys_trans`.`id`,
+        case 'listfaults':
+            $trans = sql(
+                'SELECT `sys_trans`.`id`,
                     `sys_trans`.`text`
              FROM `sys_trans`
              LEFT JOIN `sys_trans_ref`
                 ON `sys_trans`.`id`=`sys_trans_ref`.`trans_id`
              WHERE ISNULL(`sys_trans_ref`.`trans_id`)
              ORDER BY `sys_trans`.`id` DESC'
-        );
-        $tpl->assign_rs('trans', $trans);
-        sql_free_result($trans);
-        break;
+            );
+            $tpl->assign_rs('trans', $trans);
+            sql_free_result($trans);
+            break;
 
-    case 'listall':
-        $trans = sql(
-            "SELECT `sys_trans`.`id`,
+        case 'listall':
+            $trans = sql(
+                "SELECT `sys_trans`.`id`,
                     `sys_trans`.`text`,
                     `sys_trans_text`.`text` AS `trans`
              FROM `sys_trans`
@@ -170,48 +170,48 @@ 
                ON `sys_trans`.`id`=`sys_trans_text`.`trans_id`
                AND `sys_trans_text`.`lang`='&1'
              ORDER BY `sys_trans`.`id` DESC",
-            $translang
-        );
-        $tpl->assign_rs('trans', $trans);
-        sql_free_result($trans);
-        break;
+                $translang
+            );
+            $tpl->assign_rs('trans', $trans);
+            sql_free_result($trans);
+            break;
 
-    case 'remove':
-        if (!$access->mayTranslate($translang)) {
-            $tpl->error(ERROR_NO_ACCESS);
-        }
-        remove();
-        break;
-
-    case 'scan':
-        scan();
-        break;
-
-    case 'scanstart':
-        scanStart();
-        break;
-
-    case 'scanfile':
-        $filename = isset($_REQUEST['filename']) ? $_REQUEST['filename'] : '';
-        scanFile($filename);
-        exit;
-
-    case 'quicknone':
-        $cookie->un_set('translate_mode');
-        break;
-
-    case 'quicknew':
-        $cookie->set('translate_mode', 'new');
-        break;
-
-    case 'quickall':
-        $cookie->set('translate_mode', 'all');
-        break;
-
-    default:
-        $action = 'listnew';
-        $trans = sql(
-            "SELECT DISTINCT `sys_trans`.`id`,
+        case 'remove':
+            if (!$access->mayTranslate($translang)) {
+                $tpl->error(ERROR_NO_ACCESS);
+            }
+            remove();
+            break;
+
+        case 'scan':
+            scan();
+            break;
+
+        case 'scanstart':
+            scanStart();
+            break;
+
+        case 'scanfile':
+            $filename = isset($_REQUEST['filename']) ? $_REQUEST['filename'] : '';
+            scanFile($filename);
+            exit;
+
+        case 'quicknone':
+            $cookie->un_set('translate_mode');
+            break;
+
+        case 'quicknew':
+            $cookie->set('translate_mode', 'new');
+            break;
+
+        case 'quickall':
+            $cookie->set('translate_mode', 'all');
+            break;
+
+        default:
+            $action = 'listnew';
+            $trans = sql(
+                "SELECT DISTINCT `sys_trans`.`id`,
                              `sys_trans`.`text`
              FROM `sys_trans`
              LEFT JOIN `sys_trans_text`
@@ -222,10 +222,10 @@ 
              WHERE ISNULL(`sys_trans_text`.`trans_id`)
                OR `sys_trans_text`.`text`=''
              ORDER BY `sys_trans`.`id` DESC",
-            $translang
-        );
-        $tpl->assign_rs('trans', $trans);
-        sql_free_result($trans);
+                $translang
+            );
+            $tpl->assign_rs('trans', $trans);
+            sql_free_result($trans);
 }
 
 $languages = [];

local/maintenance/email_recovery.php

Spacing +1 added lines, -1 removed lines

@@ -300,7 +300,7 @@
         while ($r = sql_fetch_assoc($rs)) {
             $user = new user($r['user_id']);
             $user->sendRegistrationCode();
-            ++ $n;
+            ++$n;
         }
         sql_free_result($rs);
         self::message(0, $n . ' users have been re-sent the activation code');

Indentation +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
             'SELECT COUNT(*)
              FROM `' . $table . '`
              WHERE ' . $this->getDateCondition($table, $dateField) .
-                       ($andWhere ? ' AND (' . $andWhere . ')' : ''),
+                        ($andWhere ? ' AND (' . $andWhere . ')' : ''),
             0
         );
         self::message(1, $objectCount . ' ' . $description);

Doc Comments +9 added lines

@@ -81,6 +81,10 @@ 
         return preg_match('/^[0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:[0-9]{2}$/', $datetime);
     }
 
+    /**
+     * @param integer $level
+     * @param string $text
+     */
     private static function message($level, $text): void
     {
         echo str_repeat(' ', 2 * $level) . $text . "\n";
@@ -93,6 +97,11 @@ 
             ' AND `' . $table . '`.`' . $field . "` <= '" . sql_escape($this->toDateTime) . "'";
     }
 
+    /**
+     * @param string $table
+     * @param string $dateField
+     * @param string $description
+     */
     private function showObjectCount($table, $dateField, $description, $andWhere = ''): void
     {
         $objectCount = sql_value(

local/mailing/mailing.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
 foreach ($sendTo as $receiver) {
     $receiver = trim($receiver);
-    echo ++ $n . "/$total: $receiver";
+    echo ++$n . "/$total: $receiver";
     mail(
         $receiver,
         $subject,

htdocs/lib/settings-dist.inc.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
 
 /* default locale
  */
-$opt['template']['default']['locale'] = 'DE';   // can be overwritten by $opt['domain'][<domain>]['locale']
+$opt['template']['default']['locale'] = 'DE'; // can be overwritten by $opt['domain'][<domain>]['locale']
 
 // include all locale settings
 require_once __DIR__ . '/../config2/locale.inc.php';