Merge pull request #62 from patrickbussmann/feature/change-listing

Added change listing endpoint

Author: ricbra

resources/service.php

@@ -411,6 +411,74 @@
                 ]
             ]
         ],
+        'changeListing' => [
+            'httpMethod' => 'POST',
+            'uri' => 'marketplace/listings/{listing_id}',
+            'summary' => 'Edit the data associated with a listing.',
+            'responseModel' => 'GetResponse',
+            'parameters' => [
+	            'listing_id' => [
+		            'type' => 'string',
+		            'location' => 'uri',
+		            'required' => true,
+	            ],
+                'release_id' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'condition' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'sleeve_condition' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'price' => [
+                    'type' => 'number',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'comments' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'allow_offers' => [
+                    'type' => 'boolean',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'status' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'external_id' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'location' => [
+                    'type' => 'string',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'weight' => [
+                    'type' => 'number',
+                    'location' => 'json',
+                    'required' => false,
+                ],
+                'format_quantity' => [
+                    'type' => 'number',
+                    'location' => 'json',
+                    'required' => false,
+                ]
+            ]
+        ],
         'deleteListing' => [
             'httpMethod' => 'DELETE',
             'uri' => 'marketplace/listings/{listing_id}',

tests/Discogs/Test/ClientTest.php

@@ -277,6 +277,21 @@ public function testCreateListing()
         $this->assertSame('https://api.discogs.com/marketplace/listings', $history->getLastRequest()->getUrl());
     }
 
+    public function testChangeListing()
+    {
+        $client = $this->createClient('change_listing', $history = new History());
+        $response = $client->changeListing([
+            'listing_id' => '1',
+            'release_id' => '1',
+            'condition' => 'Mint (M)',
+            'status' => 'For Sale',
+            'price' => 5.90
+        ]);
+
+        $this->assertSame('POST', $history->getLastRequest()->getMethod());
+        $this->assertSame('https://api.discogs.com/marketplace/listings/1', $history->getLastRequest()->getUrl());
+    }
+
     public function testDeleteListing()
     {
         $client = $this->createClient('delete_listing', $history = new History());

tests/fixtures/change_listing

@@ -0,0 +1,14 @@
+HTTP/1.1 204 OK
+Reproxy-Status: yes
+Access-Control-Allow-Origin: *
+Cache-Control: public, must-revalidate
+Content-Type: application/json
+Server: lighttpd
+Content-Length: 780
+Date: Tue, 15 Jul 2014 19:59:59 GMT
+X-Varnish: 1702965334
+Age: 0
+Via: 1.1 varnish
+Connection: keep-alive
+
+{}