Passed
Push — master ( 1867d0...47d946 )
by Warwick
04:23
created
classes/class-cron.php 1 patch
Spacing   +57 added lines, -57 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
 			$scheduled = false;
89
-			if ( wp_next_scheduled( 'lsx_wetu_accommodation_images_cron' ) ) {
89
+			if (wp_next_scheduled('lsx_wetu_accommodation_images_cron')) {
90 90
 				$scheduled = true;
91 91
 			}
92 92
 
93 93
 			// If activate and its not running.
94
-			if ( false === $scheduled && 'activate' === $accommodation_cron ) {
94
+			if (false === $scheduled && 'activate' === $accommodation_cron) {
95 95
 				$schedule = 'weekly';
96
-				$this->schedule( 'lsx_wetu_accommodation_images_cron', $schedule );
97
-			} elseif ( true === $scheduled && 'deactivate' === $accommodation_cron ) {
96
+				$this->schedule('lsx_wetu_accommodation_images_cron', $schedule);
97
+			} elseif (true === $scheduled && '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 = 'weekly', $time = 'Sunday 10pm' ) {
121
-		if ( '' === $time ) {
120
+	public function schedule($task = 'lsx_wetu_accommodation_images_cron', $schedule = 'weekly', $time = 'Sunday 10pm') {
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( '+5 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('+5 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,45 +158,45 @@  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, 10 );
165
-			$this_time = array_slice( $has_accommodation, 0, 9 );
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, 10);
165
+			$this_time = array_slice($has_accommodation, 0, 9);
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
 							$accommodation_importer = new \LSX_WETU_Importer_Accommodation();
183
-							$accommodation_importer->create_main_gallery( $adata, $accommodation );
184
-							update_post_meta( $accommodation, 'lsx_wetu_modified_date', $modified_time, $last_date );
183
+							$accommodation_importer->create_main_gallery($adata, $accommodation);
184
+							update_post_meta($accommodation, 'lsx_wetu_modified_date', $modified_time, $last_date);
185 185
 						}
186 186
 					}
187 187
 				}
188 188
 			}
189 189
 
190 190
 			// Save the values for next time.
191
-			if ( ! empty( $next_time ) ) {
192
-				update_option( $task, $next_time );
193
-			} else {
194
-				delete_option( $task );
195
-				$this->deactivate( $task );
191
+			if (!empty($next_time)) {
192
+				update_option($task, $next_time);
193
+			}else {
194
+				delete_option($task);
195
+				$this->deactivate($task);
196 196
 			}
197
-		} else {
198
-			$this->deactivate( $task );
199
-			update_option( 'lsx_wetu_nexttime', $task );
197
+		}else {
198
+			$this->deactivate($task);
199
+			update_option('lsx_wetu_nexttime', $task);
200 200
 		}
201 201
 	}
202 202
 
@@ -206,14 +206,14 @@  discard block
 block discarded – undo
206 206
 	 * @param  string $task
207 207
 	 * @return void
208 208
 	 */
209
-	public function load_items_to_sync( $task = 'accommodation_images' ) {
209
+	public function load_items_to_sync($task = 'accommodation_images') {
210 210
 		$args = array(
211 211
 			'post_status'    => 'publish',
212 212
 			'posts_per_page' => -1,
213 213
 			'nopagin'        => true,
214 214
 			'fields'         => 'ids',
215 215
 		);
216
-		switch ( $task ) {
216
+		switch ($task) {
217 217
 			case 'accommodation_images':
218 218
 					$args['post_type'] = 'accommodation';
219 219
 				break;
@@ -221,9 +221,9 @@  discard block
 block discarded – undo
221 221
 			default:
222 222
 				break;
223 223
 		}
224
-		$items = new \WP_Query( $args );
225
-		if ( $items->have_posts() ) {
226
-			update_option( 'lsx_wetu_' . $task . '_sync', $items->posts );
224
+		$items = new \WP_Query($args);
225
+		if ($items->have_posts()) {
226
+			update_option('lsx_wetu_'.$task.'_sync', $items->posts);
227 227
 		}
228 228
 	}
229 229
 
@@ -236,11 +236,11 @@  discard block
 block discarded – undo
236 236
 		$api_key = false;
237 237
 		$options = lsx_wetu_get_options();
238 238
 
239
-		if ( ! defined( 'WETU_API_KEY' ) ) {
240
-			if ( isset( $options['api_key'] ) && '' !== $options['api_key'] ) {
239
+		if (!defined('WETU_API_KEY')) {
240
+			if (isset($options['api_key']) && '' !== $options['api_key']) {
241 241
 				$api_key = $options['api_key'];
242 242
 			}
243
-		} else {
243
+		}else {
244 244
 			$api_key = WETU_API_KEY;
245 245
 		}
246 246
 		return $api_key;
Please login to merge, or discard this patch.