Uninstallation should not modify file extensions

Created on 11 May 2023, over 1 year ago

Problem/Motivation

The hook_uninstall removes svg from settings.file_extensions. In my opinion this should not be done. Either there is another module that supports svg images, through an implementation of ImageToolkit or overriding the ImageWidget, which would allow svg uploading, or there is no such override. If there is no override the allowed extensions are limited in the ImageWidget.

Steps to reproduce

  • Enable svg_image module
  • Add svg to the allowed extensions of an image field.
  • Uninstall the svg_image module
  • The svg extension is removed from the allowed extensions.

Proposed resolution

Remove the part in svg_image_uninstall that removes the extension from image fields.

🐛 Bug report
Status

Active

Version

3.0

Component

Code

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024