Media 7.x-2.x Compatibility [#1896860]

Comment	File	Size	Author
#6	media_crop-1896860-6.patch	4.7 KB	Jody Lynn
#4	media_crop-7-x.1.x_media-2.x-3.patch	4.1 KB	AndrewsizZ
#3	media_crop-7-x.1.x_media-2.x-2.patch	4.43 KB	jrreid
#1	media_crop-7-x.1.x_media-2.x.patch	4.17 KB	sherakama
	media_crop-7.x-1.x_media_2.x_compatibility.patch	3.24 KB	sherakama

Comment #1

sherakama CreditAttribution: sherakama commented 23 January 2013 at 19:31

File	Size
media_crop-7-x.1.x_media-2.x.patch	4.17 KB

Slightly better one. Has tag output fixes in it as well.

Log in or register to post comments

Comment #2

jrreid CreditAttribution: jrreid commented 11 October 2013 at 16:14

Seems to work perfectly for me.

Log in or register to post comments

Comment #3

jrreid CreditAttribution: jrreid commented 19 November 2013 at 22:47

Issue summary:

View changes

File	Size
media_crop-7-x.1.x_media-2.x-2.patch	4.43 KB

Actually a few minor fixes, typo and issue with where the alt/title/height/width values are being copied from. Heres a re-roll

Log in or register to post comments

Comment #4

AndrewsizZ CreditAttribution: AndrewsizZ commented 8 February 2014 at 08:19

File	Size
media_crop-7-x.1.x_media-2.x-3.patch	4.1 KB

Hi guys,
For me your patches don't work properly with media 2.x version(tested on different git tags).
So, I created new patch, for me now all work fine. tested on drupal 7.26, media 7.x-2.0-alpha3, Wysiwyg 7.x-2.2

Log in or register to post comments

Comment #5

Jody Lynn

she/her

English

CreditAttribution: Jody Lynn commented 11 February 2014 at 22:52

The patch in #3 works for me with some notices. The patch in #4 did not work. (It gave me first an image that was of a throbber, and then it gave me the original image when I went back to edit).

I'm on Media 7.x-2.0-alpha2+8-dev with ckeditor module and doing this in a wysiwyg.

Log in or register to post comments

Comment #6

Jody Lynn

she/her

English

CreditAttribution: Jody Lynn commented 12 February 2014 at 23:35

File	Size
media_crop-1896860-6.patch	4.7 KB

This is an update from the patch in comment 3 but fixes some notices.

Log in or register to post comments

Comment #7

timbrandin CreditAttribution: timbrandin commented 14 February 2014 at 10:38

I've solved these issues in the patch I'm working on in https://drupal.org/node/2130785

Could anyone answer on that issue if this is of interest for me to upload.

// T

Log in or register to post comments

Comment #8

rodpal CreditAttribution: rodpal commented 13 March 2014 at 22:49

Hi,

Just tried patch N°3 and N°6 and it is not working. It doesn't load the crop interfase during the upload process or adds the right click action on CKeditor.

This is what I'm using:

Drupal 7.24
Media 7.x-2.0-alpha3+34-dev
File entity 7.x-2.0-alpha3+13-dev
Media Crop 7.x-1.4
CKEditor 4.3.3.7841b02

Log in or register to post comments

Comment #9

thamas

Hungarian

Hungary

CreditAttribution: thamas commented 16 June 2014 at 12:12

Priority:	Major	» Critical
Status:	Needs review	» Needs work

Tested patches in #4 and #6 but none of them worked.

I have a site where content was created using Media 7.x-2.0-unstable3+47-dev and Media crop 1.4. So I'd like to have this "old" images displayed too.

Patch in #4 generates a static quarter throbber image in case of new and old images too.

Patch in #6 does not display old images, new ones are displayed in full size, image style settings not appied after saving content. Re-editing content shows that no image style selected. I have this problem with the other patch too. And the image is displayed in the editor as a small thumb…

Tested with
Media 7.x-2.0-alpha3
Media Crop 7.x-1.x-dev
WYSIWYG 7.x-2.2
CKEditor lib 4.0.3