Fix CS

Payum · May 3, 2021 · 8331801 · 8331801
1 parent c409611
commit 8331801
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 11 deletions.
diff --git a/src/Payum/Sofort/Action/Api/CreateTransactionAction.php b/src/Payum/Sofort/Action/Api/CreateTransactionAction.php
@@ -37,10 +37,12 @@ public function execute($request)
  }
 
  $fieldsRequired = ['amount', 'currency_code', 'reason', 'success_url'];
- if ($this->api->getOption('disable_notification') == false) {
- $fieldsRequired[] = 'notification_url';
- }
- $details->validateNotEmpty($fieldsRequired);
+
+ if ($this->api->getOption('disable_notification') == false) {
+ $fieldsRequired[] = 'notification_url';
+ }
+
+ $details->validateNotEmpty($fieldsRequired);
 
  $details->replace($this->api->createTransaction((array) $details));
 

diff --git a/src/Payum/Sofort/Api.php b/src/Payum/Sofort/Api.php
@@ -164,13 +164,14 @@ public function refundTransaction(array $fields)
 
  return $fields;
  }
- 
+
  /**
- * @param string $optionName
- *
- * @return mixed
- */
- public function getOption(string $optionName){
- return $this->options[$optionName];
+ * @param string $optionName
+ *
+ * @return mixed
+ */
+ public function getOption(string $optionName)
+ {
+ return $this->options[$optionName] ?? null;
  }
 }