From 3a3cec4f9ab9a68c93efa9e696791159a46eae5c Mon Sep 17 00:00:00 2001
From: James Cole <james@firefly-iii.org>
Date: Sat, 10 Apr 2021 07:57:04 +0200
Subject: [PATCH] Store direction.

---
 app/Factory/AccountFactory.php | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/Factory/AccountFactory.php b/app/Factory/AccountFactory.php
index e4d3394f34..40336ccaa4 100644
--- a/app/Factory/AccountFactory.php
+++ b/app/Factory/AccountFactory.php
@@ -238,6 +238,10 @@ class AccountFactory
         if ($account->accountType->type !== AccountType::ASSET) {
             $accountRole = '';
         }
+        // only liability may have direction:
+        if (array_key_exists('liability_direction', $data) && !in_array($account->accountType->type, config('firefly.valid_liabilities'), true)) {
+            $data['liability_direction'] = null;
+        }
 
         $data['account_role'] = $accountRole;
         $data['currency_id']  = $currency->id;
@@ -257,7 +261,7 @@ class AccountFactory
             $fields = $this->validAssetFields;
         }
         if ($account->accountType->type === AccountType::ASSET && 'ccAsset' === $data['account_role']) {
-            $fields = $this->validCCFields; 
+            $fields = $this->validCCFields;
         }
 
         /** @var AccountMetaFactory $factory */
@@ -269,10 +273,10 @@ class AccountFactory
 
                 // convert boolean value:
                 if (is_bool($data[$field]) && false === $data[$field]) {
-                    $data[$field] = 0; 
+                    $data[$field] = 0;
                 }
                 if (is_bool($data[$field]) && true === $data[$field]) {
-                    $data[$field] = 1; 
+                    $data[$field] = 1;
                 }
 
                 $factory->crud($account, $field, (string)$data[$field]);