search

prunycb / magento-w2p

Automatically exported from code.google.com/p/magento-w2p
0 stars 0 forks source link

Image cropping err in output files #713

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Tested on /mageimage/
web_to_print_store_incl_theme   1.9.2.0beta19 (beta)

Changes I make to an uploaded image using the Image editor are not processed 
correctly for file generation.
Replication instructions:
- Navigate to:
http://d1.zetaprints.com/mageimage/index.php/magentot-03/image-editing-test.html
- Edit an image by cropping part from a corner (crop_image.png)
- Update preview, and place the order
- The preview is correct (output_prev.png)
- The generated file is not (PDF, JPG, CDR... all) same. This is the JPG output 
from the order (output_01.jpg).
It's as if the output file was generated by cropping the area that appears by 
clicking "Fit height" shortcut in the top horizontal menu.

Didn't we fix this bug a few releases ago? Customers are going live with the 
new image editor, we must fix this fast.

Original issue reported on code.google.com by agur...@gmail.com on 5 Dec 2011 at 4:06

Attachments:

GoogleCodeExporter commented 9 years ago 
Need an urgent resolution and a beta release.

Original comment by ad...@zetaprints.com on 6 Dec 2011 at 12:35

GoogleCodeExporter commented 9 years ago 
Atanas, please, find the issue where this was resolved before.

Original comment by ad...@zetaprints.com on 6 Dec 2011 at 12:37

GoogleCodeExporter commented 9 years ago 
Looks like the bug is at ZP end. Working on it.

Original comment by ad...@zetaprints.com on 6 Dec 2011 at 5:45

GoogleCodeExporter commented 9 years ago 
The problem was with image metadata copying between the saved order and the 
final one.
The list of metadata attributes was hardcoded, which is replaced with a dynamic 
list.

This affects all image attributes at that step, not just the metadata.

Testing should involve placing orders with and without fit in field, using old 
and new images, no images. The number of pages should not make any difference.
Try templates with more than one image too.

No changes to the extensions. All changes are at ZP end.

Original comment by ad...@zetaprints.com on 7 Dec 2011 at 9:19

GoogleCodeExporter commented 9 years ago 
I tried every combo I could think of. Works fine.

Original comment by agur...@gmail.com on 7 Dec 2011 at 10:01

GoogleCodeExporter commented 9 years ago

Original comment by ad...@zetaprints.com on 7 Dec 2011 at 10:06