Skip to content

flickr.photos.setDates does not set date_taken #15

Closed
JirkaChadima opened this Issue Apr 18, 2012 · 2 comments

2 participants

@JirkaChadima

My friend stumbled upon an error in https://github.com/dajobe/flickcurl/blob/master/src/photos-api.c#L1998. The date_taken parameter is converted to sql timestamp, but it is not passed to parameters and therefore not sent to Flickr.

I can not currently test it nor provide a pull request, but the correction should be simple and straightforward, on line 2033 add something like

parameters[count++][1]= *date_taken_str;
@dajobe
Owner
dajobe commented Apr 19, 2012

Oops! Yes that is a silly error. I'll fix it

@dajobe
Owner
dajobe commented Apr 19, 2012

Fixed in edf1273

@dajobe dajobe closed this Apr 19, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.