BIGFISH: importing "http:..." image locations

[Michel Schroeder]

Hello,

I am referring to the "BF Admin Module Product Loader" doc, page 4:

"URL Locations: any image reference in the input file that begins with “http” will use the
full URL reference as the file location. For example:
SMALL_IMAGE_URL = http://www.myimages.com/products/some-image.jpg
This indicates that the complete reference, without modification or appends, should be
used as the file locator."

If I understand correctly, the file:
/bf/hot-deploy/osafeadmin/src/com/osafe/services/ImportServices.java, around lines 4243 and following seams to benefit from a correction like that:

            String defaultImageDirectory = (String)imageLocationMap.get("DEFAULT_IMAGE_DIRECTORY");
            if(UtilValidate.isNotEmpty(defaultImageDirectory)) {
            if (contentValue.indexOf("http") != 0) {
                        contentValue = defaultImageDirectory + contentValue;
            }
            }
                   
Without that correction the default directory path is inserted before the url... Maybe I am missing some points?

Anyway, thanks and congratulation for Bigfish!

Michel

[Praveen Agrawal]

Hi Michel,

 

It's good to hear that you are exploring BIGFISH.

 

You are right that if you use the full URL reference in the input file, the
code will append the 'DEFAULT_IMAGE_DIRECTORY' path to the url.

To use the Full URL you can delete the 'DEFAULT_IMAGE_DIRECTORY' path using
the 'Image Location Preference' Section under the Product Load Functions and
then load the Products.

 

Currently While loading the products, Images are assumed to be already
located in the references given in the input file, so that it reduces the
time to copy all images from one location to another.

 

We are in the development to handle the 'http' url at both (Product Loader
and Product Images from Admin) places.

If you need any further help, please feel free to get in touch.

 

Again, Thanks for Keeping eyes on BIGFISH !

 

Thanks,

Praveen Agrawal

[hidden email]

 

 

---------- Forwarded message ----------
From: Michel Schroeder <[hidden email]>
Date: Mon, Sep 24, 2012 at 7:15 PM
Subject: BIGFISH: importing "http:..." image locations
To: [hidden email]


Hello,

I am referring to the "BF Admin Module Product Loader" doc, page 4:

"URL Locations: any image reference in the input file that begins with
"http" will use the
full URL reference as the file location. For example:
SMALL_IMAGE_URL = http://www.myimages.com/products/some-image.jpg
This indicates that the complete reference, without modification or appends,
should be
used as the file locator."

If I understand correctly, the file:
/bf/hot-deploy/osafeadmin/src/com/osafe/services/ImportServices.java, around
lines 4243 and following seams to benefit from a correction like that:

                String defaultImageDirectory =
(String)imageLocationMap.get("DEFAULT_IMAGE_DIRECTORY");
                if(UtilValidate.isNotEmpty(defaultImageDirectory)) {
                        if (contentValue.indexOf("http") != 0) {
                        contentValue = defaultImageDirectory + contentValue;
                        }
                }

Without that correction the default directory path is inserted before the
url... Maybe I am missing some points?

Anyway, thanks and congratulation for Bigfish!

Michel




--
View this message in context:
http://ofbiz.135035.n4.nabble.com/BIGFISH-importing-http-image-locations-tp4
636448.html
Sent from the OFBiz - User mailing list archive at Nabble.com.

 

[Christophe Noel]

Thanks for the image URL fix. As far as I can't tell the bug is still open in BigFish latest revision (1.10).

Another little problem, the following file seems to be missing, thus creating an error on the PDP page.
hot-deploy/osafe/webapp/osafe/common/eCommerceProductLargerImage.ftl

Actual file is named ecommerceProductLargerImage.ftl (with a lowercase "c")

Thanks for sharing BigFish !

Christophe

[Len Shein]

Christophe,

Thanks for the heads up, the eCommerceProductLargerImage.ftl file has been
'renamed' accordingly and is available in BF revision 1.10 which can be
downloaded at
http://www.solveda.com/website/downloads/BigFish-eCommerce-v1.10.zip.  

The image URL fix will be available in the next release of BigFish (1.11)
planned for next week.

Regards
BigFish eCommerce
Len

-----Original Message-----
From: Christophe Noel [mailto:[hidden email]]
Sent: Friday, May 03, 2013 11:23 AM
To: [hidden email]
Subject: RE: BIGFISH: importing "http:..." image locations... and new fix !

Thanks for the image URL fix. As far as I can't tell the bug is still open
in
BigFish latest revision (1.10).

Another little problem, the following file seems to be missing, thus
creating an error on the PDP page.
hot-deploy/osafe/webapp/osafe/common/eCommerceProductLargerImage.ftl

Actual file is named ecommerceProductLargerImage.ftl (with a lowercase "c")

Thanks for sharing BigFish !

Christophe



--
View this message in context:
http://ofbiz.135035.n4.nabble.com/BIGFISH-importing-http-image-locations-tp4
636448p4640998.html
Sent from the OFBiz - User mailing list archive at Nabble.com.

[praveen.agrawal]

Hi Christophe,

Image URL issue has been fixed and is available in BigFish revision 1.10
which can be downloaded at
http://www.solveda.com/website/downloads/BigFish-eCommerce-v1.10.zip.

 If you need any help, please feel free to get in touch.



Regards,

Praveen Agrawal
Email: [hidden email]


On Sat, May 4, 2013 at 1:49 AM, Len Shein <[hidden email]> wrote:

> Christophe,
>
> Thanks for the heads up, the eCommerceProductLargerImage.ftl file has been
> 'renamed' accordingly and is available in BF revision 1.10 which can be
> downloaded at
> http://www.solveda.com/website/downloads/BigFish-eCommerce-v1.10.zip.
>
> The image URL fix will be available in the next release of BigFish (1.11)
> planned for next week.
>
> Regards
> BigFish eCommerce
> Len
>
> -----Original Message-----
> From: Christophe Noel [mailto:[hidden email]]
> Sent: Friday, May 03, 2013 11:23 AM
> To: [hidden email]
> Subject: RE: BIGFISH: importing "http:..." image locations... and new fix !
>
> Thanks for the image URL fix. As far as I can't tell the bug is still open
> in
> BigFish latest revision (1.10).
>
> Another little problem, the following file seems to be missing, thus
> creating an error on the PDP page.
> hot-deploy/osafe/webapp/osafe/common/eCommerceProductLargerImage.ftl
>
> Actual file is named ecommerceProductLargerImage.ftl (with a lowercase "c")
>
> Thanks for sharing BigFish !
>
> Christophe
>
>
>
> --
> View this message in context:
>
> http://ofbiz.135035.n4.nabble.com/BIGFISH-importing-http-image-locations-tp4
> 636448p4640998.html
> Sent from the OFBiz - User mailing list archive at Nabble.com.
>
>