Passed
Push — add/image-automation ( f9204d...c29b91 )
by Warwick
03:05
created
classes/class-cron.php 1 patch
Spacing   +55 added lines, -55 removed lines patch added patch discarded remove patch
@@ -33,10 +33,10 @@  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 );
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 40
 	}
41 41
 
42 42
 	/**
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
 	 */
49 49
 	public static function get_instance() {
50 50
 		// If the single instance hasn't been set, set it now.
51
-		if ( null === self::$instance ) {
51
+		if (null === self::$instance) {
52 52
 			self::$instance = new self();
53 53
 		}
54 54
 		return self::$instance;
@@ -60,10 +60,10 @@  discard block
 block discarded – undo
60 60
 	 * @param  array $schedules
61 61
 	 * @return array
62 62
 	 */
63
-	public function register_schedule( $schedules ) {
63
+	public function register_schedule($schedules) {
64 64
 		$schedules['wetu-5-minutes'] = array(
65 65
 			'interval' => 5 * MINUTE_IN_SECONDS,
66
-			'display'  => __( 'Every 5 minutes', 'lsx-wetu-importer' ),
66
+			'display'  => __('Every 5 minutes', 'lsx-wetu-importer'),
67 67
 		);
68 68
 		return $schedules;
69 69
 	}
@@ -75,26 +75,26 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	public function watch_for_trigger() {
77 77
 
78
-		if ( isset( $_GET['page'] ) && 'lsx-wetu-importer' === $_GET['page'] && isset( $_GET['tab'] ) && 'settings' === $_GET['tab'] ) {
78
+		if (isset($_GET['page']) && 'lsx-wetu-importer' === $_GET['page'] && isset($_GET['tab']) && 'settings' === $_GET['tab']) {
79 79
 			$options = lsx_wetu_get_options();
80 80
 
81 81
 			// Check what state the option is in.
82 82
 			$accommodation_cron = 'deactivate';
83
-			if ( isset( $options['accommodation_images_cron'] ) && '' !== $options['accommodation_images_cron'] ) {
83
+			if (isset($options['accommodation_images_cron']) && '' !== $options['accommodation_images_cron']) {
84 84
 				$accommodation_cron = 'activate';
85 85
 			}
86 86
 
87 87
 			// Check what state the cron is in.
88 88
 			$schedule = false;
89
-			if ( wp_next_scheduled( 'lsx_wetu_accommodation_images_cron' ) ) {
89
+			if (wp_next_scheduled('lsx_wetu_accommodation_images_cron')) {
90 90
 				$schedule = true;
91 91
 			}
92 92
 
93 93
 			// If activate and its not running.
94
-			if ( false === $schedule && 'activate' === $accommodation_cron ) {
94
+			if (false === $schedule && 'activate' === $accommodation_cron) {
95 95
 				$schedule = 'daily';
96
-				$this->schedule( 'lsx_wetu_accommodation_images_cron', $schedule );
97
-			} elseif ( true === $schedule && 'deactivate' === $accommodation_cron ) {
96
+				$this->schedule('lsx_wetu_accommodation_images_cron', $schedule);
97
+			} elseif (true === $schedule && 'deactivate' === $accommodation_cron) {
98 98
 				$this->deactivate();
99 99
 			}
100 100
 		}
@@ -105,8 +105,8 @@  discard block
 block discarded – undo
105 105
 	 *
106 106
 	 * @return void
107 107
 	 */
108
-	public function deactivate( $task = 'lsx_wetu_accommodation_images_cron' ) {
109
-		wp_clear_scheduled_hook( $task, array( $task ) );
108
+	public function deactivate($task = 'lsx_wetu_accommodation_images_cron') {
109
+		wp_clear_scheduled_hook($task, array($task));
110 110
 	}
111 111
 
112 112
 	/**
@@ -117,11 +117,11 @@  discard block
 block discarded – undo
117 117
 	 * @param string $time
118 118
 	 * @return void
119 119
 	 */
120
-	public function schedule( $task = 'lsx_wetu_accommodation_images_cron', $schedule = 'daily', $time = '' ) {
121
-		if ( '' === $time ) {
120
+	public function schedule($task = 'lsx_wetu_accommodation_images_cron', $schedule = 'daily', $time = '') {
121
+		if ('' === $time) {
122 122
 			$time = time();
123 123
 		}
124
-		wp_schedule_event( $time, $schedule, $task, array( $task ) );
124
+		wp_schedule_event($time, $schedule, $task, array($task));
125 125
 	}
126 126
 
127 127
 	/**
@@ -129,8 +129,8 @@  discard block
 block discarded – undo
129 129
 	 *
130 130
 	 * @return void
131 131
 	 */
132
-	public function process( $task = '' ) {
133
-		switch ( $task ) {
132
+	public function process($task = '') {
133
+		switch ($task) {
134 134
 			case 'lsx_wetu_accommodation_images_cron':
135 135
 					$this->register_accommodation_images_sync();
136 136
 				break;
@@ -146,10 +146,10 @@  discard block
 block discarded – undo
146 146
 	 * @return void
147 147
 	 */
148 148
 	public function register_accommodation_images_sync() {
149
-		$time = strtotime( '+1 min' );
150
-		if ( ! wp_next_scheduled( 'lsx_wetu_accommodation_images_sync' ) ) {
151
-			$this->load_items_to_sync( 'accommodation_images' );
152
-			$this->schedule( 'lsx_wetu_accommodation_images_sync', 'wetu-5-minutes', $time );
149
+		$time = strtotime('+1 min');
150
+		if (!wp_next_scheduled('lsx_wetu_accommodation_images_sync')) {
151
+			$this->load_items_to_sync('accommodation_images');
152
+			$this->schedule('lsx_wetu_accommodation_images_sync', 'wetu-5-minutes', $time);
153 153
 		}
154 154
 	}
155 155
 
@@ -158,27 +158,27 @@  discard block
 block discarded – undo
158 158
 	 *
159 159
 	 * @return void
160 160
 	 */
161
-	public function cron_callback( $task = '' ) {
162
-		$has_accommodation = get_option( $task );
163
-		if ( false !== $has_accommodation && ! empty( $has_accommodation ) ) {
164
-			$next_time = array_slice( $has_accommodation, 5 );
165
-			$this_time = array_slice( $has_accommodation, 0, 4 );
161
+	public function cron_callback($task = '') {
162
+		$has_accommodation = get_option($task);
163
+		if (false !== $has_accommodation && !empty($has_accommodation)) {
164
+			$next_time = array_slice($has_accommodation, 5);
165
+			$this_time = array_slice($has_accommodation, 0, 4);
166 166
 
167 167
 			$api_key = $this->get_api_key();
168
-			$url     = 'https://wetu.com/API/Pins/' . $api_key . '/Get?all=include&ids=';
168
+			$url     = 'https://wetu.com/API/Pins/'.$api_key.'/Get?all=include&ids=';
169 169
 
170 170
 			// Run through the current items.
171
-			foreach ( $this_time as $accommodation ) {
172
-				$wetu_id   = get_post_meta( $accommodation, 'lsx_wetu_id', true );
173
-				$last_date = get_post_meta( $accommodation, 'lsx_wetu_modified_date', true );
171
+			foreach ($this_time as $accommodation) {
172
+				$wetu_id   = get_post_meta($accommodation, 'lsx_wetu_id', true);
173
+				$last_date = get_post_meta($accommodation, 'lsx_wetu_modified_date', true);
174 174
 
175
-				$accommodation_info = wp_remote_get( $url . $wetu_id );
176
-				if ( ! empty( $accommodation_info ) && isset( $accommodation_info['response'] ) && isset( $accommodation_info['response']['code'] ) && 200 === $accommodation_info['response']['code'] ) {
177
-					$adata = json_decode( $accommodation_info['body'], true );
175
+				$accommodation_info = wp_remote_get($url.$wetu_id);
176
+				if (!empty($accommodation_info) && isset($accommodation_info['response']) && isset($accommodation_info['response']['code']) && 200 === $accommodation_info['response']['code']) {
177
+					$adata = json_decode($accommodation_info['body'], true);
178 178
 
179
-					if ( isset( $adata[0] ) && isset( $adata[0]['last_modified'] ) && '' !== $adata[0]['last_modified'] ) {
180
-						$modified_time = strtotime( $adata[0]['last_modified'] );
181
-						if ( $modified_time > $last_date ) {
179
+					if (isset($adata[0]) && isset($adata[0]['last_modified']) && '' !== $adata[0]['last_modified']) {
180
+						$modified_time = strtotime($adata[0]['last_modified']);
181
+						if ($modified_time > $last_date) {
182 182
 							/*print_r('<pre>');
183 183
 							print_r( $wetu_id );
184 184
 							print_r('</pre>');
@@ -195,15 +195,15 @@  discard block
 block discarded – undo
195 195
 			}
196 196
 
197 197
 			// Save the values for next time.
198
-			if ( ! empty( $next_time ) ) {
199
-				update_option( $task, $next_time );
200
-			} else {
201
-				delete_option( $task );
202
-				$this->deactivate( $task );
198
+			if (!empty($next_time)) {
199
+				update_option($task, $next_time);
200
+			}else {
201
+				delete_option($task);
202
+				$this->deactivate($task);
203 203
 			}
204
-		} else {
205
-			$this->deactivate( $task );
206
-			update_option( 'lsx_wetu_nexttime', $task );
204
+		}else {
205
+			$this->deactivate($task);
206
+			update_option('lsx_wetu_nexttime', $task);
207 207
 		}
208 208
 	}
209 209
 
@@ -213,14 +213,14 @@  discard block
 block discarded – undo
213 213
 	 * @param  string $task
214 214
 	 * @return void
215 215
 	 */
216
-	public function load_items_to_sync( $task = 'accommodation_images' ) {
216
+	public function load_items_to_sync($task = 'accommodation_images') {
217 217
 		$args = array(
218 218
 			'post_status'    => 'publish',
219 219
 			'posts_per_page' => -1,
220 220
 			'nopagin'        => true,
221 221
 			'fields'         => 'ids',
222 222
 		);
223
-		switch ( $task ) {
223
+		switch ($task) {
224 224
 			case 'accommodation_images':
225 225
 					$args['post_type'] = 'accommodation';
226 226
 				break;
@@ -228,9 +228,9 @@  discard block
 block discarded – undo
228 228
 			default:
229 229
 				break;
230 230
 		}
231
-		$items = new \WP_Query( $args );
232
-		if ( $items->have_posts() ) {
233
-			update_option( 'lsx_wetu_' . $task . '_sync', $items->posts );
231
+		$items = new \WP_Query($args);
232
+		if ($items->have_posts()) {
233
+			update_option('lsx_wetu_'.$task.'_sync', $items->posts);
234 234
 		}
235 235
 	}
236 236
 
@@ -243,11 +243,11 @@  discard block
 block discarded – undo
243 243
 		$api_key = false;
244 244
 		$options = lsx_wetu_get_options();
245 245
 
246
-		if ( ! defined( 'WETU_API_KEY' ) ) {
247
-			if ( isset( $options['api_key'] ) && '' !== $options['api_key'] ) {
246
+		if (!defined('WETU_API_KEY')) {
247
+			if (isset($options['api_key']) && '' !== $options['api_key']) {
248 248
 				$api_key = $options['api_key'];
249 249
 			}
250
-		} else {
250
+		}else {
251 251
 			$api_key = WETU_API_KEY;
252 252
 		}
253 253
 		return $api_key;
Please login to merge, or discard this patch.