Completed
Pull Request — development (#794)
by Nick
07:51 queued 03:49
created
htdocs/lib2/edithelper.inc.php 3 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
  * @param $oldDescMode
32 32
  * @param $descMode
33 33
  * @param $text
34
- * @return mixed|string
34
+ * @return string
35 35
  */
36 36
 function processEditorInput($oldDescMode, $descMode, $text, &$representText)
37 37
 {
@@ -54,9 +54,9 @@  discard block
 block discarded – undo
54 54
 
55 55
 /**
56 56
  * @param $text
57
- * @param $texthtml0
57
+ * @param boolean $texthtml0
58 58
  *
59
- * @return mixed|string
59
+ * @return string
60 60
  */
61 61
 function html2plaintext($text, $texthtml0)
62 62
 {
Please login to merge, or discard this patch.
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -6,8 +6,6 @@
 block discarded – undo
6 6
 
7 7
 // used in both lib1 and lib2 code
8 8
 
9
-use OcLegacy\Editor\EditorConstants;
10
-
11 9
 require_once __DIR__ . '/smiley.inc.php';
12 10
 
13 11
 
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@
 block discarded – undo
96 96
         // REDMINE-1249: Missing log text in mail notification
97 97
         // simpler solution that converts html to text as the previous class html2text emptied the text completely
98 98
         // implementation for line wrap, url's and probably more is missing
99
-        $text = preg_replace( "/\n\s+/", "\n", rtrim(html_entity_decode(strip_tags($text))));
99
+        $text = preg_replace("/\n\s+/", "\n", rtrim(html_entity_decode(strip_tags($text))));
100 100
 
101 101
         $text = str_replace(
102 102
             [
Please login to merge, or discard this patch.
htdocs/log.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -50,8 +50,8 @@  discard block
 block discarded – undo
50 50
     $fieldNoteService = AppKernel::Container()->get(FieldNoteService::class);
51 51
 
52 52
     $fieldNote = $fieldNoteService->fetchOneBy([
53
-        'id' => (int) $_GET['fieldnoteid'],
54
-        'user_id' => (int) $user->getUserId()
53
+        'id' => (int)$_GET['fieldnoteid'],
54
+        'user_id' => (int)$user->getUserId()
55 55
     ]);
56 56
 
57 57
     if ($fieldNote !== null) {
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     // check if masslog warning is accepted (in cookie)
98 98
     $masslogCookieSet = isset($_COOKIE['ocsuppressmasslogwarn']);
99 99
     if ($masslogCookieSet) {
100
-        $masslogCookieContent = (int) $_COOKIE['ocsuppressmasslogwarn'];
100
+        $masslogCookieContent = (int)$_COOKIE['ocsuppressmasslogwarn'];
101 101
     } else {
102 102
         // save masslog acception in cookie that expires on midnight if clicked
103 103
         if (isset($_REQUEST['suppressMasslogWarning']) && $_REQUEST['suppressMasslogWarning'] == 1) {
@@ -109,17 +109,17 @@  discard block
 block discarded – undo
109 109
     //   logtext, logtype, logday, logmonth, logyear
110 110
 
111 111
     $logText = isset($_POST['logtext']) ? trim($_POST['logtext']) : '';
112
-    $logType = isset($_REQUEST['logtype']) ? (int) $_REQUEST['logtype'] : null;
112
+    $logType = isset($_REQUEST['logtype']) ? (int)$_REQUEST['logtype'] : null;
113 113
     $logDateDay = isset($_POST['logday']) ? trim($_POST['logday']) : ($datesaved ? $defaultLogDay : date('d'));
114 114
     $logDateMonth = isset($_POST['logmonth']) ? trim($_POST['logmonth']) : ($datesaved ? $defaultLogMonth : date('m'));
115 115
     $logDateYear = isset($_POST['logyear']) ? trim($_POST['logyear']) : ($datesaved ? $defaultLogYear : date('Y'));
116 116
     $logTimeHour = isset($_POST['loghour']) ? trim($_POST['loghour']) : '';
117 117
     $logTimeMinute = isset($_POST['logminute']) ? trim($_POST['logminute']) : '';
118
-    $needsMaintenance = isset($_POST['needs_maintenance2']) ? (int) $_POST['needs_maintenance2'] : (isset($_POST['needs_maintenance']) ? (int) $_POST['needs_maintenance'] : 0);
119
-    $listingOutdated = isset($_POST['listing_outdated2']) ? (int) $_POST['listing_outdated2'] : (isset($_POST['listing_outdated']) ? (int) $_POST['listing_outdated'] : 0);
120
-    $confirmListingOk = isset($_POST['confirm_listing_ok']) ? (int) $_POST['confirm_listing_ok'] : 0;
121
-    $rateOption = isset($_POST['ratingoption']) ? (int) $_POST['ratingoption'] : 0;
122
-    $rateCache = isset($_POST['rating']) ? (int) $_POST['rating'] : 0;
118
+    $needsMaintenance = isset($_POST['needs_maintenance2']) ? (int)$_POST['needs_maintenance2'] : (isset($_POST['needs_maintenance']) ? (int)$_POST['needs_maintenance'] : 0);
119
+    $listingOutdated = isset($_POST['listing_outdated2']) ? (int)$_POST['listing_outdated2'] : (isset($_POST['listing_outdated']) ? (int)$_POST['listing_outdated'] : 0);
120
+    $confirmListingOk = isset($_POST['confirm_listing_ok']) ? (int)$_POST['confirm_listing_ok'] : 0;
121
+    $rateOption = isset($_POST['ratingoption']) ? (int)$_POST['ratingoption'] : 0;
122
+    $rateCache = isset($_POST['rating']) ? (int)$_POST['rating'] : 0;
123 123
     $ocTeamComment = isset($_REQUEST['teamcomment']) ? $_REQUEST['teamcomment'] != 0 : 0;
124 124
     $suppressMasslogWarning = isset($_REQUEST['suppressMasslogWarning']) ? $_REQUEST['suppressMasslogWarning'] : ($masslogCookieSet ? $masslogCookieContent : 0);
125 125
 
@@ -165,12 +165,12 @@  discard block
 block discarded – undo
165 165
     }
166 166
 
167 167
     // if not a found log, ignore the rating
168
-    $rateOption = (int) ($logType == 1 || $logType == 7);
168
+    $rateOption = (int)($logType == 1 || $logType == 7);
169 169
 
170 170
     // get log text editor mode (from form or from userprofile)
171 171
     // 1 = text; 2 = HTML; 3 = tinyMCE
172 172
     if (isset($_POST['descMode'])) {
173
-        $descMode = (int) $_POST['descMode'];  // Ocprop: 2
173
+        $descMode = (int)$_POST['descMode']; // Ocprop: 2
174 174
         if (($descMode < EditorConstants::HTML_MODE) || ($descMode > EditorConstants::EDITOR_MODE)) {
175 175
             $descMode = EditorConstants::EDITOR_MODE;
176 176
         }
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
          * set seconds 00:00:01, means "00:00 was logged"
249 249
          * set seconds 00:00:00, means "no time was logged"
250 250
          */
251
-        $logTimeSecond = (int) ($logTimeHour . $logTimeMinute != ''
251
+        $logTimeSecond = (int)($logTimeHour . $logTimeMinute != ''
252 252
                 && $logTimeHour == 0
253 253
                 && $logTimeMinute == 0);
254 254
 
@@ -256,8 +256,8 @@  discard block
 block discarded – undo
256 256
         $logDate = date(
257 257
             $opt['db']['dateformat'],
258 258
             mktime(
259
-                (int) $logTimeHour,
260
-                (int) $logTimeMinute,
259
+                (int)$logTimeHour,
260
+                (int)$logTimeMinute,
261 261
                 $logTimeSecond,
262 262
                 $logDateMonth,
263 263
                 $logDateDay,
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
             if (isset($_REQUEST['fieldnoteid']) && $_REQUEST['fieldnoteid']) {
309 309
                 sql(
310 310
                     'DELETE FROM field_note WHERE `id` = &1 AND `user_id` = &2',
311
-                    (int) $_REQUEST['fieldnoteid'],
311
+                    (int)$_REQUEST['fieldnoteid'],
312 312
                     $user->getUserId()
313 313
                 );
314 314
 
@@ -395,8 +395,8 @@  discard block
 block discarded – undo
395 395
 
396 396
 // prepare template and display
397 397
 $tpl->assign('logtype_allows_nm', implode(',', $logtype_allows_nm));
398
-$tpl->assign('scrollposx', isset($_REQUEST['scrollposx']) ? (int) $_REQUEST['scrollposx']: 0);
399
-$tpl->assign('scrollposy', isset($_REQUEST['scrollposy']) ? (int) $_REQUEST['scrollposy']: 0);
398
+$tpl->assign('scrollposx', isset($_REQUEST['scrollposx']) ? (int)$_REQUEST['scrollposx'] : 0);
399
+$tpl->assign('scrollposy', isset($_REQUEST['scrollposy']) ? (int)$_REQUEST['scrollposy'] : 0);
400 400
 $tpl->assign('validate', $validate);
401 401
 $tpl->assign('editlog', false);
402 402
 
Please login to merge, or discard this patch.
htdocs/editlog.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -143,7 +143,7 @@
 block discarded – undo
143 143
 
144 144
 // editor mode switching
145 145
 if (isset($_POST['descMode'])) {
146
-    $descMode = (int) $_POST['descMode']; // Ocprop: 2
146
+    $descMode = (int)$_POST['descMode']; // Ocprop: 2
147 147
     if (($descMode < EditorConstants::HTML_MODE) || ($descMode > EditorConstants::EDITOR_MODE)) {
148 148
         $descMode = EditorConstants::EDITOR_MODE;
149 149
     }
Please login to merge, or discard this patch.
htdocs/mydetails.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
     if (isset($_REQUEST['save'])) {
100 100
         $purifier = new OcHTMLPurifier($opt);
101 101
         $desctext = isset($_REQUEST['desctext']) ? $purifier->purify($_REQUEST['desctext']) : "";
102
-        $desc_htmledit = isset($_REQUEST['descMode']) && ((int) $_REQUEST['descMode']) == EditorConstants::HTML_MODE ? '0' : '1';
102
+        $desc_htmledit = isset($_REQUEST['descMode']) && ((int)$_REQUEST['descMode']) == EditorConstants::HTML_MODE ? '0' : '1';
103 103
         sql(
104 104
             "
105 105
             UPDATE `user`
Please login to merge, or discard this patch.
htdocs/editdesc.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@  discard block
 block discarded – undo
24 24
         isset($_REQUEST['desclang']) &&
25 25
         !isset($_REQUEST['descid'])) {  // Ocprop
26 26
 
27
-        $cache_id = $_REQUEST['cacheid'];  // Ocprop
28
-        $desc_lang = $_REQUEST['desclang'];  // Ocprop
27
+        $cache_id = $_REQUEST['cacheid']; // Ocprop
28
+        $desc_lang = $_REQUEST['desclang']; // Ocprop
29 29
 
30 30
         $rs = sql("SELECT `id` FROM `cache_desc` WHERE `cache_id`='&1' AND `language`='&2'", $cache_id, $desc_lang);
31 31
         if (mysqli_num_rows($rs) == 1) {
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
                 //save to DB?
93 93
                 if (isset($_POST['post'])) {  // Ocprop
94 94
                     //here we read all used information from the form if submitted
95
-                    $descMode = (int) $_POST['descMode'] ?? EditorConstants::EDITOR_MODE;  // Ocprop
95
+                    $descMode = (int)$_POST['descMode'] ?? EditorConstants::EDITOR_MODE; // Ocprop
96 96
 
97 97
                     switch ($descMode) {
98 98
                         case EditorConstants::HTML_MODE:
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
                         $oldDescMode = $descMode;
116 116
                     }
117 117
 
118
-                    $short_desc = trim($_POST['short_desc']);  // Ocprop
118
+                    $short_desc = trim($_POST['short_desc']); // Ocprop
119 119
                     $hint = htmlspecialchars(trim($_POST['hints']), ENT_COMPAT, 'UTF-8');
120 120
                     $desclang = $_POST['desclang'];
121 121
                     $show_all_langs = isset($_POST['show_all_langs_value']) ? $_POST['show_all_langs_value'] : 0;
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
                     }
131 131
 
132 132
                     // Text from textarea
133
-                    $desc = trim($_POST['desc']);  // Ocprop
133
+                    $desc = trim($_POST['desc']); // Ocprop
134 134
 
135 135
                     // fuer alte Versionen von OCProp
136 136
                     if (isset($_POST['submit']) && !isset($_POST['version2'])) {
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
                 tpl_set_var('desclang', htmlspecialchars($desc_lang, ENT_COMPAT, 'UTF-8'));
288 288
                 tpl_set_var('desclang_name', htmlspecialchars(db_LanguageFromShort($desc_lang), ENT_COMPAT, 'UTF-8'));
289 289
                 tpl_set_var('cachename', htmlspecialchars($desc_record['name'], ENT_COMPAT, 'UTF-8'));
290
-                tpl_set_var('reset', $reset);  // obsolete
290
+                tpl_set_var('reset', $reset); // obsolete
291 291
                 tpl_set_var('submit', $submit);
292 292
 
293 293
                 // Text / normal HTML / HTML editor
Please login to merge, or discard this patch.
htdocs/newcache.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -108,12 +108,12 @@
 block discarded – undo
108 108
         // descMode auslesen, falls nicht gesetzt aus dem Profil laden
109 109
         if (isset($_POST['descMode'])) {
110 110
             // Ocprop
111
-            $descMode = (int) $_POST['descMode'];
111
+            $descMode = (int)$_POST['descMode'];
112 112
             if (($descMode < EditorConstants::HTML_MODE) || ($descMode > EditorConstants::EDITOR_MODE)) {
113 113
                 $descMode = EditorConstants::EDITOR_MODE;
114 114
             }
115 115
             if (isset($_POST['oldDescMode'])) {
116
-                $oldDescMode = (int) $_POST['oldDescMode'];
116
+                $oldDescMode = (int)$_POST['oldDescMode'];
117 117
                 if (($oldDescMode < EditorConstants::HTML_MODE) || ($oldDescMode > EditorConstants::EDITOR_MODE)) {
118 118
                     $oldDescMode = $descMode;
119 119
                 }
Please login to merge, or discard this patch.