[bisq-network/bisq] Coin formatter interface (#3634)

Justin Carter notifications at github.com
Tue Nov 19 14:56:59 UTC 2019 

Last step in my ongoing effort to improve the design around the `BSFormatter` class. Possible next steps are to improve the related `BsqFormatter` as well. But that would be another task.

Tracking issue: https://github.com/bisq-network/bisq/issues/3240
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/3634

-- Commit Summary --

  * Remove dead code from BSFormatter
  * Extract CoinFormatter interface
  * Use interface instead of class name for CoinFormatter
  * Rename BSFormatter to ImmutableCoinFormatter
  * Move BsqFormatter and ImmutableCoinFormatter under .coin package
  * Remove dead code
  * Inject CoinFormatter via BTC_FORMATTER_KEY

-- File Changes --

    M core/src/main/java/bisq/core/CoreModule.java (6)
    M core/src/main/java/bisq/core/app/BisqSetup.java (8)
    M core/src/main/java/bisq/core/app/WalletAppSetup.java (7)
    M core/src/main/java/bisq/core/dao/governance/asset/AssetService.java (2)
    M core/src/main/java/bisq/core/dao/governance/proposal/param/ChangeParamValidator.java (2)
    M core/src/main/java/bisq/core/dao/presentation/DaoUtil.java (6)
    M core/src/main/java/bisq/core/dao/state/DaoStateService.java (2)
    M core/src/main/java/bisq/core/notifications/alerts/market/MarketAlerts.java (7)
    M core/src/main/java/bisq/core/notifications/alerts/price/PriceAlert.java (5)
    M core/src/main/java/bisq/core/offer/CreateOfferService.java (2)
    M core/src/main/java/bisq/core/offer/OfferUtil.java (8)
    M core/src/main/java/bisq/core/presentation/BalancePresentation.java (6)
    M core/src/main/java/bisq/core/util/FormattingUtils.java (2)
    M core/src/main/java/bisq/core/util/ParsingUtils.java (5)
    R core/src/main/java/bisq/core/util/coin/BsqFormatter.java (75)
    A core/src/main/java/bisq/core/util/coin/CoinFormatter.java (20)
    R core/src/main/java/bisq/core/util/coin/CoinUtil.java (2)
    R core/src/main/java/bisq/core/util/coin/ImmutableCoinFormatter.java (56)
    M core/src/test/java/bisq/core/dao/governance/proposal/param/ChangeParamValidatorTest.java (2)
    M core/src/test/java/bisq/core/dao/state/DaoStateServiceTest.java (2)
    M core/src/test/java/bisq/core/util/CoinCryptoUtilsTest.java (2)
    M desktop/src/main/java/bisq/desktop/components/BalanceTextField.java (6)
    M desktop/src/main/java/bisq/desktop/components/BalanceWithConfirmationTextField.java (6)
    M desktop/src/main/java/bisq/desktop/components/PeerInfoIconSmall.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/AdvancedCashForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/AliPayForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/AssetsForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/BankForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/CashDepositForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/ChaseQuickPayForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/ClearXchangeForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/F2FForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/FasterPaymentsForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/GeneralAccountNumberForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/GeneralBankForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/GeneralSepaForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/HalCashForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/InteracETransferForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/JapanBankTransferForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/MoneyBeamForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/MoneyGramForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/NationalBankForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/PaymentMethodForm.java (6)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/PerfectMoneyForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/PopmoneyForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/PromptPayForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/RevolutForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/SameBankForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/SepaForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/SepaInstantForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/SpecificBankForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/SwishForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/USPostalMoneyOrderForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/UpholdForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/WeChatPayForm.java (4)
    M desktop/src/main/java/bisq/desktop/components/paymentmethods/WesternUnionForm.java (4)
    M desktop/src/main/java/bisq/desktop/main/MainView.java (8)
    M desktop/src/main/java/bisq/desktop/main/account/content/altcoinaccounts/AltCoinAccountsView.java (8)
    M desktop/src/main/java/bisq/desktop/main/account/content/fiataccounts/FiatAccountsView.java (8)
    M desktop/src/main/java/bisq/desktop/main/account/content/notifications/ManageMarketAlertsWindow.java (6)
    M desktop/src/main/java/bisq/desktop/main/account/content/notifications/MobileNotificationsView.java (7)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/BondingViewUtils.java (13)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/bonds/BondListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/bonds/BondsView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/reputation/MyReputationListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/reputation/MyReputationView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/roles/RoleDetailsWindow.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/bonding/roles/RolesView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/burnbsq/assetfee/AssetFeeView.java (10)
    M desktop/src/main/java/bisq/desktop/main/dao/burnbsq/assetfee/AssetListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/burnbsq/proofofburn/MyProofOfBurnListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/burnbsq/proofofburn/ProofOfBurnListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/burnbsq/proofofburn/ProofOfBurnView.java (10)
    M desktop/src/main/java/bisq/desktop/main/dao/economy/dashboard/BsqDashboardView.java (3)
    M desktop/src/main/java/bisq/desktop/main/dao/economy/supply/SupplyView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/ProposalDisplay.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/dashboard/GovernanceDashboardView.java (8)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/make/MakeProposalView.java (10)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/proposals/ProposalsListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/proposals/ProposalsView.java (10)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/result/CycleListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/result/ProposalListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/result/VoteListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/governance/result/VoteResultView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/news/NewsView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/wallet/BsqBalanceUtil.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/wallet/receive/BsqReceiveView.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/wallet/send/BsqSendView.java (16)
    M desktop/src/main/java/bisq/desktop/main/dao/wallet/tx/BsqTxListItem.java (2)
    M desktop/src/main/java/bisq/desktop/main/dao/wallet/tx/BsqTxView.java (2)
    M desktop/src/main/java/bisq/desktop/main/funds/deposit/DepositListItem.java (4)
    M desktop/src/main/java/bisq/desktop/main/funds/deposit/DepositView.java (8)
    M desktop/src/main/java/bisq/desktop/main/funds/locked/LockedListItem.java (6)
    M desktop/src/main/java/bisq/desktop/main/funds/locked/LockedView.java (8)
    M desktop/src/main/java/bisq/desktop/main/funds/reserved/ReservedListItem.java (6)
    M desktop/src/main/java/bisq/desktop/main/funds/reserved/ReservedView.java (8)
    M desktop/src/main/java/bisq/desktop/main/funds/transactions/TransactionListItemFactory.java (8)
    M desktop/src/main/java/bisq/desktop/main/funds/transactions/TransactionsListItem.java (6)
    M desktop/src/main/java/bisq/desktop/main/funds/transactions/TransactionsView.java (6)
    M desktop/src/main/java/bisq/desktop/main/funds/withdrawal/WithdrawalListItem.java (6)
    M desktop/src/main/java/bisq/desktop/main/funds/withdrawal/WithdrawalView.java (10)
    M desktop/src/main/java/bisq/desktop/main/market/MarketView.java (7)
    M desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartView.java (6)
    M desktop/src/main/java/bisq/desktop/main/market/offerbook/OfferBookChartViewModel.java (9)
    M desktop/src/main/java/bisq/desktop/main/market/spread/SpreadView.java (8)
    M desktop/src/main/java/bisq/desktop/main/market/spread/SpreadViewModel.java (8)
    M desktop/src/main/java/bisq/desktop/main/market/trades/TradesChartsView.java (7)
    M desktop/src/main/java/bisq/desktop/main/market/trades/TradesChartsViewModel.java (9)
    M desktop/src/main/java/bisq/desktop/main/offer/MutableOfferDataModel.java (11)
    M desktop/src/main/java/bisq/desktop/main/offer/MutableOfferView.java (8)
    M desktop/src/main/java/bisq/desktop/main/offer/MutableOfferViewModel.java (13)
    M desktop/src/main/java/bisq/desktop/main/offer/createoffer/CreateOfferDataModel.java (7)
    M desktop/src/main/java/bisq/desktop/main/offer/createoffer/CreateOfferView.java (9)
    M desktop/src/main/java/bisq/desktop/main/offer/createoffer/CreateOfferViewModel.java (9)
    M desktop/src/main/java/bisq/desktop/main/offer/offerbook/OfferBookView.java (7)
    M desktop/src/main/java/bisq/desktop/main/offer/offerbook/OfferBookViewModel.java (10)
    M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferDataModel.java (2)
    M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferView.java (8)
    M desktop/src/main/java/bisq/desktop/main/offer/takeoffer/TakeOfferViewModel.java (13)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/ContractWindow.java (7)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/DisputeSummaryWindow.java (9)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/EmptyWalletWindow.java (12)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/OfferDetailsWindow.java (7)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/ProposalResultsWindow.java (2)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/SelectProposalWindow.java (2)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/SignPaymentAccountsWindow.java (1)
    M desktop/src/main/java/bisq/desktop/main/overlays/windows/TradeDetailsWindow.java (7)
    M desktop/src/main/java/bisq/desktop/main/portfolio/closedtrades/ClosedTradesView.java (7)
    M desktop/src/main/java/bisq/desktop/main/portfolio/closedtrades/ClosedTradesViewModel.java (8)
    M desktop/src/main/java/bisq/desktop/main/portfolio/editoffer/EditOfferDataModel.java (9)
    M desktop/src/main/java/bisq/desktop/main/portfolio/editoffer/EditOfferView.java (9)
    M desktop/src/main/java/bisq/desktop/main/portfolio/editoffer/EditOfferViewModel.java (8)
    M desktop/src/main/java/bisq/desktop/main/portfolio/failedtrades/FailedTradesViewModel.java (8)
    M desktop/src/main/java/bisq/desktop/main/portfolio/openoffer/OpenOffersViewModel.java (10)
    M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java (6)
    M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java (10)
    M desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep4View.java (6)
    M desktop/src/main/java/bisq/desktop/main/presentation/MarketPricePresentation.java (7)
    M desktop/src/main/java/bisq/desktop/main/settings/network/NetworkSettingsView.java (7)
    M desktop/src/main/java/bisq/desktop/main/settings/network/P2pNetworkListItem.java (6)
    M desktop/src/main/java/bisq/desktop/main/settings/preferences/PreferencesView.java (6)
    M desktop/src/main/java/bisq/desktop/main/shared/ChatView.java (6)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/DisputeView.java (6)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/agent/DisputeAgentView.java (4)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/agent/arbitration/ArbitratorView.java (5)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/agent/mediation/MediatorView.java (5)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/agent/refund/RefundAgentView.java (5)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/client/DisputeClientView.java (4)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/client/arbitration/ArbitrationClientView.java (5)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/client/mediation/MediationClientView.java (5)
    M desktop/src/main/java/bisq/desktop/main/support/dispute/client/refund/RefundClientView.java (5)
    M desktop/src/main/java/bisq/desktop/util/DisplayUtils.java (30)
    M desktop/src/main/java/bisq/desktop/util/GUIUtil.java (12)
    M desktop/src/main/java/bisq/desktop/util/validation/BsqAddressValidator.java (2)
    M desktop/src/main/java/bisq/desktop/util/validation/BsqValidator.java (4)
    M desktop/src/main/java/bisq/desktop/util/validation/BtcValidator.java (8)
    M desktop/src/main/java/bisq/desktop/util/validation/SecurityDepositValidator.java (1)
    M desktop/src/test/java/bisq/desktop/GuiceSetupTest.java (4)
    M desktop/src/test/java/bisq/desktop/main/market/offerbook/OfferBookChartViewModelTest.java (26)
    M desktop/src/test/java/bisq/desktop/main/market/spread/SpreadViewModelTest.java (12)
    M desktop/src/test/java/bisq/desktop/main/market/trades/TradesChartsViewModelTest.java (4)
    M desktop/src/test/java/bisq/desktop/main/offer/createoffer/CreateOfferViewModelTest.java (14)
    M desktop/src/test/java/bisq/desktop/main/offer/offerbook/OfferBookViewModelTest.java (29)
    M desktop/src/test/java/bisq/desktop/main/portfolio/editoffer/EditOfferDataModelTest.java (8)
    M desktop/src/test/java/bisq/desktop/util/DisplayUtilsTest.java (6)
    R desktop/src/test/java/bisq/desktop/util/ImmutableCoinFormatterTest.java (13)
    M desktop/src/test/java/bisq/desktop/util/validation/BtcValidatorTest.java (7)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/3634.patch
https://github.com/bisq-network/bisq/pull/3634.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/3634
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20191119/6915adab/attachment-0001.html>

More information about the bisq-github mailing list