Completed
Pull Request — master (#844)
by
unknown
02:50
created
lib/timber-loader.php 1 patch
Spacing   +51 added lines, -51 removed lines patch added patch discarded remove patch
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 	 */
42 42
 	function render($file, $data = null, $expires = false, $cache_mode = self::CACHE_USE_DEFAULT) {
43 43
 		// Different $expires if user is anonymous or logged in
44
-		if (is_array($expires)) {
45
-			if (is_user_logged_in() && isset($expires[1])) {
44
+		if ( is_array($expires) ) {
45
+			if ( is_user_logged_in() && isset($expires[1]) ) {
46 46
 				$expires = $expires[1];
47 47
 			} else {
48 48
 				$expires = $expires[0];
@@ -51,15 +51,15 @@  discard block
 block discarded – undo
51 51
 
52 52
 		$key = null;
53 53
 		$output = false;
54
-		if (false !== $expires) {
54
+		if ( false !== $expires ) {
55 55
 			ksort($data);
56 56
 			$key = md5($file . json_encode($data));
57 57
 			$output = $this->get_cache($key, self::CACHEGROUP, $cache_mode);
58 58
 		}
59 59
 
60
-		if (false === $output || null === $output) {
60
+		if ( false === $output || null === $output ) {
61 61
 			$twig = $this->get_twig();
62
-			if (strlen($file)) {
62
+			if ( strlen($file) ) {
63 63
 				$loader = $this->get_loader();
64 64
 				$result = $loader->getCacheKey($file);
65 65
 				do_action('timber_loader_render_file', $result);
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 			$output = $twig->render($file, $data);
70 70
 		}
71 71
 
72
-		if (false !== $output && false !== $expires && null !== $key) {
72
+		if ( false !== $output && false !== $expires && null !== $key ) {
73 73
 			$this->set_cache($key, $output, self::CACHEGROUP, $expires, $cache_mode);
74 74
 		}
75 75
 		$output = apply_filters('timber_output', $output);
@@ -81,10 +81,10 @@  discard block
 block discarded – undo
81 81
 	 * @return bool
82 82
 	 */
83 83
 	public function choose_template($filenames) {
84
-		if (is_array($filenames)) {
84
+		if ( is_array($filenames) ) {
85 85
 			/* its an array so we have to figure out which one the dev wants */
86 86
 			foreach ($filenames as $filename) {
87
-				if (self::template_exists($filename)) {
87
+				if ( self::template_exists($filename) ) {
88 88
 					return $filename;
89 89
 				}
90 90
 			}
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	protected function template_exists($file) {
101 101
 		foreach ($this->locations as $dir) {
102 102
 			$look_for = trailingslashit($dir) . $file;
103
-			if (file_exists($look_for)) {
103
+			if ( file_exists($look_for) ) {
104 104
 				return true;
105 105
 			}
106 106
 		}
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 		$theme_locs = array();
115 115
 		$child_loc = get_stylesheet_directory();
116 116
 		$parent_loc = get_template_directory();
117
-		if (DIRECTORY_SEPARATOR == '\\') {
117
+		if ( DIRECTORY_SEPARATOR == '\\' ) {
118 118
 			$child_loc = str_replace('/', '\\', $child_loc);
119 119
 			$parent_loc = str_replace('/', '\\', $parent_loc);
120 120
 		}
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 		foreach ($this->get_locations_theme_dir() as $dirname) {
123 123
 			$theme_locs[] = trailingslashit($child_loc) . trailingslashit($dirname);
124 124
 		}
125
-		if ($child_loc != $parent_loc) {
125
+		if ( $child_loc != $parent_loc ) {
126 126
 			$theme_locs[] = $parent_loc;
127 127
 			foreach ($this->get_locations_theme_dir() as $dirname) {
128 128
 				$theme_locs[] = trailingslashit($parent_loc) . trailingslashit($dirname);
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 * @return string[] the names of directores, ie: array('templats', 'views');
139 139
 	 */
140 140
 	private function get_locations_theme_dir() {
141
-		if (is_string(Timber::$dirname)) {
141
+		if ( is_string(Timber::$dirname) ) {
142 142
 			return array(Timber::$dirname);
143 143
 		}
144 144
 		return Timber::$dirname;
@@ -149,13 +149,13 @@  discard block
 block discarded – undo
149 149
 	 */
150 150
 	function get_locations_user() {
151 151
 		$locs = array();
152
-		if (isset(Timber::$locations)) {
153
-			if (is_string(Timber::$locations)) {
152
+		if ( isset(Timber::$locations) ) {
153
+			if ( is_string(Timber::$locations) ) {
154 154
 				Timber::$locations = array(Timber::$locations);
155 155
 			}
156 156
 			foreach (Timber::$locations as $tloc) {
157 157
 				$tloc = realpath($tloc);
158
-				if (is_dir($tloc)) {
158
+				if ( is_dir($tloc) ) {
159 159
 					$locs[] = $tloc;
160 160
 				}
161 161
 			}
@@ -169,14 +169,14 @@  discard block
 block discarded – undo
169 169
 	 */
170 170
 	function get_locations_caller($caller = false) {
171 171
 		$locs = array();
172
-		if ($caller && is_string($caller)) {
172
+		if ( $caller && is_string($caller) ) {
173 173
 			$caller = trailingslashit($caller);
174
-			if (is_dir($caller)) {
174
+			if ( is_dir($caller) ) {
175 175
 				$locs[] = $caller;
176 176
 			}
177 177
 			foreach ($this->get_locations_theme_dir() as $dirname) {
178 178
 				$caller_sub = $caller . trailingslashit($dirname);
179
-				if (is_dir($caller_sub)) {
179
+				if ( is_dir($caller_sub) ) {
180 180
 					$locs[] = $caller_sub;
181 181
 				}
182 182
 			}
@@ -210,14 +210,14 @@  discard block
 block discarded – undo
210 210
 		$paths = array();
211 211
 		foreach ($this->locations as $loc) {
212 212
 			$loc = realpath($loc);
213
-			if (is_dir($loc)) {
213
+			if ( is_dir($loc) ) {
214 214
 				$loc = realpath($loc);
215 215
 				$paths[] = $loc;
216 216
 			} else {
217 217
 				//error_log($loc.' is not a directory');
218 218
 			}
219 219
 		}
220
-		if (!ini_get('open_basedir')) {
220
+		if ( !ini_get('open_basedir') ) {
221 221
 			$paths[] = '/';
222 222
 		} else {
223 223
 			$paths[] = ABSPATH;
@@ -233,15 +233,15 @@  discard block
 block discarded – undo
233 233
 	function get_twig() {
234 234
 		$loader = $this->get_loader();
235 235
 		$params = array('debug' => WP_DEBUG, 'autoescape' => false);
236
-		if (isset(Timber::$autoescape)) {
236
+		if ( isset(Timber::$autoescape) ) {
237 237
 			$params['autoescape'] = Timber::$autoescape;
238 238
 		}
239
-		if (Timber::$cache === true) {
239
+		if ( Timber::$cache === true ) {
240 240
 			Timber::$twig_cache = true;
241 241
 		}
242
-		if (Timber::$twig_cache) {
243
-			$twig_cache_loc = apply_filters( 'timber/cache/location', TIMBER_LOC . '/cache/twig' );
244
-			if (!file_exists($twig_cache_loc)) {
242
+		if ( Timber::$twig_cache ) {
243
+			$twig_cache_loc = apply_filters('timber/cache/location', TIMBER_LOC . '/cache/twig');
244
+			if ( !file_exists($twig_cache_loc) ) {
245 245
 				mkdir($twig_cache_loc, 0777, true);
246 246
 			}
247 247
 			$params['cache'] = $twig_cache_loc;
@@ -258,26 +258,26 @@  discard block
 block discarded – undo
258 258
 		return $twig;
259 259
 	}
260 260
 
261
-	public function clear_cache_timber($cache_mode = self::CACHE_USE_DEFAULT){
261
+	public function clear_cache_timber($cache_mode = self::CACHE_USE_DEFAULT) {
262 262
 		//_transient_timberloader
263 263
 		$object_cache = false;
264
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
264
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
265 265
 			$object_cache = true;
266 266
 		}
267 267
 		$cache_mode = $this->_get_cache_mode($cache_mode);
268
-		if (self::CACHE_TRANSIENT === $cache_mode) {
268
+		if ( self::CACHE_TRANSIENT === $cache_mode ) {
269 269
 			global $wpdb;
270 270
 			$query = $wpdb->prepare("DELETE FROM $wpdb->options WHERE option_name LIKE '%s'", '_transient_timberloader_%');
271
-			$wpdb->query( $query );
271
+			$wpdb->query($query);
272 272
 			return true;
273
-		} else if (self::CACHE_SITE_TRANSIENT === $cache_mode) {
273
+		} else if ( self::CACHE_SITE_TRANSIENT === $cache_mode ) {
274 274
 			global $wpdb;
275 275
 			$query = $wpdb->prepare("DELETE FROM $wpdb->options WHERE option_name LIKE '%s'", '_transient_timberloader_%');
276
-			$wpdb->query( $query );
276
+			$wpdb->query($query);
277 277
 			return true;
278
-		} else if (self::CACHE_OBJECT === $cache_mode && $object_cache) {
278
+		} else if ( self::CACHE_OBJECT === $cache_mode && $object_cache ) {
279 279
 			global $wp_object_cache;
280
-			if (isset($wp_object_cache->cache[self::CACHEGROUP])){
280
+			if ( isset($wp_object_cache->cache[self::CACHEGROUP]) ) {
281 281
 				unset($wp_object_cache->cache[self::CACHEGROUP]);
282 282
 				return true;
283 283
 			}
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
 		$twig = $this->get_twig();
290 290
 		$twig->clearCacheFiles();
291 291
 		$cache = $twig->getCache();
292
-		if ($cache){
292
+		if ( $cache ) {
293 293
 			self::rrmdir($twig->getCache());
294 294
 			return true;
295 295
 		}
@@ -300,15 +300,15 @@  discard block
 block discarded – undo
300 300
 	 * @param string|false $dirPath
301 301
 	 */
302 302
 	public static function rrmdir($dirPath) {
303
-		if (! is_dir($dirPath)) {
303
+		if ( !is_dir($dirPath) ) {
304 304
 			throw new InvalidArgumentException("$dirPath must be a directory");
305 305
 		}
306
-		if (substr($dirPath, strlen($dirPath) - 1, 1) != '/') {
306
+		if ( substr($dirPath, strlen($dirPath) - 1, 1) != '/' ) {
307 307
 			$dirPath .= '/';
308 308
 		}
309 309
 		$files = glob($dirPath . '*', GLOB_MARK);
310 310
 		foreach ($files as $file) {
311
-			if (is_dir($file)) {
311
+			if ( is_dir($file) ) {
312 312
 				self::rrmdir($file);
313 313
 			} else {
314 314
 				unlink($file);
@@ -323,9 +323,9 @@  discard block
 block discarded – undo
323 323
 	private function _get_cache_extension() {
324 324
 
325 325
 		$key_generator   = new \Timber\Cache\KeyGenerator();
326
-		$cache_provider  = new \Timber\Cache\WPObjectCacheAdapter( $this );
327
-		$cache_strategy  = new \Asm89\Twig\CacheExtension\CacheStrategy\GenerationalCacheStrategy( $cache_provider, $key_generator );
328
-		$cache_extension = new \Asm89\Twig\CacheExtension\Extension( $cache_strategy );
326
+		$cache_provider  = new \Timber\Cache\WPObjectCacheAdapter($this);
327
+		$cache_strategy  = new \Asm89\Twig\CacheExtension\CacheStrategy\GenerationalCacheStrategy($cache_provider, $key_generator);
328
+		$cache_extension = new \Asm89\Twig\CacheExtension\Extension($cache_strategy);
329 329
 
330 330
 		return $cache_extension;
331 331
 	}
@@ -339,7 +339,7 @@  discard block
 block discarded – undo
339 339
 	public function get_cache($key, $group = self::CACHEGROUP, $cache_mode = self::CACHE_USE_DEFAULT) {
340 340
 		$object_cache = false;
341 341
 
342
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
342
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
343 343
 			$object_cache = true;
344 344
 		}
345 345
 
@@ -348,13 +348,13 @@  discard block
 block discarded – undo
348 348
 		$value = false;
349 349
 
350 350
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
351
-		if (self::CACHE_TRANSIENT === $cache_mode)
351
+		if ( self::CACHE_TRANSIENT === $cache_mode )
352 352
 			$value = get_transient($trans_key);
353 353
 
354
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
354
+		elseif ( self::CACHE_SITE_TRANSIENT === $cache_mode )
355 355
 			$value = get_site_transient($trans_key);
356 356
 
357
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
357
+		elseif ( self::CACHE_OBJECT === $cache_mode && $object_cache )
358 358
 			$value = wp_cache_get($key, $group);
359 359
 
360 360
 		return $value;
@@ -371,23 +371,23 @@  discard block
 block discarded – undo
371 371
 	public function set_cache($key, $value, $group = self::CACHEGROUP, $expires = 0, $cache_mode = self::CACHE_USE_DEFAULT) {
372 372
 		$object_cache = false;
373 373
 
374
-		if (isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache'])) {
374
+		if ( isset($GLOBALS['wp_object_cache']) && is_object($GLOBALS['wp_object_cache']) ) {
375 375
 			$object_cache = true;
376 376
 		}
377 377
 
378
-		if ((int)$expires < 1)
378
+		if ( (int) $expires < 1 )
379 379
 			$expires = 0;
380 380
 
381 381
 		$cache_mode = self::_get_cache_mode($cache_mode);
382 382
 		$trans_key = substr($group . '_' . $key, 0, self::TRANS_KEY_LEN);
383 383
 
384
-		if (self::CACHE_TRANSIENT === $cache_mode)
384
+		if ( self::CACHE_TRANSIENT === $cache_mode )
385 385
 			set_transient($trans_key, $value, $expires);
386 386
 
387
-		elseif (self::CACHE_SITE_TRANSIENT === $cache_mode)
387
+		elseif ( self::CACHE_SITE_TRANSIENT === $cache_mode )
388 388
 			set_site_transient($trans_key, $value, $expires);
389 389
 
390
-		elseif (self::CACHE_OBJECT === $cache_mode && $object_cache)
390
+		elseif ( self::CACHE_OBJECT === $cache_mode && $object_cache )
391 391
 			wp_cache_set($key, $value, $group, $expires);
392 392
 
393 393
 		return $value;
@@ -398,12 +398,12 @@  discard block
 block discarded – undo
398 398
 	 * @return string
399 399
 	 */
400 400
 	private function _get_cache_mode($cache_mode) {
401
-		if (empty($cache_mode) || self::CACHE_USE_DEFAULT === $cache_mode) {
401
+		if ( empty($cache_mode) || self::CACHE_USE_DEFAULT === $cache_mode ) {
402 402
 			$cache_mode = $this->cache_mode;
403 403
 		}
404 404
 
405 405
 		// Fallback if self::$cache_mode did not get a valid value
406
-		if (!in_array($cache_mode, self::$cache_modes)) {
406
+		if ( !in_array($cache_mode, self::$cache_modes) ) {
407 407
 			$cache_mode = self::CACHE_OBJECT;
408 408
 		}
409 409
 
Please login to merge, or discard this patch.
lib/timber-image-helper.php 1 patch
Spacing   +82 added lines, -82 removed lines patch added patch discarded remove patch
@@ -43,9 +43,9 @@  discard block
 block discarded – undo
43 43
 	 * ```
44 44
 	 * @return string (ex: )
45 45
 	 */
46
-	public static function resize( $src, $w, $h = 0, $crop = 'default', $force = false ) {
47
-		if (!is_numeric($w) && is_string($w)) {
48
-			if ($sizes = self::find_wp_dimensions($w)) {
46
+	public static function resize($src, $w, $h = 0, $crop = 'default', $force = false) {
47
+		if ( !is_numeric($w) && is_string($w) ) {
48
+			if ( $sizes = self::find_wp_dimensions($w) ) {
49 49
 				$w = $sizes['w'];
50 50
 				$h = $sizes['h'];
51 51
 			} else {
@@ -66,16 +66,16 @@  discard block
 block discarded – undo
66 66
 	 *     @type int h
67 67
 	 * }
68 68
 	 */
69
-	private static function find_wp_dimensions( $size ) {
69
+	private static function find_wp_dimensions($size) {
70 70
 		global $_wp_additional_image_sizes;
71
-		if (isset($_wp_additional_image_sizes[$size])) {
71
+		if ( isset($_wp_additional_image_sizes[$size]) ) {
72 72
 			$w = $_wp_additional_image_sizes[$size]['width'];
73 73
 			$h = $_wp_additional_image_sizes[$size]['height'];
74
-		} else if (in_array($size, array('thumbnail', 'medium', 'large'))) {
75
-			$w = get_option($size.'_size_w');
76
-			$h = get_option($size.'_size_h');
74
+		} else if ( in_array($size, array('thumbnail', 'medium', 'large')) ) {
75
+			$w = get_option($size . '_size_w');
76
+			$h = get_option($size . '_size_h');
77 77
 		}
78
-		if (isset($w) && isset($h) && ($w || $h)) {
78
+		if ( isset($w) && isset($h) && ($w || $h) ) {
79 79
 			return array('w' => $w, 'h' => $h);
80 80
 		}
81 81
 		return false;
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 *
91 91
 	 * @return string url to the new image
92 92
 	 */
93
-	public static function retina_resize( $src, $multiplier = 2, $force = false ) {
93
+	public static function retina_resize($src, $multiplier = 2, $force = false) {
94 94
 		$op = new TimberImageOperationRetina($multiplier);
95 95
 		return self::_operate($src, $op, $force);
96 96
 	}
@@ -100,13 +100,13 @@  discard block
 block discarded – undo
100 100
 	 * @param  string  $file local filepath to a file, not a URL
101 101
 	 * @return boolean true if it's an animated gif, false if not
102 102
 	 */
103
-	public static function is_animated_gif( $file ) {
103
+	public static function is_animated_gif($file) {
104 104
 		if ( strpos(strtolower($file), '.gif') == -1 ) {
105 105
 			//doesn't have .gif, bail
106 106
 			return false;
107 107
 		}
108 108
 		//its a gif so test
109
-		if( !($fh = @fopen($file, 'rb')) ) {
109
+		if ( !($fh = @fopen($file, 'rb')) ) {
110 110
 		  	return false;
111 111
 	    }
112 112
 	    $count = 0;
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
 
119 119
 	    // We read through the file til we reach the end of the file, or we've found
120 120
 	    // at least 2 frame headers
121
-	    while(!feof($fh) && $count < 2) {
121
+	    while (!feof($fh) && $count < 2) {
122 122
 	        $chunk = fread($fh, 1024 * 100); //read 100kb at a time
123 123
 	        $count += preg_match_all('#\x00\x21\xF9\x04.{4}\x00[\x2C\x21]#s', $chunk, $matches);
124 124
 	    }
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 * @param bool    $force
139 139
 	 * @return mixed|null|string
140 140
 	 */
141
-	public static function letterbox( $src, $w, $h, $color = '#000000', $force = false ) {
141
+	public static function letterbox($src, $w, $h, $color = '#000000', $force = false) {
142 142
 		$op = new TimberImageOperationLetterbox($w, $h, $color);
143 143
 		return self::_operate($src, $op, $force);
144 144
 	}
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
 	 * @param string  $bghex
151 151
 	 * @return string
152 152
 	 */
153
-	public static function img_to_jpg( $src, $bghex = '#FFFFFF', $force = false ) {
153
+	public static function img_to_jpg($src, $bghex = '#FFFFFF', $force = false) {
154 154
 		$op = new TimberImageOperationToJpg($bghex);
155 155
 		return self::_operate($src, $op, $force);
156 156
 	}
@@ -159,13 +159,13 @@  discard block
 block discarded – undo
159 159
 	 * Deletes all resized versions of an image when the source is deleted
160 160
 	 */
161 161
 	protected static function add_actions() {
162
-		add_action( 'delete_attachment', function ( $post_id ) {
163
-			$post = get_post( $post_id );
164
-			$image_types = array( 'image/jpeg', 'image/png', 'image/gif', 'image/jpg' );
165
-			if ( in_array( $post->post_mime_type, $image_types ) ) {
166
-				$attachment = new TimberImage( $post_id );
162
+		add_action('delete_attachment', function($post_id) {
163
+			$post = get_post($post_id);
164
+			$image_types = array('image/jpeg', 'image/png', 'image/gif', 'image/jpg');
165
+			if ( in_array($post->post_mime_type, $image_types) ) {
166
+				$attachment = new TimberImage($post_id);
167 167
 				if ( $attachment->file_loc ) {
168
-					TimberImageHelper::delete_generated_files( $attachment->file_loc );
168
+					TimberImageHelper::delete_generated_files($attachment->file_loc);
169 169
 				}
170 170
 			}
171 171
 		} );
@@ -176,9 +176,9 @@  discard block
 block discarded – undo
176 176
 	 * for example /wp-content or /content
177 177
 	 */
178 178
 	protected static function add_constants() {
179
-		if ( !defined( 'WP_CONTENT_SUBDIR' ) ) {
180
-			$wp_content_path = str_replace( home_url(), '', WP_CONTENT_URL );
181
-			define( 'WP_CONTENT_SUBDIR', $wp_content_path );
179
+		if ( !defined('WP_CONTENT_SUBDIR') ) {
180
+			$wp_content_path = str_replace(home_url(), '', WP_CONTENT_URL);
181
+			define('WP_CONTENT_SUBDIR', $wp_content_path);
182 182
 		}
183 183
 	}
184 184
 
@@ -188,8 +188,8 @@  discard block
 block discarded – undo
188 188
 	 * @return void
189 189
 	 */
190 190
 	static function add_filters() {
191
-		add_filter( 'upload_dir', function ( $arr ) {
192
-			$arr['relative'] = str_replace( home_url(), '', $arr['baseurl'] );
191
+		add_filter('upload_dir', function($arr) {
192
+			$arr['relative'] = str_replace(home_url(), '', $arr['baseurl']);
193 193
 			return $arr;
194 194
 		} );
195 195
 	}
@@ -200,16 +200,16 @@  discard block
 block discarded – undo
200 200
 	 * @param string  $local_file   ex: /var/www/wp-content/uploads/2015/my-pic.jpg
201 201
 	 *                              or: http://example.org/wp-content/uploads/2015/my-pic.jpg
202 202
 	 */
203
-	static function delete_generated_files( $local_file ) {
204
-		if (TimberURLHelper::is_absolute( $local_file ) ) {
205
-			$local_file = TimberURLHelper::url_to_file_system( $local_file );
203
+	static function delete_generated_files($local_file) {
204
+		if ( TimberURLHelper::is_absolute($local_file) ) {
205
+			$local_file = TimberURLHelper::url_to_file_system($local_file);
206 206
 		}
207
-		$info = pathinfo( $local_file );
207
+		$info = pathinfo($local_file);
208 208
 		$dir = $info['dirname'];
209 209
 		$ext = $info['extension'];
210 210
 		$filename = $info['filename'];
211
-		self::process_delete_generated_files( $filename, $ext, $dir, '-[0-9999999]*', '-[0-9]*x[0-9]*-c-[a-z]*.' );
212
-		self::process_delete_generated_files( $filename, $ext, $dir, '-lbox-[0-9999999]*', '-lbox-[0-9]*x[0-9]*-[a-zA-Z0-9]*.' );
211
+		self::process_delete_generated_files($filename, $ext, $dir, '-[0-9999999]*', '-[0-9]*x[0-9]*-c-[a-z]*.');
212
+		self::process_delete_generated_files($filename, $ext, $dir, '-lbox-[0-9999999]*', '-lbox-[0-9]*x[0-9]*-[a-zA-Z0-9]*.');
213 213
 	}
214 214
 
215 215
 	/**
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
 	 * @param string 	$search_pattern pattern of files to pluck from
227 227
 	 * @param string 	$match_pattern pattern of files to go forth and delete
228 228
 	 */
229
-	protected static function process_delete_generated_files( $filename, $ext, $dir, $search_pattern, $match_pattern ) {
229
+	protected static function process_delete_generated_files($filename, $ext, $dir, $search_pattern, $match_pattern) {
230 230
 		$searcher = '/' . $filename . $search_pattern;
231
-		foreach ( glob( $dir . $searcher ) as $found_file ) {
232
-			$regexdir = str_replace( '/', '\/', $dir );
233
-			$pattern = '/' . ( $regexdir ) . '\/' . $filename . $match_pattern . $ext . '/';
234
-			$match = preg_match( $pattern, $found_file );
231
+		foreach (glob($dir . $searcher) as $found_file) {
232
+			$regexdir = str_replace('/', '\/', $dir);
233
+			$pattern = '/' . ($regexdir) . '\/' . $filename . $match_pattern . $ext . '/';
234
+			$match = preg_match($pattern, $found_file);
235 235
 			if ( $match ) {
236
-				unlink( $found_file );
236
+				unlink($found_file);
237 237
 			}
238 238
 		}
239 239
 	}
@@ -245,9 +245,9 @@  discard block
 block discarded – undo
245 245
 	 * @param string  $url
246 246
 	 * @return string
247 247
 	 */
248
-	public static function get_server_location( $url ) {
248
+	public static function get_server_location($url) {
249 249
 		// if we're already an absolute dir, just return
250
-		if ( 0 === strpos( $url, ABSPATH ) ) {
250
+		if ( 0 === strpos($url, ABSPATH) ) {
251 251
 			return $url;
252 252
 		}
253 253
 		// otherwise, analyze URL then build mapping path
@@ -262,15 +262,15 @@  discard block
 block discarded – undo
262 262
 	 * @param string  $file
263 263
 	 * @return string
264 264
 	 */
265
-	public static function get_sideloaded_file_loc( $file ) {
265
+	public static function get_sideloaded_file_loc($file) {
266 266
 		$upload = wp_upload_dir();
267 267
 		$dir = $upload['path'];
268 268
 		$filename = $file;
269
-		$file = parse_url( $file );
270
-		$path_parts = pathinfo( $file['path'] );
271
-		$basename = md5( $filename );
269
+		$file = parse_url($file);
270
+		$path_parts = pathinfo($file['path']);
271
+		$basename = md5($filename);
272 272
 		$ext = 'jpg';
273
-		if ( isset( $path_parts['extension'] ) ) {
273
+		if ( isset($path_parts['extension']) ) {
274 274
 			$ext = $path_parts['extension'];
275 275
 		}
276 276
 		return $dir . '/' . $basename . '.' . $ext;
@@ -282,28 +282,28 @@  discard block
 block discarded – undo
282 282
 	 * @param string  $file the URL to the original file
283 283
 	 * @return string the URL to the downloaded file
284 284
 	 */
285
-	public static function sideload_image( $file ) {
286
-		$loc = self::get_sideloaded_file_loc( $file );
287
-		if ( file_exists( $loc ) ) {
288
-			return TimberURLHelper::preslashit( TimberURLHelper::get_rel_path( $loc ) );
285
+	public static function sideload_image($file) {
286
+		$loc = self::get_sideloaded_file_loc($file);
287
+		if ( file_exists($loc) ) {
288
+			return TimberURLHelper::preslashit(TimberURLHelper::get_rel_path($loc));
289 289
 		}
290 290
 		// Download file to temp location
291
-		if ( !function_exists( 'download_url' ) ) {
291
+		if ( !function_exists('download_url') ) {
292 292
 			require_once ABSPATH . '/wp-admin/includes/file.php';
293 293
 		}
294
-		$tmp = download_url( $file );
295
-		preg_match( '/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches );
294
+		$tmp = download_url($file);
295
+		preg_match('/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches);
296 296
 		$file_array = array();
297
-		$file_array['name'] = basename( $matches[0] );
297
+		$file_array['name'] = basename($matches[0]);
298 298
 		$file_array['tmp_name'] = $tmp;
299 299
 		// If error storing temporarily, unlink
300
-		if ( is_wp_error( $tmp ) ) {
301
-			@unlink( $file_array['tmp_name'] );
300
+		if ( is_wp_error($tmp) ) {
301
+			@unlink($file_array['tmp_name']);
302 302
 			$file_array['tmp_name'] = '';
303 303
 		}
304 304
 		// do the validation and storage stuff
305
-		$locinfo = pathinfo( $loc );
306
-		$file = wp_upload_bits( $locinfo['basename'], null, file_get_contents( $file_array['tmp_name'] ) );
305
+		$locinfo = pathinfo($loc);
306
+		$file = wp_upload_bits($locinfo['basename'], null, file_get_contents($file_array['tmp_name']));
307 307
 		return $file['url'];
308 308
 	}
309 309
 
@@ -330,23 +330,23 @@  discard block
 block discarded – undo
330 330
 		if ( 0 === strpos($tmp, ABSPATH) ) { // we've been given a dir, not an url
331 331
 			$result['absolute'] = true;
332 332
 			if ( 0 === strpos($tmp, $upload_dir['basedir']) ) {
333
-				$result['base']= self::BASE_UPLOADS; // upload based
333
+				$result['base'] = self::BASE_UPLOADS; // upload based
334 334
 				$tmp = str_replace($upload_dir['basedir'], '', $tmp);
335 335
 			}
336 336
 			if ( 0 === strpos($tmp, WP_CONTENT_DIR) ) {
337
-				$result['base']= self::BASE_CONTENT; // content based
337
+				$result['base'] = self::BASE_CONTENT; // content based
338 338
 				$tmp = str_replace(WP_CONTENT_DIR, '', $tmp);
339 339
 			}
340 340
 		} else {
341
-			if (!$result['absolute']) {
342
-				$tmp = home_url().$tmp;
341
+			if ( !$result['absolute'] ) {
342
+				$tmp = home_url() . $tmp;
343 343
 			}
344
-			if (0 === strpos($tmp, $upload_dir['baseurl'])) {
345
-				$result['base']= self::BASE_UPLOADS; // upload based
344
+			if ( 0 === strpos($tmp, $upload_dir['baseurl']) ) {
345
+				$result['base'] = self::BASE_UPLOADS; // upload based
346 346
 				$tmp = str_replace($upload_dir['baseurl'], '', $tmp);
347 347
 			}
348
-			if (0 === strpos($tmp, content_url())) {
349
-				$result['base']= self::BASE_CONTENT; // content-based
348
+			if ( 0 === strpos($tmp, content_url()) ) {
349
+				$result['base'] = self::BASE_CONTENT; // content-based
350 350
 				$tmp = str_replace(content_url(), '', $tmp);
351 351
 			}
352 352
 		}
@@ -370,18 +370,18 @@  discard block
 block discarded – undo
370 370
 	 */
371 371
 	private static function _get_file_url($base, $subdir, $filename, $absolute) {
372 372
 		$url = '';
373
-		if( self::BASE_UPLOADS == $base ) {
373
+		if ( self::BASE_UPLOADS == $base ) {
374 374
 			$upload_dir = wp_upload_dir();
375 375
 			$url = $upload_dir['baseurl'];
376 376
 		}
377
-		if( self::BASE_CONTENT == $base ) {
377
+		if ( self::BASE_CONTENT == $base ) {
378 378
 			$url = content_url();
379 379
 		}
380
-		if(!empty($subdir)) {
380
+		if ( !empty($subdir) ) {
381 381
 			$url .= $subdir;
382 382
 		}
383
-		$url .= '/'.$filename;
384
-		if(!$absolute) {
383
+		$url .= '/' . $filename;
384
+		if ( !$absolute ) {
385 385
 			$url = str_replace(home_url(), '', $url);
386 386
 		}
387 387
 		// $url = TimberURLHelper::remove_double_slashes( $url);
@@ -398,17 +398,17 @@  discard block
 block discarded – undo
398 398
 	 */
399 399
 	private static function _get_file_path($base, $subdir, $filename) {
400 400
 		$path = '';
401
-		if(self::BASE_UPLOADS == $base) {
401
+		if ( self::BASE_UPLOADS == $base ) {
402 402
 			$upload_dir = wp_upload_dir();
403 403
 			$path = $upload_dir['basedir'];
404 404
 		}
405
-		if(self::BASE_CONTENT == $base) {
405
+		if ( self::BASE_CONTENT == $base ) {
406 406
 			$path = WP_CONTENT_DIR;
407 407
 		}
408
-		if(!empty($subdir)) {
408
+		if ( !empty($subdir) ) {
409 409
 			$path .= $subdir;
410 410
 		}
411
-		$path .= '/'.$filename;
411
+		$path .= '/' . $filename;
412 412
 		return $path;
413 413
 	}
414 414
 
@@ -426,13 +426,13 @@  discard block
 block discarded – undo
426 426
 	 * @return string         URL to the new image - or the source one if error
427 427
 	 *
428 428
 	 */
429
-	private static function _operate( $src, $op, $force = false ) {
430
-		if ( empty( $src ) ) {
429
+	private static function _operate($src, $op, $force = false) {
430
+		if ( empty($src) ) {
431 431
 			return '';
432 432
 		}
433 433
 		// if external image, load it first
434
-		if ( TimberURLHelper::is_external_content( $src ) ) {
435
-			$src = self::sideload_image( $src );
434
+		if ( TimberURLHelper::is_external_content($src) ) {
435
+			$src = self::sideload_image($src);
436 436
 		}
437 437
 		// break down URL into components
438 438
 		$au = self::analyze_url($src);
@@ -454,17 +454,17 @@  discard block
 block discarded – undo
454 454
 			$au['basename']
455 455
 		);
456 456
 		// if already exists...
457
-		if ( file_exists( $new_server_path ) ) {
457
+		if ( file_exists($new_server_path) ) {
458 458
 			if ( $force ) {
459 459
 				// Force operation - warning: will regenerate the image on every pageload, use for testing purposes only!
460
-				unlink( $new_server_path );
460
+				unlink($new_server_path);
461 461
 			} else {
462 462
 				// return existing file (caching)
463 463
 				return $new_url;
464 464
 			}
465 465
 		}
466 466
 		// otherwise generate result file
467
-		if($op->run($old_server_path, $new_server_path)) {
467
+		if ( $op->run($old_server_path, $new_server_path) ) {
468 468
 			return $new_url;
469 469
 		} else {
470 470
 			// in case of error, we return source file itself
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
 		);
487 487
 		return $new_url;
488 488
 	}
489
-	public static function get_letterbox_file_path($url, $w, $h, $color ) {
489
+	public static function get_letterbox_file_path($url, $w, $h, $color) {
490 490
 		$au = self::analyze_url($url);
491 491
 		$op = new TimberImageOperationLetterbox($w, $h, $color);
492 492
 		$new_path = self::_get_file_path(
Please login to merge, or discard this patch.
lib/image/timber-image-operation-retina.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -44,30 +44,30 @@
 block discarded – undo
44 44
      *                               (ex: /src/var/www/wp-content/uploads/[email protected])
45 45
      * @return bool                  true if everything went fine, false otherwise
46 46
      */
47
-    function run($load_filename, $save_filename){
48
-        $image = wp_get_image_editor( $load_filename );
49
-        if ( !is_wp_error( $image ) ) {
47
+    function run($load_filename, $save_filename) {
48
+        $image = wp_get_image_editor($load_filename);
49
+        if ( !is_wp_error($image) ) {
50 50
             $current_size = $image->get_size();
51 51
             $src_w = $current_size['width'];
52 52
             $src_h = $current_size['height'];
53 53
             // Get ratios
54 54
             $w = $src_w * $this->factor;
55 55
             $h = $src_h * $this->factor;
56
-            $image->crop( 0, 0, $src_w, $src_h, $w, $h );
57
-            $result = $image->save( $save_filename );
58
-            if ( is_wp_error( $result ) ) {
56
+            $image->crop(0, 0, $src_w, $src_h, $w, $h);
57
+            $result = $image->save($save_filename);
58
+            if ( is_wp_error($result) ) {
59 59
             	// @codeCoverageIgnoreStart
60
-				TimberHelper::error_log( 'Error resizing image' );
61
-				TimberHelper::error_log( $result );
60
+				TimberHelper::error_log('Error resizing image');
61
+				TimberHelper::error_log($result);
62 62
 				return false;
63 63
 				// @codeCoverageIgnoreEnd
64 64
             } else {
65 65
                 return true;
66 66
             }
67
-        } else if ( isset( $image->error_data['error_loading_image'] ) ) {
68
-            TimberHelper::error_log( 'Error loading ' . $image->error_data['error_loading_image'] );
67
+        } else if ( isset($image->error_data['error_loading_image']) ) {
68
+            TimberHelper::error_log('Error loading ' . $image->error_data['error_loading_image']);
69 69
         } else {
70
-            TimberHelper::error_log( $image );
70
+            TimberHelper::error_log($image);
71 71
         }
72 72
         return false;
73 73
     }
Please login to merge, or discard this patch.
lib/image/timber-image-operation-resize.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -21,8 +21,8 @@  discard block
 block discarded – undo
21 21
 		$this->w = $w;
22 22
 		$this->h = $h;
23 23
 		// Sanitize crop position
24
-		$allowed_crop_positions = array( 'default', 'center', 'top', 'bottom', 'left', 'right' );
25
-		if ( $crop !== false && !in_array( $crop, $allowed_crop_positions ) ) {
24
+		$allowed_crop_positions = array('default', 'center', 'top', 'bottom', 'left', 'right');
25
+		if ( $crop !== false && !in_array($crop, $allowed_crop_positions) ) {
26 26
 			$crop = $allowed_crop_positions[0];
27 27
 		}
28 28
 		$this->crop = $crop;
@@ -42,15 +42,15 @@  discard block
 block discarded – undo
42 42
 		if ( $this->h ) {
43 43
 			$h = $this->h;
44 44
 		}
45
-		$result = $src_filename . '-' . $w . 'x' . $h . '-c-' . ( $this->crop ? $this->crop : 'f' ); // Crop will be either user named or f (false)
46
-		if($src_extension) {
47
-			$result .= '.'.$src_extension;
45
+		$result = $src_filename . '-' . $w . 'x' . $h . '-c-' . ($this->crop ? $this->crop : 'f'); // Crop will be either user named or f (false)
46
+		if ( $src_extension ) {
47
+			$result .= '.' . $src_extension;
48 48
 		}
49 49
 		return $result;
50 50
 	}
51 51
 
52
-	protected function run_animated_gif( $load_filename, $save_filename ) {
53
-		$image = wp_get_image_editor( $load_filename );
52
+	protected function run_animated_gif($load_filename, $save_filename) {
53
+		$image = wp_get_image_editor($load_filename);
54 54
 		$current_size = $image->get_size();
55 55
 		$src_w = $current_size['width'];
56 56
 		$src_h = $current_size['height'];
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 		}
62 62
 		$image = new Imagick($load_filename);
63 63
 		$image = $image->coalesceImages();
64
-		$crop = self::get_target_sizes( $load_filename );
64
+		$crop = self::get_target_sizes($load_filename);
65 65
 		foreach ($image as $frame) {
66 66
 			$frame->cropImage($crop['src_w'], $crop['src_h'], $crop['x'], $crop['y']);
67 67
 			$frame->thumbnailImage($w, $h);
@@ -71,8 +71,8 @@  discard block
 block discarded – undo
71 71
 		return $image->writeImages($save_filename, true);
72 72
 	}
73 73
 
74
-	protected function get_target_sizes( $load_filename ) {
75
-		$image = wp_get_image_editor( $load_filename );
74
+	protected function get_target_sizes($load_filename) {
75
+		$image = wp_get_image_editor($load_filename);
76 76
 		$w = $this->w;
77 77
 		$h = $this->h;
78 78
 		$crop = $this->crop;
@@ -82,11 +82,11 @@  discard block
 block discarded – undo
82 82
 		$src_h = $current_size['height'];
83 83
 		$src_ratio = $src_w / $src_h;
84 84
 		if ( !$h ) {
85
-			$h = round( $w / $src_ratio );
85
+			$h = round($w / $src_ratio);
86 86
 		}
87 87
 		if ( !$w ) {
88 88
 			//the user wants to resize based on constant height
89
-			$w = round( $h * $src_ratio );
89
+			$w = round($h * $src_ratio);
90 90
 		}
91 91
 		if ( !$crop ) {
92 92
 			return array(
@@ -100,12 +100,12 @@  discard block
 block discarded – undo
100 100
 		$src_wt = $src_h * $dest_ratio;
101 101
 		$src_ht = $src_w / $dest_ratio;
102 102
 		$src_x = $src_w / 2 - $src_wt / 2;
103
-		$src_y = ( $src_h - $src_ht ) / 6;
103
+		$src_y = ($src_h - $src_ht) / 6;
104 104
 		//now specific overrides based on options:
105 105
 		if ( $crop == 'center' ) {
106 106
 			// Get source x and y
107
-			$src_x = round( ( $src_w - $src_wt ) / 2 );
108
-			$src_y = round( ( $src_h - $src_ht ) / 2 );
107
+			$src_x = round(($src_w - $src_wt) / 2);
108
+			$src_y = round(($src_h - $src_ht) / 2);
109 109
 		} else if ( $crop == 'top' ) {
110 110
 			$src_y = 0;
111 111
 		} else if ( $crop == 'bottom' ) {
@@ -147,35 +147,35 @@  discard block
 block discarded – undo
147 147
 			//return if successful
148 148
 			//proceed if not
149 149
 			$gif = self::run_animated_gif($load_filename, $save_filename);
150
-			if ($gif) {
150
+			if ( $gif ) {
151 151
 				return true;
152 152
 			}
153 153
 		}
154
-		$image = wp_get_image_editor( $load_filename );
155
-		if ( !is_wp_error( $image ) ) {
156
-			$crop = self::get_target_sizes( $load_filename );
157
-			$image->crop( 	$crop['x'],
154
+		$image = wp_get_image_editor($load_filename);
155
+		if ( !is_wp_error($image) ) {
156
+			$crop = self::get_target_sizes($load_filename);
157
+			$image->crop($crop['x'],
158 158
 							$crop['y'],
159 159
 							$crop['src_w'],
160 160
 							$crop['src_h'],
161 161
 							$crop['target_w'],
162 162
 							$crop['target_h']
163 163
 			);
164
-			$result = $image->save( $save_filename );
165
-			if ( is_wp_error( $result ) ) {
164
+			$result = $image->save($save_filename);
165
+			if ( is_wp_error($result) ) {
166 166
 				// @codeCoverageIgnoreStart
167
-				TimberHelper::error_log( 'Error resizing image' );
168
-				TimberHelper::error_log( $result );
167
+				TimberHelper::error_log('Error resizing image');
168
+				TimberHelper::error_log($result);
169 169
 				return false;
170 170
 				// @codeCoverageIgnoreEnd
171 171
 			} else {
172 172
 				return true;
173 173
 			}
174
-		} else if ( isset( $image->error_data['error_loading_image'] ) ) {
174
+		} else if ( isset($image->error_data['error_loading_image']) ) {
175 175
 			// @codeCoverageIgnoreStart
176
-			TimberHelper::error_log( 'Error loading ' . $image->error_data['error_loading_image'] );
176
+			TimberHelper::error_log('Error loading ' . $image->error_data['error_loading_image']);
177 177
 		} else {
178
-			TimberHelper::error_log( $image );
178
+			TimberHelper::error_log($image);
179 179
 			// @codeCoverageIgnoreEnd
180 180
 		}
181 181
 	}
Please login to merge, or discard this patch.
lib/timber-function-wrapper.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
 
9 9
 	public function __toString() {
10 10
 		try {
11
-			return (string)$this->call();
11
+			return (string) $this->call();
12 12
 		} catch (Exception $e) {
13 13
 			return 'Caught exception: ' . $e->getMessage() . "\n";
14 14
 		}
@@ -21,12 +21,12 @@  discard block
 block discarded – undo
21 21
 	 * @param array   $args
22 22
 	 * @param bool    $return_output_buffer
23 23
 	 */
24
-	public function __construct( $function, $args = array(), $return_output_buffer = false ) {
24
+	public function __construct($function, $args = array(), $return_output_buffer = false) {
25 25
 		$this->_function = $function;
26 26
 		$this->_args = $args;
27 27
 		$this->_use_ob = $return_output_buffer;
28 28
 
29
-		add_filter( 'get_twig', array( &$this, 'add_to_twig' ) );
29
+		add_filter('get_twig', array(&$this, 'add_to_twig'));
30 30
 	}
31 31
 
32 32
 	/**
@@ -35,12 +35,12 @@  discard block
 block discarded – undo
35 35
 	 * @param Twig_Environment $twig
36 36
 	 * @return Twig_Environment
37 37
 	 */
38
-	public function add_to_twig( $twig ) {
38
+	public function add_to_twig($twig) {
39 39
 		$wrapper = $this;
40 40
 
41
-		$twig->addFunction( new Twig_SimpleFunction( $this->_function, function () use ( $wrapper ) {
42
-					return call_user_func_array( array( $wrapper, 'call' ), func_get_args() );
43
-				} ) );
41
+		$twig->addFunction(new Twig_SimpleFunction($this->_function, function() use ($wrapper) {
42
+					return call_user_func_array(array($wrapper, 'call'), func_get_args());
43
+				} ));
44 44
 
45 45
 		return $twig;
46 46
 	}
@@ -51,12 +51,12 @@  discard block
 block discarded – undo
51 51
 	 * @return string
52 52
 	 */
53 53
 	public function call() {
54
-		$args = $this->_parse_args( func_get_args(), $this->_args );
54
+		$args = $this->_parse_args(func_get_args(), $this->_args);
55 55
 
56 56
 		if ( $this->_use_ob ) {
57
-			return TimberHelper::ob_function( $this->_function, $args );
57
+			return TimberHelper::ob_function($this->_function, $args);
58 58
 		} else {
59
-			return call_user_func_array( $this->_function, $args );
59
+			return call_user_func_array($this->_function, $args);
60 60
 		}
61 61
 	}
62 62
 
@@ -67,12 +67,12 @@  discard block
 block discarded – undo
67 67
 	 * @param array   $defaults
68 68
 	 * @return array
69 69
 	 */
70
-	private function _parse_args( $args, $defaults ) {
71
-		$_arg = reset( $defaults );
70
+	private function _parse_args($args, $defaults) {
71
+		$_arg = reset($defaults);
72 72
 
73
-		foreach ( $args as $index => $arg ) {
74
-			$defaults[$index] = is_null( $arg ) ? $_arg : $arg;
75
-			$_arg = next( $defaults );
73
+		foreach ($args as $index => $arg) {
74
+			$defaults[$index] = is_null($arg) ? $_arg : $arg;
75
+			$_arg = next($defaults);
76 76
 		}
77 77
 
78 78
 		return $defaults;
Please login to merge, or discard this patch.
lib/timber-comment.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	function init($cid) {
56 56
 		$comment_data = $cid;
57
-		if (is_integer($cid)) {
57
+		if ( is_integer($cid) ) {
58 58
 			$comment_data = get_comment($cid);
59 59
 		}
60 60
 		$this->import($comment_data);
@@ -86,11 +86,11 @@  discard block
 block discarded – undo
86 86
 	 * @return TimberUser
87 87
 	 */
88 88
 	public function author() {
89
-		if ($this->user_id) {
89
+		if ( $this->user_id ) {
90 90
 			return new TimberUser($this->user_id);
91 91
 		} else {
92 92
 			$author = new TimberUser(0);
93
-			if (isset($this->comment_author) && $this->comment_author) {
93
+			if ( isset($this->comment_author) && $this->comment_author ) {
94 94
 				$author->name = $this->comment_author;
95 95
 			} else {
96 96
 				$author->name = 'Anonymous';
@@ -114,21 +114,21 @@  discard block
 block discarded – undo
114 114
 	 * @return bool|mixed|string
115 115
 	 */
116 116
 	public function avatar($size = 92, $default = '') {
117
-		if (!get_option('show_avatars')) {
117
+		if ( !get_option('show_avatars') ) {
118 118
 			return false;
119 119
 		}
120
-		if (!is_numeric($size)) {
120
+		if ( !is_numeric($size) ) {
121 121
 			$size = '92';
122 122
 		}
123 123
 
124 124
 		$email = $this->avatar_email();
125 125
 		$email_hash = '';
126
-		if (!empty($email)) {
126
+		if ( !empty($email) ) {
127 127
 			$email_hash = md5(strtolower(trim($email)));
128 128
 		}
129 129
 		$host = $this->avatar_host($email_hash);
130 130
 		$default = $this->avatar_default($default, $email, $size, $host);
131
-		if (!empty($email)) {
131
+		if ( !empty($email) ) {
132 132
 			$avatar = $this->avatar_out($default, $host, $email_hash, $size);
133 133
 		} else {
134 134
 			$avatar = $default;
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
 	 * ```
180 180
 	 * @return string
181 181
 	 */
182
-	public function date( $date_format = '' ) {
182
+	public function date($date_format = '') {
183 183
 		$df = $date_format ? $date_format : get_option('date_format');
184
-		$the_date = (string)mysql2date($df, $this->comment_date);
184
+		$the_date = (string) mysql2date($df, $this->comment_date);
185 185
 		return apply_filters('get_comment_date ', $the_date, $df);
186 186
 	}
187 187
 
@@ -204,9 +204,9 @@  discard block
 block discarded – undo
204 204
 	 * ```
205 205
 	 * @return string
206 206
 	 */
207
-	public function time( $time_format = '' ) {
207
+	public function time($time_format = '') {
208 208
 		$tf = $time_format ? $time_format : get_option('time_format');
209
-		$the_time = (string)mysql2date($tf, $this->comment_date);
209
+		$the_time = (string) mysql2date($tf, $this->comment_date);
210 210
 		return apply_filters('get_comment_time', $the_time, $tf);
211 211
 	}
212 212
 
@@ -232,14 +232,14 @@  discard block
 block discarded – undo
232 232
 	 * @return mixed
233 233
 	 */
234 234
 	protected function get_meta_fields($comment_id = null) {
235
-		if ($comment_id === null) {
235
+		if ( $comment_id === null ) {
236 236
 			$comment_id = $this->ID;
237 237
 		}
238 238
 		//Could not find a WP function to fetch all comment meta data, so I made one.
239 239
 		apply_filters('timber_comment_get_meta_pre', array(), $comment_id);
240 240
 		$comment_metas = get_comment_meta($comment_id);
241 241
 		foreach ($comment_metas as &$cm) {
242
-			if (is_array($cm) && count($cm) == 1) {
242
+			if ( is_array($cm) && count($cm) == 1 ) {
243 243
 				$cm = $cm[0];
244 244
 			}
245 245
 		}
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
 	 */
255 255
 	protected function get_meta_field($field_name) {
256 256
 		$value = apply_filters('timber_comment_get_meta_field_pre', null, $this->ID, $field_name, $this);
257
-		if ($value === null) {
257
+		if ( $value === null ) {
258 258
 			$value = get_comment_meta($this->ID, $field_name, true);
259 259
 		}
260 260
 		$value = apply_filters('timber_comment_get_meta_field', $value, $this->ID, $field_name, $this);
@@ -267,9 +267,9 @@  discard block
 block discarded – undo
267 267
 	 * @api
268 268
 	 * @return string
269 269
 	 */
270
-	public function reply_link( $reply_text = 'Reply' ) {
270
+	public function reply_link($reply_text = 'Reply') {
271 271
 		if ( is_singular() && comments_open() && get_option('thread_comments') ) {
272
-			wp_enqueue_script( 'comment-reply' );
272
+			wp_enqueue_script('comment-reply');
273 273
 		}
274 274
 
275 275
 		// Get the comments depth option from the admin panel
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
 			'max_depth' => $max_depth,
285 285
 		);
286 286
 
287
-		return get_comment_reply_link( $args, $this->ID, $this->post_id );
287
+		return get_comment_reply_link($args, $this->ID, $this->post_id);
288 288
 	}
289 289
 
290 290
 	/* AVATAR Stuff
@@ -295,9 +295,9 @@  discard block
 block discarded – undo
295 295
 	 * @return string
296 296
 	 */
297 297
 	protected function avatar_email() {
298
-		$id = (int)$this->user_id;
298
+		$id = (int) $this->user_id;
299 299
 		$user = get_userdata($id);
300
-		if ($user) {
300
+		if ( $user ) {
301 301
 			$email = $user->user_email;
302 302
 		} else {
303 303
 			$email = $this->comment_author_email;
@@ -311,10 +311,10 @@  discard block
 block discarded – undo
311 311
 	 * @return string
312 312
 	 */
313 313
 	protected function avatar_host($email_hash) {
314
-		if (is_ssl()) {
314
+		if ( is_ssl() ) {
315 315
 			$host = 'https://secure.gravatar.com';
316 316
 		} else {
317
-			if (!empty($email_hash)) {
317
+			if ( !empty($email_hash) ) {
318 318
 				$host = sprintf("http://%d.gravatar.com", (hexdec($email_hash[0]) % 2));
319 319
 			} else {
320 320
 				$host = 'http://0.gravatar.com';
@@ -333,28 +333,28 @@  discard block
 block discarded – undo
333 333
 	 * @return string
334 334
 	 */
335 335
 	protected function avatar_default($default, $email, $size, $host) {
336
-		if (substr($default, 0, 1) == '/') {
336
+		if ( substr($default, 0, 1) == '/' ) {
337 337
 			$default = home_url() . $default;
338 338
 		}
339 339
 
340
-		if (empty($default)) {
340
+		if ( empty($default) ) {
341 341
 			$avatar_default = get_option('avatar_default');
342
-			if (empty($avatar_default)) {
342
+			if ( empty($avatar_default) ) {
343 343
 				$default = 'mystery';
344 344
 			} else {
345 345
 				$default = $avatar_default;
346 346
 			}
347 347
 		}
348
-		if ('mystery' == $default) {
348
+		if ( 'mystery' == $default ) {
349 349
 			$default = $host . '/avatar/ad516503a11cd5ca435acc9bb6523536?s=' . $size;
350 350
 			// ad516503a11cd5ca435acc9bb6523536 == md5('[email protected]')
351
-		} else if ('blank' == $default) {
351
+		} else if ( 'blank' == $default ) {
352 352
 			$default = $email ? 'blank' : includes_url('images/blank.gif');
353
-		} else if (!empty($email) && 'gravatar_default' == $default) {
353
+		} else if ( !empty($email) && 'gravatar_default' == $default ) {
354 354
 			$default = '';
355
-		} else if ('gravatar_default' == $default) {
355
+		} else if ( 'gravatar_default' == $default ) {
356 356
 			$default = $host . '/avatar/?s=' . $size;
357
-		} else if (empty($email) && !strstr($default, 'http://')) {
357
+		} else if ( empty($email) && !strstr($default, 'http://') ) {
358 358
 			$default = $host . '/avatar/?d=' . $default . '&amp;s=' . $size;
359 359
 		}
360 360
 		return $default;
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
371 371
 	protected function avatar_out($default, $host, $email_hash, $size) {
372 372
 		$out = $host . '/avatar/' . $email_hash . '?s=' . $size . '&amp;d=' . urlencode($default);
373 373
 		$rating = get_option('avatar_rating');
374
-		if (!empty($rating)) {
374
+		if ( !empty($rating) ) {
375 375
 			$out .= '&amp;r=' . $rating;
376 376
 		}
377 377
 		return str_replace('&#038;', '&amp;', esc_url($out));
Please login to merge, or discard this patch.
lib/timber-image.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -141,27 +141,27 @@  discard block
 block discarded – undo
141 141
 	 * @internal
142 142
 	 * @param  int $iid the id number of the image in the WP database
143 143
 	 */
144
-	protected function get_image_info( $iid ) {
144
+	protected function get_image_info($iid) {
145 145
 		$image_info = $iid;
146
-		if (is_numeric($iid)) {
146
+		if ( is_numeric($iid) ) {
147 147
 			$image_info = wp_get_attachment_metadata($iid);
148
-			if (!is_array($image_info)) {
148
+			if ( !is_array($image_info) ) {
149 149
 				$image_info = array();
150 150
 			}
151 151
 			$image_custom = get_post_custom($iid);
152 152
 			$basic = get_post($iid);
153
-			if ($basic) {
154
-				if (isset($basic->post_excerpt)) {
153
+			if ( $basic ) {
154
+				if ( isset($basic->post_excerpt) ) {
155 155
 					$this->caption = $basic->post_excerpt;
156 156
 				}
157 157
 				$image_custom = array_merge($image_custom, get_object_vars($basic));
158 158
 			}
159 159
 			return array_merge($image_info, $image_custom);
160 160
 		}
161
-		if (is_array($image_info) && isset($image_info['image'])) {
161
+		if ( is_array($image_info) && isset($image_info['image']) ) {
162 162
 			return $image_info['image'];
163 163
 		}
164
-		if (is_object($image_info)) {
164
+		if ( is_object($image_info) ) {
165 165
 		   return get_object_vars($image_info);
166 166
 		}
167 167
 		return $iid;
@@ -193,9 +193,9 @@  discard block
 block discarded – undo
193 193
 	 * @internal
194 194
 	 * @param int $iid
195 195
 	 */
196
-	function init( $iid = false ) {
197
-		if ( !is_numeric( $iid ) && is_string( $iid ) ) {
198
-			if (strstr($iid, '://')) {
196
+	function init($iid = false) {
197
+		if ( !is_numeric($iid) && is_string($iid) ) {
198
+			if ( strstr($iid, '://') ) {
199 199
 				$this->init_with_url($iid);
200 200
 				return;
201 201
 			}
@@ -244,9 +244,9 @@  discard block
 block discarded – undo
244 244
 	 * @internal
245 245
 	 * @param string $relative_path
246 246
 	 */
247
-	protected function init_with_relative_path( $relative_path ) {
248
-		$this->abs_url = home_url( $relative_path );
249
-		$file_path = TimberURLHelper::get_full_path( $relative_path );
247
+	protected function init_with_relative_path($relative_path) {
248
+		$this->abs_url = home_url($relative_path);
249
+		$file_path = TimberURLHelper::get_full_path($relative_path);
250 250
 		$this->file_loc = $file_path;
251 251
 		$this->file = $file_path;
252 252
 	}
@@ -255,8 +255,8 @@  discard block
 block discarded – undo
255 255
 	 * @internal
256 256
 	 * @param string $file_path
257 257
 	 */
258
-	protected function init_with_file_path( $file_path ) {
259
-		$url = TimberURLHelper::file_system_to_url( $file_path );
258
+	protected function init_with_file_path($file_path) {
259
+		$url = TimberURLHelper::file_system_to_url($file_path);
260 260
 		$this->abs_url = $url;
261 261
 		$this->file_loc = $file_path;
262 262
 		$this->file = $file_path;
@@ -455,8 +455,8 @@  discard block
 block discarded – undo
455 455
 	 * @param string $size
456 456
 	 * @return bool|string
457 457
 	 */
458
-	function get_src( $size = '' ) {
459
-		return $this->src( $size );
458
+	function get_src($size = '') {
459
+		return $this->src($size);
460 460
 	}
461 461
 
462 462
 	/**
Please login to merge, or discard this patch.
lib/image/timber-image-operation-tojpg.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -36,14 +36,14 @@  discard block
 block discarded – undo
36 36
 	 * @return bool                  true if everything went fine, false otherwise
37 37
 	 */
38 38
 	function run($load_filename, $save_filename) {
39
-		$input = self::image_create( $load_filename );
40
-		list( $width, $height ) = getimagesize( $load_filename );
41
-		$output = imagecreatetruecolor( $width, $height );
42
-		$c = self::hexrgb( $this->color );
43
-		$color = imagecolorallocate( $output, $c['red'], $c['green'], $c['blue'] );
44
-		imagefilledrectangle( $output, 0, 0, $width, $height, $color );
45
-		imagecopy( $output, $input, 0, 0, 0, 0, $width, $height );
46
-		imagejpeg( $output, $save_filename );
39
+		$input = self::image_create($load_filename);
40
+		list($width, $height) = getimagesize($load_filename);
41
+		$output = imagecreatetruecolor($width, $height);
42
+		$c = self::hexrgb($this->color);
43
+		$color = imagecolorallocate($output, $c['red'], $c['green'], $c['blue']);
44
+		imagefilledrectangle($output, 0, 0, $width, $height, $color);
45
+		imagecopy($output, $input, 0, 0, 0, 0, $width, $height);
46
+		imagejpeg($output, $save_filename);
47 47
 		return true;
48 48
 	}
49 49
 
@@ -52,10 +52,10 @@  discard block
 block discarded – undo
52 52
 	 * @return resource an image identifier representing the image obtained from the given filename
53 53
 	 *                  will return the same data type regardless of whether the source is gif or png
54 54
 	 */
55
-	function image_create( $filename, $ext = 'auto' ) {
55
+	function image_create($filename, $ext = 'auto') {
56 56
 		if ( $ext == 'auto' ) {
57 57
 			$ext = wp_check_filetype($filename);
58
-			if (isset($ext['ext'])) {
58
+			if ( isset($ext['ext']) ) {
59 59
 				$ext = $ext['ext'];
60 60
 			}
61 61
 		}
@@ -69,6 +69,6 @@  discard block
 block discarded – undo
69 69
 		if ( $ext == 'jpg' || $ext == 'jpeg' ) {
70 70
 			return imagecreatefromjpeg($filename);
71 71
 		}
72
-		throw new InvalidArgumentException( 'image_create only accepts PNG, GIF and JPGs. File extension was: '.$ext );
72
+		throw new InvalidArgumentException('image_create only accepts PNG, GIF and JPGs. File extension was: ' . $ext);
73 73
 	}
74 74
 }
Please login to merge, or discard this patch.
lib/timber-archives.php 1 patch
Spacing   +25 added lines, -25 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 * }
55 55
 	 * @param string $base any additional paths that need to be prepended to the URLs that are generated, for example: "tags"
56 56
 	 */
57
-	function __construct( $args = null, $base = '' ) {
57
+	function __construct($args = null, $base = '') {
58 58
 		$this->init($args, $base);
59 59
 	}
60 60
 
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 	 * @param array|string $args
64 64
 	 * @param string $base
65 65
 	 */
66
-	function init( $args = null, $base = '' ) {
66
+	function init($args = null, $base = '') {
67 67
 		$this->base = $base;
68 68
 		$this->items = $this->get_items($args);
69 69
 	}
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 	 * @param string $text
75 75
 	 * @return mixed
76 76
 	 */
77
-	protected function get_archives_link( $url, $text ) {
77
+	protected function get_archives_link($url, $text) {
78 78
 		$ret = array();
79 79
 		$ret['text'] = $ret['title'] = $ret['name'] = wptexturize($text);
80 80
 		$ret['url'] = $ret['link'] = esc_url(TimberURLHelper::prepend_to_url($url, $this->base));
@@ -91,19 +91,19 @@  discard block
 block discarded – undo
91 91
 	 * @param string $limit
92 92
 	 * @return array
93 93
 	 */
94
-	protected function get_items_yearly( $args, $last_changed, $join, $where, $order, $limit ) {
94
+	protected function get_items_yearly($args, $last_changed, $join, $where, $order, $limit) {
95 95
 		global $wpdb;
96 96
 		$output = array();
97 97
 		$query = "SELECT YEAR(post_date) AS `year`, count(ID) as posts FROM {$wpdb->posts} $join $where GROUP BY YEAR(post_date) ORDER BY post_date $order $limit";
98 98
 		$key = md5($query);
99 99
 		$key = "wp_get_archives:$key:$last_changed";
100
-		if (!$results = wp_cache_get($key, 'posts')) {
100
+		if ( !$results = wp_cache_get($key, 'posts') ) {
101 101
 			$results = $wpdb->get_results($query);
102 102
 			wp_cache_set($key, $results, 'posts');
103 103
 		}
104
-		if ($results) {
105
-			foreach ( (array)$results as $result ) {
106
-				$url = get_year_link( $result->year );
104
+		if ( $results ) {
105
+			foreach ((array) $results as $result) {
106
+				$url = get_year_link($result->year);
107 107
 				$text = sprintf('%d', $result->year);
108 108
 				$output[] = $this->get_archives_link($url, $text);
109 109
 			}
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 	 * @param bool $nested
123 123
 	 * @return array
124 124
 	 */
125
-	protected function get_items_monthly( $args, $last_changed, $join, $where, $order, $limit = 1000, $nested = true ) {
125
+	protected function get_items_monthly($args, $last_changed, $join, $where, $order, $limit = 1000, $nested = true) {
126 126
 		global $wpdb, $wp_locale;
127 127
 		$output = array();
128 128
 		$defaults = array(
@@ -137,26 +137,26 @@  discard block
 block discarded – undo
137 137
 			. "ORDER BY post_date $order $limit";
138 138
 		$key = md5($query);
139 139
 		$key = "wp_get_archives:$key:$last_changed";
140
-		if (!$results = wp_cache_get($key, 'posts')) {
140
+		if ( !$results = wp_cache_get($key, 'posts') ) {
141 141
 			$results = $wpdb->get_results($query);
142 142
 			wp_cache_set($key, $results, 'posts');
143 143
 		}
144
-		if ($results) {
145
-			foreach ((array)$results as $result) {
144
+		if ( $results ) {
145
+			foreach ((array) $results as $result) {
146 146
 				$url = get_month_link($result->year, $result->month);
147
-				if ($show_year && !$nested) {
147
+				if ( $show_year && !$nested ) {
148 148
 					$text = sprintf(__('%1$s %2$d'), $wp_locale->get_month($result->month), $result->year);
149 149
 				} else {
150 150
 					$text = sprintf(__('%1$s'), $wp_locale->get_month($result->month));
151 151
 				}
152
-				if ($nested) {
152
+				if ( $nested ) {
153 153
 					$output[$result->year][] = $this->get_archives_link($url, $text);
154 154
 				} else {
155 155
 					$output[] = $this->get_archives_link($url, $text);
156 156
 				}
157 157
 			}
158 158
 		}
159
-		if ($nested) {
159
+		if ( $nested ) {
160 160
 			$out2 = array();
161 161
 			foreach ($output as $year => $months) {
162 162
 				$out2[] = array('name' => $year, 'children' => $months);
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
 	 * @param array|string $args
172 172
 	 * @return array|string
173 173
 	 */
174
-	function get_items( $args = null ) {
174
+	function get_items($args = null) {
175 175
 		global $wpdb;
176 176
 
177 177
 		$defaults = array(
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
 		$archive_week_start_date_format = 'Y/m/d';
221 221
 		$archive_week_end_date_format = 'Y/m/d';
222 222
 
223
-		if (!$archive_date_format_over_ride) {
223
+		if ( !$archive_date_format_over_ride ) {
224 224
 			$archive_day_date_format = get_option('date_format');
225 225
 			$archive_week_start_date_format = get_option('date_format');
226 226
 			$archive_week_end_date_format = get_option('date_format');
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
 
233 233
 		$output = array();
234 234
 		$last_changed = wp_cache_get('last_changed', 'posts');
235
-		if (!$last_changed) {
235
+		if ( !$last_changed ) {
236 236
 			$last_changed = microtime();
237 237
 			wp_cache_set('last_changed', $last_changed, 'posts');
238 238
 		}
@@ -246,14 +246,14 @@  discard block
 block discarded – undo
246 246
 			$query = "SELECT YEAR(post_date) AS `year`, MONTH(post_date) AS `month`, DAYOFMONTH(post_date) AS `dayofmonth`, count(ID) as posts FROM $wpdb->posts $join $where GROUP BY YEAR(post_date), MONTH(post_date), DAYOFMONTH(post_date) ORDER BY post_date $order $limit";
247 247
 			$key = md5($query);
248 248
 			$key = "wp_get_archives:$key:$last_changed";
249
-			if (!$results = wp_cache_get($key, 'posts')) {
249
+			if ( !$results = wp_cache_get($key, 'posts') ) {
250 250
 				$results = $wpdb->get_results($query);
251 251
 				$cache = array();
252 252
 				$cache[$key] = $results;
253 253
 				wp_cache_set($key, $results, 'posts');
254 254
 			}
255 255
 			if ( $results ) {
256
-				foreach ( (array)$results as $result ) {
256
+				foreach ((array) $results as $result) {
257 257
 					$url = get_day_link($result->year, $result->month, $result->dayofmonth);
258 258
 					$date = sprintf('%1$d-%2$02d-%3$02d 00:00:00', $result->year, $result->month, $result->dayofmonth);
259 259
 					$text = mysql2date($archive_day_date_format, $date);
@@ -266,13 +266,13 @@  discard block
 block discarded – undo
266 266
 				. "count( `ID` ) AS `posts` FROM `$wpdb->posts` $join $where GROUP BY $week, YEAR( `post_date` ) ORDER BY `post_date` $order $limit";
267 267
 			$key = md5($query);
268 268
 			$key = "wp_get_archives:$key:$last_changed";
269
-			if (!$results = wp_cache_get($key, 'posts')) {
269
+			if ( !$results = wp_cache_get($key, 'posts') ) {
270 270
 				$results = $wpdb->get_results($query);
271 271
 				wp_cache_set($key, $results, 'posts');
272 272
 			}
273 273
 			$arc_w_last = '';
274 274
 			if ( $results ) {
275
-				foreach ( (array)$results as $result ) {
275
+				foreach ((array) $results as $result) {
276 276
 					if ( $result->week != $arc_w_last ) {
277 277
 						$arc_year = $result->yr;
278 278
 						$arc_w_last = $result->week;
@@ -295,10 +295,10 @@  discard block
 block discarded – undo
295 295
 				wp_cache_set($key, $results, 'posts');
296 296
 			}
297 297
 			if ( $results ) {
298
-				foreach ( (array)$results as $result ) {
299
-					if ($result->post_date != '0000-00-00 00:00:00') {
298
+				foreach ((array) $results as $result) {
299
+					if ( $result->post_date != '0000-00-00 00:00:00' ) {
300 300
 						$url = get_permalink($result);
301
-						if ($result->post_title) {
301
+						if ( $result->post_title ) {
302 302
 							/** This filter is documented in wp-includes/post-template.php */
303 303
 							$text = strip_tags(apply_filters('the_title', $result->post_title, $result->ID));
304 304
 						} else {
Please login to merge, or discard this patch.