Opened 6 years ago

Closed 3 years ago

#830 closed defect (fixed)

Blog post and exif

Reported by: Jeremy Pope Owned by:
Priority: major Milestone:
Component: programming Keywords: blogpost, exif
Cc: Parent Tickets: #5438

Description

Looks like ./mediagoblin/templates/mediagoblin/user_pages/blog_media.html contains a reference to mediagoblin/utils/exif.html which causes blog posts to crash the server when being loaded if exif_visible = true is set in the config.

Removing the reference fixes the issue.

Anyone have insight as to why it may need to stay?

Subtickets

Change History (3)

comment:1 Changed 4 years ago by ayleph

Parent Tickets: 5438

comment:2 Changed 4 years ago by ayleph

I've got exif_visible = true in my config, and blog posts load without crashing. It seems like this was addressed at some point in the past. However, I don't see any reason at all for blog_media.html to pull in the exif page. I assume that blog_media.html was created as a copy of media.html and the exif code should not have been copied over. I'll create a patch which removes it.

comment:3 Changed 3 years ago by ayleph

Resolution: fixed
Status: newclosed

The exif reference has been removed from blog_media.html as of 6f5fcc5.

Note: See TracTickets for help on using tickets.