Letractively / magento-w2p

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

Blocked image field BUG #641

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
A "disabled" image field makes the product page fall apart:
http://d1.zetaprints.com/mageimage/index.php/magentot-01/template.html

If a template originally has an image field in the template file but the 
"upload" option, "color picker" and internet images are disabled from the 
template page on w2p portal:
http://cname.zetaprints.com/?page=template;TemplateID=FDE4C716-93F6-404B-A38B-14
BF86C9B4D8
(notice the second image field has no options)
leaving no image field for that variable image, the product page in magento 
falls apart.

Original issue reported on code.google.com by agur...@gmail.com on 2 Sep 2011 at 10:30

GoogleCodeExporter commented 8 years ago
I fixed HTML code for empty image fields. But I think we have to decide what to 
do with such fields.

Diff: http://code.google.com/p/magento-w2p/source/detail?r=1789

Original comment by Anatoly....@gmail.com on 2 Sep 2011 at 5:00

GoogleCodeExporter commented 8 years ago
Please, test it with 1.9.1.1beta1 release.

Original comment by Anatoly....@gmail.com on 2 Sep 2011 at 5:31

GoogleCodeExporter commented 8 years ago

Original comment by ad...@zetaprints.com on 2 Sep 2011 at 7:47

GoogleCodeExporter commented 8 years ago
Tested on /mageimage/
web_to_print_store_incl_theme   1.9.1.1beta2 (beta)
http://d1.zetaprints.com/mageimage/index.php/magentot-01/template.html

I think we should just not show image fields in the form if they have all their 
options disabled.

Original comment by agur...@gmail.com on 5 Sep 2011 at 9:33

GoogleCodeExporter commented 8 years ago
Let's leave it for the future. Now it's just a quick fix for current stable 
release.

I think we can't hide such image fields because there are two options (Blank 
and Default) user can select.

Original comment by Anatoly....@gmail.com on 5 Sep 2011 at 10:34

GoogleCodeExporter commented 8 years ago

Original comment by ad...@zetaprints.com on 16 Sep 2011 at 12:09