Merge pull request #13 from omise/release/1.5.0.2

Upgrade Omise-PHP version to 2.4.0 & Improve the checkout process.

Author: guzzilar

CHANGELOG.md

@@ -1,26 +1,27 @@
 Change Log
 ==========
 
-[1.5.0.1] - 2015-08-03
+[1.5.0.2] 2015-11-16
 ----------------------
+#### Updates
+- *`Added`* Added `OMISE_USER_AGENT_SUFFIX` and `OMISE_API_VERSION` into the `user-agent` when request to the OMISE APIs
+- *`Updated`* Updated checkout display, add `secured by Omise` logo into checkout form.
+- *`Updated`* Upgrade `omise-php` library from 2.3.1 to 2.4.0.
+- *`Fixed`* Improved the checkout process.
 
-#### Versioning & Document
-- *`Updated`* Updated the omise-opencart version number to `1.5.0.1`
-
-#### Dependencies
-- *`Updated`* Updated `omise-php` library from 2.1.2 to 2.3.1
-
-
-[1.5.0.0] - 2015-07-06
+[1.5.0.1] 2015-08-03
 ----------------------
+#### Updates
+- *`Updated`* Updated the omise-opencart version number to `1.5.0.1`.
+- *`Updated`* Updated `omise-php` library from 2.1.2 to 2.3.1.
 
-#### Versioning & Document
-- *`Updated`* Updated the omise-opencart version number to `1.5.0.0`
-
+[1.5.0.0] 2015-07-06
+----------------------
+#### Updates
+- *`Updated`* Updated the omise-opencart version number to `1.5.0.0`.
 
-[1.0] - 2015-05-18
+[1.0] 2015-05-18
 ------------------
-
 #### Featured
 Implemented the Omise services into OpenCart 1.5.x, The features are as follows:
 - *`Added`* Implemented **Omise Dashboard** into OpenCart's admin page.
@@ -31,7 +32,7 @@ Implemented the Omise services into OpenCart 1.5.x, The features are as follows:
 - *`Added`* Implemented **Omise Keys Setting page** into OpenCart's admin page.
 - *`Added`* Added **Omise menu** into top bar menu of OpenCart's admin page.
 - *`Added`* Added **Omise Payment Gateway Module Configuration** into OpenCart's admin page in Extensions > Payments page.
-- *`Added`* Implemented **Omise Charge API** (supported for `Authorize & Capture` method only)
+- *`Added`* Implemented **Omise Charge API** (supported for `Authorize & Capture` method only).
 - *`Added`* Added **Omise Checkout Form** into OpenCart's checkout page.
 - *`Added`* Added [omise-php](https://github.com/omise/omise-php) library *(v2.1.2)* into this extension.
 - *`Added`* Added **README.md** file.

README.md

@@ -4,7 +4,7 @@
 - OpenCart 1.5.x
 
 ## Dependencies (Already included in Omise-OpenCart)
-- [omise-php](https://github.com/omise/omise-php) (v2.3.1)
+- [omise-php](https://github.com/omise/omise-php) (v2.4.0)
 - [Jquery](https://github.com/jquery/jquery) (v1.7.1 from OpenCart (v1.5.6.4)'s dependency)
 - [vQmod](https://github.com/vqmod/vqmod) (v2.5.1 with OpenCart integration edition)
 
@@ -14,8 +14,8 @@ Follow these steps to install **omise-opencart**:
 1. Download this repository and unzip it into your `local machine` (or directly to your server)
 
   Download links: 
-  [omise-opencart-v1.5.0.1.zip](https://github.com/omise/omise-opencart/archive/v1.5.0.1.zip) or 
-  [omise-opencart-v1.5.0.1.tar.gz](https://github.com/omise/omise-opencart/archive/v1.5.0.1.tar.gz)
+  [omise-opencart-v1.5.0.2.zip](https://github.com/omise/omise-opencart/archive/v1.5.0.2.zip) or
+  [omise-opencart-v1.5.0.2.tar.gz](https://github.com/omise/omise-opencart/archive/v1.5.0.2.tar.gz)
 
   ![omise-opencart Folder Structure](https://omise-cdn.s3.amazonaws.com/assets/omise-opencart/omise-opencart-install-01.png)
 

src/admin/model/payment/omise.php

@@ -1,4 +1,11 @@
 <?php
+// Define 'OMISE_USER_AGENT_SUFFIX'
+if(!defined('OMISE_USER_AGENT_SUFFIX') && defined('VERSION'))
+    define('OMISE_USER_AGENT_SUFFIX', 'OmiseOpenCart/1.5.0.2 OpenCart/'.VERSION);
+
+// Define 'OMISE_API_VERSION'
+if(!defined('OMISE_API_VERSION'))
+    define('OMISE_API_VERSION', '2014-07-27');
 
 class ModelPaymentOmise extends Model
 {

src/catalog/controller/payment/omise.php

@@ -8,6 +8,14 @@ class ControllerPaymentOmise extends Controller
      */
     public function checkout()
     {
+        // Define 'OMISE_USER_AGENT_SUFFIX'
+        if(!defined('OMISE_USER_AGENT_SUFFIX') && defined('VERSION'))
+            define('OMISE_USER_AGENT_SUFFIX', 'OmiseOpenCart/1.5.0.2 OpenCart/'.VERSION);
+
+        // Define 'OMISE_API_VERSION'
+        if(!defined('OMISE_API_VERSION'))
+            define('OMISE_API_VERSION', '2014-07-27');
+
         // If has a `post['omise_token']` request.
         if (isset($this->request->post['omise_token'])) {
             // Load `omise-php` library.
@@ -25,43 +33,85 @@ public function checkout()
 
             // Create a order history with `Processing` status
             $this->load->model('checkout/order');
-            $this->model_checkout_order->confirm($this->session->data['order_id'], 2);
-
-            try {
-                // Try to create a charge and capture it.
-                $omise_charge = OmiseCharge::create(
-                    array(
-                        "amount"        => $this->request->post['amount'],
-                        "currency"      => "thb",
-                        "description"   => $this->request->post['description'],
-                        "card"          => $this->request->post['omise_token']
-                    ),
-                    $omise['public_key'],
-                    $omise['secret_key']
-                );
-
-                if (is_null($omise_charge['failure_code']) && is_null($omise_charge['failure_code']) && $omise_charge['captured'])
-                    $this->model_checkout_order->update($this->session->data['order_id'], 15);
-                else
-                    $this->model_checkout_order->update($this->session->data['order_id'], 10);
+            $order_id       = $this->session->data['order_id'];
+            $order_info     = $this->model_checkout_order->getOrder($order_id);
+            $order_total    = number_format($order_info['total'], 2, '', '');
+
+            if ($order_info) {
+                try {
+                    // Try to create a charge and capture it.
+                    $omise_charge = OmiseCharge::create(
+                        array(
+                            "amount"        => $order_total,
+                            "currency"      => 'thb',
+                            "description"   => $this->request->post['description'],
+                            "card"          => $this->request->post['omise_token']
+                        ),
+                        $omise['public_key'],
+                        $omise['secret_key']
+                    );
+
+                    if (is_null($omise_charge['failure_code']) && is_null($omise_charge['failure_code']) && $omise_charge['captured']) {
+                        // Status: processed.
+                        $this->model_checkout_order->confirm($order_id, 15);
+                    } else {
+                        // Status: failed.
+                        $this->model_checkout_order->update($order_id, 10);
+                    }
 
-                echo json_encode(
-                    array(
-                        'failure_code'      => $omise_charge['failure_code'],
-                        'failure_message'   => $omise_charge['failure_message'],
-                        'captured'          => $omise_charge['captured'],
-                        'omise'             => $omise_charge
-                    )
-                );
-            } catch (Exception $e) {
-                $this->model_checkout_order->update($this->session->data['order_id'], 10);
-                echo json_encode(array('error' => $e->getMessage()));
+                    echo json_encode(
+                        array(
+                            'failure_code'      => $omise_charge['failure_code'],
+                            'failure_message'   => $omise_charge['failure_message'],
+                            'captured'          => $omise_charge['captured'],
+                            'omise'             => $omise_charge
+                        )
+                    );
+                } catch (Exception $e) {
+                    // Status: failed.
+                    $this->model_checkout_order->update($this->session->data['order_id'], 10);
+                    echo json_encode(array('error' => $e->getMessage()));
+                }
+            } else {
+                echo json_encode(array('error' => 'Cannot find your order, please try again.'));
             }
         } else {
             return 'not authorized';
         }
     }
 
+    /**
+     * Retrieve list of months translation
+     *
+     * @return array
+     */
+    public function getMonths()
+    {
+        $months = array();
+        for ($index=1; $index <= 12; $index++) {
+            $month = ($index < 10) ? '0'.$index : $index;
+            $months[$month] = $month;
+        }
+        return $months;
+    }
+
+    /**
+     * Retrieve array of available years
+     *
+     * @return array
+     */
+    public function getYears()
+    {
+        $years = array();
+        $first = date("Y");
+
+        for ($index=0; $index <= 10; $index++) {
+            $year = $first + $index;
+            $years[$year] = $year;
+        }
+        return $years;
+    }
+    
     /**
      * Omise card information form
      * @return void

src/catalog/view/theme/default/image/secured_by_omise.png

@@ -53,7 +53,6 @@
                     // Token was created. Then, charge a card with token.
                     var posting = $.post("<?php echo $checkout_url; ?>", {
                         "omise_token": response.id,
-                        "amount": "<?php echo str_replace('.', '', $orderamount); ?>",
                         "description": "Charge a card from OpenCart that order id is <?php echo $orderid; ?> from <?php echo $billemail; ?>"
                     });
 
@@ -88,7 +87,7 @@
 
 <!-- Omise's checkout form -->
 <form id="omise-form-checkout" method="post" action="<?php echo $success_url; ?>">
-
+    <img src="catalog/view/theme/default/image/secured_by_omise.png" width="200">
     <!-- Collect a customer's card -->
     <div class="omise-payment">
         <h3>Card Information</h3>
@@ -120,8 +119,16 @@
         <div class="input-group clearfix">
             <div class="left"><label>Expire Date</label></div>
             <div class="right">
-                <input type="text" data-omise="expiration_month" size="4" style="width: 10%;" placeholder="dd" value="" class="input-omise-collect-expiration-month"> /
-                <input type="text" data-omise="expiration_year" size="8" style="width: 10%;" placeholder="YYYY" value="" class="input-omise-collect-expiration-year">
+                <select data-omise="expiration_month" class="input-omise-collect-expiration-month"> >
+                <?php foreach ($this->getMonths() as $k=>$v): ?>
+                    <option value="<?php echo $k?$k:'' ?>"><?php echo $v ?></option>
+                <?php endforeach ?>
+                </select>
+                <select data-omise="expiration_year" class="input-omise-collect-expiration-year">
+                <?php foreach ($this->getYears() as $k=>$v): ?>
+                    <option value="<?php echo $k?$k:'' ?>"><?php echo $v ?></option>
+                <?php endforeach ?>
+                </select>
             </div>
         </div>
 

src/catalog/view/theme/default/template/payment/omise.tpl

@@ -1,6 +1,11 @@
 # Change Log
 
-An [unreleased] version is not available on `master` branch. Elements of unreleased list may be edited or removed at any time.
+## [2.4.0] 2015-11-13
+- [Added] Add Omise-Version header to request.
+- [Added] Add `OMISE_USER_AGENT_SUFFIX` constant that let's people add the suffix into the `user-agent` string.
+
+## [2.3.2] 2015-09-30
+- [Fixed] Pass key values into the OmiseRefundList object when call a refunds() method inside the OmiseCharge instance.
 
 ## [2.3.1] 2015-07-29
 - [Added] New DigiCert CA certificates.

src/system/library/omise/omise-php/.gitignore

@@ -56,6 +56,24 @@ require_once dirname(__FILE__).'/omise-php/lib/Omise.php';
 
 Please see usage section below for usage examples.
 
+### API version
+
+You can choose which API version to use with Omise. Each API version has new features and might not be compatible with previous versions. You can change the default version by visiting your Omise Dashboard.
+
+To overwrite the API version to use, you can specify it by defining OMISE_API_VERSION.
+The version configured here will have higher priority than the version set in your Omise account.
+This is useful if you have multiple environments with different API versions for testing.
+(e.g. Development on the latest version but production is on an older version).
+
+```php
+require_once dirname(__FILE__).'/omise-php/lib/Omise.php';
+
+define('OMISE_API_VERSION', '2014-07-27');
+```
+
+It is highly recommended to set this version to the current version
+you're using.
+
 ## Usage
 
 ### 1. Flow