sergiotarxz
/
Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Media: Remove an extra quote when sending a link of a media file to the editor. 

Introduced in [37035].

Props joemcgill, swissspidy, boonebgorges.
Fixes #36578.

git-svn-id: https://develop.svn.wordpress.org/trunk@37288 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Dominik Schilling (ocean90) 2016-04-21 21:20:26 +00:00
parent d434c58404
commit 153610292d
3 changed files with 114 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/wp-admin/includes/ajax-actions.php
View File

@ -2602,7 +2602,7 @@ function wp_ajax_send_attachment_to_editor() {
$rel = $rel ? ' rel="attachment wp-att-' . $id . '"' : ''; // Hard-coded string, $id is already sanitized
if ( ! empty( $url ) ) {
$html = '<a href="' . esc_url( $url ) . '"' . $rel . '">' . $html . '</a>';
$html = '<a href="' . esc_url( $url ) . '"' . $rel . '>' . $html . '</a>';
}
}

2
tests/phpunit/includes/testcase-ajax.php
View File

@ -50,7 +50,7 @@ abstract class WP_Ajax_UnitTestCase extends WP_UnitTestCase {
'menu-locations-save', 'menu-quick-search', 'meta-box-order', 'get-permalink',
'sample-permalink', 'inline-save', 'inline-save-tax', 'find_posts', 'widgets-order',
'save-widget', 'set-post-thumbnail', 'date_format', 'time_format', 'wp-fullscreen-save-post',
'wp-remove-post-lock', 'dismiss-wp-pointer', 'heartbeat', 'nopriv_heartbeat', 'get-revision-diffs',
'wp-remove-post-lock', 'dismiss-wp-pointer', 'send-attachment-to-editor', 'heartbeat', 'nopriv_heartbeat', 'get-revision-diffs',
'save-user-color-scheme', 'update-widget', 'query-themes', 'parse-embed', 'set-attachment-thumbnail',
'parse-media-shortcode', 'destroy-sessions', 'install-plugin', 'update-plugin', 'press-this-save-post',
'press-this-add-category', 'crop-image', 'generate-password',

112
tests/phpunit/tests/ajax/Attachments.php Normal file
View File

@ -0,0 +1,112 @@
<?php
/**
* Admin ajax functions to be tested
*/
require_once( ABSPATH . 'wp-admin/includes/ajax-actions.php' );
/**
* Testing ajax attachment handling.
*
* @group ajax
*/
class Tests_Ajax_Attachments extends WP_Ajax_UnitTestCase {
/**
* @ticket 36578
*/
public function test_wp_ajax_send_attachment_to_editor_should_return_an_image() {
// Become an administrator
$post = $_POST;
$user_id = self::factory()->user->create( array(
'role' => 'administrator',
'user_login' => 'user_36578_administrator',
'user_email' => 'user_36578_administrator@example.com',
) );
wp_set_current_user( $user_id );
$_POST = array_merge($_POST, $post);
$filename = DIR_TESTDATA . '/images/canola.jpg';
$contents = file_get_contents( $filename );
$upload = wp_upload_bits( basename( $filename ), null, $contents );
$attachment = $this->_make_attachment( $upload );
// Set up a default request
$_POST['nonce'] = wp_create_nonce( 'media-send-to-editor' );
$_POST['html'] = 'Bar Baz';
$_POST['post_id'] = 0;
$_POST['attachment'] = array(
'id' => $attachment,
'align' => 'left',
'image-size' => 'large',
'image_alt' => 'Foo bar',
'url' => 'http://example.com/',
);
// Make the request
try {
$this->_handleAjax( 'send-attachment-to-editor' );
} catch ( WPAjaxDieContinueException $e ) {
unset( $e );
}
// Get the response.
$response = json_decode( $this->_last_response, true );
$expected = get_image_send_to_editor( $attachment, '', '', 'left', 'http://example.com/', false, 'large', 'Foo bar' );
// Ensure everything is correct
$this->assertTrue( $response['success'] );
$this->assertEquals( $expected, $response['data'] );
}
/**
* @ticket 36578
*/
public function test_wp_ajax_send_attachment_to_editor_should_return_a_link() {
// Become an administrator
$post = $_POST;
$user_id = self::factory()->user->create( array(
'role' => 'administrator',
'user_login' => 'user_36578_administrator',
'user_email' => 'user_36578_administrator@example.com',
) );
wp_set_current_user( $user_id );
$_POST = array_merge($_POST, $post);
$filename = DIR_TESTDATA . '/formatting/entities.txt';
$contents = file_get_contents( $filename );
$upload = wp_upload_bits( basename( $filename ), null, $contents );
$attachment = $this->_make_attachment( $upload );
// Set up a default request
$_POST['nonce'] = wp_create_nonce( 'media-send-to-editor' );
$_POST['html'] = 'Bar Baz';
$_POST['post_id'] = 0;
$_POST['attachment'] = array(
'id' => $attachment,
'post_title' => 'Foo bar',
'url' => get_attachment_link( $attachment ),
);
// Make the request
try {
$this->_handleAjax( 'send-attachment-to-editor' );
} catch ( WPAjaxDieContinueException $e ) {
unset( $e );
}
// Get the response.
$response = json_decode( $this->_last_response, true );
$expected = sprintf(
'<a href="%s" rel="attachment wp-att-%d">Foo bar</a>',
get_attachment_link( $attachment ),
$attachment
);
// Ensure everything is correct
$this->assertTrue( $response['success'] );
$this->assertEquals( $expected, $response['data'] );
}
}