Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: David_Rothstein commentedRelated to the issue at #1932118: Image links broken after upgrade to 7.20?
Comment #5
wayne57 CreditAttribution: 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 CreditAttribution: caw67 commentedthis works for me
this not!
the origanal path works!
Comment #7
deggertsen CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: deggertsen commentedForgot to mark as needs review.
Comment #11
jamesbenison CreditAttribution: jamesbenison commentedPatch in #9 committed.
Thanks deggertsen.
Comment #12
jamesbenison CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: jamesbenison commentedImagecache reflect is now a real drupal module.
You can find it here.
Comment #15
jamesbenison CreditAttribution: jamesbenison commentedMarking as fixed.