Closed (fixed)
Project:
Imageflow
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
12 Oct 2012 at 13:28 UTC
Updated:
1 Aug 2013 at 20:01 UTC
Jump to comment: Most recent file
in ../imageflow/theme/theme.inc/theme_imageflow_image the image src is sent to drupal_attributes as /sites/default/... thus for sites in subdirectories, the image does not show up. Example if my site is accessed through localhost/folder/site, the image src output by the module is localhost/sites/defaults/... instead of localhost/folder/site/sites/defaults/...; thus the image does not show up. You might want to concat $GLOBALS['base_url'] to the current image src
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | image_src_url-1811050-9.patch | 711 bytes | deggertsen |
Comments
Comment #1
jamesbenison commentedProblem is that the imageflow library does not work with reflections unless the base url is stripped off. It's not missing by accident.
The best solution would be to integrate imagecache actions to create the reflection and do away with the module reflect script.
Comment #2
kengichan commentedafter debug, i found
1. in ./imageflow/theme/theme.inc theme_views_view_imageflow line 59
my site is with 2 languages, , the $lang = 'english', but the "image_field" , i setup it without lang.
the item must fix to:
2. after upgrade drupal to 7.20
the image src broke again.
/sites/all/libraries/imageflow/reflect2.php?img=../../../../sites/default/files/styles/c400/public/images/certificates/2.jpg?itok=EYu3d-vY
remove “?itok=EYu3d-vY”, become to work.
so , in in ./imageflow/theme/theme.inc theme_views_view_imageflow line 127
Comment #3
jrreid commentedDid a couple changes to make it work for me:
In theme/theme.inc, theme_imageflow_image_style()
This does bypass the changes in 7.20 about adding the token but that doesn't matter as reflect[2/3].php look for the file on disk anyways. This does leave the issue that the image may not exist in the image style, but I've solved that by also using the images on another page anyways so it does get created.
In imageflow.module, imageflow_add_js()
Instead of ../../../..
Comment #4
David_Rothstein commentedRelated to the issue at #1932118: Image links broken after upgrade to 7.20?
Comment #5
wayne57 commentedI have tried the above but the images still do not appear, only the outlines. I have colorbox configured and when the blank image is clicked the clolorbox image appears ok, but the image flow presenter does not work. I will have to find another way to display clickable images.
Comment #6
caw67 commentedthis works for me
this not!
the origanal path works!
Comment #7
deggertsen commented#6 worked as explained worked for me as well. We should probably put this into a patch so that it can be reviewed and committed... I'll try to get to that.
Comment #8
kdmarks commentedSame here. #6 worked for me. The first change (to theme.inc) fixed the issue of images not appearing. No need for the second change to imageflow.module.
Thanks for the fix!
Comment #9
deggertsen commentedFirst attempt at a patch on this issue. Let me know if I messed anything up (I always seem to mess something up in the first patch attempt).
Comment #10
deggertsen commentedForgot to mark as needs review.
Comment #11
jamesbenison commentedPatch in #9 committed.
Thanks deggertsen.
Comment #12
jamesbenison commentedReopening this issue. The fix above removes the token and allows the reflect script to work. However the image module will not create the new styled image without the token.
One workaround is to put the line
$conf['image_allow_insecure_derivatives'] = TRUE;in the settings.php file. But that's a hack.Ultimately fixing this problem is going to require adding a reflection action to the image module and doing away with the reflect script in the imageflow library. Unfortunately there is no module right now that provides that action.
Comment #13
jamesbenison commentedI just pushed a commit that changes the reflections to use the imagecache_reflect module. You can find that module here: https://drupal.org/node/1366668#comment-7633607
I recommend installing it before doing the update.
Also beware that imagecache_reflect is not technically a real drupal module yet. I'm hoping to get it incorporated into the imagecache_actions library. Things could change.
These changes should take care of all problems related to paths (including base paths), image security tokens, and reflection-effects. Let me know of any trouble that you run into.
Comment #14
jamesbenison commentedImagecache reflect is now a real drupal module.
You can find it here.
Comment #15
jamesbenison commentedMarking as fixed.