lightspeeddevelopment / wetu-importer

classes/class-lsx-wetu-importer-banner-integration.php

Doc Comments +3 added lines

@@ -229,6 +229,9 @@
 		}
 	}
 
+	/**
+	 * @param string $url
+	 */
 	public function attach_external_image2( $url = null, $post_data = array(), $post_id = '' ) {
 		if ( ! $url ) { return new WP_Error( 'missing', 'Need a valid URL' ); }
 

Spacing +69 added lines, -69 removed lines

@@ -26,8 +26,8 @@ 
 	 * @access private
 	 */
 	public function __construct() {
-		add_action( 'wp_ajax_lsx_import_sync_banners',array( $this, 'sync_new_banner' ) );
-		add_action( 'wp_ajax_nopriv_lsx_import_sync_banners',array( $this, 'sync_new_banner' ) );
+		add_action('wp_ajax_lsx_import_sync_banners', array($this, 'sync_new_banner'));
+		add_action('wp_ajax_nopriv_lsx_import_sync_banners', array($this, 'sync_new_banner'));
 	}
 
 	/**
@@ -36,13 +36,13 @@ 
 	public function display_page() {
 		?>
 		<div class="wrap">
-			<h2><?php esc_html_e( 'Download new banners straight from WETU','lsx-wetu-importer' ); ?></h2>
+			<h2><?php esc_html_e('Download new banners straight from WETU', 'lsx-wetu-importer'); ?></h2>
 
 			<form method="get" action="" id="banners-filter">
-				<input type="hidden" name="post_type" class="post_type" value="<?php echo esc_attr( $this->tab_slug ); ?>" />
+				<input type="hidden" name="post_type" class="post_type" value="<?php echo esc_attr($this->tab_slug); ?>" />
 
 			   <div class="ajax-loader-small" style="display:none;width:100%;text-align:center;">
-					<img style="width:32px;" src="<?php echo esc_url( LSX_WETU_IMPORTER_URL . 'assets/images/ajaxloader.gif' ); ?>" />
+					<img style="width:32px;" src="<?php echo esc_url(LSX_WETU_IMPORTER_URL.'assets/images/ajaxloader.gif'); ?>" />
 				</div>
 
 				<table class="wp-list-table widefat fixed posts">
@@ -60,7 +60,7 @@ 
 					<?php
 						$accommodation_args = array(
 							'post_type' => 'accommodation',
-							'post_status' => array( 'publish','pending','draft','future','private' ),
+							'post_status' => array('publish', 'pending', 'draft', 'future', 'private'),
 							'nopagin' => 'true',
 							'posts_per_page' => '1000',
 							'meta_query' => array(
@@ -80,13 +80,13 @@ 
 								),
 							),
 						);
-						$accommodation = new WP_Query( $accommodation_args );
+						$accommodation = new WP_Query($accommodation_args);
 					?>
 
 					<tbody id="the-list">
 						<?php
-							if ( $accommodation->have_posts() ) {
-								while ( $accommodation->have_posts() ) {
+							if ($accommodation->have_posts()) {
+								while ($accommodation->have_posts()) {
 									$accommodation->the_post();
 									?>
 									<tr class="post-<?php the_ID(); ?> type-tour status-none" id="post-<?php the_ID(); ?>">
@@ -95,26 +95,26 @@ 
 									$min_width = '1920';
 									$min_height = '500';
 
-									$img_group = get_post_meta( get_the_ID(),'image_group',true );
+									$img_group = get_post_meta(get_the_ID(), 'image_group', true);
 
 									$thumbnails_html = false;
 
-									if ( false !== $img_group ) {
-										foreach ( $img_group['banner_image'] as $banner_image ) {
-											$large = wp_get_attachment_image_src( $banner_image,'full' );
+									if (false !== $img_group) {
+										foreach ($img_group['banner_image'] as $banner_image) {
+											$large = wp_get_attachment_image_src($banner_image, 'full');
 											$real_width = $large[1];
 											$real_height = $large[2];
 
 											$status = 'optimized';
-											if ( $real_width < intval( $real_width ) ) {
+											if ($real_width < intval($real_width)) {
 												$status = 'width not enough.';
 												}
 
-											$thumbnail = wp_get_attachment_image_src( $banner_image,'thumbnail' );
+											$thumbnail = wp_get_attachment_image_src($banner_image, 'thumbnail');
 											$thumbnails_html[] = '
 													<div style="display:block;float:left;">
-														<img src="' . $thumbnail[0] . '" />
-														<p style="text-align:center;">' . $real_width . 'px by ' . $real_height . 'px</p>
+														<img src="' . $thumbnail[0].'" />
+														<p style="text-align:center;">' . $real_width.'px by '.$real_height.'px</p>
 													</div>';
 											}
 										}
@@ -124,15 +124,15 @@ 
 										<input type="checkbox" data-identifier="<?php the_ID(); ?>" value="<?php the_ID(); ?>" name="post[]" id="cb-select-<?php the_ID(); ?>">
 										</th>
 
-										<td class="post-title page-title column-title"><?php echo '<a href="' . esc_url( admin_url( '/post.php?post=' . get_the_ID() . '&action=edit' ) ) . '" target="_blank">';
+										<td class="post-title page-title column-title"><?php echo '<a href="'.esc_url(admin_url('/post.php?post='.get_the_ID().'&action=edit')).'" target="_blank">';
 										the_title();
 										echo '</a>'; ?></td>
 
 										<td colspan="2" class="thumbnails column-thumbnails">
 										<?php
-											if ( false !== $thumbnails_html ) {
-												echo wp_kses_post( implode( '', $thumbnails_html ) );
-											} else {
+											if (false !== $thumbnails_html) {
+												echo wp_kses_post(implode('', $thumbnails_html));
+											}else {
 												echo '<p>There was an error retrieving your images.</p>';
 											}
 										?>
@@ -156,7 +156,7 @@ 
 
 				</table>
 
-				<p><input class="button button-primary download" type="button" value="<?php esc_html_e( 'Download new Banners','lsx-wetu-importer' ); ?>" />
+				<p><input class="button button-primary download" type="button" value="<?php esc_html_e('Download new Banners', 'lsx-wetu-importer'); ?>" />
 				</p>
 			</form>
 		</div>
@@ -167,32 +167,32 @@ 
 	 * Creates the main gallery data
 	 */
 	public function sync_new_banner() {
-		check_ajax_referer( 'lsx_wetu_ajax_action', 'security' );
-		if ( isset( $_POST['action'] ) && 'lsx_import_sync_banners' === $_POST['action'] && isset( $_POST['post_id'] ) ) {
+		check_ajax_referer('lsx_wetu_ajax_action', 'security');
+		if (isset($_POST['action']) && 'lsx_import_sync_banners' === $_POST['action'] && isset($_POST['post_id'])) {
 
-			$post_id = wp_unslash( $_POST['post_id'] );
-			$banners       = get_post_meta( $post_id, 'image_group', true );
-			$this->wetu_id = get_post_meta( $post_id, 'lsx_wetu_id', true );
+			$post_id = wp_unslash($_POST['post_id']);
+			$banners       = get_post_meta($post_id, 'image_group', true);
+			$this->wetu_id = get_post_meta($post_id, 'lsx_wetu_id', true);
 
 			$new_banner_array = false;
 			$array_index = 0;
 
-			foreach ( $banners['banner_image'] as $banner_image ) {
-				$image_id = $this->attach_external_image2( $this->format_wetu_url( $banner_image ), array(), $post_id );
-				if ( null !== $image_id && '' !== $image_id ) {
-					$new_banner_array['banner_image'][ 'cmb-field-' . $array_index ] = $image_id;
+			foreach ($banners['banner_image'] as $banner_image) {
+				$image_id = $this->attach_external_image2($this->format_wetu_url($banner_image), array(), $post_id);
+				if (null !== $image_id && '' !== $image_id) {
+					$new_banner_array['banner_image']['cmb-field-'.$array_index] = $image_id;
 					$array_index++;
 				}
 			}
 
-			if ( false !== $new_banner_array ) {
-				delete_post_meta( $post_id, 'image_group' );
-				add_post_meta( $post_id, 'image_group', $new_banner_array, true );
+			if (false !== $new_banner_array) {
+				delete_post_meta($post_id, 'image_group');
+				add_post_meta($post_id, 'image_group', $new_banner_array, true);
 				echo true;
-			} else {
+			}else {
 				echo false;
 			}
-		} else {
+		}else {
 			echo false;
 		}
 
@@ -202,77 +202,77 @@ 
 	/**
 	 * formats the url
 	 */
-	public function format_wetu_url( $post_id ) {
-		return 'https://wetu.com/ImageHandler/c1920x800/' . $this->wetu_id . '/' . $this->format_filename( $post_id );
+	public function format_wetu_url($post_id) {
+		return 'https://wetu.com/ImageHandler/c1920x800/'.$this->wetu_id.'/'.$this->format_filename($post_id);
 	}
 
 	/**
 	 * formats the filename
 	 */
-	public function format_filename( $post_id ) {
-		$base = str_replace( '_',' ',get_the_title( $post_id ) );
-		$base = rawurlencode( $base );
-		$type = get_post_mime_type( $post_id );
+	public function format_filename($post_id) {
+		$base = str_replace('_', ' ', get_the_title($post_id));
+		$base = rawurlencode($base);
+		$type = get_post_mime_type($post_id);
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'image/jpeg':
-				return $base . '.jpg';
+				return $base.'.jpg';
 			break;
 			case 'image/png':
-				return $base . '.png';
+				return $base.'.png';
 			break;
 			case 'image/gif':
-				return $base . '.gif';
+				return $base.'.gif';
 			break;
 			default:
 				return false;
 		}
 	}
 
-	public function attach_external_image2( $url = null, $post_data = array(), $post_id = '' ) {
-		if ( ! $url ) { return new WP_Error( 'missing', 'Need a valid URL' ); }
+	public function attach_external_image2($url = null, $post_data = array(), $post_id = '') {
+		if (!$url) { return new WP_Error('missing', 'Need a valid URL'); }
 
-		require_once( ABSPATH . 'wp-admin/includes/file.php' );
-		require_once( ABSPATH . 'wp-admin/includes/media.php' );
-		require_once( ABSPATH . 'wp-admin/includes/image.php' );
+		require_once(ABSPATH.'wp-admin/includes/file.php');
+		require_once(ABSPATH.'wp-admin/includes/media.php');
+		require_once(ABSPATH.'wp-admin/includes/image.php');
 
-		$tmp = tempnam( '/tmp', 'FOO' );
-		$image = file_get_contents( $url );
-		file_put_contents( $tmp, $image );
-		chmod( $tmp, '777' );
+		$tmp = tempnam('/tmp', 'FOO');
+		$image = file_get_contents($url);
+		file_put_contents($tmp, $image);
+		chmod($tmp, '777');
 
-		preg_match( '/[^\?]+\.(tif|TIFF|jpg|JPG|jpe|JPE|jpeg|JPEG|gif|GIF|png|PNG|pdf|PDF|bmp|BMP)/', $url, $matches );
-		$url_filename = basename( $matches[0] );
-		$url_filename = str_replace( '%20','_', $url_filename );
+		preg_match('/[^\?]+\.(tif|TIFF|jpg|JPG|jpe|JPE|jpeg|JPEG|gif|GIF|png|PNG|pdf|PDF|bmp|BMP)/', $url, $matches);
+		$url_filename = basename($matches[0]);
+		$url_filename = str_replace('%20', '_', $url_filename);
 		// extract filename from url for title.
-		$url_type = wp_check_filetype( $url_filename );
+		$url_type = wp_check_filetype($url_filename);
 
 		// assemble file data (should be built like $_FILES since wp_handle_sideload() will be using).
 		$file_array['tmp_name'] = $tmp;
 
-		if ( ! empty( $filename ) && ' ' != $filename ) {
-			$file_array['name'] = $filename . '.' . $url_type['ext'];
-		} else {
+		if (!empty($filename) && ' ' != $filename) {
+			$file_array['name'] = $filename.'.'.$url_type['ext'];
+		}else {
 			$file_array['name'] = $url_filename;
 		}
 
 		// set additional wp_posts columns.
-		if ( empty( $post_data['post_title'] ) ) {
-			$url_filename = str_replace( '%20', ' ', $url_filename );
-			$post_data['post_title'] = basename( $url_filename, '.' . $url_type['ext'] );
+		if (empty($post_data['post_title'])) {
+			$url_filename = str_replace('%20', ' ', $url_filename);
+			$post_data['post_title'] = basename($url_filename, '.'.$url_type['ext']);
 		}
 
 		// make sure gets tied to parent.
-		if ( empty( $post_data['post_parent'] ) ) {
+		if (empty($post_data['post_parent'])) {
 			$post_data['post_parent'] = $post_id;
 		}
 
 		// do the validation and storage stuff.
-		$att_id = media_handle_sideload( $file_array, $post_id, null, $post_data );
+		$att_id = media_handle_sideload($file_array, $post_id, null, $post_data);
 
 		// If error storing permanently, unlink.
-		if ( is_wp_error( $att_id ) ) {
-			unlink( $file_array['tmp_name'] );
+		if (is_wp_error($att_id)) {
+			unlink($file_array['tmp_name']);
 			return false;
 		}