Needs review
Project:
Website Screenshot
Version:
6.x-5.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Aug 2010 at 23:04 UTC
Updated:
11 Feb 2016 at 04:55 UTC
Jump to comment: Most recent
Comments
Comment #1
Methos76 commentedI looked over the patch, cause i recognized a bug concerning imagecache handling for imagecache presets containing a '_' in there names. An some preset names where used bei the image function right before they were set in the function.
So this patch is an improvment for the filters.
Comment #2
puravida commentedI have reviewed the codebase for 6.x-5.x-dev and also for the 7.x branch.
However, the patch references the following file:
link_screenshot/link_screenshot.module
...which I do not see anywhere. Am I missing something?
The 6.x branch is no longer supported, but I am going to make some obvious bug fixes and add a couple of small enhancements anyway. I am also considering provided patches but this one confuses me. Please elaborate.
Comment #3
puravida commentedUpdate: I was checking out the "master" branch, which I assumed was the latest code. It may be, but there is no link_screenshot module in it. However, I see the link_screenshot module when I pull the 6.x-5.x branch.
So it may have been deprecated or the master branch may be out-of-sync. I am comparing versions and will get to the bottom of this. If it's been deprecated, then I'll close this ticket as "won't fix". Otherwise, I will look at getting the patch incorporated.