diff --git a/app/Factory/TransactionJournalFactory.php b/app/Factory/TransactionJournalFactory.php index 63a26b31e9..92f18e7dd0 100644 --- a/app/Factory/TransactionJournalFactory.php +++ b/app/Factory/TransactionJournalFactory.php @@ -596,8 +596,8 @@ class TransactionJournalFactory $this->accountValidator->setTransactionType($transactionType); // validate source account. - $sourceId = property_exists($data, 'source_id') ? (int) $data['source_id'] : null; - $sourceName = property_exists($data, 'source_name') ? (string) $data['source_name'] : null; + $sourceId = $data['source_id'] ? (int) $data['source_id'] : null; + $sourceName = $data['source_name'] ? (string) $data['source_name'] : null; $validSource = $this->accountValidator->validateSource($sourceId, $sourceName, null); // do something with result: @@ -606,8 +606,8 @@ class TransactionJournalFactory } Log::debug('Source seems valid.'); // validate destination account - $destinationId = property_exists($data, 'destination_id') ? (int) $data['destination_id'] : null; - $destinationName = property_exists($data, 'destination_name') ? (string) $data['destination_name'] : null; + $destinationId = $data['destination_id'] ? (int) $data['destination_id'] : null; + $destinationName = $data['destination_name'] ? (string) $data['destination_name'] : null; $validDestination = $this->accountValidator->validateDestination($destinationId, $destinationName, null); // do something with result: if (false === $validDestination) {