Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In the EditorImageDialog, buildFom(), list of valid file extensions are hardcoded and limited to: gif, png, jpg, and jpeg.
'file_validate_extensions' => array('gif png jpg jpeg'),
Just like 'Maximum file size' or 'Maximum dimensions', file extension, would be set for each text format. If not specified to use default extensions.
Comment | File | Size | Author |
---|---|---|---|
#39 | 2733615-39.patch | 4.98 KB | pashupathi nath gajawada |
#36 | image_extension-2733615-5.patch | 4.94 KB | pericxc |
#32 | image_extension-2733615-4.patch | 5.1 KB | pericxc |
#31 | interdiff-image_extension-2733615-1-4.txt | 1.56 KB | pericxc |
#28 | image_extension-2733615-3.patch | 5.5 KB | pericxc |
Comments
Comment #2
Wim LeersOut of curiosity: what are you missing?
Comment #3
Wim LeersComment #4
pericxc CreditAttribution: pericxc commentedpdf and svg
Comment #5
Wim Leers<img>
tag.Comment #6
pericxc CreditAttribution: pericxc commentedstill prefer to have an option to add a file extension.
Comment #7
Wim LeersI understand. This is as low priority as it gets though:
So, if you want to see this happen in the near future, you'll have to work on the patch that adds support for this. "Scratch your own itch" and all that :)
Comment #8
rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commentedComment #9
rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commentedI have added the extensions.
Thanks!!
Comment #11
rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commentedHi Wim Leers,
In the patch, I have added the extension-pdf and svg as per your discussion. But why it has failed,I am not able to get it.
Please can you review the patch and explain me.
Thanks!!
Comment #12
Wim LeersThat's not per our discussion. It's also not what this feature request is requesting.
Please re-read the issue, you misread it :)
Comment #13
pericxc CreditAttribution: pericxc commentedI would expected something like this.
Comment #14
rajeshwari10 CreditAttribution: rajeshwari10 as a volunteer and at Blisstering Solutions commentedComment #15
pericxc CreditAttribution: pericxc commentedComment #16
pericxc CreditAttribution: pericxc commentedComment #18
Wim Leers#13: yep. Next steps:
Comment #19
pericxc CreditAttribution: pericxc commentedComment #20
pericxc CreditAttribution: pericxc commentedComment #22
mayurjadhav CreditAttribution: mayurjadhav commentedI just updated schema file label, let's try this patch for now, will work on this if patch fails to apply successfully.
Comment #24
pericxc CreditAttribution: pericxc commentedComment #25
pericxc CreditAttribution: pericxc commentedComment #27
pericxc CreditAttribution: pericxc commentedComment #28
pericxc CreditAttribution: pericxc commentedComment #30
Wim LeersCan you please provide interdiffs? https://www.drupal.org/documentation/git/interdiff
Comment #31
pericxc CreditAttribution: pericxc commentedinterdiff for patch-4 and patch-1: interdiff-image_extension-2733615-1-4.txt
Comment #32
pericxc CreditAttribution: pericxc commentedComment #33
pericxc CreditAttribution: pericxc commentedComment #35
mondrakerelated, #1014816: Allow image fields to use any extensions the current image toolkit supports (instead of hard-coding jpg, png and gif only)
Comment #36
pericxc CreditAttribution: pericxc commentedComment #37
pericxc CreditAttribution: pericxc commentedComment #39
pashupathi nath gajawada CreditAttribution: pashupathi nath gajawada as a volunteer and at Melity commentedPlease find the updated patch #39.
Comment #54
Wim LeersWe're not going to do this, because: #3231341: Deprecate EditorLinkDialog, EditorImageDialog and EditorMediaDialog in Drupal 10.1 for removal in Drupal 11.