From 0961ef4fe87fad156ce5b022116fbb001243094d Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 13 Aug 2015 17:17:23 +0100
Subject: [PATCH 01/25] vip

---
 helpers.php                         | 68 ++++++++++++++++++++++++++++
 inc/class-resource-space-loader.php | 70 +----------------------------
 resourcespace.php                   |  1 +
 3 files changed, 71 insertions(+), 68 deletions(-)
 create mode 100644 helpers.php

diff --git a/helpers.php b/helpers.php
new file mode 100644
index 0000000..b53d425
--- /dev/null
+++ b/helpers.php
@@ -0,0 +1,68 @@
+<?php
+
+if ( ! function_exists( 'wpcom_vip_download_image' ) ) {
+
+	/**
+	 * Downloads an external image and optionally attaches it to a post.
+	 *
+	 * Contains most of core's media_sideload_image() but returns an attachment ID instead of HTML.
+	 *
+	 * Note: this function does not validate the domain that the image is coming from. Please make sure
+	 *   to validate this before downloading the image. Should only pull down images from trusted sources.
+	 *
+	 * @param string $image_url URL of the image.
+	 * @param int $post_ID ID of the post it should be attached to.
+	 * @return $thumbnail_id id of the thumbnail attachment post id
+	 */
+	function wpcom_vip_download_image( $image_url, $post_id = 0, $description = '' ) {
+		if ( isset( $_SERVER['REQUEST_METHOD'] ) && strtoupper( $_SERVER['REQUEST_METHOD'] ) == 'GET' ) {
+			return new WP_Error( 'invalid-request-method', 'Media sideloading is not supported via GET. Use POST.' );
+		}
+
+		if ( ! is_admin() ) {
+			return new WP_Error( 'not-in-admin', 'Media sideloading can only be done in when `true === is_admin()`.' );
+		}
+
+		if ( $post_id < 0 ) {
+			return new WP_Error( 'invalid-post-id', 'Please specify a valid post ID.' );
+		}
+
+		if ( ! filter_var( $image_url, FILTER_VALIDATE_URL ) ) {
+			return new WP_Error( 'not-a-url', 'Please specify a valid URL.' );
+		}
+
+		$image_url_path = parse_url( $image_url, PHP_URL_PATH );
+		$image_path_info = pathinfo( $image_url_path );
+
+		if ( ! in_array( strtolower( $image_path_info['extension'] ), array( 'jpg', 'jpe', 'jpeg', 'gif', 'png' ) ) ) {
+			return new WP_Error( 'not-an-image', 'Specified URL does not have a valid image extension.' );
+		}
+
+		// Download file to temp location; short timeout, because we don't have all day.
+		$downloaded_url = download_url( $image_url, 30 );
+
+		// We couldn't download and store to a temporary location, so bail.
+		if ( is_wp_error( $downloaded_url ) ) {
+			return $downloaded_url;
+		}
+
+		$file_array['name'] = $image_path_info['basename'];
+		$file_array['tmp_name'] = $downloaded_url;
+
+		if ( empty( $description ) ) {
+			$description = $image_path_info['filename'];
+		}
+
+		// Now, let's sideload it.
+		$attachment_id = media_handle_sideload( $file_array, $post_id, $description );
+
+		// If error storing permanently, unlink and return the error
+		if ( is_wp_error( $attachment_id ) ) {
+			@unlink( $file_array['tmp_name'] ); // unlink can throw errors if the file isn't there
+			return $attachment_id;
+		}
+
+		return $attachment_id;
+	}
+
+}
diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index f53203b..cff4f35 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -66,10 +66,10 @@ function ajax_get_image() {
 		}
 
 		// Request original URL.
-		// $attachment_id = $this->sideload_image( $data[0]->original );
+		// $attachment_id = wpcom_vip_download_image( $data[0]->original );
 
 		// Request preview size.
-		$attachment_id = $this->sideload_image( $data[0]->preview );
+		$attachment_id = wpcom_vip_download_image( $data[0]->preview );
 
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );
@@ -81,70 +81,4 @@ function ajax_get_image() {
 
 	}
 
-	private function sideload_image( $url ) {
-
-		$request_args = array( 'headers' => array() );
-
-		// Pass basic auth header if available.
-		if ( defined( 'PJ_RESOURCE_SPACE_AUTHL' ) &&  defined( 'PJ_RESOURCE_SPACE_AUTHP' ) ) {
-			$request_args['headers']['Authorization'] = 'Basic ' . base64_encode( PJ_RESOURCE_SPACE_AUTHL . ':' . PJ_RESOURCE_SPACE_AUTHP );
-		}
-
-		// TODO test. Advice from Kirill was to use the users cookie.
-		// Hopefully it isn't required as this isn't as robust as using basic auth.
-		$response = wp_remote_get( $url, $request_args );
-
-		if ( 200 == wp_remote_retrieve_response_code( $response ) ) {
-
-			$file = get_temp_dir() . sanitize_file_name( $data[0]->Original_filename );
-			file_put_contents( $file, wp_remote_retrieve_body( $response ) );
-
-			$filename = basename( $file );
-
-			$upload_file = wp_upload_bits( $filename, null, file_get_contents( $file ) );
-
-			if ( ! $upload_file['error'] ) {
-
-				$wp_filetype = wp_check_filetype( $filename, null );
-
-				$attachment = array(
-					'post_mime_type' => $wp_filetype['type'],
-					'post_parent'    => 0,
-					'post_title'     => $data[0]->{'Légende'},
-					'post_content'   => 'Downloaded ' . current_time( 'd/m/Y \a\t H:i:s' ),
-					'post_status'    => 'inherit',
-				);
-
-				$attachment_id = wp_insert_attachment( $attachment, $upload_file['file'], $parent_post_id );
-
-				if ( ! is_wp_error( $attachment_id ) ) {
-
-					require_once( trailingslashit( ABSPATH ) . 'wp-admin/includes/image.php' );
-
-					$attachment_data = wp_generate_attachment_metadata( $attachment_id, $upload_file['file'] );
-					$attachment_data['image_meta']['created_timestamp'] = current_time( 'Y-m-d H:i:s', true );
-
-					wp_update_attachment_metadata( $attachment_id, $attachment_data );
-
-					add_post_meta( $attachment_id, 'resource_space', true, true );
-
-					return $attachment_id;
-
-				} else {
-					unlink( $file );
-					return new WP_Error( 'broke', __( 'Could not create attachment', 'resourcespace' ) );
-				}
-			} else {
-				unlink( $file );
-				return new WP_Error( 'broke', __( 'Upload error', 'resourcespace' ) );
-			}
-
-			unlink( $file );
-
-		} else {
-			return new WP_Error( 'broke', __( 'Unable to retrieve image', 'resourcespace' ) );
-		}
-
-	}
-
 }
diff --git a/resourcespace.php b/resourcespace.php
index 02e563d..2ccf0cb 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -19,6 +19,7 @@
 	wp_die( __( 'Media Explorer plugin must be enabled.', 'resourcespace' ) );
 }
 
+require_once( __DIR__ . '/helpers.php' );
 require_once( __DIR__ . '/inc/class-resource-space-loader.php' );
 require_once( __DIR__ . '/inc/class-resource-space-admin.php' );
 require_once( __DIR__ . '/inc/class-mexp-resource-space-service.php' );

From 946e1ad3e2dbf65381cb123227e1f4a47a274aea Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 13 Aug 2015 17:20:57 +0100
Subject: [PATCH 02/25] pass parent id

---
 inc/class-resource-space-loader.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index cff4f35..bbfd0a1 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -69,7 +69,7 @@ function ajax_get_image() {
 		// $attachment_id = wpcom_vip_download_image( $data[0]->original );
 
 		// Request preview size.
-		$attachment_id = wpcom_vip_download_image( $data[0]->preview );
+		$attachment_id = wpcom_vip_download_image( $data[0]->preview, $parent_post_id );
 
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );

From 9a4328b73ed68568f0a856db0b8226c88c28617a Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 13 Aug 2015 17:55:41 +0100
Subject: [PATCH 03/25] bugfixes

---
 js/mexp.js        |  2 +-
 resourcespace.php | 24 ++++++++++++++----------
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/js/mexp.js b/js/mexp.js
index 4aff878..58c7a60 100644
--- a/js/mexp.js
+++ b/js/mexp.js
@@ -150,7 +150,7 @@
 
 	var view = media.view.MEXP
 
-	media.view.MEXP = view.extend( {
+	view = view.extend( {
 
 		noMorePosts: false,
 
diff --git a/resourcespace.php b/resourcespace.php
index 2ccf0cb..fefd7bf 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -15,18 +15,22 @@
 
 defined( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE' ) or define( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE', 10 );
 
-if ( ! class_exists( 'MEXP_Service' ) ) {
-	wp_die( __( 'Media Explorer plugin must be enabled.', 'resourcespace' ) );
-}
+add_action( 'plugins_loaded', function() {
+
+	if ( ! class_exists( 'MEXP_Service' ) ) {
+		wp_die( __( 'Media Explorer plugin must be enabled.', 'resourcespace' ) );
+	}
+
+	require_once( __DIR__ . '/helpers.php' );
+	require_once( __DIR__ . '/inc/class-resource-space-loader.php' );
+	require_once( __DIR__ . '/inc/class-resource-space-admin.php' );
+	require_once( __DIR__ . '/inc/class-mexp-resource-space-service.php' );
+	require_once( __DIR__ . '/inc/class-mexp-resource-space-template.php' );
 
-require_once( __DIR__ . '/helpers.php' );
-require_once( __DIR__ . '/inc/class-resource-space-loader.php' );
-require_once( __DIR__ . '/inc/class-resource-space-admin.php' );
-require_once( __DIR__ . '/inc/class-mexp-resource-space-service.php' );
-require_once( __DIR__ . '/inc/class-mexp-resource-space-template.php' );
+	Resource_Space_Loader::get_instance();
+	Resource_Space_Admin::get_instance();
 
-Resource_Space_Loader::get_instance();
-Resource_Space_Admin::get_instance();
+});
 
 add_filter( 'mexp_services', function( array $services ) {
 

From 45ea0a537e7120ee1b4cfd74d884dfeaee2f6541 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 13 Aug 2015 18:04:19 +0100
Subject: [PATCH 04/25] More bugfixes

---
 inc/class-resource-space-loader.php | 8 ++------
 js/mexp.js                          | 2 ++
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index bbfd0a1..f682911 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -42,8 +42,7 @@ function ajax_get_image() {
 			'key'              => $key,
 			'search'           => $resource_id,
 			'prettyfieldnames' => 1,
-			'previewsize'      => 'pre',
-			'original'         => true,
+			'previewsize'      => 'scr',
 		), $url );
 
 		$request_args = array( 'headers' => array() );
@@ -66,10 +65,7 @@ function ajax_get_image() {
 		}
 
 		// Request original URL.
-		// $attachment_id = wpcom_vip_download_image( $data[0]->original );
-
-		// Request preview size.
-		$attachment_id = wpcom_vip_download_image( $data[0]->preview, $parent_post_id );
+		$attachment_id = wpcom_vip_download_image( $data[0]->preview );
 
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );
diff --git a/js/mexp.js b/js/mexp.js
index 58c7a60..d134293 100644
--- a/js/mexp.js
+++ b/js/mexp.js
@@ -114,6 +114,8 @@
 							alert( 'Failed to import image.' );
 						}
 
+						toggleLoading( false );
+
 						return;
 					}
 

From e11750d09611a406869c2da333410a5a2acfb85d Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 13 Aug 2015 18:11:30 +0100
Subject: [PATCH 05/25] instructions

---
 resourcespace.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/resourcespace.php b/resourcespace.php
index fefd7bf..3f3b1b3 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -21,6 +21,14 @@
 		wp_die( __( 'Media Explorer plugin must be enabled.', 'resourcespace' ) );
 	}
 
+	if ( ! defined( 'PJ_RESOURCE_SPACE_DOMAIN' ) ) {
+		wp_die( __( 'You must define PJ_RESOURCE_SPACE_DOMAIN', 'resourcespace' ) );
+	}
+
+	if ( ! defined( 'PJ_RESOURCE_SPACE_KEY' ) ) {
+		wp_die( __( 'You must define PJ_RESOURCE_SPACE_KEY', 'resourcespace' ) );
+	}
+
 	require_once( __DIR__ . '/helpers.php' );
 	require_once( __DIR__ . '/inc/class-resource-space-loader.php' );
 	require_once( __DIR__ . '/inc/class-resource-space-admin.php' );

From d1c6e14ce76ad0b2d5670fb76cb0cdee3b0eb3a3 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Fri, 14 Aug 2015 12:03:55 +0100
Subject: [PATCH 06/25] disable admin

---
 resourcespace.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resourcespace.php b/resourcespace.php
index 3f3b1b3..fc6566b 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -36,7 +36,7 @@
 	require_once( __DIR__ . '/inc/class-mexp-resource-space-template.php' );
 
 	Resource_Space_Loader::get_instance();
-	Resource_Space_Admin::get_instance();
+	// Resource_Space_Admin::get_instance();
 
 });
 

From b05d994447727948a54f99e4e60126d0410f7df7 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Fri, 14 Aug 2015 12:04:16 +0100
Subject: [PATCH 07/25] use POST not REQUEST

---
 inc/class-resource-space-admin.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/inc/class-resource-space-admin.php b/inc/class-resource-space-admin.php
index 54f6336..26b0f10 100644
--- a/inc/class-resource-space-admin.php
+++ b/inc/class-resource-space-admin.php
@@ -71,7 +71,7 @@ function admin_enqueue_scripts() {
 
 	function ajax_query_attachments_args( $query ) {
 
-		if ( isset( $_REQUEST['query']['meta_query'] ) && 'resource_space' === $_REQUEST['query']['meta_query'] ) {
+		if ( isset( $_POST['query']['meta_query'] ) && 'resource_space' === $_POST['query']['meta_query'] ) {
 			$query['meta_query'] = array(
 				array(
 					'key'     => 'resource_space',

From a4cb55e9b1c17c4fb0c2af4efe1b5c147e91e679 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Fri, 14 Aug 2015 12:04:25 +0100
Subject: [PATCH 08/25] store meta data

---
 inc/class-resource-space-loader.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index f682911..407e928 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -67,6 +67,8 @@ function ajax_get_image() {
 		// Request original URL.
 		$attachment_id = wpcom_vip_download_image( $data[0]->preview );
 
+		update_post_meta( $attachment_id, 'resource_space', 1 );
+
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );
 		} else {

From 1d1d6c7d05894da43822a7b9703226313cbaf295 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Fri, 14 Aug 2015 12:04:39 +0100
Subject: [PATCH 09/25] fix JS error

---
 js/admin.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/js/admin.js b/js/admin.js
index 66bf0b9..322db90 100644
--- a/js/admin.js
+++ b/js/admin.js
@@ -1,6 +1,6 @@
 (function ( $ ) {
 
-	var $oldContainer = ('#resource-space-images');
+	var $oldContainer = $('#resource-space-images');
 	var $newContainer = $('#resource-space-new-images');
 
 	jQuery('#resource-space-add-new').on( 'click', function( event ) {

From 136d66f102796aef0c5dedaec65bd9360d6d0dd9 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 26 Aug 2015 10:49:06 +0100
Subject: [PATCH 10/25] Get the admin frame working

---
 inc/class-resource-space-admin.php |  5 +++++
 js/admin.js                        | 36 ++++++------------------------
 resourcespace.php                  |  2 +-
 templates/admin-page.php           |  4 ----
 4 files changed, 13 insertions(+), 34 deletions(-)

diff --git a/inc/class-resource-space-admin.php b/inc/class-resource-space-admin.php
index 26b0f10..e3dbade 100644
--- a/inc/class-resource-space-admin.php
+++ b/inc/class-resource-space-admin.php
@@ -65,6 +65,11 @@ function admin_enqueue_scripts() {
 			$deps = array( 'jquery', 'backbone', 'media' );
 			wp_enqueue_script( 'resource-space-admin', PJ_RESOURCE_SPACE_PLUGIN_URL . '/js/admin.js', $deps, PJ_RESOURCESPACE_PLUGIN_VERSION, true );
 
+			wp_localize_script( 'media-grid', '_wpMediaGridSettings', array(
+				'adminUrl' => parse_url( self_admin_url(), PHP_URL_PATH ),
+				'queryVars' => (object) array( 'meta_query' => 'resource_space' ),
+			) );
+
 		}
 
 	}
diff --git a/js/admin.js b/js/admin.js
index 322db90..5d35590 100644
--- a/js/admin.js
+++ b/js/admin.js
@@ -3,6 +3,12 @@
 	var $oldContainer = $('#resource-space-images');
 	var $newContainer = $('#resource-space-new-images');
 
+	var library = window.wp.media({
+		frame: 'manage',
+		container: $oldContainer,
+		library: _wpMediaGridSettings.queryVars,
+	}).open();
+
 	jQuery('#resource-space-add-new').on( 'click', function( event ) {
 
 		event.preventDefault();
@@ -13,30 +19,8 @@
 				$newContainer.show();
 			}
 
-			// Loop through attachments and build image element HTML.
 			_.each( this.attachments, function( attachment ) {
-
-				var $img = $('<img />');
-				var size = 'medium';
-
-				if ( attachment.sizes[ size ] ) {
-					$img.attr( 'src', attachment.sizes[ size ].url );
-					$img.attr( 'width', attachment.sizes[ size ].width );
-					$img.attr( 'height', attachment.sizes[ size ].height );
-				} else {
-					$img.attr( 'src', attachment.url );
-					$img.attr( 'width', attachment.width );
-					$img.attr( 'height', attachment.height );
-				}
-
-				$img.attr( 'alt', attachment.title );
-
-				$img.addClass( 'alignnone' );
-				$img.addClass( 'size-' + size );
-				$img.addClass( 'wp-image-' + attachment.id );
-
-				$newContainer.append( $img );
-
+				library.state().attributes.library.add( attachment );
 			});
 
 			this.complete();
@@ -54,10 +38,4 @@
 
 	});
 
-	window.wp.media({
-		frame: 'manage',
-		container: $oldContainer,
-		library: { 'meta_query': 'resource_space' },
-	}).open();
-
 }( jQuery ));
diff --git a/resourcespace.php b/resourcespace.php
index fc6566b..3f3b1b3 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -36,7 +36,7 @@
 	require_once( __DIR__ . '/inc/class-mexp-resource-space-template.php' );
 
 	Resource_Space_Loader::get_instance();
-	// Resource_Space_Admin::get_instance();
+	Resource_Space_Admin::get_instance();
 
 });
 
diff --git a/templates/admin-page.php b/templates/admin-page.php
index 724152c..97183dd 100644
--- a/templates/admin-page.php
+++ b/templates/admin-page.php
@@ -9,10 +9,6 @@
 		<a href="#" id="resource-space-add-new" class="add-new-h2"><?php esc_html_e( 'Add new', 'resourcespace' ); ?></a>
 	</h2>
 
-	<div id="resource-space-new-images" style="display: none;">
-		<h3><?php esc_html_e( 'Recently Imported Stock Images', 'resourcespace' ); ?></h3>
-	</div>
-
 	<div class="wrap" id="resource-space-images" data-search="<?php _admin_search_query() ?>">
 
 	<style>

From 91e4a3a2cd233d0b07dc3d8af88e849684dd5d63 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 26 Aug 2015 11:12:20 +0100
Subject: [PATCH 11/25] sync

---
 inc/class-mexp-resource-space-service.php | 2 +-
 inc/class-resource-space-loader.php       | 1 +
 resourcespace.php                         | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/inc/class-mexp-resource-space-service.php b/inc/class-mexp-resource-space-service.php
index 353a338..e8e8fe5 100644
--- a/inc/class-mexp-resource-space-service.php
+++ b/inc/class-mexp-resource-space-service.php
@@ -88,7 +88,7 @@ public function request( array $request ) {
 				'date'        => strtotime( $resource->creation_date ),
 				'id'          => $resource->ref,
 				'thumbnail'   => $resource->preview,
-				'url'         => $resource->original_link,
+				'url'         => null,
 			);
 
 			$dirty_data = apply_filters( 'resourcespace_parse_raw_image_data', $dirty_data, $resource );
diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index 407e928..4a34aab 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -42,6 +42,7 @@ function ajax_get_image() {
 			'key'              => $key,
 			'search'           => $resource_id,
 			'prettyfieldnames' => 1,
+			'original'         => true,
 			'previewsize'      => 'scr',
 		), $url );
 
diff --git a/resourcespace.php b/resourcespace.php
index 3f3b1b3..4c4eb5a 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -82,7 +82,7 @@ function resource_space_vc_script() {
  */
 add_action( 'admin_init', function() {
 
-	if ( version_compare( PJ_RESOURCESPACE_PLUGIN_VERSION, get_option( 'pj_resourcespace_version'), '<=' ) ) {
+	if ( version_compare( PJ_RESOURCESPACE_PLUGIN_VERSION, get_option( 'pj_resourcespace_version' ), '<=' ) ) {
 		return;
 	}
 

From 687b2cf660f547cc56261b54ca9ae2f7a1c5c44f Mon Sep 17 00:00:00 2001
From: Theo Savage <tcrsavage@gmail.com>
Date: Fri, 4 Sep 2015 13:16:28 +0100
Subject: [PATCH 12/25] Switched plugins_loaded hook to init, required VIP

---
 resourcespace.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/resourcespace.php b/resourcespace.php
index 4c4eb5a..b682e2d 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -15,7 +15,7 @@
 
 defined( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE' ) or define( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE', 10 );
 
-add_action( 'plugins_loaded', function() {
+add_action( 'init', function() {
 
 	if ( ! class_exists( 'MEXP_Service' ) ) {
 		wp_die( __( 'Media Explorer plugin must be enabled.', 'resourcespace' ) );
@@ -38,7 +38,7 @@
 	Resource_Space_Loader::get_instance();
 	Resource_Space_Admin::get_instance();
 
-});
+}, 9 );
 
 add_filter( 'mexp_services', function( array $services ) {
 

From abdf4d6770838906cec06c34bb7541d976fa9b05 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Tue, 29 Sep 2015 20:40:50 +0100
Subject: [PATCH 13/25] Featured image tab

---
 js/featured-image.js | 98 ++++++++++++++++++++++++++++++++++++++++++++
 resourcespace.php    | 17 ++++++++
 2 files changed, 115 insertions(+)
 create mode 100644 js/featured-image.js

diff --git a/js/featured-image.js b/js/featured-image.js
new file mode 100644
index 0000000..e4ffc8b
--- /dev/null
+++ b/js/featured-image.js
@@ -0,0 +1,98 @@
+(function ( $ ) {
+
+	var resourceSpaceFeaturedImage = function() {
+
+		var controller = wp.media.controller.FeaturedImage;
+
+		wp.media.controller.FeaturedImage = controller.extend({
+
+			initialize: function() {
+
+				var self = this;
+
+				wp.media.controller.FeaturedImage.__super__.initialize.apply( this );
+
+				if ( ! wp.media.frames.resourceSpaceFeaturedImageFrame ) {
+
+					// Bit of an odd hack. But we have to set this to a non-falsey value
+					// in order to prevent infinite loop when creating the new frame.
+					wp.media.frames.resourceSpaceFeaturedImageFrame = 1;
+
+					wp.media.frames.resourceSpaceFeaturedImageFrame = wp.media({
+						frame : "post",
+						state : 'mexp-service-resource-space',
+						resourceSpaceInsertCallback: function() {
+							var selection   = self.get( 'selection' );
+							var attachments = ( this.attachments.length ) ? [ this.attachments[0] ] : [];
+							selection.reset( attachments );
+
+							this.complete();
+						},
+					});
+				}
+
+			},
+
+			activate: function() {
+
+				wp.media.controller.FeaturedImage.__super__.activate.apply( this, arguments );
+
+				// Add the stock images tab.
+				this.frame.on( 'router:render:browse', this.resourceSpaceCreateTab );
+				this.frame.on( 'content:render:resourceSpace', this.resourceSpaceRenderTab );
+
+			},
+
+			resourceSpaceCreateTab: function( routerView ) {
+
+				routerView.set({
+					resourceSpace: {
+						text:     'Stock Images',
+						priority: 60
+					}
+				});
+
+				routerView.controller.content.mode('browse');
+
+			},
+
+			resourceSpaceRenderTab: function() {
+
+				var resourceSpaceFrame = wp.media.frames.resourceSpaceFeaturedImageFrame;
+
+				resourceSpaceFrame.open();
+
+				// Switch content view back to browse in the original frame.
+				var routerView = wp.media.frame.views.get( '.media-frame-router' )[0];
+				routerView.controller.content.mode('browse');
+
+				window.setTimeout( function() {
+
+					// Ensure that the resource space frame is on top.
+					resourceSpaceFrame.$el.closest('.media-modal').parent().appendTo( 'body' );
+
+					// Hide all other menu options in the frame.
+					resourceSpaceFrame.$el.addClass( 'hide-menu' );
+
+				}, 1 );
+
+				// Slightly hcky workaround because for some reason the load more
+				// button doesn't exist when the event callback is attached.
+				$('#resource-space-loadmore').on('click', function(e) {
+					var view = wp.media.frames.resourceSpaceFrame.views.get('.media-frame-content' );
+					if ( view.length ) {
+						view[0].paginate(e);
+					}
+				} );
+
+			},
+
+		} );
+
+	}
+
+	$(document).ready( function() {
+		resourceSpaceFeaturedImage()
+	} );
+
+}( jQuery ));
diff --git a/resourcespace.php b/resourcespace.php
index b682e2d..a797d0d 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -76,6 +76,23 @@ function resource_space_vc_script() {
 add_action( 'admin_print_scripts-post.php', 'resource_space_vc_script' );
 add_action( 'admin_print_scripts-post-new.php', 'resource_space_vc_script' );
 
+/**
+ * Resourcespace featured image Script.
+ * @return null
+ */
+function resource_space_featured_image_script() {
+
+	if ( ! current_user_can( 'insert_from_resourcespace' ) ) {
+		return;
+	}
+
+	wp_enqueue_script( 'resource-space-featured-image', plugins_url( 'js/featured-image.js', __FILE__ ), array( 'backbone' ), null, true );
+
+}
+
+add_action( 'admin_print_scripts-post.php', 'resource_space_featured_image_script' );
+add_action( 'admin_print_scripts-post-new.php', 'resource_space_featured_image_script' );
+
 /**
  * Add the resourcespace MEXP capability for required roles.
  * Note - version check to ensure this isn't fired always as it writes to the database.d

From 378df93b1cc89c62d11be894ca6b97bcdde78e25 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 12:41:44 +0100
Subject: [PATCH 14/25] Handle setting featured image

---
 js/featured-image.js | 54 +++++++++++++++++++++++++-------------------
 1 file changed, 31 insertions(+), 23 deletions(-)

diff --git a/js/featured-image.js b/js/featured-image.js
index e4ffc8b..2b240aa 100644
--- a/js/featured-image.js
+++ b/js/featured-image.js
@@ -7,30 +7,8 @@
 		wp.media.controller.FeaturedImage = controller.extend({
 
 			initialize: function() {
-
-				var self = this;
-
 				wp.media.controller.FeaturedImage.__super__.initialize.apply( this );
-
-				if ( ! wp.media.frames.resourceSpaceFeaturedImageFrame ) {
-
-					// Bit of an odd hack. But we have to set this to a non-falsey value
-					// in order to prevent infinite loop when creating the new frame.
-					wp.media.frames.resourceSpaceFeaturedImageFrame = 1;
-
-					wp.media.frames.resourceSpaceFeaturedImageFrame = wp.media({
-						frame : "post",
-						state : 'mexp-service-resource-space',
-						resourceSpaceInsertCallback: function() {
-							var selection   = self.get( 'selection' );
-							var attachments = ( this.attachments.length ) ? [ this.attachments[0] ] : [];
-							selection.reset( attachments );
-
-							this.complete();
-						},
-					});
-				}
-
+				this.initResourceSpaceFrame();
 			},
 
 			activate: function() {
@@ -87,6 +65,36 @@
 
 			},
 
+			initResourceSpaceFrame: function() {
+
+				if ( ! wp.media.frames.resourceSpaceFeaturedImageFrame ) {
+
+					var self = this;
+
+					// Bit of an odd hack. But we have to set this to a non-falsey value
+					// in order to prevent infinite loop when creating the new frame.
+					wp.media.frames.resourceSpaceFeaturedImageFrame = 1;
+
+					wp.media.frames.resourceSpaceFeaturedImageFrame = wp.media({
+						frame : "post",
+						state : 'mexp-service-resource-space',
+						resourceSpaceInsertCallback: function() {
+							var selection   = self.get( 'selection' );
+							var attachments = ( this.attachments.length ) ? [ this.attachments[0] ] : [];
+							selection.reset( attachments );
+							this.complete();
+
+							if ( attachments.length ) {
+								wp.media.featuredImage.set( attachments[0].id );
+							}
+
+							wp.media.frame.close();
+						},
+					});
+				}
+
+			}
+
 		} );
 
 	}

From fbf6c9753ddb17583931faf3b9c16a8827bfb06d Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 14:56:51 +0100
Subject: [PATCH 15/25] Allow plugins to hook in and add do stuff on import

---
 inc/class-resource-space-loader.php |  4 ++++
 resourcespace.php                   | 17 +++++++++++++++++
 2 files changed, 21 insertions(+)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index 4a34aab..1ee753f 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -68,8 +68,12 @@ function ajax_get_image() {
 		// Request original URL.
 		$attachment_id = wpcom_vip_download_image( $data[0]->preview );
 
+		// Update Metadata.
 		update_post_meta( $attachment_id, 'resource_space', 1 );
 
+		// Allow plugins to hook in here.
+		do_action( 'resourcespace_import_complete', $attachment_id, $resource );
+
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );
 		} else {
diff --git a/resourcespace.php b/resourcespace.php
index a797d0d..c8f9ceb 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -115,3 +115,20 @@ function resource_space_featured_image_script() {
     }
 
 } );
+
+add_filter( 'resourcespace_import_complete', function( $data, $resource ) {
+
+	$post_array = array(
+		'ID' => $attachment_id,
+		// 'post_content' => isset( $resource->Title ) ? sanitize_text_field( $resource->Title ),
+	);
+
+	if ( isset( $resource->Title ) ) {
+		$post_array['post_title'] = sanitize_text_field( $resource->Title );
+	}
+
+	if ( count( $post_array ) > 1 ) {
+		wp_update_post( $post_array );
+	}
+
+}, 10, 2 );

From 2a26f56e712d10969978357e4174667b5cb44aac Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 14:57:50 +0100
Subject: [PATCH 16/25] remove test

---
 resourcespace.php | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/resourcespace.php b/resourcespace.php
index c8f9ceb..a797d0d 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -115,20 +115,3 @@ function resource_space_featured_image_script() {
     }
 
 } );
-
-add_filter( 'resourcespace_import_complete', function( $data, $resource ) {
-
-	$post_array = array(
-		'ID' => $attachment_id,
-		// 'post_content' => isset( $resource->Title ) ? sanitize_text_field( $resource->Title ),
-	);
-
-	if ( isset( $resource->Title ) ) {
-		$post_array['post_title'] = sanitize_text_field( $resource->Title );
-	}
-
-	if ( count( $post_array ) > 1 ) {
-		wp_update_post( $post_array );
-	}
-
-}, 10, 2 );

From 66be5fbf9b40b9ed28a492b1f1902e1579c8bf9c Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 20:00:01 +0100
Subject: [PATCH 17/25] fix php error

---
 inc/class-resource-space-loader.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index 1ee753f..908b6b7 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -72,7 +72,7 @@ function ajax_get_image() {
 		update_post_meta( $attachment_id, 'resource_space', 1 );
 
 		// Allow plugins to hook in here.
-		do_action( 'resourcespace_import_complete', $attachment_id, $resource );
+		do_action( 'resourcespace_import_complete', $attachment_id, $data[0] );
 
 		if ( is_wp_error( $attachment_id ) ) {
 			wp_send_json_error( $attachment_id->get_error_message() );

From a70b3bd446da2cc6e26f9754e69e4699c8048f9d Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 20:12:52 +0100
Subject: [PATCH 18/25] Don't use prettyfieldnames

---
 inc/class-resource-space-loader.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index 908b6b7..f942e55 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -41,7 +41,7 @@ function ajax_get_image() {
 		$url = add_query_arg( array(
 			'key'              => $key,
 			'search'           => $resource_id,
-			'prettyfieldnames' => 1,
+			'prettyfieldnames' => false,
 			'original'         => true,
 			'previewsize'      => 'scr',
 		), $url );

From 66adf06c1647d248b368b1eb615bee85118936e6 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Wed, 30 Sep 2015 20:22:02 +0100
Subject: [PATCH 19/25] allow defining custom path/urls

---
 resourcespace.php | 72 +++++++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 70 insertions(+), 2 deletions(-)

diff --git a/resourcespace.php b/resourcespace.php
index a797d0d..bdfa352 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -10,9 +10,9 @@
 */
 
 define( 'PJ_RESOURCESPACE_PLUGIN_VERSION', '0.1' );
-define( 'PJ_RESOURCE_SPACE_PLUGIN_DIR', untrailingslashit( plugin_dir_path( __FILE__ ) ) );
-define( 'PJ_RESOURCE_SPACE_PLUGIN_URL', untrailingslashit( plugin_dir_url( __FILE__ ) ) );
 
+defined( 'PJ_RESOURCE_SPACE_PLUGIN_DIR' ) OR define( 'PJ_RESOURCE_SPACE_PLUGIN_DIR', untrailingslashit( plugin_dir_path( __FILE__ ) ) );
+defined( 'PJ_RESOURCE_SPACE_PLUGIN_URL' ) OR define( 'PJ_RESOURCE_SPACE_PLUGIN_URL', untrailingslashit( plugin_dir_url( __FILE__ ) ) );
 defined( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE' ) or define( 'PJ_RESOURCE_SPACE_RESULTS_PER_PAGE', 10 );
 
 add_action( 'init', function() {
@@ -115,3 +115,71 @@ function resource_space_featured_image_script() {
     }
 
 } );
+
+add_filter( 'resourcespace_import_complete', function( $attachment_id, $resource ) {
+
+	hm_log( $resource );
+
+	$post_array = array( 'ID' => $attachment_id );
+
+	if ( isset( $resource->field8 ) ) {
+		$post_array['post_title'] = sanitize_text_field( $resource->field8 );
+	}
+
+	if ( isset( $resource->field18 ) ) {
+		$post_array['post_content'] = wp_kses_post( $resource->field18 );
+	}
+
+	// Only update if there is more data than just the ID.
+	if ( count( $post_array ) > 1 ) {
+		wp_update_post( $post_array );
+	}
+
+	if ( isset( $resource->field10 ) ) {
+		update_post_meta( $attachment_id, 'aleteia_media_copyright', sanitize_text_field( $resource->field10 ) );
+	}
+
+}, 10, 2 );
+
+/**
+ * Add custom attachment meta fields
+ *
+ * @param array $form_fields
+ * @param object $post
+ * @return array
+ */
+function aleteia_attachment_fields_to_edit( $form_fields, $post ) {
+	// Copyright text
+	$form_fields['aleteia_media_copyright']['label'] = __( 'Copyright Text', 'aleteia' );
+	$form_fields['aleteia_media_copyright']['input'] = 'textarea';
+	$form_fields['aleteia_media_copyright']['value'] = get_post_meta( $post->ID, 'aleteia_media_copyright', true );
+	// Copyright Link
+	$form_fields['aleteia_media_copyright_link']['label'] = __( 'Copyright Link', 'aleteia' );
+	$form_fields['aleteia_media_copyright_link']['input'] = 'text';
+	$form_fields['aleteia_media_copyright_link']['value'] = get_post_meta( $post->ID, 'aleteia_media_copyright_link', true );
+	return $form_fields;
+}
+add_filter( 'attachment_fields_to_edit', 'aleteia_attachment_fields_to_edit', null, 2 );
+/**
+ * Save custom attachment meta fields
+ *
+ * @param array $post
+ * @param array $attachment
+ * @return array
+ */
+function aleteia_attachment_fields_to_save( $post, $attachment ) {
+	// Copyright text
+	if ( isset( $attachment['aleteia_media_copyright'] ) && $attachment['aleteia_media_copyright'] ) {
+		update_post_meta( $post['ID'], 'aleteia_media_copyright', sanitize_text_field( $attachment['aleteia_media_copyright'] ) );
+	} else if ( isset( $attachment['aleteia_media_copyright'] ) ) {
+		delete_post_meta( $post['ID'], 'aleteia_media_copyright' );
+	}
+	// Copyright Link
+	if ( isset( $attachment['aleteia_media_copyright_link'] ) && $attachment['aleteia_media_copyright_link'] ) {
+		update_post_meta( $post['ID'], 'aleteia_media_copyright_link', sanitize_text_field( $attachment['aleteia_media_copyright_link'] ) );
+	} else if ( isset( $attachment['aleteia_media_copyright_link'] ) ) {
+		delete_post_meta( $post['ID'], 'aleteia_media_copyright_link' );
+	}
+	return $post;
+}
+add_filter( 'attachment_fields_to_save','aleteia_attachment_fields_to_save', null, 2 );

From d50687ae8621036d7d76c11e5f0e4ba547f6b899 Mon Sep 17 00:00:00 2001
From: Paul de Wouters <paul@hmn.md>
Date: Mon, 2 Nov 2015 12:15:16 +0000
Subject: [PATCH 20/25] remove unnecessary function

---
 inc/class-resource-space-loader.php | 66 -----------------------------
 1 file changed, 66 deletions(-)

diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index bee394f..a168ad2 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -84,70 +84,4 @@ function ajax_get_image() {
 
 	}
 
-	private function sideload_image( $url ) {
-
-		$request_args = array( 'headers' => array() );
-
-		// Pass basic auth header if available.
-		if ( defined( 'PJ_RESOURCE_SPACE_AUTHL' ) &&  defined( 'PJ_RESOURCE_SPACE_AUTHP' ) ) {
-			$request_args['headers']['Authorization'] = 'Basic ' . base64_encode( PJ_RESOURCE_SPACE_AUTHL . ':' . PJ_RESOURCE_SPACE_AUTHP );
-		}
-
-		// TODO test. Advice from Kirill was to use the users cookie.
-		// Hopefully it isn't required as this isn't as robust as using basic auth.
-		$response = wp_remote_get( $url, $request_args );
-
-		if ( 200 == wp_remote_retrieve_response_code( $response ) ) {
-
-			$file = get_temp_dir() . sanitize_file_name( $data[0]->Original_filename );
-			file_put_contents( $file, wp_remote_retrieve_body( $response ) );
-
-			$filename = basename( $file );
-
-			$upload_file = wp_upload_bits( $filename, null, file_get_contents( $file ) );
-
-			if ( ! $upload_file['error'] ) {
-
-				$wp_filetype = wp_check_filetype( $filename, null );
-
-				$attachment = array(
-					'post_mime_type' => $wp_filetype['type'],
-					'post_parent'    => 0,
-					'post_title'     => $data[0]->{'Légende'},
-					'post_content'   => 'Downloaded ' . current_time( 'd/m/Y \a\t H:i:s' ),
-					'post_status'    => 'inherit',
-				);
-
-				$attachment_id = wp_insert_attachment( $attachment, $upload_file['file'], $parent_post_id );
-
-				if ( ! is_wp_error( $attachment_id ) ) {
-
-					require_once( trailingslashit( ABSPATH ) . 'wp-admin/includes/image.php' );
-
-					$attachment_data = wp_generate_attachment_metadata( $attachment_id, $upload_file['file'] );
-					$attachment_data['image_meta']['created_timestamp'] = current_time( 'Y-m-d H:i:s', true );
-
-					wp_update_attachment_metadata( $attachment_id, $attachment_data );
-
-					add_post_meta( $attachment_id, 'resource_space', true, true );
-
-					return $attachment_id;
-
-				} else {
-					unlink( $file );
-					return new WP_Error( 'broke', esc_html__( 'Could not create attachment', 'resourcespace' ) );
-				}
-			} else {
-				unlink( $file );
-				return new WP_Error( 'broke', esc_html__( 'Upload error', 'resourcespace' ) );
-			}
-
-			unlink( $file );
-
-		} else {
-			return new WP_Error( 'broke', esc_html__( 'Unable to retrieve image', 'resourcespace' ) );
-		}
-
-	}
-
 }

From a071e17727bf281ff63d167f77a2092a43776b85 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 12 Nov 2015 12:36:17 +0000
Subject: [PATCH 21/25] Remove debug

---
 resourcespace.php | 2 --
 1 file changed, 2 deletions(-)

diff --git a/resourcespace.php b/resourcespace.php
index 3be7e04..a1921f3 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -118,8 +118,6 @@ function resource_space_featured_image_script() {
 
 add_filter( 'resourcespace_import_complete', function( $attachment_id, $resource ) {
 
-	hm_log( $resource );
-
 	$post_array = array( 'ID' => $attachment_id );
 
 	if ( isset( $resource->field8 ) ) {

From 23380947914f38d1743536a3e5494734537d3273 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Thu, 12 Nov 2015 12:36:28 +0000
Subject: [PATCH 22/25] bump version to ensure user caps are correct

---
 resourcespace.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resourcespace.php b/resourcespace.php
index a1921f3..65a29d5 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -9,7 +9,7 @@
 Domain Path: /languages
 */
 
-define( 'PJ_RESOURCESPACE_PLUGIN_VERSION', '0.1' );
+define( 'PJ_RESOURCESPACE_PLUGIN_VERSION', '0.2' );
 
 defined( 'PJ_RESOURCE_SPACE_PLUGIN_DIR' ) OR define( 'PJ_RESOURCE_SPACE_PLUGIN_DIR', untrailingslashit( plugin_dir_path( __FILE__ ) ) );
 defined( 'PJ_RESOURCE_SPACE_PLUGIN_URL' ) OR define( 'PJ_RESOURCE_SPACE_PLUGIN_URL', untrailingslashit( plugin_dir_url( __FILE__ ) ) );

From 0c93063f86af0cd058d10e3cce398e6af2b93b50 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Tue, 17 Nov 2015 14:13:50 +0000
Subject: [PATCH 23/25] Encode request args - breaks 2 word searches

---
 inc/class-mexp-resource-space-service.php |  9 ++++-----
 inc/class-resource-space-loader.php       | 10 ++++------
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/inc/class-mexp-resource-space-service.php b/inc/class-mexp-resource-space-service.php
index 937a65e..9edae1b 100644
--- a/inc/class-mexp-resource-space-service.php
+++ b/inc/class-mexp-resource-space-service.php
@@ -50,8 +50,7 @@ public function request( array $request ) {
 		$request['page'] = ( $request['page'] < 1 ) ? 1 : $request['page'];
 
 		// Build the request URL.
-		$api_url = add_query_arg(
-			apply_filters( 'resourcespace_request_args', array(
+		$args = array_map( 'rawurlencode', apply_filters( 'resourcespace_request_args', array(
 				'search'           => sanitize_text_field( $request['params']['q'] ),
 				'key'              => PJ_RESOURCE_SPACE_KEY,
 				'previewsize'      => 'pre',
@@ -60,9 +59,9 @@ public function request( array $request ) {
 				'results_per_page' => PJ_RESOURCE_SPACE_RESULTS_PER_PAGE,
 				'page'             => absint( $request['page'] ),
 				'restypes'         => 1, // Restrict to images only.
-			) ),
-			sprintf( '%s/plugins/api_search/', PJ_RESOURCE_SPACE_DOMAIN )
-		);
+		) ) );
+
+		$api_url = add_query_arg( $args, sprintf( '%s/plugins/api_search/', PJ_RESOURCE_SPACE_DOMAIN ) );
 
 		$request_args = array(
 			'headers' => array()
diff --git a/inc/class-resource-space-loader.php b/inc/class-resource-space-loader.php
index a168ad2..9039f5f 100644
--- a/inc/class-resource-space-loader.php
+++ b/inc/class-resource-space-loader.php
@@ -35,17 +35,15 @@ function ajax_get_image() {
 			wp_send_json_error( esc_html__( 'Empty resource id', 'resourcespace' ) );
 		}
 
-		$url = PJ_RESOURCE_SPACE_DOMAIN . '/plugins/api_search/';
-		$key = PJ_RESOURCE_SPACE_KEY;
-
-		$url = add_query_arg( array(
-			'key'              => $key,
+		$args = array_map( 'rawurlencode', array(
+			'key'              => PJ_RESOURCE_SPACE_KEY,
 			'search'           => $resource_id,
 			'prettyfieldnames' => false,
 			'original'         => true,
 			'previewsize'      => 'scr',
-		), $url );
+		) );
 
+		$url          = add_query_arg( $args, PJ_RESOURCE_SPACE_DOMAIN . '/plugins/api_search/' );
 		$request_args = array( 'headers' => array() );
 
 		// Pass basic auth header if available.

From 103193d62942e9e3abb16506c494429a152830ee Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Mon, 7 Dec 2015 14:06:25 +0000
Subject: [PATCH 24/25] remove site specific code

---
 inc/class-mexp-resource-space-service.php |  1 -
 resourcespace.php                         | 66 -----------------------
 2 files changed, 67 deletions(-)

diff --git a/inc/class-mexp-resource-space-service.php b/inc/class-mexp-resource-space-service.php
index 9edae1b..e3bdcf2 100644
--- a/inc/class-mexp-resource-space-service.php
+++ b/inc/class-mexp-resource-space-service.php
@@ -54,7 +54,6 @@ public function request( array $request ) {
 				'search'           => sanitize_text_field( $request['params']['q'] ),
 				'key'              => PJ_RESOURCE_SPACE_KEY,
 				'previewsize'      => 'pre',
-				'prettyfieldnames' => true,
 				'original'         => true,
 				'results_per_page' => PJ_RESOURCE_SPACE_RESULTS_PER_PAGE,
 				'page'             => absint( $request['page'] ),
diff --git a/resourcespace.php b/resourcespace.php
index 65a29d5..f656829 100644
--- a/resourcespace.php
+++ b/resourcespace.php
@@ -115,69 +115,3 @@ function resource_space_featured_image_script() {
     }
 
 } );
-
-add_filter( 'resourcespace_import_complete', function( $attachment_id, $resource ) {
-
-	$post_array = array( 'ID' => $attachment_id );
-
-	if ( isset( $resource->field8 ) ) {
-		$post_array['post_title'] = sanitize_text_field( $resource->field8 );
-	}
-
-	if ( isset( $resource->field18 ) ) {
-		$post_array['post_content'] = wp_kses_post( $resource->field18 );
-	}
-
-	// Only update if there is more data than just the ID.
-	if ( count( $post_array ) > 1 ) {
-		wp_update_post( $post_array );
-	}
-
-	if ( isset( $resource->field10 ) ) {
-		update_post_meta( $attachment_id, 'aleteia_media_copyright', sanitize_text_field( $resource->field10 ) );
-	}
-
-}, 10, 2 );
-
-/**
- * Add custom attachment meta fields
- *
- * @param array $form_fields
- * @param object $post
- * @return array
- */
-function aleteia_attachment_fields_to_edit( $form_fields, $post ) {
-	// Copyright text
-	$form_fields['aleteia_media_copyright']['label'] = __( 'Copyright Text', 'aleteia' );
-	$form_fields['aleteia_media_copyright']['input'] = 'textarea';
-	$form_fields['aleteia_media_copyright']['value'] = get_post_meta( $post->ID, 'aleteia_media_copyright', true );
-	// Copyright Link
-	$form_fields['aleteia_media_copyright_link']['label'] = __( 'Copyright Link', 'aleteia' );
-	$form_fields['aleteia_media_copyright_link']['input'] = 'text';
-	$form_fields['aleteia_media_copyright_link']['value'] = get_post_meta( $post->ID, 'aleteia_media_copyright_link', true );
-	return $form_fields;
-}
-add_filter( 'attachment_fields_to_edit', 'aleteia_attachment_fields_to_edit', null, 2 );
-/**
- * Save custom attachment meta fields
- *
- * @param array $post
- * @param array $attachment
- * @return array
- */
-function aleteia_attachment_fields_to_save( $post, $attachment ) {
-	// Copyright text
-	if ( isset( $attachment['aleteia_media_copyright'] ) && $attachment['aleteia_media_copyright'] ) {
-		update_post_meta( $post['ID'], 'aleteia_media_copyright', sanitize_text_field( $attachment['aleteia_media_copyright'] ) );
-	} else if ( isset( $attachment['aleteia_media_copyright'] ) ) {
-		delete_post_meta( $post['ID'], 'aleteia_media_copyright' );
-	}
-	// Copyright Link
-	if ( isset( $attachment['aleteia_media_copyright_link'] ) && $attachment['aleteia_media_copyright_link'] ) {
-		update_post_meta( $post['ID'], 'aleteia_media_copyright_link', sanitize_text_field( $attachment['aleteia_media_copyright_link'] ) );
-	} else if ( isset( $attachment['aleteia_media_copyright_link'] ) ) {
-		delete_post_meta( $post['ID'], 'aleteia_media_copyright_link' );
-	}
-	return $post;
-}
-add_filter( 'attachment_fields_to_save','aleteia_attachment_fields_to_save', null, 2 );

From fcc6d54af97a37763474d537a02995c86f644ea2 Mon Sep 17 00:00:00 2001
From: Matth_eu <matthew@matth.eu>
Date: Mon, 7 Dec 2015 15:28:41 +0000
Subject: [PATCH 25/25] fix admin frame loading bug

---
 js/admin.js | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/js/admin.js b/js/admin.js
index 5d35590..e96d8ea 100644
--- a/js/admin.js
+++ b/js/admin.js
@@ -3,6 +3,8 @@
 	var $oldContainer = $('#resource-space-images');
 	var $newContainer = $('#resource-space-new-images');
 
+	var wp_media_frame;
+
 	var library = window.wp.media({
 		frame: 'manage',
 		container: $oldContainer,
@@ -27,11 +29,21 @@
 
 		}
 
-		var wp_media_frame = wp.media.frames.wp_media_frame = wp.media({
-			frame : "post",
-			state : 'mexp-service-resource-space',
-			resourceSpaceInsertCallback: insertImages,
-		});
+		if ( ! wp_media_frame ) {
+			wp_media_frame = wp.media.frames.wp_media_frame = wp.media({
+				frame : "post",
+				state : 'mexp-service-resource-space',
+				resourceSpaceInsertCallback: insertImages,
+			});
+
+			// Hack to get load more working.
+			wp_media_frame.on( 'open', function() {
+				jQuery( '#resource-space-loadmore' ).click( function( event ) {
+					wp.media.frame.views.get('.media-frame-content' )[0].paginate( event );
+				} );
+			} );
+
+		}
 
 		wp_media_frame.open();
 		wp_media_frame.$el.addClass( 'hide-menu' );