Merge pull request #14 from esion/fix_throttle

ricbra · ricbra · commit 98bca8431e5b · 2014-07-18T08:18:25.000+02:00
wait a lil' bit more between two calls
diff --git a/lib/Discogs/Service.php b/lib/Discogs/Service.php
@@ -264,14 +264,14 @@ protected function call($path, array $parameters = array())
             $rawData = $this->client->convertResponse($json);
         } else {
             if ($this->isEnableThrottle) {
-                $timestamp = round(microtime(true) * 1000);
+                $timestamp = floor(microtime(true) * 1000);
                 $wait = static::$lastApiRequest + 1000000 - $timestamp;
 
                 if ($wait > 0) {
                     usleep($wait);
                 }
 
-                static::$lastApiRequest = round(microtime(true) * 1000);
+                static::$lastApiRequest = ceil(microtime(true) * 1000);
             }
 
             $rawData = $this->client->call($path, $parameters);

Original file line number	Diff line number	Diff line change
`@@ -264,14 +264,14 @@ protected function call($path, array $parameters = array())`
`264`	`264`	`$rawData = $this->client->convertResponse($json);`
`265`	`265`	`} else {`
`266`	`266`	`if ($this->isEnableThrottle) {`
`267`		`- $timestamp = round(microtime(true) * 1000);`
	`267`	`+ $timestamp = floor(microtime(true) * 1000);`
`268`	`268`	`$wait = static::$lastApiRequest + 1000000 - $timestamp;`
`269`	`269`
`270`	`270`	`if ($wait > 0) {`
`271`	`271`	`usleep($wait);`
`272`	`272`	`}`
`273`	`273`
`274`		`- static::$lastApiRequest = round(microtime(true) * 1000);`
	`274`	`+ static::$lastApiRequest = ceil(microtime(true) * 1000);`
`275`	`275`	`}`
`276`	`276`
`277`	`277`	`$rawData = $this->client->call($path, $parameters);`