@@ -15,10 +15,16 @@ discard block |
||
| 15 | 15 | $min_title_length = (int) $_POST["min_title_length"]; |
| 16 | 16 | $enable_globally = checkbox_to_sql_bool($_POST["enable_globally"]); |
| 17 | 17 | |
| 18 | - if ($similarity < 0) $similarity = 0; |
|
| 19 | - if ($similarity > 1) $similarity = 1; |
|
| 18 | + if ($similarity < 0) { |
|
| 19 | + $similarity = 0; |
|
| 20 | + } |
|
| 21 | + if ($similarity > 1) { |
|
| 22 | + $similarity = 1; |
|
| 23 | + } |
|
| 20 | 24 | |
| 21 | - if ($min_title_length < 0) $min_title_length = 0; |
|
| 25 | + if ($min_title_length < 0) { |
|
| 26 | + $min_title_length = 0; |
|
| 27 | + } |
|
| 22 | 28 | |
| 23 | 29 | $similarity = sprintf("%.2f", $similarity); |
| 24 | 30 | |
@@ -121,7 +127,9 @@ discard block |
||
| 121 | 127 | } |
| 122 | 128 | |
| 123 | 129 | public function hook_prefs_tab($args) { |
| 124 | - if ($args != "prefFeeds") return; |
|
| 130 | + if ($args != "prefFeeds") { |
|
| 131 | + return; |
|
| 132 | + } |
|
| 125 | 133 | |
| 126 | 134 | print "<div dojoType=\"dijit.layout.AccordionPane\" |
| 127 | 135 | title=\"<i class='material-icons'>extension</i> ".__('Mark similar articles as read')."\">"; |
@@ -140,8 +148,12 @@ discard block |
||
| 140 | 148 | $min_title_length = $this->host->get($this, "min_title_length"); |
| 141 | 149 | $enable_globally = $this->host->get($this, "enable_globally"); |
| 142 | 150 | |
| 143 | - if (!$similarity) $similarity = '0.75'; |
|
| 144 | - if (!$min_title_length) $min_title_length = '32'; |
|
| 151 | + if (!$similarity) { |
|
| 152 | + $similarity = '0.75'; |
|
| 153 | + } |
|
| 154 | + if (!$min_title_length) { |
|
| 155 | + $min_title_length = '32'; |
|
| 156 | + } |
|
| 145 | 157 | |
| 146 | 158 | print "<form dojoType=\"dijit.form.Form\">"; |
| 147 | 159 | |
@@ -198,7 +210,9 @@ discard block |
||
| 198 | 210 | print "</form>"; |
| 199 | 211 | |
| 200 | 212 | $enabled_feeds = $this->host->get($this, "enabled_feeds"); |
| 201 | - if (!array($enabled_feeds)) $enabled_feeds = array(); |
|
| 213 | + if (!array($enabled_feeds)) { |
|
| 214 | + $enabled_feeds = array(); |
|
| 215 | + } |
|
| 202 | 216 | |
| 203 | 217 | $enabled_feeds = $this->filter_unknown_feeds($enabled_feeds); |
| 204 | 218 | $this->host->set($this, "enabled_feeds", $enabled_feeds); |
@@ -225,7 +239,9 @@ discard block |
||
| 225 | 239 | print "<section>"; |
| 226 | 240 | |
| 227 | 241 | $enabled_feeds = $this->host->get($this, "enabled_feeds"); |
| 228 | - if (!array($enabled_feeds)) $enabled_feeds = array(); |
|
| 242 | + if (!array($enabled_feeds)) { |
|
| 243 | + $enabled_feeds = array(); |
|
| 244 | + } |
|
| 229 | 245 | |
| 230 | 246 | $key = array_search($feed_id, $enabled_feeds); |
| 231 | 247 | $checked = $key !== FALSE ? "checked" : ""; |
@@ -242,7 +258,9 @@ discard block |
||
| 242 | 258 | |
| 243 | 259 | public function hook_prefs_save_feed($feed_id) { |
| 244 | 260 | $enabled_feeds = $this->host->get($this, "enabled_feeds"); |
| 245 | - if (!is_array($enabled_feeds)) $enabled_feeds = array(); |
|
| 261 | + if (!is_array($enabled_feeds)) { |
|
| 262 | + $enabled_feeds = array(); |
|
| 263 | + } |
|
| 246 | 264 | |
| 247 | 265 | $enable = checkbox_to_sql_bool($_POST["trgm_similarity_enabled"]); |
| 248 | 266 | $key = array_search($feed_id, $enabled_feeds); |
@@ -262,24 +280,34 @@ discard block |
||
| 262 | 280 | |
| 263 | 281 | public function hook_article_filter($article) { |
| 264 | 282 | |
| 265 | - if (DB_TYPE != "pgsql") return $article; |
|
| 283 | + if (DB_TYPE != "pgsql") { |
|
| 284 | + return $article; |
|
| 285 | + } |
|
| 266 | 286 | |
| 267 | 287 | $res = $this->pdo->query("select 'similarity'::regproc"); |
| 268 | - if (!$res->fetch()) return $article; |
|
| 288 | + if (!$res->fetch()) { |
|
| 289 | + return $article; |
|
| 290 | + } |
|
| 269 | 291 | |
| 270 | 292 | $enable_globally = $this->host->get($this, "enable_globally"); |
| 271 | 293 | |
| 272 | 294 | if (!$enable_globally) { |
| 273 | 295 | $enabled_feeds = $this->host->get($this, "enabled_feeds"); |
| 274 | 296 | $key = array_search($article["feed"]["id"], $enabled_feeds); |
| 275 | - if ($key === FALSE) return $article; |
|
| 297 | + if ($key === FALSE) { |
|
| 298 | + return $article; |
|
| 299 | + } |
|
| 276 | 300 | } |
| 277 | 301 | |
| 278 | 302 | $similarity = (float) $this->host->get($this, "similarity"); |
| 279 | - if ($similarity < 0.01) return $article; |
|
| 303 | + if ($similarity < 0.01) { |
|
| 304 | + return $article; |
|
| 305 | + } |
|
| 280 | 306 | |
| 281 | 307 | $min_title_length = (int) $this->host->get($this, "min_title_length"); |
| 282 | - if (mb_strlen($article["title"]) < $min_title_length) return $article; |
|
| 308 | + if (mb_strlen($article["title"]) < $min_title_length) { |
|
| 309 | + return $article; |
|
| 310 | + } |
|
| 283 | 311 | |
| 284 | 312 | $owner_uid = $article["owner_uid"]; |
| 285 | 313 | $entry_guid = $article["guid_hashed"]; |
@@ -38,7 +38,9 @@ discard block |
||
| 38 | 38 | // fallback on HTTP parameters |
| 39 | 39 | if ($input) { |
| 40 | 40 | $input = json_decode($input, true); |
| 41 | - if ($input) $_REQUEST = $input; |
|
| 41 | + if ($input) { |
|
| 42 | + $_REQUEST = $input; |
|
| 43 | + } |
|
| 42 | 44 | } |
| 43 | 45 | } else { |
| 44 | 46 | // Accept JSON only |
@@ -55,7 +57,9 @@ discard block |
||
| 55 | 57 | |
| 56 | 58 | startup_gettext(); |
| 57 | 59 | |
| 58 | -if (!init_plugins()) return; |
|
| 60 | +if (!init_plugins()) { |
|
| 61 | + return; |
|
| 62 | +} |
|
| 59 | 63 | |
| 60 | 64 | if ($_SESSION["uid"]) { |
| 61 | 65 | if (!validate_session()) { |