Advanced search  

News:

cpg1.5.36 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.34 or older update to this latest version as soon as possible.
[more]

Pages: [1]   Go Down

Author Topic: Suggestion on the exposure time  (Read 8924 times)

0 Members and 1 Guest are viewing this topic.

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Suggestion on the exposure time
« on: March 26, 2012, 02:59:20 PM »

I noticed that exposures longer than 1s are shown in tenths rather than in seconds, as shown here (20/10s instead of 2s) http://www.flapane.com/gallery/displayimage.php?album=search&cat=0&pid=896#top_display_media
Also, another pic in that gallery shows "10/10s", while a simple "1s" imho would be better.
« Last Edit: November 10, 2014, 11:48:38 AM by flapane »
Logged

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #1 on: March 26, 2012, 03:47:17 PM »

I just tested with the latest version of the EXIF library (you can download it here) and it seems to fix that issue. Unfortunately the formatted data is stored directly in the database, so you'll either have to delete the EXIF data from the database (Coppermine will read it automatically on the next view) or reload it manually for each file.

Please replace Coppermine's include/exif.php file and report any unexpected behavior.
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Re: Suggestion on the exposure time
« Reply #2 on: March 26, 2012, 04:12:01 PM »

Thanks, I tried by manually deleting exif data just for the photo I linked above, and it works. I'll report any strange behaviour.
I can't manage to batch delete EXIF data for all the images in Admin Tools, should I manually delete some sql table?

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #3 on: March 26, 2012, 04:19:57 PM »

Don't delete any table, but just clear the exif table (= remove all data). Something like
Quote
DELETE FROM cpg15x_exif
will do the trick.
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Re: Suggestion on the exposure time
« Reply #4 on: March 26, 2012, 04:29:58 PM »

Thanks, it worked.
There's only one thing left: whenever the exposure time is 1sec, it will show 1/1sec.

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #5 on: March 26, 2012, 04:41:45 PM »

Updated EXIF library in SVN revision 8349.
Logged

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #6 on: March 26, 2012, 04:59:10 PM »

Please post a link to a picture with 1s exposure time.
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
« Last Edit: November 10, 2014, 11:49:11 AM by flapane »
Logged

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #8 on: March 26, 2012, 05:10:56 PM »

Open include/exif.php, find
Code: [Select]
if ($data > 1) {and replace with
Code: [Select]
if ($data >= 1) {
Please confirm if it works as expected.
« Last Edit: March 26, 2012, 05:57:47 PM by Αndré »
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Re: Suggestion on the exposure time
« Reply #9 on: March 26, 2012, 05:25:04 PM »

I just found
Code: [Select]
if ($data == 1)         $data = gettext('sRGB');without the {

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #10 on: March 26, 2012, 05:57:50 PM »

Sorry, my mistake. I just updated the code.
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Re: Suggestion on the exposure time
« Reply #11 on: March 26, 2012, 06:01:10 PM »

It seems to work :)

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #12 on: March 27, 2012, 11:02:25 AM »

Issue reported to Zenphoto (ticket #2143). They maintain the EXIF library.
Logged

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #13 on: March 27, 2012, 01:48:41 PM »

Committed fix in SVN revision 8351.
Logged

flapane

  • Contributor
  • Coppermine frequent poster
  • ***
  • Country: it
  • Offline Offline
  • Gender: Male
  • Posts: 248
    • Flavio's page
Re: Suggestion on the exposure time
« Reply #14 on: March 27, 2012, 02:51:41 PM »

Glad to hear it

Αndré

  • Administrator
  • Coppermine addict
  • *****
  • Country: de
  • Offline Offline
  • Gender: Male
  • Posts: 15038
Re: Suggestion on the exposure time
« Reply #15 on: March 29, 2012, 04:30:08 PM »

The fix is now also included in the source library: http://www.zenphoto.org/trac/changeset/9640
Logged
Pages: [1]   Go Up
 

Page created in 0.079 seconds with 21 queries.