diff --git a/.ci/phpmd/composer.lock b/.ci/phpmd/composer.lock
index 1828f38202..50f1d6138e 100644
--- a/.ci/phpmd/composer.lock
+++ b/.ci/phpmd/composer.lock
@@ -396,16 +396,16 @@
},
{
"name": "symfony/config",
- "version": "v6.3.2",
+ "version": "v6.3.8",
"source": {
"type": "git",
"url": "https://github.com/symfony/config.git",
- "reference": "b47ca238b03e7b0d7880ffd1cf06e8d637ca1467"
+ "reference": "b7a63887960359e5b59b15826fa9f9be10acbe88"
},
"dist": {
"type": "zip",
- "url": "https://api.github.com/repos/symfony/config/zipball/b47ca238b03e7b0d7880ffd1cf06e8d637ca1467",
- "reference": "b47ca238b03e7b0d7880ffd1cf06e8d637ca1467",
+ "url": "https://api.github.com/repos/symfony/config/zipball/b7a63887960359e5b59b15826fa9f9be10acbe88",
+ "reference": "b7a63887960359e5b59b15826fa9f9be10acbe88",
"shasum": ""
},
"require": {
@@ -451,7 +451,7 @@
"description": "Helps you find, load, combine, autofill and validate configuration values of any kind",
"homepage": "https://symfony.com",
"support": {
- "source": "https://github.com/symfony/config/tree/v6.3.2"
+ "source": "https://github.com/symfony/config/tree/v6.3.8"
},
"funding": [
{
@@ -467,20 +467,20 @@
"type": "tidelift"
}
],
- "time": "2023-07-19T20:22:16+00:00"
+ "time": "2023-11-09T08:28:21+00:00"
},
{
"name": "symfony/dependency-injection",
- "version": "v6.3.5",
+ "version": "v6.3.8",
"source": {
"type": "git",
"url": "https://github.com/symfony/dependency-injection.git",
- "reference": "2ed62b3bf98346e1f45529a7b6be2196739bb993"
+ "reference": "1f30f545c4151f611148fc19e28d54d39e0a00bc"
},
"dist": {
"type": "zip",
- "url": "https://api.github.com/repos/symfony/dependency-injection/zipball/2ed62b3bf98346e1f45529a7b6be2196739bb993",
- "reference": "2ed62b3bf98346e1f45529a7b6be2196739bb993",
+ "url": "https://api.github.com/repos/symfony/dependency-injection/zipball/1f30f545c4151f611148fc19e28d54d39e0a00bc",
+ "reference": "1f30f545c4151f611148fc19e28d54d39e0a00bc",
"shasum": ""
},
"require": {
@@ -532,7 +532,7 @@
"description": "Allows you to standardize and centralize the way objects are constructed in your application",
"homepage": "https://symfony.com",
"support": {
- "source": "https://github.com/symfony/dependency-injection/tree/v6.3.5"
+ "source": "https://github.com/symfony/dependency-injection/tree/v6.3.8"
},
"funding": [
{
@@ -548,11 +548,11 @@
"type": "tidelift"
}
],
- "time": "2023-09-25T16:46:40+00:00"
+ "time": "2023-10-31T08:07:48+00:00"
},
{
"name": "symfony/deprecation-contracts",
- "version": "v3.3.0",
+ "version": "v3.4.0",
"source": {
"type": "git",
"url": "https://github.com/symfony/deprecation-contracts.git",
@@ -599,7 +599,7 @@
"description": "A generic function and convention to trigger deprecation notices",
"homepage": "https://symfony.com",
"support": {
- "source": "https://github.com/symfony/deprecation-contracts/tree/v3.3.0"
+ "source": "https://github.com/symfony/deprecation-contracts/tree/v3.4.0"
},
"funding": [
{
@@ -847,16 +847,16 @@
},
{
"name": "symfony/service-contracts",
- "version": "v3.3.0",
+ "version": "v3.4.0",
"source": {
"type": "git",
"url": "https://github.com/symfony/service-contracts.git",
- "reference": "40da9cc13ec349d9e4966ce18b5fbcd724ab10a4"
+ "reference": "b3313c2dbffaf71c8de2934e2ea56ed2291a3838"
},
"dist": {
"type": "zip",
- "url": "https://api.github.com/repos/symfony/service-contracts/zipball/40da9cc13ec349d9e4966ce18b5fbcd724ab10a4",
- "reference": "40da9cc13ec349d9e4966ce18b5fbcd724ab10a4",
+ "url": "https://api.github.com/repos/symfony/service-contracts/zipball/b3313c2dbffaf71c8de2934e2ea56ed2291a3838",
+ "reference": "b3313c2dbffaf71c8de2934e2ea56ed2291a3838",
"shasum": ""
},
"require": {
@@ -909,7 +909,7 @@
"standards"
],
"support": {
- "source": "https://github.com/symfony/service-contracts/tree/v3.3.0"
+ "source": "https://github.com/symfony/service-contracts/tree/v3.4.0"
},
"funding": [
{
@@ -925,7 +925,7 @@
"type": "tidelift"
}
],
- "time": "2023-05-23T14:45:45+00:00"
+ "time": "2023-07-30T20:28:31+00:00"
},
{
"name": "symfony/var-exporter",
diff --git a/.ci/phpmd/phpmd.xml b/.ci/phpmd/phpmd.xml
index 7f94b2dba2..42328dc44f 100644
--- a/.ci/phpmd/phpmd.xml
+++ b/.ci/phpmd/phpmd.xml
@@ -32,10 +32,22 @@
-
+
+
+
+
+
+
+
+
+
+
+
+
+
@@ -55,7 +67,7 @@
-
+
@@ -67,7 +79,7 @@
-
+
diff --git a/app/Repositories/UserGroups/Budget/AvailableBudgetRepository.php b/app/Repositories/UserGroups/Budget/AvailableBudgetRepository.php
index 40555a096a..cfadc6e422 100644
--- a/app/Repositories/UserGroups/Budget/AvailableBudgetRepository.php
+++ b/app/Repositories/UserGroups/Budget/AvailableBudgetRepository.php
@@ -59,7 +59,7 @@ class AvailableBudgetRepository implements AvailableBudgetRepositoryInterface
'currency_code' => $availableBudget->transactionCurrency->code,
'currency_symbol' => $availableBudget->transactionCurrency->symbol,
'currency_name' => $availableBudget->transactionCurrency->name,
- 'currency_decimal_places' => (int)$availableBudget->transactionCurrency->decimal_places,
+ 'currency_decimal_places' => $availableBudget->transactionCurrency->decimal_places,
'native_id' => $default->id,
'native_code' => $default->code,
'native_symbol' => $default->symbol,
diff --git a/app/Services/Internal/Support/CreditRecalculateService.php b/app/Services/Internal/Support/CreditRecalculateService.php
index 6a06a40982..63d2a98431 100644
--- a/app/Services/Internal/Support/CreditRecalculateService.php
+++ b/app/Services/Internal/Support/CreditRecalculateService.php
@@ -197,7 +197,7 @@ class CreditRecalculateService
$startOfDebt = $this->repository->getOpeningBalanceAmount($account) ?? '0';
$leftOfDebt = app('steam')->positive($startOfDebt);
$currency = $this->repository->getAccountCurrency($account);
- $decimals = (int)($currency?->decimal_places ?? 2);
+ $decimals = $currency?->decimal_places ?? 2;
app('log')->debug(sprintf('Start of debt is "%s", so initial left of debt is "%s"', app('steam')->bcround($startOfDebt, $decimals), app('steam')->bcround($leftOfDebt, $decimals)));
/** @var AccountMetaFactory $factory */
@@ -274,7 +274,7 @@ class CreditRecalculateService
$foreignCurrency = $transaction->foreignCurrency;
$accountCurrency = $this->repository->getAccountCurrency($account);
$groupId = $journal->transaction_group_id;
- $decimals = (int)$accountCurrency->decimal_places;
+ $decimals = $accountCurrency->decimal_places;
$type = $journal->transactionType->type;
/** @var Transaction $destTransaction */
$destTransaction = $journal->transactions()->where('amount', '>', '0')->first();
diff --git a/app/Support/Amount.php b/app/Support/Amount.php
index 9055f79117..6d08315871 100644
--- a/app/Support/Amount.php
+++ b/app/Support/Amount.php
@@ -51,7 +51,7 @@ class Amount
*/
public function formatAnything(TransactionCurrency $format, string $amount, bool $coloured = null): string
{
- return $this->formatFlat($format->symbol, (int)$format->decimal_places, $amount, $coloured);
+ return $this->formatFlat($format->symbol, $format->decimal_places, $amount, $coloured);
}
/**
diff --git a/app/Transformers/RecurrenceTransformer.php b/app/Transformers/RecurrenceTransformer.php
index a455f337da..e7e9a1aa84 100644
--- a/app/Transformers/RecurrenceTransformer.php
+++ b/app/Transformers/RecurrenceTransformer.php
@@ -174,7 +174,7 @@ class RecurrenceTransformer extends AbstractTransformer
$foreignCurrencyId = (int)$transaction->foreign_currency_id;
$foreignCurrencyCode = $transaction->foreignCurrency->code;
$foreignCurrencySymbol = $transaction->foreignCurrency->symbol;
- $foreignCurrencyDp = (int)$transaction->foreignCurrency->decimal_places;
+ $foreignCurrencyDp = $transaction->foreignCurrency->decimal_places;
}
// source info:
@@ -208,7 +208,7 @@ class RecurrenceTransformer extends AbstractTransformer
'currency_id' => (string)$transaction->transaction_currency_id,
'currency_code' => $transaction->transactionCurrency->code,
'currency_symbol' => $transaction->transactionCurrency->symbol,
- 'currency_decimal_places' => (int)$transaction->transactionCurrency->decimal_places,
+ 'currency_decimal_places' => $transaction->transactionCurrency->decimal_places,
'foreign_currency_id' => null === $foreignCurrencyId ? null : (string)$foreignCurrencyId,
'foreign_currency_code' => $foreignCurrencyCode,
'foreign_currency_symbol' => $foreignCurrencySymbol,
diff --git a/app/Transformers/V2/AccountTransformer.php b/app/Transformers/V2/AccountTransformer.php
index f8aab9b8ac..262ea013c3 100644
--- a/app/Transformers/V2/AccountTransformer.php
+++ b/app/Transformers/V2/AccountTransformer.php
@@ -149,7 +149,7 @@ class AccountTransformer extends AbstractTransformer
'native_currency_id' => (string)$this->default->id,
'native_currency_code' => $this->default->code,
'native_currency_symbol' => $this->default->symbol,
- 'native_currency_decimal_places' => (int)$this->default->decimal_places,
+ 'native_currency_decimal_places' => $this->default->decimal_places,
// balance:
'current_balance' => $balance,
diff --git a/app/Transformers/V2/BillTransformer.php b/app/Transformers/V2/BillTransformer.php
index d5119483f5..887bdbd2fd 100644
--- a/app/Transformers/V2/BillTransformer.php
+++ b/app/Transformers/V2/BillTransformer.php
@@ -136,7 +136,7 @@ class BillTransformer extends AbstractTransformer
$foreignCurrencyCode = $currencies[$foreignCurrencyId]->code;
$foreignCurrencyName = $currencies[$foreignCurrencyId]->name;
$foreignCurrencySymbol = $currencies[$foreignCurrencyId]->symbol;
- $foreignCurrencyDp = (int)$currencies[$foreignCurrencyId]->decimal_places;
+ $foreignCurrencyDp = $currencies[$foreignCurrencyId]->decimal_places;
}
$this->paidDates[$billId][] = [
@@ -147,11 +147,11 @@ class BillTransformer extends AbstractTransformer
'currency_code' => $currencies[$currencyId]->code,
'currency_name' => $currencies[$currencyId]->name,
'currency_symbol' => $currencies[$currencyId]->symbol,
- 'currency_decimal_places' => (int)$currencies[$currencyId]->decimal_places,
+ 'currency_decimal_places' => $currencies[$currencyId]->decimal_places,
'native_currency_id' => $currencies[$currencyId]->id,
'native_currency_code' => $currencies[$currencyId]->code,
'native_currency_symbol' => $currencies[$currencyId]->symbol,
- 'native_currency_decimal_places' => (int)$currencies[$currencyId]->decimal_places,
+ 'native_currency_decimal_places' => $currencies[$currencyId]->decimal_places,
'foreign_currency_id' => $foreignCurrencyId,
'foreign_currency_code' => $foreignCurrencyCode,
'foreign_currency_name' => $foreignCurrencyName,
@@ -212,7 +212,7 @@ class BillTransformer extends AbstractTransformer
'native_currency_code' => $this->default->code,
'native_currency_name' => $this->default->name,
'native_currency_symbol' => $this->default->symbol,
- 'native_currency_decimal_places' => (int)$this->default->decimal_places,
+ 'native_currency_decimal_places' => $this->default->decimal_places,
'date' => $bill->date->toAtomString(),
'end_date' => $bill->end_date?->toAtomString(),
'extension_date' => $bill->extension_date?->toAtomString(),
diff --git a/app/Transformers/V2/PiggyBankTransformer.php b/app/Transformers/V2/PiggyBankTransformer.php
index 98dceed838..a6e200f6c7 100644
--- a/app/Transformers/V2/PiggyBankTransformer.php
+++ b/app/Transformers/V2/PiggyBankTransformer.php
@@ -212,7 +212,7 @@ class PiggyBankTransformer extends AbstractTransformer
'native_currency_id' => (string)$this->default->id,
'native_currency_code' => $this->default->code,
'native_currency_symbol' => $this->default->symbol,
- 'native_currency_decimal_places' => (int)$this->default->decimal_places,
+ 'native_currency_decimal_places' => $this->default->decimal_places,
'current_amount' => $currentAmount,
'native_current_amount' => $nativeCurrentAmount,
'target_amount' => $targetAmount,
diff --git a/app/Transformers/V2/TransactionGroupTransformer.php b/app/Transformers/V2/TransactionGroupTransformer.php
index f87bb81a92..1b297cdca5 100644
--- a/app/Transformers/V2/TransactionGroupTransformer.php
+++ b/app/Transformers/V2/TransactionGroupTransformer.php
@@ -193,7 +193,7 @@ class TransactionGroupTransformer extends AbstractTransformer
'native_currency_code' => $this->default->code,
'native_currency_name' => $this->default->name,
'native_currency_symbol' => $this->default->symbol,
- 'native_currency_decimal_places' => (int)$this->default->decimal_places,
+ 'native_currency_decimal_places' => $this->default->decimal_places,
// foreign currency amount:
'foreign_currency_id' => $this->stringFromArray($transaction, 'foreign_currency_id', null),