Active
Project:
Website Screenshot
Version:
6.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
5 Jun 2011 at 23:11 UTC
Updated:
11 Feb 2016 at 05:09 UTC
Jump to comment: Most recent
Uninstall does not delete website_screenshot imagecache preset
Comments
Comment #2
puravida commentedI see references in 6.x-5.x-dev to using imagecache presets, but I don't see anywhere that they are defined. I will have to dig in deeper to figure that out.
However, I see in the 6.x-3.0-beta3 branch where the imagecache preset is defined and the uninstall is missing the proper imagecache removal call. I have added it and committed the fix. Unfortunately, I am new to "git" and it is not intuitive on how to update an existing stable release. It's already been tagged, so I don't know how to update it.
Also, I noticed that the BETA3 version is NOT really the 6.x-3.x branch anymore. It looks like it's been updated to the 6.x-5.x branch but the stable version tag was not properly updated to reflect that. I need to make sure that I update the correct code before I update the stable release, so not to break anything. Might take me a few days to figure it out. Leaving this ticket open until I know it's fixed and available.
Comment #3
puravida commentedAfter further comparison, it appears that the 6.x-3.0-beta3 is an outdated 6.x-3.x version. There have been numerous updates to that branch since the "stable" release, so I cannot know if it is fully tested and stable now.
This is likely why simple fixes have not been pushed in all these years.
After comparing the 6.x branches, I will have to decide which one has the most potential and not support the others. With only 17 6.x users (and dropping), it doesn't make sense to spend a lot of time on a soon-to-be unsupported Drupal version.