Bug fixing in the form data

The undefined fields were throwing error. So used a condition to skip those fields

Author: psimanta

README.md

@@ -1,4 +1,4 @@
-# SSLCOMMERZ Payment Gateway implementation in NodeJs
+# SSLCOMMERZ Payment Gateway itegration in NodeJs
 It is npm package which provides functionalities to implement SSLCommerze Payment Gateway in Node Based Apps.
 
 ## Installation
@@ -7,7 +7,7 @@ Via NPM
 npm i ssl-commerz-node
 ```
 
-## Instruction
+## Set the parameters to make a request
 ```js
 const SSLCommerz = require('ssl-commerze-node');
 const PaymentSession = SSLCommerz.PaymentSession;
@@ -27,12 +27,82 @@ payment.setOrderInfo({
     total_amount: 1570, // Number field
     currency: 'BDT', // Must be three character string
     tran_id: 'ref12345667', // Unique Transaction id 
-    emi_option: 0 // 1 or 0,
+    emi_option: 0, // 1 or 0
     multi_card_name: 'internetbank', // Do not Use! If you do not customize the gateway list, 
     allowed_bin: '371598,371599,376947,376948,376949', // Do not Use! If you do not control on transaction 
     emi_max_inst_option: 3,  // Max instalment Option
     emi_allow_only: 0 // Value is 1/0, if value is 1 then only EMI transaction is possible
 });
 
+// Set customer info
+payment.setCusInfo({
+    name: 'Simanta Paul',
+    email: 'simanta@bohubrihi.com',
+    add1: '66/A Midtown',
+    add2: 'Andarkilla',
+    city: 'Chittagong',
+    state: 'Optional',
+    postcode: 4000,
+    country: 'Bangladesh',
+    phone: '010000000000',
+    fax: 'Customer_fax_id'
+});
+
+// Set shipping info
+payment.setShippingInfo({
+    method: 'Courier', //Shipping method of the order. Example: YES or NO or Courier
+    num_item: 2,
+    name: 'Simanta Paul',
+    add1: '66/A Midtown',
+    add2: 'Andarkilla',
+    city: 'Chittagong',
+    state: 'Optional',
+    postcode: 4000,
+    country: 'Bangladesh',
+});
+
+// Set Product Profile
+payment.setProductInfo({
+    product_name: 'Computer',
+    product_category: 'Electronics',
+    product_profile: 'general'
+});
+```
+**See this for details:** https://developer.sslcommerz.com/doc/v4/#ready-the-parameters
+
+## After setting the parameters initialize the payment
+```js
+// Initiate Payment and Get session key
+payment.paymentInit()
+    .then(response => {
+        console.log(response);
+    });
 ```
-See this for details: https://developer.sslcommerz.com/doc/v4/#ready-the-parameters
+**This link containes details about the response parameters:** https://developer.sslcommerz.com/doc/v4/#returned-parameters
+
+## Response Parameters Examples
+### After Success
+```js
+console.log(response['status']);
+``` 
+> SUCCESS
+```js
+console.log(response['sessionkey']);
+```       
+> D37CD2C0A0D322991531D217E194F981
+```js
+console.log(response['GatewayPageURL']);
+```       
+> https://sandbox.sslcommerz.com/EasyCheckOut/testcded37cd2c0a0d322991531d217e194f981
+
+### After Failure (Wrong Store ID)
+```js
+console.log(response['status']);
+``` 
+> FAILED
+```js
+console.log(response['failedreason']);
+```       
+> Store Credential Error Or Store is De-active
+
+The **GatewayPageURL** is the url of the payment page.

SSLCommerz/PaymentSession.js

@@ -9,6 +9,7 @@ class PaymentSession extends SSLCommerz {
     }
 
     setSSLdata() {
+        this.setSSLdata();
         this.postData['store_id'] = this.store_id;
         this.postData['store_passwd'] = this.store_pass;
     }
@@ -65,17 +66,21 @@ class PaymentSession extends SSLCommerz {
     }
 
     async paymentInit() {
-        const response = await fetch('https://sandbox.sslcommerz.com/gwprocess/v4/api.php', {
-            method: 'POST', // *GET, POST, PUT, DELETE, etc.
-            mode: 'cors', // no-cors, cors, *same-origin
-            cache: 'no-cache', // *default, no-cache, reload, force-cache, only-if-cached
-            credentials: 'same-origin', // include, *same-origin, omit
-            redirect: 'follow', // manual, *follow, error
-            referrer: 'no-referrer', // no-referrer, *client
-            body: getFormData(this.postData), // body data type must match "Content-Type" header
-        })
-        const data = await response.json();
-        return data;
+        try {
+            const response = await fetch(this.session_api, {
+                method: 'POST', // *GET, POST, PUT, DELETE, etc.
+                mode: 'cors', // no-cors, cors, *same-origin
+                cache: 'no-cache', // *default, no-cache, reload, force-cache, only-if-cached
+                credentials: 'same-origin', // include, *same-origin, omit
+                redirect: 'follow', // manual, *follow, error
+                referrer: 'no-referrer', // no-referrer, *client
+                body: getFormData(this.postData), // body data type must match "Content-Type" header
+            })
+            const data = await response.json();
+            return data;
+        } catch (error) {
+            return error;
+        }
     }
 }
 

SSLCommerz/processFormData.js

@@ -1,9 +1,10 @@
 const FormData = require('form-data');
 
 module.exports = (postData) => {
-    let fdata = new FormData();
-    for (var property in postData) {
-        fdata.append(property, postData[property])
+    let form = new FormData();
+    for (let key in postData) {
+        if (!postData[key]) continue;
+        form.append(key, postData[key]);
     }
-    return fdata;
+    return form;
 }