• Hi sewpafly,

    I love your plugin! Congrats!

    Unfortunately, I just realize that is not working with ImageMagick Engine.

    I’m using your plugin last version 1.0.7, on wp 3.3.1 with ImageMagick Engine 1.5.0.

    I know that for a lot of user is not a problem, but for those who have to crop big thumbnails and do care about qualaty, it will be limititative.

    You can see the diferencs, with and without Imagemacick: here and here.

    Do you think will be possible to fix it? (would be awesome!) or there is any similar alternative to re-crop the thumbnails you would recommend?

    Thank you!

    http://wordpress.org/extend/plugins/post-thumbnail-editor/

Viewing 7 replies - 16 through 22 (of 22 total)
  • Plugin Author sewpafly

    (@sewpafly)

    Sorry, that doesn’t really help. I’m not sure what the problem actually is (all the imagemagick code is written by wordpress). Unfortunately I don’t have imagemagick on my test systems, so I can’t test this myself, but adding this line to ‘php/functions.php’ (line 403) might help:

    add_filter('jpeg_quality', 'pte_get_jpeg_quality');
    // this is the new line
    add_filter('wp_editor_set_quality', 'pte_get_jpeg_quality');

    I’ll see if I can get imagemagick installed. Does wordpress seem to crop your other pictures correctly? How were you generating the “correct” imagemagick picture above?

    Thread Starter lc

    (@lortnoc)

    the correct imagemagick picture above is a custom thumbnail (using this) automatic generated every-time i upload a image on my media library, or (when I’m testing like now) through the regenerate thumbnails plugin.

    I will test your function lines and I will get back to you.

    Thread Starter lc

    (@lortnoc)

    i removed:
    add_filter('jpeg_quality', 'pte_get_jpeg_quality');
    I had:
    add_filter('wp_editor_set_quality', 'pte_get_jpeg_quality');
    I croped the image with your plugin:

    20130225 16:31:40 - [string] ===== STARTUP PHP MESSAGES =====
    20130225 16:31:40 - [string] ===== SERVER LOG =====
    20130225 16:31:40 - [string] [DEBUG] - PARAMETERS: Array
    (
        [action] => pte_ajax
        [pte-action] => launch
        [id] => 7791
    )
    
    20130225 16:31:40 - [string] [DEBUG] - PTE-VERSION: 2.0.0-alpha
    20130225 16:31:40 - [string] [DEBUG] - USER-AGENT: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_5) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.99 Safari/537.22
    20130225 16:31:40 - [string] [DEBUG] - WORDPRESS: 3.5.1
    20130225 16:31:40 - [string] [DEBUG] - SIZER: 0.33333333
    20130225 16:31:40 - [string] [DEBUG] - SIZES: Array
    (
        [thumbnail] => Array
            (
                [width] => 350
                [height] => 900
                [crop] =>
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-350x232.jpg
                        [width] => 350
                        [height] => 232
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-350x232.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-350x232.jpg
                    )
    
            )
    
        [medium] => Array
            (
                [width] => 342
                [height] => 700
                [crop] =>
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-342x227.jpg
                        [width] => 342
                        [height] => 227
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-342x227.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-342x227.jpg
                    )
    
            )
    
        [large] => Array
            (
                [width] => 1050
                [height] => 1600
                [crop] =>
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x698.jpg
                        [width] => 1050
                        [height] => 698
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x698.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x698.jpg
                    )
    
            )
    
        [150x100crop] => Array
            (
                [width] => 175
                [height] => 117
                [crop] => 1
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-175x117.jpg
                        [width] => 175
                        [height] => 117
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-175x117.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-175x117.jpg
                    )
    
            )
    
        [700xfree] => Array
            (
                [width] => 700
                [height] => 1400
                [crop] => 0
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-700x465.jpg
                        [width] => 700
                        [height] => 465
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-700x465.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-700x465.jpg
                    )
    
            )
    
        [1050x436slider] => Array
            (
                [width] => 1050
                [height] => 436
                [crop] => 1
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                        [width] => 1050
                        [height] => 436
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                    )
    
            )
    
    )
    
    20130225 16:31:43 - [string] ===== REFLOW =====
    20130225 16:31:43 - [string] ===== FIXING THICKBOX =====
    20130225 16:31:43 - [string] WINDOW WIDTH: 779
    20130225 16:31:48 - [string] ENABLING SUBMIT
    20130225 16:31:51 - [string] ===== SETTING ASPECTRATIO: 525:218 =====
    20130225 16:31:51 - [string] DISABLING SUBMIT
    20130225 16:31:54 - [string] DISABLING SUBMIT
    20130225 16:32:00 - [string] DISABLING SUBMIT
    20130225 16:32:02 - [string] DISABLING SUBMIT
    20130225 16:32:06 - [string] DISABLING SUBMIT
    20130225 16:32:09 - [string] DISABLING SUBMIT
    20130225 16:32:45 - [string] DISABLING SUBMIT
    20130225 16:32:47 - [string] ===== RESIZE-IMAGES =====
    20130225 16:32:47 - [object] {"id":"7791","action":"pte_ajax","pte-action":"resize-images","pte-sizes[]":["1050x436slider"],"x":0,"y":177,"w":1200,"h":498}
    20130225 16:32:53 - [string] ===== RESIZE-IMAGES SUCCESS =====
    20130225 16:32:53 - [object] {"thumbnails":{"1050x436slider":{"url":"http://cityburns.com/wp-content/uploads/ptetmp/7791/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg","file":"urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg"}},"pte-nonce":"b6ca21a349","pte-delete-nonce":"4e1f559840","error":[],"log":["[DEBUG] - PARAMETERS: Array\n(\n    [id] => 7791\n    [action] => pte_ajax\n    [pte-action] => resize-images\n    [pte-sizes] => Array\n        (\n            [0] => 1050x436slider\n        )\n\n    [x] => 0\n    [y] => 177\n    [w] => 1200\n    [h] => 498\n)\n","[DEBUG] - BASE FILE DIMENSIONS/INFO: Array\n(\n    [0] => 1200\n    [1] => 798\n    [2] => 2\n    [3] => width=\"1200\" height=\"798\"\n    [bits] => 8\n    [channels] => 3\n    [mime] => image/jpeg\n)\n","[DEBUG] - GETwidthheightCROPPED","[DEBUG] - WIDTHxHEIGHT: 1050 x 436","[DEBUG] - COMPRESSION: 96","[DEBUG] - EDITOR: ImageMagick"]}
    20130225 16:32:53 - [string] ===== SERVER LOG =====
    20130225 16:32:53 - [string] [DEBUG] - PARAMETERS: Array
    (
        [id] => 7791
        [action] => pte_ajax
        [pte-action] => resize-images
        [pte-sizes] => Array
            (
                [0] => 1050x436slider
            )
    
        [x] => 0
        [y] => 177
        [w] => 1200
        [h] => 498
    )
    
    20130225 16:32:53 - [string] [DEBUG] - BASE FILE DIMENSIONS/INFO: Array
    (
        [0] => 1200
        [1] => 798
        [2] => 2
        [3] => width="1200" height="798"
        [bits] => 8
        [channels] => 3
        [mime] => image/jpeg
    )
    
    20130225 16:32:53 - [string] [DEBUG] - GETwidthheightCROPPED
    20130225 16:32:53 - [string] [DEBUG] - WIDTHxHEIGHT: 1050 x 436
    20130225 16:32:53 - [string] [DEBUG] - COMPRESSION: 96
    20130225 16:32:53 - [string] [DEBUG] - EDITOR: ImageMagick
    20130225 16:32:53 - [array] [-1,-779,true]
    20130225 16:32:54 - [array] [-1,0,false]
    20130225 16:32:54 - [string] running callback with no arguments
    20130225 16:32:54 - [string] finished running callback
    20130225 16:32:55 - [string] ===== CHECK SUBMIT BUTTON =====
    20130225 16:32:55 - [string] ENABLE
    20130225 16:32:56 - [string] ===== CONFIRM-IMAGES =====
    20130225 16:32:56 - [object] {"id":"7791","action":"pte_ajax","pte-action":"confirm-images","pte-nonce":"b6ca21a349","pte-confirm":{"1050x436slider":"urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg"}}
    20130225 16:32:58 - [string] ===== CONFIRM-IMAGES SUCCESS =====
    20130225 16:32:58 - [object] {"success":"Yay!","error":[],"log":["[DEBUG] - PARAMETERS: Array\n(\n    [id] => 7791\n    [action] => pte_ajax\n    [pte-action] => confirm-images\n    [pte-nonce] => b6ca21a349\n    [pte-confirm] => Array\n        (\n            [1050x436slider] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg\n        )\n\n)\n","[DEBUG] - pte_get_all_alternate_size_information returned: Array\n(\n    [1050x436slider] => Array\n        (\n            [width] => 1050\n            [height] => 436\n            [crop] => 1\n            [current] => Array\n                (\n                    [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg\n                    [width] => 1050\n                    [height] => 436\n                    [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg\n                    [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg\n                )\n\n        )\n\n)\n","[DEBUG] - Deleting old thumbnail: /home/tiagogon/public_html/cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg","[DEBUG] - Moving '/home/tiagogon/public_html/cityburns.com/wp-content/uploads/ptetmp/7791/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg' to '/home/tiagogon/public_html/cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg'","[DEBUG] - Updating '1050x436slider' metadata: Array\n(\n    [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg\n    [width] => 1050\n    [height] => 436\n)\n"]}
    20130225 16:32:58 - [string] ===== SERVER LOG =====
    20130225 16:32:58 - [string] [DEBUG] - PARAMETERS: Array
    (
        [id] => 7791
        [action] => pte_ajax
        [pte-action] => confirm-images
        [pte-nonce] => b6ca21a349
        [pte-confirm] => Array
            (
                [1050x436slider] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
            )
    
    )
    
    20130225 16:32:58 - [string] [DEBUG] - pte_get_all_alternate_size_information returned: Array
    (
        [1050x436slider] => Array
            (
                [width] => 1050
                [height] => 436
                [crop] => 1
                [current] => Array
                    (
                        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                        [width] => 1050
                        [height] => 436
                        [path] => 2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                        [url] => http://cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
                    )
    
            )
    
    )
    
    20130225 16:32:58 - [string] [DEBUG] - Deleting old thumbnail: /home/tiagogon/public_html/cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
    20130225 16:32:58 - [string] [DEBUG] - Moving '/home/tiagogon/public_html/cityburns.com/wp-content/uploads/ptetmp/7791/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg' to '/home/tiagogon/public_html/cityburns.com/wp-content/uploads/2013/02/urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg'
    20130225 16:32:58 - [string] [DEBUG] - Updating '1050x436slider' metadata: Array
    (
        [file] => urbanoid_dot_cityburns-dot-com_CC-BY-NC-ND_by-lortnoc_Aug2012_kg_016744-1050x436.jpg
        [width] => 1050
        [height] => 436
    )
    
    20130225 16:32:58 - [array] [-1,-779,true]
    20130225 16:32:58 - [array] [-1,0,false]

    it still lose considerable quality for a big image, comparative to the one by imagemagick engine plugin… :/

    would great if you could test with imagemagick engine yourself. anything i can help or test, please let me know. i would love to be able to custom crop with the same setting of IM engine (“96” on quality value).

    Plugin Author sewpafly

    (@sewpafly)

    Why don’t you try the latest version now? I’ve been testing it using the imagemagick engine and I haven’t been able to duplicate your problems. Let me know how it goes.

    (WordPress did not send me these latest posts by you, or I would have responded sooner, sorry)

    Thread Starter lc

    (@lortnoc)

    sorry, I still get on the output the same file quality then before. :/ this time there is a new edition painel and no debug button available.

    Plugin Author sewpafly

    (@sewpafly)

    Yeah all that debug code was getting in the way. The new interface really just uses the same backend, so I guess it isn’t surprising that its still not working for you. 🙁

    Let me see if I can recap your current situation.

    It appears that imagemagick plugin IS working for you as the color profiles are being preserved. However the image quality isn’t being affected when you change the quality value. Is this correct?

    Thread Starter lc

    (@lortnoc)

    correct: imagemagick engine plugin is working fine to me, when it automatically resizes the images right after upload on my library library. this is a good exemple how aswome it works. i could not be more satisfied: it preservs the color porfile and keeep the original quality, just like it the original. (in my case using 96 value on the jpeg compression of Imagemagick engine settings)

    the problem is when i tried your plugin for reposition the crop area, it stil loses significant quality even if is respecting the color profile, as i showed you here before.

    would be great if you could fixe it. I thank you a lot for your time!

Viewing 7 replies - 16 through 22 (of 22 total)
  • The topic ‘Not working with "ImageMagick Engine" plugin’ is closed to new replies.