Passed
Push — master ( 3e62f5...f6ab3e )
by Warwick
03:37
created
classes/class-cron.php 1 patch
Spacing   +71 added lines, -71 removed lines patch added patch discarded remove patch
@@ -33,11 +33,11 @@  discard block
 block discarded – undo
33 33
 	 * @access private
34 34
 	 */
35 35
 	public function __construct() {
36
-		add_filter( 'cron_schedules', array( $this, 'register_schedule' ), 10, 1 );
37
-		add_action( 'lsx_wetu_importer_settings_before', array( $this, 'watch_for_trigger' ), 200 );
38
-		add_action( 'lsx_wetu_accommodation_images_cron', array( $this, 'process' ), 10, 1 );
39
-		add_action( 'lsx_wetu_accommodation_images_sync', array( $this, 'cron_callback' ), 10, 1 );
40
-		add_filter( 'cmb_meta_boxes', array( $this, 'metaboxes' ) );
36
+		add_filter('cron_schedules', array($this, 'register_schedule'), 10, 1);
37
+		add_action('lsx_wetu_importer_settings_before', array($this, 'watch_for_trigger'), 200);
38
+		add_action('lsx_wetu_accommodation_images_cron', array($this, 'process'), 10, 1);
39
+		add_action('lsx_wetu_accommodation_images_sync', array($this, 'cron_callback'), 10, 1);
40
+		add_filter('cmb_meta_boxes', array($this, 'metaboxes'));
41 41
 	}
42 42
 
43 43
 	/**
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
 	 */
50 50
 	public static function get_instance() {
51 51
 		// If the single instance hasn't been set, set it now.
52
-		if ( null === self::$instance ) {
52
+		if (null === self::$instance) {
53 53
 			self::$instance = new self();
54 54
 		}
55 55
 		return self::$instance;
@@ -61,26 +61,26 @@  discard block
 block discarded – undo
61 61
 	 * @param  array $meta_boxes
62 62
 	 * @return array
63 63
 	 */
64
-	public function metaboxes( array $meta_boxes ) {
64
+	public function metaboxes(array $meta_boxes) {
65 65
 		// Allowed post types.
66
-		$allowed_post_types = array( 'accommodation' );
66
+		$allowed_post_types = array('accommodation');
67 67
 
68 68
 		$fields = array();
69 69
 
70 70
 		$fields[] = array(
71 71
 			'id'   => 'wetu_skip_banner',
72
-			'name' => esc_html__( 'Skip Banner Image', 'lsx-banners' ),
72
+			'name' => esc_html__('Skip Banner Image', 'lsx-banners'),
73 73
 			'type' => 'checkbox',
74 74
 		);
75 75
 
76 76
 		$fields[] = array(
77 77
 			'id'   => 'wetu_skip_featured',
78
-			'name' => esc_html__( 'Skip Featured Image', 'lsx-banners' ),
78
+			'name' => esc_html__('Skip Featured Image', 'lsx-banners'),
79 79
 			'type' => 'checkbox',
80 80
 		);
81 81
 
82 82
 		$meta_boxes[] = array(
83
-			'title'  => esc_html__( 'WETU Settings', 'lsx-banners' ),
83
+			'title'  => esc_html__('WETU Settings', 'lsx-banners'),
84 84
 			'pages'  => $allowed_post_types,
85 85
 			'fields' => $fields,
86 86
 			'context'    => 'side',
@@ -96,10 +96,10 @@  discard block
 block discarded – undo
96 96
 	 * @param  array $schedules
97 97
 	 * @return array
98 98
 	 */
99
-	public function register_schedule( $schedules ) {
99
+	public function register_schedule($schedules) {
100 100
 		$schedules['wetu-5-minutes'] = array(
101 101
 			'interval' => 5 * MINUTE_IN_SECONDS,
102
-			'display'  => __( 'Every 5 minutes', 'lsx-wetu-importer' ),
102
+			'display'  => __('Every 5 minutes', 'lsx-wetu-importer'),
103 103
 		);
104 104
 		return $schedules;
105 105
 	}
@@ -111,26 +111,26 @@  discard block
 block discarded – undo
111 111
 	 */
112 112
 	public function watch_for_trigger() {
113 113
 
114
-		if ( isset( $_GET['page'] ) && 'lsx-wetu-importer' === $_GET['page'] && isset( $_GET['tab'] ) && 'settings' === $_GET['tab'] ) {
114
+		if (isset($_GET['page']) && 'lsx-wetu-importer' === $_GET['page'] && isset($_GET['tab']) && 'settings' === $_GET['tab']) {
115 115
 			$options = lsx_wetu_get_options();
116 116
 
117 117
 			// Check what state the option is in.
118 118
 			$accommodation_cron = 'deactivate';
119
-			if ( isset( $options['accommodation_images_cron'] ) && '' !== $options['accommodation_images_cron'] ) {
119
+			if (isset($options['accommodation_images_cron']) && '' !== $options['accommodation_images_cron']) {
120 120
 				$accommodation_cron = 'activate';
121 121
 			}
122 122
 
123 123
 			// Check what state the cron is in.
124 124
 			$scheduled = false;
125
-			if ( wp_next_scheduled( 'lsx_wetu_accommodation_images_cron' ) ) {
125
+			if (wp_next_scheduled('lsx_wetu_accommodation_images_cron')) {
126 126
 				$scheduled = true;
127 127
 			}
128 128
 
129 129
 			// If activate and its not running.
130
-			if ( false === $scheduled && 'activate' === $accommodation_cron ) {
130
+			if (false === $scheduled && 'activate' === $accommodation_cron) {
131 131
 				$schedule = 'weekly';
132
-				$this->schedule( 'lsx_wetu_accommodation_images_cron', $schedule );
133
-			} elseif ( true === $scheduled && 'deactivate' === $accommodation_cron ) {
132
+				$this->schedule('lsx_wetu_accommodation_images_cron', $schedule);
133
+			} elseif (true === $scheduled && 'deactivate' === $accommodation_cron) {
134 134
 				$this->deactivate();
135 135
 			}
136 136
 		}
@@ -141,8 +141,8 @@  discard block
 block discarded – undo
141 141
 	 *
142 142
 	 * @return void
143 143
 	 */
144
-	public function deactivate( $task = 'lsx_wetu_accommodation_images_cron' ) {
145
-		wp_clear_scheduled_hook( $task, array( $task ) );
144
+	public function deactivate($task = 'lsx_wetu_accommodation_images_cron') {
145
+		wp_clear_scheduled_hook($task, array($task));
146 146
 	}
147 147
 
148 148
 	/**
@@ -153,16 +153,16 @@  discard block
 block discarded – undo
153 153
 	 * @param string $time
154 154
 	 * @return void
155 155
 	 */
156
-	public function schedule( $task = 'lsx_wetu_accommodation_images_cron', $schedule = 'weekly', $time = 'Sunday 10pm' ) {
157
-		$args = array( $task );
158
-		if ( '' === $time ) {
156
+	public function schedule($task = 'lsx_wetu_accommodation_images_cron', $schedule = 'weekly', $time = 'Sunday 10pm') {
157
+		$args = array($task);
158
+		if ('' === $time) {
159 159
 			$time = time();
160 160
 		}
161 161
 
162
-		if ( isset( $_GET['accommodation_images_cron_featured'] ) && '' !== $_GET['accommodation_images_cron_featured'] ) {
162
+		if (isset($_GET['accommodation_images_cron_featured']) && '' !== $_GET['accommodation_images_cron_featured']) {
163 163
 			$args[] = 'featured_image';
164 164
 		}
165
-		wp_schedule_event( $time, $schedule, $task, $args );
165
+		wp_schedule_event($time, $schedule, $task, $args);
166 166
 	}
167 167
 
168 168
 	/**
@@ -170,8 +170,8 @@  discard block
 block discarded – undo
170 170
 	 *
171 171
 	 * @return void
172 172
 	 */
173
-	public function process( $task = '' ) {
174
-		switch ( $task ) {
173
+	public function process($task = '') {
174
+		switch ($task) {
175 175
 			case 'lsx_wetu_accommodation_images_cron':
176 176
 					$this->register_accommodation_images_sync();
177 177
 				break;
@@ -187,10 +187,10 @@  discard block
 block discarded – undo
187 187
 	 * @return void
188 188
 	 */
189 189
 	public function register_accommodation_images_sync() {
190
-		$time = strtotime( '+5 min' );
191
-		if ( ! wp_next_scheduled( 'lsx_wetu_accommodation_images_sync' ) ) {
192
-			$this->load_items_to_sync( 'accommodation_images' );
193
-			$this->schedule( 'lsx_wetu_accommodation_images_sync', 'wetu-5-minutes', $time );
190
+		$time = strtotime('+5 min');
191
+		if (!wp_next_scheduled('lsx_wetu_accommodation_images_sync')) {
192
+			$this->load_items_to_sync('accommodation_images');
193
+			$this->schedule('lsx_wetu_accommodation_images_sync', 'wetu-5-minutes', $time);
194 194
 		}
195 195
 	}
196 196
 
@@ -199,58 +199,58 @@  discard block
 block discarded – undo
199 199
 	 *
200 200
 	 * @return void
201 201
 	 */
202
-	public function cron_callback( $task = '', $featured_image = '' ) {
203
-		$has_accommodation = get_option( $task );
204
-		if ( false !== $has_accommodation && ! empty( $has_accommodation ) ) {
205
-			$next_time = array_slice( $has_accommodation, 3 );
206
-			$this_time = array_slice( $has_accommodation, 0, 2 );
202
+	public function cron_callback($task = '', $featured_image = '') {
203
+		$has_accommodation = get_option($task);
204
+		if (false !== $has_accommodation && !empty($has_accommodation)) {
205
+			$next_time = array_slice($has_accommodation, 3);
206
+			$this_time = array_slice($has_accommodation, 0, 2);
207 207
 
208 208
 			$api_key = $this->get_api_key();
209
-			$url     = 'https://wetu.com/API/Pins/' . $api_key . '/Get?all=include&ids=';
209
+			$url     = 'https://wetu.com/API/Pins/'.$api_key.'/Get?all=include&ids=';
210 210
 
211 211
 			// Run through the current items.
212
-			foreach ( $this_time as $accommodation ) {
213
-				$wetu_id   = get_post_meta( $accommodation, 'lsx_wetu_id', true );
214
-				$last_date = get_post_meta( $accommodation, 'lsx_wetu_modified_date', true );
212
+			foreach ($this_time as $accommodation) {
213
+				$wetu_id   = get_post_meta($accommodation, 'lsx_wetu_id', true);
214
+				$last_date = get_post_meta($accommodation, 'lsx_wetu_modified_date', true);
215 215
 
216 216
 				// Grabbing the image sync.
217
-				$featured_image = get_post_meta( $accommodation, 'wetu_skip_featured', true );
218
-				$banner_image   = get_post_meta( $accommodation, 'wetu_skip_banner', true );
217
+				$featured_image = get_post_meta($accommodation, 'wetu_skip_featured', true);
218
+				$banner_image   = get_post_meta($accommodation, 'wetu_skip_banner', true);
219 219
 
220
-				$accommodation_info = wp_remote_get( $url . $wetu_id );
221
-				if ( ! empty( $accommodation_info ) && isset( $accommodation_info['response'] ) && isset( $accommodation_info['response']['code'] ) && 200 === $accommodation_info['response']['code'] ) {
222
-					$adata = json_decode( $accommodation_info['body'], true );
220
+				$accommodation_info = wp_remote_get($url.$wetu_id);
221
+				if (!empty($accommodation_info) && isset($accommodation_info['response']) && isset($accommodation_info['response']['code']) && 200 === $accommodation_info['response']['code']) {
222
+					$adata = json_decode($accommodation_info['body'], true);
223 223
 
224
-					if ( isset( $adata[0] ) && isset( $adata[0]['last_modified'] ) && '' !== $adata[0]['last_modified'] ) {
225
-						$modified_time = strtotime( $adata[0]['last_modified'] );
226
-						if ( $modified_time > $last_date ) {
224
+					if (isset($adata[0]) && isset($adata[0]['last_modified']) && '' !== $adata[0]['last_modified']) {
225
+						$modified_time = strtotime($adata[0]['last_modified']);
226
+						if ($modified_time > $last_date) {
227 227
 							$accommodation_importer = new \LSX_WETU_Importer_Accommodation();
228 228
 
229
-							if ( false === $banner_image || '' === $banner_image ) {
230
-								$accommodation_importer->set_banner_image( $adata, $accommodation );
229
+							if (false === $banner_image || '' === $banner_image) {
230
+								$accommodation_importer->set_banner_image($adata, $accommodation);
231 231
 							}
232 232
 
233
-							if ( false === $featured_image || '' === $featured_image ) {
234
-								$accommodation_importer->set_featured_image( $adata, $accommodation );
233
+							if (false === $featured_image || '' === $featured_image) {
234
+								$accommodation_importer->set_featured_image($adata, $accommodation);
235 235
 							}
236 236
 
237
-							$accommodation_importer->create_main_gallery( $adata, $accommodation );
238
-							update_post_meta( $accommodation, 'lsx_wetu_modified_date', $modified_time, $last_date );
237
+							$accommodation_importer->create_main_gallery($adata, $accommodation);
238
+							update_post_meta($accommodation, 'lsx_wetu_modified_date', $modified_time, $last_date);
239 239
 						}
240 240
 					}
241 241
 				}
242 242
 			}
243 243
 
244 244
 			// Save the values for next time.
245
-			if ( ! empty( $next_time ) ) {
246
-				update_option( $task, $next_time );
247
-			} else {
248
-				delete_option( $task );
249
-				$this->deactivate( $task );
245
+			if (!empty($next_time)) {
246
+				update_option($task, $next_time);
247
+			}else {
248
+				delete_option($task);
249
+				$this->deactivate($task);
250 250
 			}
251
-		} else {
252
-			$this->deactivate( $task );
253
-			update_option( 'lsx_wetu_nexttime', $task );
251
+		}else {
252
+			$this->deactivate($task);
253
+			update_option('lsx_wetu_nexttime', $task);
254 254
 		}
255 255
 	}
256 256
 
@@ -260,14 +260,14 @@  discard block
 block discarded – undo
260 260
 	 * @param  string $task
261 261
 	 * @return void
262 262
 	 */
263
-	public function load_items_to_sync( $task = 'accommodation_images' ) {
263
+	public function load_items_to_sync($task = 'accommodation_images') {
264 264
 		$args = array(
265 265
 			'post_status'    => 'publish',
266 266
 			'posts_per_page' => -1,
267 267
 			'nopagin'        => true,
268 268
 			'fields'         => 'ids',
269 269
 		);
270
-		switch ( $task ) {
270
+		switch ($task) {
271 271
 			case 'accommodation_images':
272 272
 					$args['post_type'] = 'accommodation';
273 273
 				break;
@@ -275,9 +275,9 @@  discard block
 block discarded – undo
275 275
 			default:
276 276
 				break;
277 277
 		}
278
-		$items = new \WP_Query( $args );
279
-		if ( $items->have_posts() ) {
280
-			update_option( 'lsx_wetu_' . $task . '_sync', $items->posts );
278
+		$items = new \WP_Query($args);
279
+		if ($items->have_posts()) {
280
+			update_option('lsx_wetu_'.$task.'_sync', $items->posts);
281 281
 		}
282 282
 	}
283 283
 
@@ -290,11 +290,11 @@  discard block
 block discarded – undo
290 290
 		$api_key = false;
291 291
 		$options = lsx_wetu_get_options();
292 292
 
293
-		if ( ! defined( 'WETU_API_KEY' ) ) {
294
-			if ( isset( $options['api_key'] ) && '' !== $options['api_key'] ) {
293
+		if (!defined('WETU_API_KEY')) {
294
+			if (isset($options['api_key']) && '' !== $options['api_key']) {
295 295
 				$api_key = $options['api_key'];
296 296
 			}
297
-		} else {
297
+		}else {
298 298
 			$api_key = WETU_API_KEY;
299 299
 		}
300 300
 		return $api_key;
Please login to merge, or discard this patch.