Merge pull request #105 from Payum/klarna-checkout

Add Klarna Checkout.
Payum · Mar 12, 2014 · eda4d61 · eda4d61
2 parents 5f8d9a9 + 4265622
commit eda4d61
Show file tree

Hide file tree

Showing 7 changed files with 93 additions and 25 deletions.
diff --git a/Request/PostRedirectUrlInteractiveRequest.php b/Request/PostRedirectUrlInteractiveRequest.php
@@ -1,34 +1,28 @@
 <?php
-
 namespace Payum\Core\Request;
 
-class PostRedirectUrlInteractiveRequest extends BaseInteractiveRequest
+class PostRedirectUrlInteractiveRequest extends ResponseInteractiveRequest
 {
     /**
-     * @var string
+     * @param string $content
+     * @param array $fields
      */
-    protected $url;
-
-    /**
-     * @var array
-     */
-    protected $post;
-
-    /**
-     * @param string $url
-     * @param array $post
-     */
-    public function __construct($url, array $post = array())
+    public function __construct($content, array $fields = array())
     {
-        $this->url = $url;
-        $this->post = $post;
+        parent::__construct($this->prepareContent($content, $fields));
     }
 
-    public function getContent()
+    /**
+     * @param $url
+     * @param array $fields
+     *
+     * @return string
+     */
+    protected function prepareContent($url, array $fields)
     {
-        $formFields = '';
-        foreach ($this->post as $name => $value) {
-            $formFields .= sprintf(
+        $formInputs = '';
+        foreach ($fields as $name => $value) {
+            $formInputs .= sprintf(
                 '<input type="hidden" name="%1$s" value="%2$s" />',
                 htmlspecialchars($name, ENT_QUOTES, 'UTF-8'),
                 htmlspecialchars($value, ENT_QUOTES, 'UTF-8')
@@ -50,6 +44,6 @@ public function getContent()
 </html>
 HTML;
 
-       return sprintf($content, htmlspecialchars($this->url, ENT_QUOTES, 'UTF-8'), $formFields);
+       return sprintf($content, htmlspecialchars($url, ENT_QUOTES, 'UTF-8'), $formInputs);
     }
 }
diff --git a/Request/RedirectUrlInteractiveRequest.php b/Request/RedirectUrlInteractiveRequest.php
@@ -9,11 +9,11 @@ class RedirectUrlInteractiveRequest extends BaseInteractiveRequest
     protected $url;
 
     /**
-     * @param string $url
+     * @param string $content
      */
-    public function __construct($url)
+    public function __construct($content)
     {
-        $this->url = $url;
+        $this->url = $content;
     }
 
     /**

diff --git a/Request/ResponseInteractiveRequest.php b/Request/ResponseInteractiveRequest.php
@@ -0,0 +1,26 @@
+<?php
+namespace Payum\Core\Request;
+
+class ResponseInteractiveRequest extends BaseInteractiveRequest
+{
+    /**
+     * @var string
+     */
+    protected $content;
+
+    /**
+     * @param string $content
+     */
+    public function __construct($content)
+    {
+        $this->content = $content;
+    }
+
+    /**
+     * @return string
+     */
+    public function getContent()
+    {
+        return $this->content;
+    }
+}
diff --git a/Resources/docs/supported-payments.md b/Resources/docs/supported-payments.md
@@ -10,6 +10,7 @@
 * [Be2Bill Onsite](https://github.com/Payum/Be2Bill/blob/master/Resources/docs/index.md)
 * [Payex](https://github.com/Payum/Payex/blob/master/Resources/docs/index.md)
 * [Offline](https://github.com/Payum/Offline/blob/master/Resources/docs/index.md)
+* [Klarna Checkout](https://github.com/Payum/KlarnaCheckout/blob/master/Resources/docs/get-it-started.md)
 
 ## Community
 

diff --git a/Tests/Request/PostRedirectUrlInterativeRequestTest.php b/Tests/Request/PostRedirectUrlInterativeRequestTest.php
@@ -15,6 +15,16 @@ public function shouldImplementInteractiveRequestInterface()
         $this->assertTrue($rc->implementsInterface('Payum\Core\Request\InteractiveRequestInterface'));
     }
 
+    /**
+     * @test
+     */
+    public function shouldBeSubClassOfResponseInteractiveRequest()
+    {
+        $rc = new \ReflectionClass('Payum\Core\Request\PostRedirectUrlInteractiveRequest');
+
+        $this->assertTrue($rc->isSubclassOf('Payum\Core\Request\ResponseInteractiveRequest'));
+    }
+
     /**
      * @test
      */

diff --git a/...uest/RedirectUrlInterativeRequestTest.php → ...est/RedirectUrlInteractiveRequestTest.php b/...uest/RedirectUrlInterativeRequestTest.php → ...est/RedirectUrlInteractiveRequestTest.php
diff --git a/Tests/Request/ResponseInterativeRequestTest.php b/Tests/Request/ResponseInterativeRequestTest.php
@@ -0,0 +1,37 @@
+<?php
+namespace Payum\Core\Tests\Request;
+
+use Payum\Core\Request\ResponseInteractiveRequest;
+
+class ResponseInteractiveRequestTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * @test
+     */
+    public function shouldImplementInteractiveRequestInterface()
+    {
+        $rc = new \ReflectionClass('Payum\Core\Request\ResponseInteractiveRequest');
+
+        $this->assertTrue($rc->implementsInterface('Payum\Core\Request\InteractiveRequestInterface'));
+    }
+
+    /**
+     * @test
+     */
+    public function couldBeConstructedWithUrlAsArgument()
+    {
+        new ResponseInteractiveRequest('html page');
+    }
+
+    /**
+     * @test
+     */
+    public function shouldAllowGetUrlSetInConstructor()
+    {
+        $expectedContent = 'html page';
+
+        $request = new ResponseInteractiveRequest($expectedContent);
+
+        $this->assertEquals($expectedContent, $request->getContent());
+    }
+}