[
https://issues.apache.org/jira/browse/OFBIZ-5658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14040093#comment-14040093
]
Jacques Le Roux commented on OFBIZ-5658:
----------------------------------------
Hi chunlinyao,
Did you not encouters errors (reported in UI for instance)? Are you using a
*nix OS (I use Windows hence the question)?
>The Image Management can not upload at first, I created a tmp folder under
>runtime to fix it.
But we have already a tmp folder under runtime
>The images for products is not processed consistently. We can upload from
>Product -> Content view OR upload from Image Management. And they saved in
>different places.
Yes the content way existed before and is reliable. I initially thought that
the Image Management was an improvement upon it, but it seems it's totally
dissociated, which is a pity (the content way is a bit tedious). It would be
great if we could combine them. But anyway I was just curious, the project I'm
currently working on does not use the OFBiz UI, only OFBiz services. So I'm not
that interested for now, thanks to share your experience!
> checkExistsImage method in ImageManagementServices
> ---------------------------------------------------
>
> Key: OFBIZ-5658
> URL: https://issues.apache.org/jira/browse/OFBIZ-5658
> Project: OFBiz
> Issue Type: Bug
> Components: product
> Affects Versions: Release Branch 13.07
> Environment: any
> Reporter: chunlinyao
> Priority: Trivial
>
> I think indexOf(".") in these code should be lastIndexOf(".")
> public static File checkExistsImage(File file) {
> if (!file.exists()) {
> imageCount = 0;
> imagePath = null;
> return file;
> }
> imageCount++;
> String filePath = imagePath.substring(0, imagePath.indexOf("."));
> String type = imagePath.substring(imagePath.indexOf(".") + 1);
> file = new File(filePath + "(" + imageCount + ")." + type);
> return checkExistsImage(file);
> }
--
This message was sent by Atlassian JIRA
(v6.2#6252)
