Browse Source

Task/improve validation of currency management in Admin Control panel (#5973)

* Improve validation (disallow GBp)

* Update changelog
pull/5018/merge
Thomas Kaul 7 days ago
committed by GitHub
parent
commit
8c989c112b
No known key found for this signature in database GPG Key ID: B5690EEEBB952194
  1. 1
      CHANGELOG.md
  2. 10
      apps/client/src/app/components/admin-market-data/create-asset-profile-dialog/create-asset-profile-dialog.component.ts

1
CHANGELOG.md

@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Changed
- Disabled the action to delete activities if the activities table is empty
- Improved the validation of the currency management in the admin control panel
- Resolved the data source of the `GHOSTFOLIO` data provider in the export functionality
- Resolved the data source of the `GHOSTFOLIO` data provider in the import functionality
- Refreshed the cryptocurrencies list

10
apps/client/src/app/components/admin-market-data/create-asset-profile-dialog/create-asset-profile-dialog.component.ts

@ -107,9 +107,8 @@ export class GfCreateAssetProfileDialogComponent implements OnDestroy, OnInit {
symbol: this.createAssetProfileForm.get('searchSymbol').value.symbol
});
} else if (this.mode === 'currency') {
const currency = (
this.createAssetProfileForm.get('addCurrency').value as string
).toUpperCase();
const currency = this.createAssetProfileForm.get('addCurrency')
.value as string;
const currencies = Array.from(
new Set([...this.customCurrencies, currency])
@ -201,7 +200,10 @@ export class GfCreateAssetProfileDialogComponent implements OnDestroy, OnInit {
private iso4217CurrencyCodeValidator(): ValidatorFn {
return (control: AbstractControl): ValidationErrors | null => {
if (!isISO4217CurrencyCode(control.value?.toUpperCase())) {
if (
control.value !== control.value?.toUpperCase() ||
!isISO4217CurrencyCode(control.value)
) {
return { invalidCurrency: true };
}

Loading…
Cancel
Save