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.I'm not sure if I should be posting this as a bug under media_flickr. But when I add a flickr image using that module, I get these notices:
Undefined index: width in file_entity_file_update() line 120...
Undefined index: height in file_entity_file_update() line 120...
I should indicate that the image successfully gets added and any metadata about the image is saved as well. So this doesn't seem to prevent the content from being created. FYI the version of media_flickr is 7.x-2.0-alpha3 if that's important.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | add-filesize-checks-2081713-13.patch | 1.14 KB | Devin Carlson |
| #9 | file_entity-empty-width-error-2081713-7.patch | 1.32 KB | Delphine Lepers |
| #7 | file_entity-empty-width-error-2081713-7.patch | 1.31 KB | Delphine Lepers |
| #4 | file_entity.file_.inc_.patch | 1.39 KB | Delphine Lepers |
| #1 | Screen Shot 2013-09-05 at 4.29.09 PM.png | 143.4 KB | jlk4p |
Comments
Comment #1
jlk4p CreditAttribution: jlk4p commentedI think this is a File Entity problem because I see a similar type of error within the module I'm developing. But I don't have a field called "metadata" as one of those created in the module.
Comment #2
SocialNicheGuru CreditAttribution: SocialNicheGuru commentedI am getting undefined width and height also.
I am using the pdf module
Notice: Undefined index: width in file_entity_file_update() (line 120 of /modules/all/file_entity/file_entity.file.inc).
Notice: Undefined index: width in file_entity_file_update() (line 120 of /modules/all/file_entity/file_entity.file.inc).
Notice: Undefined index: height in file_entity_file_update() (line 120 of /modules/all/file_entity/file_entity.file.inc).
Notice: Undefined index: height in file_entity_file_update() (line 120 of /modules/all/file_entity/file_entity.file.inc).
Comment #3
ditcheva CreditAttribution: ditcheva commentedSame here after adding the media and file_entity modules and running cron...
Comment #4
Delphine Lepers CreditAttribution: Delphine Lepers commentedMoved the closing bracket down so that all metadata management is only made on files that contain metadata.
Comment #5
Delphine Lepers CreditAttribution: Delphine Lepers commentedComment #7
Delphine Lepers CreditAttribution: Delphine Lepers commentedFixed path and file name
Comment #9
Delphine Lepers CreditAttribution: Delphine Lepers commentedFix file path
Comment #10
jlk4p CreditAttribution: jlk4p commentedI just installed and tested the patch. It cleared up the problem with the flicker module. Thank you for fixing this!
Comment #11
cosmicdreams CreditAttribution: cosmicdreams commentedMaybe I have to apply this patch to really undestand what was changed but it seems like the only thing that checked was the removal of a bracket and more indentation. No visible changes to the file path are in that patch.
Comment #12
NancyDru#2182693: Numerous errors on first Cron run marked as duplicate.
Comment #13
Devin Carlson CreditAttribution: Devin Carlson commentedIt looks like there are a few places where a check to ensure that the file is not empty is missing.
The attached patch adds a check to all of those locations.
Comment #14
Devin Carlson CreditAttribution: Devin Carlson commentedTested #13 with a number of empty files to confirm the issue and verify that the patch fixed it while still allowing non-empty files files to have their dimensions stored as metadata.
Committed #13 to File entity 7.x-2x.