phpFlickr » Bug Fixes

Bug Fix: Post-Upload Problems

Dan — Fri, 29 Sep 2006 13:10:57 +0000

Here's a quick bugfix for those of you who don't plan to upgrade to version 2.0.0 right away (because it'll break your apps). If you upload a file, you'll get an error on any Flickr methods you try to call after this. This is because the HTTP_Request object changes the post encoding when you upload a file and there's no systematic way to change it back. You can, however, throw the switch manually. Just stick the following code before the return statement in all of your upload functions in phpFlickr

$this->req->_requestHeaders['content-type'] = 'application/x-www-form-urlencoded';
unset($this->req->_postFiles['photo']);

setLicense Bug Fix

Dan — Sat, 11 Mar 2006 17:06:51 +0000

There is a bug in photos_licenses.setLicense() (thanks, Christian!). Line 1034 in phpFlickr.php should read:
$this->request("flickr.photos.licenses.setLicense", array("photo_id"=>$photo_id, "license_id"=>$license_id), TRUE);

The problem was with capitalization of $photo_id. If you've been having problems setting licenses, this would be why. I've updated CVS and the fix will be in the next release.

Minor bug in photosets_getPhotos()

Dan — Sat, 17 Dec 2005 03:13:51 +0000

I forgot to add 7 characters to the function definition. Line 1085 in phpFlickr.php should be changed from:
function photosets_getPhotos($photoset_id, $extras)
to:
function photosets_getPhotos($photoset_id, $extras = NULL)

Sorry for the inconvenience.

Quasi-bug: Old photo URLs

Dan — Tue, 22 Nov 2005 22:14:55 +0000

It has come to my attention that if your photo's "server" number is 35 and you try to use the depreciated image URL format (i.e. "http://photos35.flickr.com/…." it will fail to resolve that server. You should use the new url format, which is:
http://static.flickr.com/{server-id}/{id}_{secret}_[mstb].jpg

If you're using phpFlickr::buildPhotoURL(), then you'll need to fix the format for that. In phpFlickr.php, you can replace line 262 with:
$url = "http://static.flickr.com/" . $photo['server'] . "/" . $photo['id'] . "_" . $photo['secret'];

Note: The old, depreciated method is supposed to continue to work, but there's apparently something wrong with the DNS resolution for that server. You may or may not be running into this problem and I imagine that it'll clear up at some point.

Bug fix: PEAR Include delimiter

Dan — Tue, 23 Aug 2005 18:33:12 +0000

Ok, I changed this once in 1.3.2, but now I realize the way I did it didn't really work either. The result is that if you're using PHP on a Windows server, it's not including the PEAR folder correctly. I think that the way I've got it now will work just fine for you. If you're using anything but Windows and not having a problem with this, you won't need to touch this code.

Fix:
Change line 24 of phpFlickr.php from:

if (strpos($_SERVER['SERVER_SOFTWARE'], "Windows") !== false) {

To:

if (strpos(__FILE__, ":") !== false) {

phpFlickr 1.3.2 released!

Dan — Thu, 18 Aug 2005 02:37:59 +0000

I recently discovered (thanks to Eliot Shepard) two fairly major bugs. Turns out that since I added the new Auth API (version 1.3), caching hasn't been working. Also, in version 1.3.1, your include folders probably got royally messed up, thanks to a dumb mistake on my part. The good news is that I just put out a bug fix release, version 1.3.2, to get you back on your feet. Hope these haven't caused too many problems for you guys. There was also a fairly minor bug in the XML class that caused newlines to get stripped out of photo descriptions. I've gone ahead and fixed that too, just 'cause I like you guys.

Bug fix: Newlines in XML

Dan — Thu, 11 Aug 2005 15:26:44 +0000

There was a bug in the XML class that was causing it to strip newlines out of fields (a photo's description, for example). Below is the fix that you can apply to xml.php to fix it. I've also updated it in CVS so that it will appear in the next update. The only place where this doesn't quite work is if someone puts a newline at the very beginning of the description. That will still get chopped out, but that's a fairly minor issue.

xml.php
Change lines 27-28 to:
var $_replace = array('°','&',"\n");
var $_replaceWith = array('{deg}', '{amp}', '{lf}');

After line 42 which reads as:
$xml = str_replace($this->_replace, $this->_replaceWith, $xml);
Add this line:
$xml = str_replace(">{lf}", ">\n", $xml);

Bug fix: photos_getNotInSet()

Dan — Sat, 06 Aug 2005 16:34:38 +0000

All versions up through 1.3.1 have had a bug in photos_getNotInSet(). If a call is made while there is only one photo not in a set, the array structure of the result won't be quite the same. This is because of the way that the xml parser arranges things. If this is a concern, replace the entire function (starting on line 596 in phpFlickr.php) with the code below. I've fixed the problem in CVS and the next update will have the correct code.

Photos search change

Dan — Sat, 30 Jul 2005 03:22:14 +0000

In the documentation for photos_search() function, you'll see that I tell you to use a space delimited string if you're searching for tags. Unless I am sorely mistaken, that's the way it used to work. What I know for sure is that Flickr now wants a comma-delimited string for tags. Which means that it should read:

$photos = $f->photos_search(array("tags"=>"brown,cow", "tag_mode"=>"any"));

README.txt bug

Dan — Thu, 30 Jun 2005 13:43:49 +0000

Alx has just pointed out that the README.txt file tells you that you need to install the HTTP_Response package. Unfortunately, this package doesn't exist. What I meant to type was that you need the HTTP_Request package. If you've been having trouble installing it for that reason, installing that package should fix it. Fortunately, I think that it's a common package to come preinstalled with some packaging systems (go Debian!)