PDA

View Full Version : 'use exif date' doesn't work?


Mav
04-22-2007, 07:29 PM
Just updated to 1.6, looks very nice. But I noticed somehow the "use exif date" no longer works. It's just using "now" as date when adding a photo.

http://kzoiks.com/
I added the <EXIF_CAPTURE_DATE> to the photo stats, so you can see it's really off. As are all the photos.

I've tried to find the "bug", and figured it's doing the date processing around line 298 in new_image.php? Looks like the splitting of the date is okay... At least for this format: "2006:09:11 17:30:32". Maybe it's has been changed somewhere, but it used to work fine on my previous older pixelpost version.

Oh I also did a search here, but only found the original feature request for this option.

Connie
04-22-2007, 07:51 PM
one question: did you follow the update-readme?

did you upgrade the EXIF?

After the upgrade is complete, login to the admin panel, and go to the addons page.
Scroll down near the bottom and press the button labeled "Update EXIF".

Mav
04-22-2007, 08:50 PM
Sorry, forgot to mention that. I didn't do an upgrade from the older version, I decided to do a clean start. Deleted all pixelpost files & photos, and dropped the DB too. So I followed the install-readme. And added all photo's again, there weren't that many anyway.

Note that it's not the photo stats that are off, it's the posting date when you add a new photo and choose "use exif date" under "Date and Time for the entry". (Just mentioning it, because I believe the upgrade exif is for storing exif data in the DB from older photos)

Dennis
04-23-2007, 05:46 AM
We recently discovered this bug and updated the code. Version 1.6.1 will cover this.

Mav
04-23-2007, 02:59 PM
Awesome, but is there a svn repository somewhere? Or a pointer to where the bug is at, I'd probably be able to fix it myself.

Dennis
04-23-2007, 03:24 PM
I send you an email with instructions.

For those of you interested:

Modified: trunk/admin/new_image.php
================================================== ============================
--- trunk/admin/new_image.php (original)
+++ trunk/admin/new_image.php Tue Apr 17 09:07:21 2007
@@ -291,7 +291,10 @@
$exif_info_db = serialize_exif ($uploadfile);
if($postdatefromexif == TRUE)
{
- $exif_result=unserialize_exif($exif_info_db);
+ // since we all ready escaped everything for database commit we have
+ // strip the slashes before we can use the exif again.
+ $exif_info = stripslashes($exif_info_db);
+ $exif_result=unserialize_exif($exif_info);
$exposuredatetime = $exif_result['DateTimeOriginalSubIFD'];
if ($exposuredatetime!='')
{

Mav
04-23-2007, 03:25 PM
Cool, thanks! :)

*edit: Just updated my site, and it works.

Dennis
04-24-2007, 03:26 PM
Issue resolved, closed and moved to 1.6.