Completed
Push — master ( e872a0...7fdc80 )
by Arthur
9s
created
src/WebThumbnailer/Utils/ImageUtils.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -32,11 +32,11 @@  discard block
 block discarded – undo
32 32
      */
33 33
     public static function generateThumbnail($imageStr, $target, $maxWidth, $maxHeight, $crop = false)
34 34
     {
35
-        if (! touch($target)) {
35
+        if (!touch($target)) {
36 36
             throw new ImageConvertException('Target file is not writable.');
37 37
         }
38 38
 
39
-        if ($crop && ($maxWidth == 0  || $maxHeight == 0)) {
39
+        if ($crop && ($maxWidth == 0 || $maxHeight == 0)) {
40 40
             throw new ImageConvertException('Both width and height must be provided for cropping');
41 41
         }
42 42
 
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         );
64 64
 
65 65
         $targetImg = imagecreatetruecolor($finalWidth, $finalHeight);
66
-        if (! imagecopyresized(
66
+        if (!imagecopyresized(
67 67
                 $targetImg, $sourceImg,
68 68
                 0, 0, 0, 0,
69 69
                 $finalWidth, $finalHeight, $originalWidth, $originalHeight
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
         $diffHeight = !empty($maxHeight) ? $originalHeight - $maxHeight : false;
98 98
 
99 99
         if (($diffHeight === false && $diffWidth !== false)
100
-            || ($diffWidth > $diffHeight && ! $crop)
100
+            || ($diffWidth > $diffHeight && !$crop)
101 101
             || ($diffWidth < $diffHeight && $crop)
102 102
         ) {
103 103
             $finalWidth = $maxWidth;
Please login to merge, or discard this patch.
src/WebThumbnailer/Utils/DataUtils.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,14 +26,14 @@
 block discarded – undo
26 26
      */
27 27
     public static function loadJson($jsonFile)
28 28
     {
29
-        if (! file_exists($jsonFile) || ! is_readable($jsonFile)) {
29
+        if (!file_exists($jsonFile) || !is_readable($jsonFile)) {
30 30
             throw new IOException('JSON resource file not found or not readable.');
31 31
         }
32 32
         $data = json_decode(file_get_contents($jsonFile), true);
33 33
         if ($data === null) {
34 34
             $error = json_last_error();
35 35
             $msg = json_last_error_msg();
36
-            throw new BadRulesException('An error occured while parsing JSON file: error code #'. $error .': '. $msg);
36
+            throw new BadRulesException('An error occured while parsing JSON file: error code #'.$error.': '.$msg);
37 37
         }
38 38
         return $data;
39 39
     }
Please login to merge, or discard this patch.
src/WebThumbnailer/Utils/SizeUtils.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
      */
51 51
     public static function isMetaSize($size)
52 52
     {
53
-        $metaSize = array (
53
+        $metaSize = array(
54 54
             WebThumbnailer::SIZE_SMALL,
55 55
             WebThumbnailer::SIZE_MEDIUM,
56 56
             WebThumbnailer::SIZE_LARGE
Please login to merge, or discard this patch.
src/WebThumbnailer/Utils/FileUtils.php 2 patches
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -51,12 +51,12 @@
 block discarded – undo
51 51
 
52 52
         foreach (
53 53
             new \RecursiveIteratorIterator(
54
-                 new \RecursiveDirectoryIterator(
55
-                     $path,
56
-                     \FilesystemIterator::SKIP_DOTS |
57
-                     \FilesystemIterator::UNIX_PATHS
58
-                 ),
59
-                 \RecursiveIteratorIterator::CHILD_FIRST
54
+                    new \RecursiveDirectoryIterator(
55
+                        $path,
56
+                        \FilesystemIterator::SKIP_DOTS |
57
+                        \FilesystemIterator::UNIX_PATHS
58
+                    ),
59
+                    \RecursiveIteratorIterator::CHILD_FIRST
60 60
             )
61 61
             as $value
62 62
         ) {
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     /**
13 13
      * @var string Path to resources folder.
14 14
      */
15
-    const RESOURCES_PATH = __DIR__ . DIRECTORY_SEPARATOR .'..'. DIRECTORY_SEPARATOR .'resources'. DIRECTORY_SEPARATOR;
15
+    const RESOURCES_PATH = __DIR__.DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'resources'.DIRECTORY_SEPARATOR;
16 16
 
17 17
     /**
18 18
      * Build the real path from all given folders, with a trailing /.
@@ -29,11 +29,11 @@  discard block
 block discarded – undo
29 29
         if ($nbArgs == 0) {
30 30
             return false;
31 31
         }
32
-        for($i = 0; $i < $nbArgs; $i++) {
33
-            $out .= rtrim(rtrim(func_get_arg($i), '/'), '\\') . DIRECTORY_SEPARATOR;
32
+        for ($i = 0; $i < $nbArgs; $i++) {
33
+            $out .= rtrim(rtrim(func_get_arg($i), '/'), '\\').DIRECTORY_SEPARATOR;
34 34
         }
35 35
         $out = realpath($out);
36
-        return $out !== false ? $out . DIRECTORY_SEPARATOR : false;
36
+        return $out !== false ? $out.DIRECTORY_SEPARATOR : false;
37 37
     }
38 38
 
39 39
     /**
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
      */
46 46
     public static function rmdir($path)
47 47
     {
48
-        if (empty($path) || $path == '/' || ! self::getPath($path)) {
48
+        if (empty($path) || $path == '/' || !self::getPath($path)) {
49 49
             return false;
50 50
         }
51 51
 
Please login to merge, or discard this patch.
src/WebThumbnailer/Utils/ApplicationUtils.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
     public static function checkExtensionRequirements($required)
17 17
     {
18 18
         foreach ($required as $extension) {
19
-            if (! extension_loaded($extension)) {
19
+            if (!extension_loaded($extension)) {
20 20
                 throw new MissingRequirementException(sprintf(
21 21
                     'PHP extension php-%s is required and must be loaded',
22 22
                     $extension
Please login to merge, or discard this patch.
src/WebThumbnailer/Application/WebAccess/WebAccessFactory.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
     public static function getWebAccess($url = null)
22 22
     {
23 23
         // Local file
24
-        if (! empty($url) && $url[0] === '/') {
24
+        if (!empty($url) && $url[0] === '/') {
25 25
             return new WebAccessLocal();
26 26
         }
27 27
 
Please login to merge, or discard this patch.
src/WebThumbnailer/Finder/FinderCommon.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
     protected function replaceOption($thumbnailUrl, $option)
59 59
     {
60 60
         $chosenOption = $this->getOptionValue($option);
61
-        return str_replace('${'. $option .'}', $chosenOption, $thumbnailUrl);
61
+        return str_replace('${'.$option.'}', $chosenOption, $thumbnailUrl);
62 62
     }
63 63
 
64 64
     /**
@@ -72,13 +72,13 @@  discard block
 block discarded – undo
72 72
     protected function getOptionValue($option)
73 73
     {
74 74
         // If the provided option is not defined in the Finder rules.
75
-        if (empty($this->finderOptions) || ! in_array($option, array_keys($this->finderOptions))) {
76
-            throw new BadRulesException('Unknown option "'. $option .'" for the finder "'. $this->getName() .'"');
75
+        if (empty($this->finderOptions) || !in_array($option, array_keys($this->finderOptions))) {
76
+            throw new BadRulesException('Unknown option "'.$option.'" for the finder "'.$this->getName().'"');
77 77
         }
78 78
 
79 79
         // User option is defined.
80 80
         // Any defined option must provide a replacement value in rules under the `param` key.
81
-        if (! empty($this->userOptions[$option])
81
+        if (!empty($this->userOptions[$option])
82 82
             && is_string($this->userOptions[$option])
83 83
             && isset($this->finderOptions[$option][$this->userOptions[$option]]['param'])
84 84
         ) {
@@ -87,15 +87,15 @@  discard block
 block discarded – undo
87 87
         }
88 88
 
89 89
         // If no user option has been found, and no default value is provided: error.
90
-        if (! isset($this->finderOptions[$option]['default'])) {
91
-            $error = 'No default set for option "'. $option .'" for the finder "'. $this->getName() .'"';
90
+        if (!isset($this->finderOptions[$option]['default'])) {
91
+            $error = 'No default set for option "'.$option.'" for the finder "'.$this->getName().'"';
92 92
             throw new BadRulesException($error);
93 93
         }
94 94
 
95 95
         // Use default option replacement.
96 96
         $default = $this->finderOptions[$option]['default'];
97 97
         if (!isset($this->finderOptions[$option][$default]['param'])) {
98
-            $error = 'No default parameter set for option "'. $option .'" for the finder "'. $this->getName() .'"';
98
+            $error = 'No default parameter set for option "'.$option.'" for the finder "'.$this->getName().'"';
99 99
             throw new BadRulesException($error);
100 100
         }
101 101
 
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
     public function isHotlinkAllowed()
109 109
     {
110 110
         if (
111
-            ! isset($this->finderOptions['hotlink_allowed'])
111
+            !isset($this->finderOptions['hotlink_allowed'])
112 112
             ||  $this->finderOptions['hotlink_allowed'] === true
113 113
         ) {
114 114
             return true;
Please login to merge, or discard this patch.
src/WebThumbnailer/Finder/DefaultFinder.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -62,16 +62,16 @@  discard block
 block discarded – undo
62 62
             return $this->url;
63 63
         }
64 64
 
65
-        if (empty($thumbnail) && ! empty($headers) && strpos($headers[0], '200') === false) {
65
+        if (empty($thumbnail) && !empty($headers) && strpos($headers[0], '200') === false) {
66 66
             return false;
67 67
         }
68 68
 
69 69
         // With curl, the thumb is extracted during the download
70
-        if ($this->webAccess instanceof WebAccessCUrl && ! empty($thumbnail)) {
70
+        if ($this->webAccess instanceof WebAccessCUrl && !empty($thumbnail)) {
71 71
             return $thumbnail;
72 72
         }
73 73
 
74
-        return ! empty($content) ? self::extractMetaTag($content) : false;
74
+        return !empty($content) ? self::extractMetaTag($content) : false;
75 75
     }
76 76
 
77 77
     /**
@@ -144,10 +144,10 @@  discard block
 block discarded – undo
144 144
     {
145 145
         $propertiesKey = ['property', 'name', 'itemprop'];
146 146
         // Try to retrieve OpenGraph image.
147
-        $ogRegex = '#<meta[^>]+(?:'. implode('|', $propertiesKey) .')=["\']?og:image["\'\s][^>]*content=["\']?(.*?)["\'\s>]#';
147
+        $ogRegex = '#<meta[^>]+(?:'.implode('|', $propertiesKey).')=["\']?og:image["\'\s][^>]*content=["\']?(.*?)["\'\s>]#';
148 148
         // If the attributes are not in the order property => content (e.g. Github)
149 149
         // New regex to keep this readable... more or less.
150
-        $ogRegexReverse = '#<meta[^>]+content=["\']?([^"\'\s]+)[^>]+(?:'. implode('|', $propertiesKey) .')=["\']?og:image["\'\s/>]#';
150
+        $ogRegexReverse = '#<meta[^>]+content=["\']?([^"\'\s]+)[^>]+(?:'.implode('|', $propertiesKey).')=["\']?og:image["\'\s/>]#';
151 151
 
152 152
         if (preg_match($ogRegex, $content, $matches) > 0
153 153
             || preg_match($ogRegexReverse, $content, $matches) > 0
Please login to merge, or discard this patch.
src/WebThumbnailer/Application/CacheManager.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
     /**
27 27
      * Thumbnails image cache.
28 28
      */
29
-    const TYPE_THUMB  = 'thumb';
29
+    const TYPE_THUMB = 'thumb';
30 30
     /**
31 31
      * Finder cache.
32 32
      */
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
             self::rebuildCacheFolders();
57 57
             return self::getCachePath($type, true);
58 58
         } else if (!$path) {
59
-            throw new IOException('Cache folders are not writable: '. $cache);
59
+            throw new IOException('Cache folders are not writable: '.$cache);
60 60
         }
61 61
         return $path;
62 62
     }
@@ -78,11 +78,11 @@  discard block
 block discarded – undo
78 78
         self::createDomainThumbCacheFolder($domain, $type);
79 79
         $domainFolder = FileUtils::getPath(self::getCachePath($type), $domain);
80 80
         if ($type === self::TYPE_THUMB) {
81
-            $suffix = $width . $height . ($crop ? '1' : '0') .'.png';
81
+            $suffix = $width.$height.($crop ? '1' : '0').'.png';
82 82
         } else {
83
-            $suffix = $width . $height;
83
+            $suffix = $width.$height;
84 84
         }
85
-        return $domainFolder . self::getThumbFilename($url) . $suffix;
85
+        return $domainFolder.self::getThumbFilename($url).$suffix;
86 86
     }
87 87
 
88 88
     /**
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
      */
100 100
     public static function isCacheValid($cacheFile, $domain, $type) {
101 101
         $out = false;
102
-        $cacheDuration = ConfigManager::get('settings.cache_duration', 3600*24*31);
102
+        $cacheDuration = ConfigManager::get('settings.cache_duration', 3600 * 24 * 31);
103 103
 
104 104
         if (is_readable($cacheFile)
105 105
             && ($cacheDuration < 0 || (time() - filemtime($cacheFile)) < $cacheDuration)
@@ -120,10 +120,10 @@  discard block
 block discarded – undo
120 120
      */
121 121
     protected static function createDomainThumbCacheFolder($domain, $type)
122 122
     {
123
-        $domainFolder = self::getCachePath($type) . $domain;
123
+        $domainFolder = self::getCachePath($type).$domain;
124 124
         if (!file_exists($domainFolder)) {
125 125
             mkdir($domainFolder, 0775, false);
126
-            touch($domainFolder . '/' . self::$CLEAN_FILE);
126
+            touch($domainFolder.'/'.self::$CLEAN_FILE);
127 127
         }
128 128
     }
129 129
 
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
     protected static function checkCacheType($type)
151 151
     {
152 152
         if ($type != self::TYPE_THUMB && $type != self::TYPE_FINDER) {
153
-            throw new CacheException('Unknown cache type '. $type);
153
+            throw new CacheException('Unknown cache type '.$type);
154 154
         }
155 155
     }
156 156
 
@@ -160,19 +160,19 @@  discard block
 block discarded – undo
160 160
     protected static function rebuildCacheFolders()
161 161
     {
162 162
         $mainFolder = ConfigManager::get('settings.path.cache', 'cache/');
163
-        if (! is_dir($mainFolder)) {
163
+        if (!is_dir($mainFolder)) {
164 164
             mkdir($mainFolder, 0755);
165 165
         }
166
-        if (! is_dir($mainFolder.self::TYPE_THUMB)) {
166
+        if (!is_dir($mainFolder.self::TYPE_THUMB)) {
167 167
             mkdir($mainFolder.self::TYPE_THUMB, 0755);
168 168
         }
169
-        if (! is_readable($mainFolder . self::TYPE_THUMB . DIRECTORY_SEPARATOR . '.gitkeep')) {
170
-            touch($mainFolder . self::TYPE_THUMB . DIRECTORY_SEPARATOR . '.gitkeep');
169
+        if (!is_readable($mainFolder.self::TYPE_THUMB.DIRECTORY_SEPARATOR.'.gitkeep')) {
170
+            touch($mainFolder.self::TYPE_THUMB.DIRECTORY_SEPARATOR.'.gitkeep');
171 171
         }
172
-        if (! is_dir($mainFolder.self::TYPE_FINDER)) {
173
-            mkdir($mainFolder . self::TYPE_FINDER, 0755);
172
+        if (!is_dir($mainFolder.self::TYPE_FINDER)) {
173
+            mkdir($mainFolder.self::TYPE_FINDER, 0755);
174 174
         }
175
-        if (! is_readable($mainFolder . self::TYPE_THUMB . DIRECTORY_SEPARATOR . '.gitkeep')) {
175
+        if (!is_readable($mainFolder.self::TYPE_THUMB.DIRECTORY_SEPARATOR.'.gitkeep')) {
176 176
             touch($mainFolder.self::TYPE_FINDER.DIRECTORY_SEPARATOR.'.gitkeep');
177 177
         }
178 178
     }
Please login to merge, or discard this patch.