@@ -1,6 +1,8 @@ discard block |
||
| 1 | 1 | <?php declare(strict_types=1); |
| 2 | 2 | |
| 3 | -if(!extension_loaded('gd')) die('GD ext is required to run this!'); |
|
| 3 | +if(!extension_loaded('gd')) { |
|
| 4 | + die('GD ext is required to run this!'); |
|
| 5 | +} |
|
| 4 | 6 | |
| 5 | 7 | chdir(dirname(__FILE__).'/../'); //Just to make things easier, change dir to project root. |
| 6 | 8 | |
@@ -9,7 +11,7 @@ discard block |
||
| 9 | 11 | private $className; |
| 10 | 12 | |
| 11 | 13 | public function __construct() { |
| 12 | - if(isset($_SERVER['argv']) && count($_SERVER['argv']) === 3){ |
|
| 14 | + if(isset($_SERVER['argv']) && count($_SERVER['argv']) === 3) { |
|
| 13 | 15 | $this->baseURL = rtrim($_SERVER['argv'][1], '/'); |
| 14 | 16 | $this->className = $_SERVER['argv'][2]; |
| 15 | 17 | |
@@ -82,7 +84,9 @@ discard block |
||
| 82 | 84 | $baseFile = file_get_contents('./public/userscripts/manga-tracker.user.js'); |
| 83 | 85 | |
| 84 | 86 | $parse = parse_url($this->baseURL); |
| 85 | - if(strpos($baseFile, $parse['host']) !== false) die("Domain already exists in userscript?"); |
|
| 87 | + if(strpos($baseFile, $parse['host']) !== false) { |
|
| 88 | + die("Domain already exists in userscript?"); |
|
| 89 | + } |
|
| 86 | 90 | |
| 87 | 91 | preg_match('/\@updated ([0-9\-]+)\r.*?\@version ([0-9\.]+)/s', $baseFile, $matches); |
| 88 | 92 | |
@@ -179,7 +183,9 @@ discard block |
||
| 179 | 183 | } |
| 180 | 184 | } |
| 181 | 185 | |
| 182 | - if(empty($titleArr)) die("API isn't returning any titles?"); |
|
| 186 | + if(empty($titleArr)) { |
|
| 187 | + die("API isn't returning any titles?"); |
|
| 188 | + } |
|
| 183 | 189 | return $titleArr; |
| 184 | 190 | } |
| 185 | 191 | } |
@@ -53,7 +53,8 @@ |
||
| 53 | 53 | $this->session->keep_flashdata('referred_from'); |
| 54 | 54 | if($prevURL = $this->session->flashdata('referred_from')) { |
| 55 | 55 | redirect($prevURL); |
| 56 | - } else { //@codeCoverageIgnore |
|
| 56 | + } else { |
|
| 57 | +//@codeCoverageIgnore |
|
| 57 | 58 | redirect('/'); //TODO (CHECK): Should this be refresh? |
| 58 | 59 | } //@codeCoverageIgnore |
| 59 | 60 | } else { |