lubusIN / laravel-mojo

src/Mojo.php

Doc Comments +3 added lines

@@ -134,6 +134,9 @@
 		}
 	}
 
+	/**
+	 * @param resource $curl
+	 */
 	private static function closeCurl($curl,$payload)
 	{
 		curl_setopt($curl, CURLOPT_POST, true);

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Lubusin\Mojo\Models\MojoRefundDetails;
 use Exception;
 use App\User;
-use DB;
 
 class Mojo
 {

Indentation +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 
 		$payload = self::createPaymentPayload($user, $amount, $purpose, $phone);
 
-        $response = self::closeCurl($curl,$payload); 
+		$response = self::closeCurl($curl,$payload); 
 
 		$finalResponse = json_decode($response);
 		
@@ -66,8 +66,8 @@ 
 		$curl = static::setupCURL("https://{$sub}.instamojo.com/api/1.1/refunds/");
 
 		$payload = ['payment_id' => $payment_id,
-				    'type' => $type,
-				    'body' => $reason ];
+					'type' => $type,
+					'body' => $reason ];
 
 		$response = self::closeCurl($curl,$payload);
 		$afterDecoding = json_decode($response);
@@ -125,7 +125,7 @@ 
 			curl_setopt($ch, CURLOPT_RETURNTRANSFER, TRUE);
 			curl_setopt($ch, CURLOPT_FOLLOWLOCATION, TRUE);
 			curl_setopt($ch, CURLOPT_HTTPHEADER,["X-Api-Key:{$api_key}",
-			                                     "X-Auth-Token:{$api_token}"]);
+												 "X-Auth-Token:{$api_token}"]);
 			return $ch;
 
 		}

Spacing +20 added lines, -20 removed lines

@@ -26,7 +26,7 @@ 
 
 		$payload = self::createPaymentPayload($user, $amount, $purpose, $phone);
 
-        $response = self::closeCurl($curl,$payload); 
+        $response = self::closeCurl($curl, $payload); 
 
 		$finalResponse = json_decode($response);
 		
@@ -59,7 +59,7 @@ 
 	 * To process the refund's
 	 */
 
-	public static function refund($payment_id,$type,$reason)
+	public static function refund($payment_id, $type, $reason)
 	{
 		$sub = config('laravelmojo.subdomain_for_endpoints');
 
@@ -67,16 +67,16 @@ 
 
 		$payload = ['payment_id' => $payment_id,
 				    'type' => $type,
-				    'body' => $reason ];
+				    'body' => $reason];
 
-		$response = self::closeCurl($curl,$payload);
+		$response = self::closeCurl($curl, $payload);
 		$afterDecoding = json_decode($response);
 		$refund = $afterDecoding->refund;
 
-		$transaction = MojoPaymentDetails::where('payment_id',$payment_id)->first();
+		$transaction = MojoPaymentDetails::where('payment_id', $payment_id)->first();
 		$user_id = $transaction->user_id;
 
-		$refund_record = self::createRefundInDB($user_id,$refund,$payment_id);
+		$refund_record = self::createRefundInDB($user_id, $refund, $payment_id);
 			
 		return $refund_record;
 	}
@@ -87,23 +87,23 @@ 
 			throw new Exception('Please set the Instamojo API key in your env file');
 		}
 
-		elseif(!config('laravelmojo.token')) {
+		elseif (!config('laravelmojo.token')) {
 			throw new Exception('Please set the Instamojo token in your env file');
 		}
 
-		elseif(!config('laravelmojo.redirect_url_after_payment')) {
+		elseif (!config('laravelmojo.redirect_url_after_payment')) {
 			throw new Exception('Please set the redirect url in your env file');
 		}
 
-		elseif(!config('laravelmojo.subdomain_for_endpoints')) {
+		elseif (!config('laravelmojo.subdomain_for_endpoints')) {
 			throw new Exception('Please set the subdomain for Instamojo api endpoint in your env file');
 		}
 
-		elseif(!config('laravelmojo.webhook_url')) {
+		elseif (!config('laravelmojo.webhook_url')) {
 			throw new Exception('Please set the webhook url in your env file');
 		}
 
-		elseif(!config('laravelmojo.salt')) {
+		elseif (!config('laravelmojo.salt')) {
 			throw new Exception('Please set the instamojo salt in your env file');
 		}
 
@@ -124,7 +124,7 @@ 
 			curl_setopt($ch, CURLOPT_HEADER, FALSE);
 			curl_setopt($ch, CURLOPT_RETURNTRANSFER, TRUE);
 			curl_setopt($ch, CURLOPT_FOLLOWLOCATION, TRUE);
-			curl_setopt($ch, CURLOPT_HTTPHEADER,["X-Api-Key:{$api_key}",
+			curl_setopt($ch, CURLOPT_HTTPHEADER, ["X-Api-Key:{$api_key}",
 			                                     "X-Auth-Token:{$api_token}"]);
 			return $ch;
 
@@ -134,7 +134,7 @@ 
 		}
 	}
 
-	private static function closeCurl($curl,$payload)
+	private static function closeCurl($curl, $payload)
 	{
 		curl_setopt($curl, CURLOPT_POST, true);
 		curl_setopt($curl, CURLOPT_POSTFIELDS, http_build_query($payload));
@@ -146,7 +146,7 @@ 
 
 	private static function createPaymentPayload(User $user, $amount, $purpose, $phone = null)
 	{
-		if(is_null($phone)) {
+		if (is_null($phone)) {
 			$phone = $user->phone;
 		}
 
@@ -159,12 +159,12 @@ 
 					'webhook' => config('laravelmojo.webhook_url'),
 					'send_sms' => false,
 					'email' => $user->email,
-					'allow_repeated_payments' => false ];
+					'allow_repeated_payments' => false];
 
 		return $payload;
 	} 
 
-	private static function createRefundInDB($user_id,$refund,$payment_id)
+	private static function createRefundInDB($user_id, $refund, $payment_id)
 	{
 		$refund_record = MojoRefundDetails::create(['user_id' => $user_id,
 												   'refund_id' => $refund->id,
@@ -185,15 +185,15 @@ 
 	}
 	public static function allPaymentsFor(User $user)
 	{
-		return MojoPaymentDetails::where('user_id',$user->id)->get();
+		return MojoPaymentDetails::where('user_id', $user->id)->get();
 	}
 	public static function failedPayments()
 	{
-		return MojoPaymentDetails::where('payment_status','!=','credit')->get();
+		return MojoPaymentDetails::where('payment_status', '!=', 'credit')->get();
 	}
 	public static function successfulPayments()
 	{
-		return MojoPaymentDetails::where('payment_status','credit')->get();
+		return MojoPaymentDetails::where('payment_status', 'credit')->get();
 	}
 	public static function myAndMojosIncome()
 	{
@@ -216,7 +216,7 @@ 
 	}
 	public static function allRefundsFor(User $user)
 	{
-		return MojoRefundDetails::where('user_id',$user->id)->get();
+		return MojoRefundDetails::where('user_id', $user->id)->get();
 	}
 
 }

Braces +7 added lines, -20 removed lines

@@ -85,29 +85,17 @@ 
 	{
 		if (!config('laravelmojo.key')) {
 			throw new Exception('Please set the Instamojo API key in your env file');
-		}
-
-		elseif(!config('laravelmojo.token')) {
+		} elseif(!config('laravelmojo.token')) {
 			throw new Exception('Please set the Instamojo token in your env file');
-		}
-
-		elseif(!config('laravelmojo.redirect_url_after_payment')) {
+		} elseif(!config('laravelmojo.redirect_url_after_payment')) {
 			throw new Exception('Please set the redirect url in your env file');
-		}
-
-		elseif(!config('laravelmojo.subdomain_for_endpoints')) {
+		} elseif(!config('laravelmojo.subdomain_for_endpoints')) {
 			throw new Exception('Please set the subdomain for Instamojo api endpoint in your env file');
-		}
-
-		elseif(!config('laravelmojo.webhook_url')) {
+		} elseif(!config('laravelmojo.webhook_url')) {
 			throw new Exception('Please set the webhook url in your env file');
-		}
-
-		elseif(!config('laravelmojo.salt')) {
+		} elseif(!config('laravelmojo.salt')) {
 			throw new Exception('Please set the instamojo salt in your env file');
-		}
-
-		else {
+		} else {
 			return true;
 		}
 	}
@@ -128,8 +116,7 @@ 
 			                                     "X-Auth-Token:{$api_token}"]);
 			return $ch;
 
-		}
-		else {
+		} else {
 			throw new Exception('CURL extension is not loaded');
 		}
 	}

src/controllers/WebhookController.php

Indentation +11 added lines, -11 removed lines

@@ -19,19 +19,19 @@ 
 		$major = (int) $ver[0];
 		$minor = (int) $ver[1];
 		if($major >= 5 and $minor >= 4){
-		     ksort($data, SORT_STRING | SORT_FLAG_CASE);
+			 ksort($data, SORT_STRING | SORT_FLAG_CASE);
 		}
 		else{
-		     uksort($data, 'strcasecmp');
+			 uksort($data, 'strcasecmp');
 		}
 		
 		$mac_calculated = hash_hmac("sha1", implode("|", $data), config('laravelmojo.salt'));
 		if($mac_provided == $mac_calculated){
-		    if($data['status'] == "Credit"){
+			if($data['status'] == "Credit"){
 
-		    $user = User::where('email',$data['buyer'])->first();
+			$user = User::where('email',$data['buyer'])->first();
 			$user_id = $user->id;
-	    	MojoPaymentDetails::create(['user_id' => $user_id,
+			MojoPaymentDetails::create(['user_id' => $user_id,
 									   'buyer_email' => $data['buyer'],
 									   'buyer_name' => $data['buyer_name'],
 									   'buyer_phone' => $data['buyer_phone'],
@@ -46,13 +46,13 @@ 
 									   'request_status' => 'completed',
 									   'payment_status' => $data['status'],
 									 ]);
-		    }
+			}
 
-		    else{
+			else{
 		        
-		    	$user = User::where('email',$data['buyer'])->first();
+				$user = User::where('email',$data['buyer'])->first();
 				$user_id = $user->id;
-		    	MojoPaymentDetails::create(['user_id' => $user_id,
+				MojoPaymentDetails::create(['user_id' => $user_id,
 										   'buyer_email' => $data['buyer'],
 										   'buyer_name' => $data['buyer_name'],
 										   'buyer_phone' => $data['buyer_phone'],
@@ -68,10 +68,10 @@ 
 										   'payment_status' => $data['status'],
 										 ]);
 
-		    }
+			}
 		}
 		else{
-		    echo "MAC mismatch";
+			echo "MAC mismatch";
 		}
 	}
 }

Spacing +8 added lines, -8 removed lines

@@ -18,18 +18,18 @@ 
 		$ver = explode('.', phpversion());
 		$major = (int) $ver[0];
 		$minor = (int) $ver[1];
-		if($major >= 5 and $minor >= 4){
+		if ($major >= 5 and $minor >= 4) {
 		     ksort($data, SORT_STRING | SORT_FLAG_CASE);
 		}
-		else{
+		else {
 		     uksort($data, 'strcasecmp');
 		}
 		
 		$mac_calculated = hash_hmac("sha1", implode("|", $data), config('laravelmojo.salt'));
-		if($mac_provided == $mac_calculated){
-		    if($data['status'] == "Credit"){
+		if ($mac_provided == $mac_calculated) {
+		    if ($data['status'] == "Credit") {
 
-		    $user = User::where('email',$data['buyer'])->first();
+		    $user = User::where('email', $data['buyer'])->first();
 			$user_id = $user->id;
 	    	MojoPaymentDetails::create(['user_id' => $user_id,
 									   'buyer_email' => $data['buyer'],
@@ -48,9 +48,9 @@ 
 									 ]);
 		    }
 
-		    else{
+		    else {
 		        
-		    	$user = User::where('email',$data['buyer'])->first();
+		    	$user = User::where('email', $data['buyer'])->first();
 				$user_id = $user->id;
 		    	MojoPaymentDetails::create(['user_id' => $user_id,
 										   'buyer_email' => $data['buyer'],
@@ -70,7 +70,7 @@ 
 
 		    }
 		}
-		else{
+		else {
 		    echo "MAC mismatch";
 		}
 	}

Braces +3 added lines, -7 removed lines

@@ -20,8 +20,7 @@ 
 		$minor = (int) $ver[1];
 		if($major >= 5 and $minor >= 4){
 		     ksort($data, SORT_STRING | SORT_FLAG_CASE);
-		}
-		else{
+		} else{
 		     uksort($data, 'strcasecmp');
 		}
 		
@@ -46,9 +45,7 @@ 
 									   'request_status' => 'completed',
 									   'payment_status' => $data['status'],
 									 ]);
-		    }
-
-		    else{
+		    } else{
 		        
 		    	$user = User::where('email',$data['buyer'])->first();
 				$user_id = $user->id;
@@ -69,8 +66,7 @@ 
 										 ]);
 
 		    }
-		}
-		else{
+		} else{
 		    echo "MAC mismatch";
 		}
 	}

src/MojoServiceProvider.php

Indentation +7 added lines, -7 removed lines

@@ -8,14 +8,14 @@
 {
 	public function boot()
 	{
-	    $this->loadMigrationsFrom(__DIR__.'/../migrations');
+		$this->loadMigrationsFrom(__DIR__.'/../migrations');
 
-	    $this->publishes([
-        	__DIR__.'/config/laravelmojo.php' => config_path('laravelmojo.php'),
-    	]);
+		$this->publishes([
+			__DIR__.'/config/laravelmojo.php' => config_path('laravelmojo.php'),
+		]);
 	}
-    public function register()
-    {
+	public function register()
+	{
         
-    }
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -8,10 +8,10 @@
 {
 	public function boot()
 	{
-	    $this->loadMigrationsFrom(__DIR__.'/../migrations');
+	    $this->loadMigrationsFrom(__DIR__ . '/../migrations');
 
 	    $this->publishes([
-        	__DIR__.'/config/laravelmojo.php' => config_path('laravelmojo.php'),
+        	__DIR__ . '/config/laravelmojo.php' => config_path('laravelmojo.php'),
     	]);
 	}
     public function register()

src/config/laravelmojo.php

Indentation +12 added lines, -12 removed lines

@@ -2,24 +2,24 @@ 
 
 return [
 
-    /*
+	/*
      * Specify the Private AUTH KEY in your .env file provided
      * by Instamojo in the API & Plugin's section
      * of your account's integration page.
      */
     
-    'key' => env('INSTAMOJO_KEY', null),
+	'key' => env('INSTAMOJO_KEY', null),
 
-    /*
+	/*
      * Specify your Private AUTH TOKEN in your .env file provided by Instamojo in the 
      * API & Plugins section of your integration page.
      * Also I haven't watched a single episode of FRIENDS yet, please
      * don't judge me for that. It's on my list.
      */
     
-    'token' => env('INSTAMOJO_TOKEN', null), 
+	'token' => env('INSTAMOJO_TOKEN', null), 
 
-    /*
+	/*
      * The URL of your app to which the user will be redirected after the  
      * payment process at Instamojo's part will end.
      * Tip : If you are testing on localhost , create a alias in your host
@@ -27,9 +27,9 @@ 
      * as your base URL. And add it to your routes.
      */
 
-    'redirect_url_after_payment' => env('INSTAMOJO_REDIRECT_URL', null),
+	'redirect_url_after_payment' => env('INSTAMOJO_REDIRECT_URL', null),
 
-    /*
+	/*
      * These took me an hour to figure out , for testing purposers you will  
      * most probably use the instamojo sandbox testing account , if thats
      * the case, then the subdomain for all your endpoints will be "test"
@@ -38,23 +38,23 @@ 
      * then it will be "www"  
      */
 
-    'subdomain_for_endpoints' => env('INSTAMOJO_SUBDOMAIN', null),
+	'subdomain_for_endpoints' => env('INSTAMOJO_SUBDOMAIN', null),
 
-    /* 
+	/* 
      * WEBHOOK is your app's URL to which Instamojo sends
      * payment details as a POST request. This package 
      * handles everything , you just have to define
      * the POST route & add it your route's file
      */
     
-    'webhook_url' => env('INSTAMOJO_WEBHOOK_URL', null),
+	'webhook_url' => env('INSTAMOJO_WEBHOOK_URL', null),
 
-    /*
+	/*
      * Specify the Private SALT in your .env file provided
      * by Instamojo in the API & Plugin's section
      * of your account's integration page.
      */
     
-    'salt' => env('INSTAMOJO_SALT', null),
+	'salt' => env('INSTAMOJO_SALT', null),
     
 ];