WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#23733 closed defect (bug) (fixed)

Not all images have metadata, data should be checked first

Reported by: trusktr Owned by: markjaquith
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.5.1
Component: Media Keywords:
Focuses: Cc:

Description

I'm getting many errors like these. Is this a problem with a plugin, or with WordPress core?

The errors seems to be happening in core files, so I thought I'd post it here.

To see the actual errors, see the following site (at the time of this writing), in the right sidebar (scroll down):
http://creationofsociety.com

Attachments (3)

23733.diff (817 bytes) - added by wonderboymusic 9 years ago.
Fixes another instance
23733.2.diff (2.7 KB) - added by wonderboymusic 9 years ago.
More of them
23733.3.diff (8.9 KB) - added by wonderboymusic 9 years ago.

Download all attachments as: .zip

Change History (11)

#1 @markoheijnen
9 years ago

  • Keywords reporter-feedback added
  • Severity changed from major to normal

I need more information then this. Seems like the image metadata is corrupted but no clue why and what. Seems like a plugin is doing something wrong but hard to tell.

#2 @trusktr
9 years ago

The plugin that generates the sidebar widget where the errors show up is eshop, but the error doesn't point to any files from the eshop plugin.

#3 @markoheijnen
9 years ago

If you look at the code you see that the error isn't on that place. The data from wp_get_attachment_metadata() isn't how it should. Maybe a plugin does something wrong or you did a replace query on urls. However last one would most likely not show these errors.

#5 @SergeyBiryukov
9 years ago

  • Component changed from General to Media

@wonderboymusic
9 years ago

Fixes another instance

#6 @wonderboymusic
9 years ago

  • Keywords reporter-feedback removed
  • Milestone changed from Awaiting Review to 3.6
  • Summary changed from Warning: Illegal string offset 'width' in /wp-includes/media.php on line 440 to Not all images have metadata, data should be checked first

@wonderboymusic
9 years ago

More of them

#8 @markjaquith
9 years ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In 23873:

Sanity checks on image metadata to avoid warnings, etc.

fixes #23733. props wonderboymusic.

Note: See TracTickets for help on using tickets.