Uninstall does not delete website_screenshot imagecache preset

Comments

  • puravida committed 183ec16 on 6.x-3.x
    Issue #1179554: Uninstall does not delete website_screenshot imagecache...
puravida’s picture

Assigned: Unassigned » puravida
Issue summary: View changes

I 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.

puravida’s picture

After 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.