Comments on: WebThumb is back up http://blog.joshuaeichorn.com/archives/2008/11/04/webthumb-is-back-up/ The weblog of Joshua Eichorn, AJAX, PHP and Open Source Mon, 30 Nov 2009 18:24:26 -0700 http://wordpress.org/?v=2.8.4 hourly 1 By: Joshua Eichorn http://blog.joshuaeichorn.com/archives/2008/11/04/webthumb-is-back-up/comment-page-1/#comment-592094 Joshua Eichorn Wed, 19 Nov 2008 17:24:23 +0000 http://blog.joshuaeichorn.com/?p=359#comment-592094 Jonas: it should be, fix commited thanks Jonas: it should be, fix commited thanks

]]> By: Jonas Eriksson http://blog.joshuaeichorn.com/archives/2008/11/04/webthumb-is-back-up/comment-page-1/#comment-592089 Jonas Eriksson Wed, 19 Nov 2008 16:53:43 +0000 http://blog.joshuaeichorn.com/?p=359#comment-592089 ok, perhaps it should look like $this->jobs[$job->options->url] = $job; even... ok, perhaps it should look like
$this->jobs[$job->options->url] = $job;
even…

]]> By: Jonas Eriksson http://blog.joshuaeichorn.com/archives/2008/11/04/webthumb-is-back-up/comment-page-1/#comment-592087 Jonas Eriksson Wed, 19 Nov 2008 16:48:53 +0000 http://blog.joshuaeichorn.com/?p=359#comment-592087 Great work, however... There seems to be a minor bug in API class Bluga_Webthumb::addJob (checked out from svn). The function only contains on line (line 140): $this->jobs[$job->url]; Shouldn't the line be $this->jobs[$job->url] = $job; / jonas Great work, however…

There seems to be a minor bug in API class Bluga_Webthumb::addJob (checked out from svn). The function only contains on line (line 140):
$this->jobs[$job->url];
Shouldn’t the line be
$this->jobs[$job->url] = $job;

/ jonas

]]>