Inspection of "Release 1.2.0" - MultinetInteractive/eduadmin-wp-klarna-checkout - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 5c12d3...f60c20 )

by Chris

created 2020-01-15 13:07 UTC

Status

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -29,9 +29,9 @@  discard block
 block discarded – undo
 	along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
-add_action( 'admin_init', function () {
-	if ( is_admin() && current_user_can( 'activate_plugins' ) && ( ! is_plugin_active( 'eduadmin-booking/eduadmin.php' ) && ! is_plugin_active( 'eduadmin/eduadmin.php' ) ) ) {
-		add_action( 'admin_notices', function () {
+add_action( 'admin_init', function() {
+	if ( is_admin() && current_user_can( 'activate_plugins' ) && ( !is_plugin_active( 'eduadmin-booking/eduadmin.php' ) && !is_plugin_active( 'eduadmin/eduadmin.php' ) ) ) {
+		add_action( 'admin_notices', function() {
 			?>
             <div class="error">
                 <p><?php esc_html_e( 'This plugin requires the EduAdmin-WordPress-plugin to be installed and activated.', 'eduadmin-wp-klarna-checkout' ); ?></p>
@@ -45,14 +45,14 @@  discard block
 block discarded – undo
 	}
 } );
 
-if ( ! class_exists( 'EDU_KlarnaCheckout_Loader' ) ) {
+if ( !class_exists( 'EDU_KlarnaCheckout_Loader' ) ) {
 	final class EDU_KlarnaCheckout_Loader {
 		public function __construct() {
 			add_action( 'plugins_loaded', array( $this, 'init' ) );
 		}
 
 		public function init() {
-			if ( ! class_exists( 'Klarna_Checkout_Connector' ) ) {
+			if ( !class_exists( 'Klarna_Checkout_Connector' ) ) {
 				require_once __DIR__ . '/klarna_kco_php_4.0.0/src/Klarna/Checkout.php';
 			}
 

Please login to merge, or discard this patch.

class-edu-klarnacheckout.php 1 patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -1,7 +1,7 @@  discard block
 block discarded – undo
 <?php
 defined( 'ABSPATH' ) || die( 'This plugin must be run within the scope of WordPress.' );
 
-if ( ! class_exists( 'EDU_KlarnaCheckout' ) ) {
+if ( !class_exists( 'EDU_KlarnaCheckout' ) ) {
 	class EDU_KlarnaCheckout extends EDU_Integration {
 		public function __construct() {
 			$this->id          = 'eduadmin-klarnacheckout';
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
 
 			$ebi = new EduAdmin_BookingInfo( $event_booking, $_customer, $_contact );
 
-			if ( ! empty( EDU()->session['klarna-order-id'] ) && ! empty( $_GET['klarna_order_id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
+			if ( !empty( EDU()->session['klarna-order-id'] ) && !empty( $_GET['klarna_order_id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
 				do_action( 'eduadmin-bookingcompleted', $ebi );
 			} else {
 				do_action( 'eduadmin-processbooking', $ebi );
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
 				return;
 			}
 
-			if ( ! empty( $_POST['act'] ) && ( 'bookCourse' === $_POST['act'] || 'bookProgramme' === $_POST['act'] ) ) {
+			if ( !empty( $_POST['act'] ) && ( 'bookCourse' === $_POST['act'] || 'bookProgramme' === $_POST['act'] ) ) {
 				$ebi->NoRedirect = true;
 			}
 		}
@@ -107,10 +107,10 @@  discard block
 block discarded – undo
 			if ( 'no' === $this->get_option( 'enabled', 'no' ) ) {
 				return;
 			}
-			$checkout_url  = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
+			$checkout_url  = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
 			$shared_secret = $this->get_option( 'shared_secret', '' );
 
-			if ( ! empty( $_GET['klarna_order_id'] ) && ! empty( EDU()->session['klarna-order-id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
+			if ( !empty( $_GET['klarna_order_id'] ) && !empty( EDU()->session['klarna-order-id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
 				try {
 					$connector = Klarna_Checkout_Connector::create(
 						$shared_secret,
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
 		 */
 		public function create_checkout( $ebi = null ) {
 
-			$checkout_url  = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
+			$checkout_url  = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
 			$shared_secret = $this->get_option( 'shared_secret', '' );
 
 			$create = array();
@@ -184,10 +184,10 @@  discard block
 block discarded – undo
 			$organization     = EDUAPIHelper()->GetOrganization();
 			$purchase_country = $organization["CountryCode"];
 
-			if ( ! empty( $ebi->Customer["CountryCode"] ) ) {
+			if ( !empty( $ebi->Customer["CountryCode"] ) ) {
 				$purchase_country = $ebi->Customer["CountryCode"];
 
-				if ( ! empty( $ebi->Customer["BillingInfo"]["CountryCode"] ) ) {
+				if ( !empty( $ebi->Customer["BillingInfo"]["CountryCode"] ) ) {
 					$purchase_country = $ebi->Customer["BillingInfo"]["CountryCode"];
 				}
 			}
@@ -209,14 +209,14 @@  discard block
 block discarded – undo
 
 			$_event = null;
 
-			if ( ! empty( $ebi->EventBooking['BookingId'] ) ) {
+			if ( !empty( $ebi->EventBooking['BookingId'] ) ) {
 				$booking_id   = intval( $ebi->EventBooking['BookingId'] );
 				$reference_id = $booking_id;
 
 				$_event = EDUAPI()->OData->Events->GetItem( $ebi->EventBooking['EventId'] );
 			}
 
-			if ( ! empty( $ebi->EventBooking['ProgrammeBookingId'] ) ) {
+			if ( !empty( $ebi->EventBooking['ProgrammeBookingId'] ) ) {
 				$programme_booking_id = intval( $ebi->EventBooking['ProgrammeBookingId'] );
 				$reference_id         = $programme_booking_id;
 
@@ -226,15 +226,15 @@  discard block
 block discarded – undo
 			$rowExtraInfo = "";
 
 			if ( null != $_event ) {
-				if ( ! empty( $_event['City'] ) ) {
+				if ( !empty( $_event['City'] ) ) {
 					$rowExtraInfo .= ';' . $_event['City'];
 				}
 
-				if ( ! empty( $_event['StartDate'] ) ) {
+				if ( !empty( $_event['StartDate'] ) ) {
 					$rowExtraInfo .= ';' . date( "Y-m-d", strtotime( $_event['StartDate'] ) );
 				}
 
-				if ( ! empty( $_event['EndDate'] ) ) {
+				if ( !empty( $_event['EndDate'] ) ) {
 					$rowExtraInfo .= ';' . date( "Y-m-d", strtotime( $_event['EndDate'] ) );
 				}
 			}
@@ -281,7 +281,7 @@  discard block
 block discarded – undo
 				$cart_item['name']      = $order_row['Description'] . $rowExtraInfo;
 				$cart_item['quantity']  = intval( $order_row['Quantity'] );
 
-				if ( ! $order_row['PriceIncVat'] ) {
+				if ( !$order_row['PriceIncVat'] ) {
 					$price_per_unit = $order_row['PricePerUnit'] * ( 1 + ( $order_row['VatPercent'] / 100 ) ) * 100;
 				} else {
 					$price_per_unit = $order_row['PricePerUnit'] * 100;
@@ -318,8 +318,8 @@  discard block
 block discarded – undo
 		}
 
 		public function process_paymentstatus() {
-			if ( ! empty( $_GET['klarna_order_id'] ) && ! empty( $_GET['status'] ) ) {
-				$checkout_url  = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
+			if ( !empty( $_GET['klarna_order_id'] ) && !empty( $_GET['status'] ) ) {
+				$checkout_url  = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
 				$shared_secret = $this->get_option( 'shared_secret', '' );
 
 				try {

Please login to merge, or discard this patch.

		@@ -29,9 +29,9 @@ discard block
		block discarded – undo
29	29	along with this program. If not, see <http://www.gnu.org/licenses/>.
30	30	*/
31	31
32		-add_action( 'admin_init', function () {
33		- if ( is_admin() && current_user_can( 'activate_plugins' ) && ( ! is_plugin_active( 'eduadmin-booking/eduadmin.php' ) && ! is_plugin_active( 'eduadmin/eduadmin.php' ) ) ) {
34		- add_action( 'admin_notices', function () {
	32	+add_action( 'admin_init', function() {
	33	+ if ( is_admin() && current_user_can( 'activate_plugins' ) && ( !is_plugin_active( 'eduadmin-booking/eduadmin.php' ) && !is_plugin_active( 'eduadmin/eduadmin.php' ) ) ) {
	34	+ add_action( 'admin_notices', function() {
35	35	?>
36	36	<div class="error">
37	37	<p><?php esc_html_e( 'This plugin requires the EduAdmin-WordPress-plugin to be installed and activated.', 'eduadmin-wp-klarna-checkout' ); ?></p>
		@@ -45,14 +45,14 @@ discard block
		block discarded – undo
45	45	}
46	46	} );
47	47
48		-if ( ! class_exists( 'EDU_KlarnaCheckout_Loader' ) ) {
	48	+if ( !class_exists( 'EDU_KlarnaCheckout_Loader' ) ) {
49	49	final class EDU_KlarnaCheckout_Loader {
50	50	public function __construct() {
51	51	add_action( 'plugins_loaded', array( $this, 'init' ) );
52	52	}
53	53
54	54	public function init() {
55		- if ( ! class_exists( 'Klarna_Checkout_Connector' ) ) {
	55	+ if ( !class_exists( 'Klarna_Checkout_Connector' ) ) {
56	56	require_once __DIR__ . '/klarna_kco_php_4.0.0/src/Klarna/Checkout.php';
57	57	}
58	58

		@@ -1,7 +1,7 @@ discard block
		block discarded – undo
1	1	<?php
2	2	defined( 'ABSPATH' ) \|\| die( 'This plugin must be run within the scope of WordPress.' );
3	3
4		-if ( ! class_exists( 'EDU_KlarnaCheckout' ) ) {
	4	+if ( !class_exists( 'EDU_KlarnaCheckout' ) ) {
5	5	class EDU_KlarnaCheckout extends EDU_Integration {
6	6	public function __construct() {
7	7	$this->id = 'eduadmin-klarnacheckout';
		@@ -65,7 +65,7 @@ discard block
		block discarded – undo
65	65
66	66	$ebi = new EduAdmin_BookingInfo( $event_booking, $_customer, $_contact );
67	67
68		- if ( ! empty( EDU()->session['klarna-order-id'] ) && ! empty( $_GET['klarna_order_id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
	68	+ if ( !empty( EDU()->session['klarna-order-id'] ) && !empty( $_GET['klarna_order_id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
69	69	do_action( 'eduadmin-bookingcompleted', $ebi );
70	70	} else {
71	71	do_action( 'eduadmin-processbooking', $ebi );
		@@ -80,7 +80,7 @@ discard block
		block discarded – undo
80	80	return;
81	81	}
82	82
83		- if ( ! empty( $_POST['act'] ) && ( 'bookCourse' === $_POST['act'] \|\| 'bookProgramme' === $_POST['act'] ) ) {
	83	+ if ( !empty( $_POST['act'] ) && ( 'bookCourse' === $_POST['act'] \|\| 'bookProgramme' === $_POST['act'] ) ) {
84	84	$ebi->NoRedirect = true;
85	85	}
86	86	}
		@@ -107,10 +107,10 @@ discard block
		block discarded – undo
107	107	if ( 'no' === $this->get_option( 'enabled', 'no' ) ) {
108	108	return;
109	109	}
110		- $checkout_url = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
	110	+ $checkout_url = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
111	111	$shared_secret = $this->get_option( 'shared_secret', '' );
112	112
113		- if ( ! empty( $_GET['klarna_order_id'] ) && ! empty( EDU()->session['klarna-order-id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
	113	+ if ( !empty( $_GET['klarna_order_id'] ) && !empty( EDU()->session['klarna-order-id'] ) && EDU()->session['klarna-order-id'] === $_GET['klarna_order_id'] ) {
114	114	try {
115	115	$connector = Klarna_Checkout_Connector::create(
116	116	$shared_secret,
		@@ -176,7 +176,7 @@ discard block
		block discarded – undo
176	176	*/
177	177	public function create_checkout( $ebi = null ) {
178	178
179		- $checkout_url = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
	179	+ $checkout_url = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
180	180	$shared_secret = $this->get_option( 'shared_secret', '' );
181	181
182	182	$create = array();
		@@ -184,10 +184,10 @@ discard block
		block discarded – undo
184	184	$organization = EDUAPIHelper()->GetOrganization();
185	185	$purchase_country = $organization["CountryCode"];
186	186
187		- if ( ! empty( $ebi->Customer["CountryCode"] ) ) {
	187	+ if ( !empty( $ebi->Customer["CountryCode"] ) ) {
188	188	$purchase_country = $ebi->Customer["CountryCode"];
189	189
190		- if ( ! empty( $ebi->Customer["BillingInfo"]["CountryCode"] ) ) {
	190	+ if ( !empty( $ebi->Customer["BillingInfo"]["CountryCode"] ) ) {
191	191	$purchase_country = $ebi->Customer["BillingInfo"]["CountryCode"];
192	192	}
193	193	}
		@@ -209,14 +209,14 @@ discard block
		block discarded – undo
209	209
210	210	$_event = null;
211	211
212		- if ( ! empty( $ebi->EventBooking['BookingId'] ) ) {
	212	+ if ( !empty( $ebi->EventBooking['BookingId'] ) ) {
213	213	$booking_id = intval( $ebi->EventBooking['BookingId'] );
214	214	$reference_id = $booking_id;
215	215
216	216	$_event = EDUAPI()->OData->Events->GetItem( $ebi->EventBooking['EventId'] );
217	217	}
218	218
219		- if ( ! empty( $ebi->EventBooking['ProgrammeBookingId'] ) ) {
	219	+ if ( !empty( $ebi->EventBooking['ProgrammeBookingId'] ) ) {
220	220	$programme_booking_id = intval( $ebi->EventBooking['ProgrammeBookingId'] );
221	221	$reference_id = $programme_booking_id;
222	222
		@@ -226,15 +226,15 @@ discard block
		block discarded – undo
226	226	$rowExtraInfo = "";
227	227
228	228	if ( null != $_event ) {
229		- if ( ! empty( $_event['City'] ) ) {
	229	+ if ( !empty( $_event['City'] ) ) {
230	230	$rowExtraInfo .= ';' . $_event['City'];
231	231	}
232	232
233		- if ( ! empty( $_event['StartDate'] ) ) {
	233	+ if ( !empty( $_event['StartDate'] ) ) {
234	234	$rowExtraInfo .= ';' . date( "Y-m-d", strtotime( $_event['StartDate'] ) );
235	235	}
236	236
237		- if ( ! empty( $_event['EndDate'] ) ) {
	237	+ if ( !empty( $_event['EndDate'] ) ) {
238	238	$rowExtraInfo .= ';' . date( "Y-m-d", strtotime( $_event['EndDate'] ) );
239	239	}
240	240	}
		@@ -281,7 +281,7 @@ discard block
		block discarded – undo
281	281	$cart_item['name'] = $order_row['Description'] . $rowExtraInfo;
282	282	$cart_item['quantity'] = intval( $order_row['Quantity'] );
283	283
284		- if ( ! $order_row['PriceIncVat'] ) {
	284	+ if ( !$order_row['PriceIncVat'] ) {
285	285	$price_per_unit = $order_row['PricePerUnit'] * ( 1 + ( $order_row['VatPercent'] / 100 ) ) * 100;
286	286	} else {
287	287	$price_per_unit = $order_row['PricePerUnit'] * 100;
		@@ -318,8 +318,8 @@ discard block
		block discarded – undo
318	318	}
319	319
320	320	public function process_paymentstatus() {
321		- if ( ! empty( $_GET['klarna_order_id'] ) && ! empty( $_GET['status'] ) ) {
322		- $checkout_url = ! checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
	321	+ if ( !empty( $_GET['klarna_order_id'] ) && !empty( $_GET['status'] ) ) {
	322	+ $checkout_url = !checked( $this->get_option( 'test_mode', 'no' ), '1', false ) ? Klarna_Checkout_Connector::BASE_URL : Klarna_Checkout_Connector::BASE_TEST_URL;
323	323	$shared_secret = $this->get_option( 'shared_secret', '' );
324	324
325	325	try {

MultinetInteractive / eduadmin-wp-klarna-checkout

Push — master ( 5c12d3...f60c20 )

Status

Category

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch