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.
Problem/Motivation
File field has a "Enable Display" field. Field settings:
When you uploading files, file widget has "Display" checkbox. If it checked file formatter shows files for site visitors.
ImageItem extends FileItem, but "Display" checkbox was removed from code.
Comment | File | Size | Author |
---|---|---|---|
#24 | 2040033-24.patch | 4.96 KB | ivnish |
#22 | 2040033-22.patch | 4.55 KB | _utsavsharma |
#22 | interdiff_20-22.txt | 4.55 KB | _utsavsharma |
#20 | core-2040033-20.patch | 5.26 KB | ivnish |
Issue fork drupal-2040033
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #1
GiorgosKYes I vouch for this feature as well
I think it was part of the imagefield in drupal 6 then probably got dropped in drupal 7
Comment #2
dcam CreditAttribution: dcam commentedNew features must be added to the current development version. I'm going to have to assume this hasn't been added D8. I can't check due to Image module installation problems of the on Windows platforms.
Comment #7
ao2 CreditAttribution: ao2 as a volunteer commentedIf there is any chance for this feature to be accepted in core I could spend some time looking into it.
Comment #8
ao2 CreditAttribution: ao2 as a volunteer commentedJFTR for my D8 site I decided to hide the image field by default and use
entity_embed
to show images when needed instead of hardcoding<img/>
elements like I used to do in D6.So I have no interest anymore in this feature.
Thanks,
Antonio
Comment #20
ivnish CreditAttribution: ivnish commentedI made a quick and dirty patch for Drupal 9.5
Don't work with existing fields. Needs to remove image field from content type and add again
Comment #21
smustgrave CreditAttribution: smustgrave at Mobomo commentedThanks for working on this old issue!
Will need test coverage and upgrade path for existing fields.
UI changes should have before/after screenshots added to issue summary.
Got the issue summary started but will need updating
Comment #22
_utsavsharma CreditAttribution: _utsavsharma commentedRerolled for 11.x as #20 failed to apply.
Comment #23
ivnish CreditAttribution: ivnish commentedComment #24
ivnish CreditAttribution: ivnish commented