When changing the Normal image style from flexslider_full to None (original image) the file path is wrong and no images show up.

File path ends up being:

http://domain/sites/default/files/styles/public/file.jpg

Instead of:

http://domain/sites/default/files/file.jpg

CommentFileSizeAuthor
#4 flexslider-original_image_path_fix-1971696-7434066.patch1.46 KBpiyuesh23
#2 flexslider-original_image_path_fix-1971696-7434042.patch1.15 KBpiyuesh23

Comments

piyuesh23’s picture

piyuesh23 commented
Assigned: Unassigned » piyuesh23
piyuesh23’s picture

piyuesh23 commented
StatusFileSize
new flexslider-original_image_path_fix-1971696-7434042.patch1.15 KB

There is no condition set to handle the original image case. Attaching a patch for this fix.

piyuesh23’s picture

piyuesh23 commented
Status: Active » Needs review
piyuesh23’s picture

piyuesh23 commented
StatusFileSize
new flexslider-original_image_path_fix-1971696-7434066.patch1.46 KB

Adding another patch with couple of warnings fixed for the original image.

caktux’s picture

caktux commented

Works great, the patch just needs to be based in the module's folder and not Drupal's root.

ThaJoa’s picture

ThaJoa commented

The patch shows the images now but it still ignores the original image style, it keeps showing it as flexslider_full

minoroffense’s picture

minoroffense commented
Status: Needs review » Closed (fixed)

I've committed the code to dev. I think the image style selection is another issue in the queue.

Once I have that code fixed too, we can revisit this one to ensure it's all working.

As for the path to the image, that should be set by the file ID itself. I don't see anything in here which is forcing it to be in a certain path (but I could be wrong).

minoroffense’s picture

minoroffense commented

#2002146: Image Style selection is ignored