Project:Imagecache Effects
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:reviewed & tested by the community

Issue Summary

When using a bottom or right reflection a line is added between the original image and the reflection (more visible on non-transparent images).

This line is actually the result of the first row of pixels right outside the original image (which doesn't have any content) being copied on top of the new image. This patch fixes the issue.

AttachmentSize
remove_line.patch1.45 KB

Comments

#1

The bug exists. It's annoying.

but the patch doesn't work...

A

#2

I had the same problem and the patch worked for me.
Thanks @flobruit

#3

Status:needs review» reviewed & tested by the community

Patch works.

#4

This patch really didn't worked for me. First of all i used another patch already which renamed $image->res to $image->resource ... this patch doesn't use that patch so it didn't worked. Another issue is this patch is created from Git, so it have weird file paths if i'm not wrong ... like (a/sites/... and b/sites).
But on the other hand, if this patch is properly used (copy, paste, rewrite res to resource) it works for me.

#5

@morf2009 You can just use:

patch -p N < name-of-patch.patch

to remove the N path components from the start of the patch, so on the one in this issue, you'd want to use:

patch -p 2 < remove_line.patch
most likely.

nobody click here