diff --git a/src/main/java/io/bitsquare/gui/main/account/AccountModel.java b/src/main/java/io/bitsquare/gui/main/account/AccountModel.java index 75669899dd..e86ee55faa 100644 --- a/src/main/java/io/bitsquare/gui/main/account/AccountModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/AccountModel.java @@ -30,21 +30,11 @@ class AccountModel extends UIModel { private final User user; - - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private AccountModel(User user) { this.user = user; } - - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -69,11 +59,6 @@ class AccountModel extends UIModel { super.terminate(); } - - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - boolean getNeedRegistration() { return user.getAccountId() == null; diff --git a/src/main/java/io/bitsquare/gui/main/account/AccountPM.java b/src/main/java/io/bitsquare/gui/main/account/AccountPM.java index d26d84c1ee..7002e71fba 100644 --- a/src/main/java/io/bitsquare/gui/main/account/AccountPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/AccountPM.java @@ -28,20 +28,12 @@ class AccountPM extends PresentationModel { private static final Logger log = LoggerFactory.getLogger(AccountPM.class); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private AccountPM(AccountModel model) { super(model); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -67,10 +59,6 @@ class AccountPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - boolean getNeedRegistration() { return model.getNeedRegistration(); } diff --git a/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordModel.java b/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordModel.java index 7fb89bc26d..781a8dd664 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordModel.java @@ -27,19 +27,10 @@ import org.slf4j.LoggerFactory; class ChangePasswordModel extends UIModel { private static final Logger log = LoggerFactory.getLogger(ChangePasswordModel.class); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private ChangePasswordModel() { } - - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -64,11 +55,6 @@ class ChangePasswordModel extends UIModel { super.terminate(); } - - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") void savePassword(String password) { diff --git a/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordPM.java b/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordPM.java index 729f5fc3b8..ca5f1c7679 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/changepassword/ChangePasswordPM.java @@ -43,10 +43,6 @@ class ChangePasswordPM extends PresentationModel { final BooleanProperty saveButtonDisabled = new SimpleBooleanProperty(true); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private ChangePasswordPM(ChangePasswordModel model, PasswordValidator passwordValidator) { super(model); @@ -57,10 +53,6 @@ class ChangePasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -86,10 +78,6 @@ class ChangePasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - boolean requestSavePassword() { if (validate()) { model.savePassword(passwordField.get()); @@ -103,10 +91,6 @@ class ChangePasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private boolean validate() { InputValidator.ValidationResult result = passwordValidator.validate(passwordField.get()); if (result.isValid) { diff --git a/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountModel.java b/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountModel.java index adb9f9905a..37698fb8af 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountModel.java @@ -70,10 +70,6 @@ class FiatAccountModel extends UIModel { final ObservableList allRegions = FXCollections.observableArrayList(CountryUtil.getAllRegions()); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject FiatAccountModel(User user, Persistence persistence, AccountSettings accountSettings) { this.persistence = persistence; @@ -82,10 +78,6 @@ class FiatAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -112,10 +104,6 @@ class FiatAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - void saveBankAccount() { BankAccount bankAccount = new BankAccount(type.get(), currency.get(), @@ -168,19 +156,11 @@ class FiatAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getAllCountriesFor(Region selectedRegion) { return FXCollections.observableArrayList(CountryUtil.getAllCountriesFor(selectedRegion)); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setType(BankAccountType type) { this.type.set(type); @@ -203,10 +183,6 @@ class FiatAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private void reset() { title.set(null); holderName.set(null); diff --git a/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountPm.java b/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountPm.java index f0dddf5802..e7f15ba08d 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountPm.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/fiat/FiatAccountPm.java @@ -62,10 +62,6 @@ class FiatAccountPm extends PresentationModel { final ObjectProperty currency = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject FiatAccountPm(FiatAccountModel model, BankAccountNumberValidator bankAccountNumberValidator) { super(model); @@ -73,10 +69,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { // input @@ -123,10 +115,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - InputValidator.ValidationResult requestSaveBankAccount() { InputValidator.ValidationResult result = validateInput(); if (result.isValid) { @@ -148,10 +136,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Converters - /////////////////////////////////////////////////////////////////////////////////////////// - StringConverter getTypesConverter() { return new StringConverter() { @Override @@ -225,10 +209,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getAllTypes() { return model.allTypes; } @@ -258,10 +238,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setType(BankAccountType type) { model.setType(type); validateInput(); @@ -278,10 +254,6 @@ class FiatAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private void applyAllBankAccounts() { if (model.allBankAccounts.isEmpty()) { selectionPrompt.set("No bank account available"); diff --git a/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountModel.java b/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountModel.java index c2edd19365..d5b9e30f3f 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountModel.java @@ -72,10 +72,6 @@ class IrcAccountModel extends UIModel { final ObservableList allBankAccounts = FXCollections.observableArrayList(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject IrcAccountModel(User user, Persistence persistence, AccountSettings accountSettings, MessageService messageService) { @@ -86,10 +82,6 @@ class IrcAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -118,10 +110,6 @@ class IrcAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - void saveBankAccount() { BankAccount bankAccount = new BankAccount(type.get(), currency.get(), @@ -137,19 +125,11 @@ class IrcAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getAllCountriesFor(Region selectedRegion) { return FXCollections.observableArrayList(CountryUtil.getAllCountriesFor(selectedRegion)); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setType(BankAccountType type) { this.type.set(type); } @@ -159,10 +139,6 @@ class IrcAccountModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private void reset() { nickName.set(null); diff --git a/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountPm.java b/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountPm.java index 70af7ad6cc..45a1a96d62 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountPm.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/irc/IrcAccountPm.java @@ -51,10 +51,6 @@ class IrcAccountPm extends PresentationModel { final ObjectProperty currency = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject IrcAccountPm(IrcAccountModel model, BankAccountNumberValidator nickNameValidator) { super(model); @@ -62,10 +58,6 @@ class IrcAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { // input @@ -96,10 +88,6 @@ class IrcAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - InputValidator.ValidationResult requestSaveBankAccount() { InputValidator.ValidationResult result = validateInput(); if (result.isValid) { @@ -112,10 +100,6 @@ class IrcAccountPm extends PresentationModel { return model.allBankAccounts; } - /////////////////////////////////////////////////////////////////////////////////////////// - // Converters - /////////////////////////////////////////////////////////////////////////////////////////// - StringConverter getTypesConverter() { return new StringConverter() { @Override @@ -149,10 +133,6 @@ class IrcAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getAllTypes() { return model.allTypes; } @@ -166,10 +146,6 @@ class IrcAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setType(BankAccountType type) { model.setType(type); validateInput(); @@ -181,10 +157,6 @@ class IrcAccountPm extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private InputValidator.ValidationResult validateInput() { InputValidator.ValidationResult result = nickNameValidator.validate(model.nickName.get()); diff --git a/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordModel.java b/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordModel.java index 0bdd5da15b..12cc8bd54a 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordModel.java @@ -28,19 +28,11 @@ class PasswordModel extends UIModel { private static final Logger log = LoggerFactory.getLogger(PasswordModel.class); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private PasswordModel() { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -66,10 +58,6 @@ class PasswordModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") void savePassword(String password) { //TODO Implement password encryption for wallet diff --git a/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordPM.java b/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordPM.java index 549a4406f6..b447743997 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/password/PasswordPM.java @@ -43,10 +43,6 @@ class PasswordPM extends PresentationModel { final BooleanProperty saveButtonDisabled = new SimpleBooleanProperty(true); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private PasswordPM(PasswordModel model, PasswordValidator passwordValidator) { super(model); @@ -57,10 +53,6 @@ class PasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -86,10 +78,6 @@ class PasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - boolean requestSavePassword() { if (validate()) { model.savePassword(passwordField.get()); @@ -103,10 +91,6 @@ class PasswordPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private boolean validate() { InputValidator.ValidationResult result = passwordValidator.validate(passwordField.get()); if (result.isValid) { diff --git a/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationModel.java b/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationModel.java index 1c0b5b77e0..798eacb3ce 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationModel.java @@ -60,10 +60,6 @@ class RegistrationModel extends UIModel { final StringProperty payFeeErrorMessage = new SimpleStringProperty(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private RegistrationModel(WalletService walletService, User user, Persistence persistence) { @@ -73,10 +69,6 @@ class RegistrationModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -112,10 +104,6 @@ class RegistrationModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - void payFee() { FutureCallback callback = new FutureCallback() { @Override @@ -147,10 +135,6 @@ class RegistrationModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - WalletService getWalletService() { return walletService; } @@ -168,10 +152,6 @@ class RegistrationModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void updateBalance(@NotNull Coin balance) { isWalletFunded.set(balance.compareTo(getFeeAsCoin()) >= 0); } diff --git a/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationPM.java b/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationPM.java index c966024f53..7f15e7d003 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/registration/RegistrationPM.java @@ -51,10 +51,6 @@ class RegistrationPM extends PresentationModel { private final BSFormatter formatter; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private RegistrationPM(RegistrationModel model, BSFormatter formatter) { super(model); @@ -62,10 +58,6 @@ class RegistrationPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -113,10 +105,6 @@ class RegistrationPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // UI actions - /////////////////////////////////////////////////////////////////////////////////////////// - void payFee() { model.payFeeErrorMessage.set(null); model.payFeeSuccess.set(false); @@ -128,10 +116,6 @@ class RegistrationPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - WalletService getWalletService() { return model.getWalletService(); } @@ -161,10 +145,6 @@ class RegistrationPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void validateInput() { isPayButtonDisabled.set(!(model.isWalletFunded.get())); } diff --git a/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsModel.java b/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsModel.java index 1dadea3662..7e5a4272bd 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsModel.java @@ -62,10 +62,6 @@ class RestrictionsModel extends UIModel { final ObservableList allRegions = FXCollections.observableArrayList(CountryUtil.getAllRegions()); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private RestrictionsModel(User user, AccountSettings accountSettings, Persistence persistence, MessageService messageService) { @@ -76,10 +72,6 @@ class RestrictionsModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -123,10 +115,6 @@ class RestrictionsModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getAllCountriesFor(Region selectedRegion) { return FXCollections.observableArrayList(CountryUtil.getAllCountriesFor(selectedRegion)); } @@ -178,10 +166,6 @@ class RestrictionsModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void saveSettings() { persistence.write(accountSettings); } diff --git a/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsPM.java b/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsPM.java index f14c750985..eccd95ee5e 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/restrictions/RestrictionsPM.java @@ -39,20 +39,12 @@ class RestrictionsPM extends PresentationModel { final BooleanProperty doneButtonDisable = new SimpleBooleanProperty(true); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private RestrictionsPM(RestrictionsModel model) { super(model); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -74,10 +66,6 @@ class RestrictionsPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public - /////////////////////////////////////////////////////////////////////////////////////////// - void addLanguage(Locale locale) { model.addLanguage(locale); updateDoneButtonDisableState(); @@ -109,10 +97,6 @@ class RestrictionsPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getListWithAllEuroCountries() { return model.getListWithAllEuroCountries(); } @@ -142,10 +126,6 @@ class RestrictionsPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - //TODO Revert size() > -1 to 0(2 later). For mock testing disabled arbitratorList test private void updateDoneButtonDisableState() { boolean isValid = model.languageList != null && model.languageList.size() > 0 && diff --git a/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsModel.java b/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsModel.java index 33438b5f78..c08820ad62 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsModel.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsModel.java @@ -33,10 +33,6 @@ class SeedWordsModel extends UIModel { private List mnemonicCode; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private SeedWordsModel(WalletService walletService) { if (walletService != null && walletService.getWallet() != null) @@ -44,10 +40,6 @@ class SeedWordsModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -73,10 +65,6 @@ class SeedWordsModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getter - /////////////////////////////////////////////////////////////////////////////////////////// - List getMnemonicCode() { return mnemonicCode; } diff --git a/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsPM.java b/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsPM.java index 3ada05f144..acaee53384 100644 --- a/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsPM.java +++ b/src/main/java/io/bitsquare/gui/main/account/content/seedwords/SeedWordsPM.java @@ -35,10 +35,6 @@ class SeedWordsPM extends PresentationModel { private final BSFormatter formatter; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject private SeedWordsPM(SeedWordsModel model, BSFormatter formatter) { super(model); @@ -46,10 +42,6 @@ class SeedWordsPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesModel.java b/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesModel.java index db32851af5..7be3178730 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesModel.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesModel.java @@ -43,10 +43,6 @@ class ClosedTradesModel extends UIModel { private MapChangeListener mapChangeListener; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject public ClosedTradesModel(TradeManager tradeManager, User user) { this.tradeManager = tradeManager; @@ -54,10 +50,6 @@ class ClosedTradesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { mapChangeListener = change -> { @@ -97,10 +89,6 @@ class ClosedTradesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - public ObservableList getList() { return list; } diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesPM.java b/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesPM.java index fce241c8ac..c1c4d8291a 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesPM.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/closed/ClosedTradesPM.java @@ -33,10 +33,6 @@ class ClosedTradesPM extends PresentationModel { private final BSFormatter formatter; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject public ClosedTradesPM(ClosedTradesModel model, BSFormatter formatter) { super(model); @@ -45,10 +41,6 @@ class ClosedTradesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -74,10 +66,6 @@ class ClosedTradesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - public ObservableList getList() { return model.getList(); } diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersModel.java b/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersModel.java index 450706e62c..a76f87da40 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersModel.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersModel.java @@ -44,10 +44,6 @@ class OffersModel extends UIModel { private MapChangeListener offerMapChangeListener; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject public OffersModel(TradeManager tradeManager, User user) { this.tradeManager = tradeManager; @@ -55,10 +51,6 @@ class OffersModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { offerMapChangeListener = change -> { @@ -98,19 +90,11 @@ class OffersModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public methods - /////////////////////////////////////////////////////////////////////////////////////////// - void removeOffer(OfferListItem item) { tradeManager.removeOffer(item.getOffer()); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - public ObservableList getList() { return list; } diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersPM.java b/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersPM.java index 7d7a1aca53..f5aae62b9a 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersPM.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/offer/OffersPM.java @@ -33,10 +33,6 @@ class OffersPM extends PresentationModel { private final BSFormatter formatter; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject public OffersPM(OffersModel model, BSFormatter formatter) { super(model); @@ -45,10 +41,6 @@ class OffersPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -74,19 +66,11 @@ class OffersPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public methods - /////////////////////////////////////////////////////////////////////////////////////////// - void removeOffer(OfferListItem item) { model.removeOffer(item); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - public ObservableList getList() { return model.getList(); } diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesModel.java b/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesModel.java index f40ee878a8..2358cac110 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesModel.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesModel.java @@ -77,10 +77,6 @@ class PendingTradesModel extends UIModel { final ObjectProperty fault = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject PendingTradesModel(TradeManager tradeManager, WalletService walletService, User user) { this.tradeManager = tradeManager; @@ -89,10 +85,6 @@ class PendingTradesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { stateChangeListener = (ov, oldValue, newValue) -> tradeState.set(newValue); @@ -143,10 +135,6 @@ class PendingTradesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - void selectTrade(PendingTradesListItem item) { // clean up previous selectedItem cleanUpSelectedTrade(); @@ -249,10 +237,6 @@ class PendingTradesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getList() { return list; } @@ -301,10 +285,6 @@ class PendingTradesModel extends UIModel { return getTrade().getSecurityDeposit(); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void updateConfidence(TransactionConfidence confidence) { log.trace("updateConfidence confidence " + confidence); log.trace("updateConfidence getTrade().getState() " + getTrade().getState()); diff --git a/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesPM.java b/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesPM.java index 6697aa33ea..e9aa4d9f6f 100644 --- a/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesPM.java +++ b/src/main/java/io/bitsquare/gui/main/portfolio/pending/PendingTradesPM.java @@ -68,10 +68,6 @@ public class PendingTradesPM extends PresentationModel { final BooleanProperty withdrawalButtonDisable = new SimpleBooleanProperty(true); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject PendingTradesPM(PendingTradesModel model, BSFormatter formatter, BtcAddressValidator btcAddressValidator) { @@ -82,10 +78,6 @@ public class PendingTradesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { stateChangeListener = (ov) -> updateState(); @@ -116,10 +108,6 @@ public class PendingTradesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - void selectTrade(PendingTradesListItem item) { model.selectTrade(item); updateState(); @@ -145,10 +133,6 @@ public class PendingTradesPM extends PresentationModel { return formatter.formatCoinWithCode(model.getAmountToWithdraw()); //.subtract(FeePolicy.TX_FEE)); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - ObservableList getList() { return model.getList(); } @@ -242,10 +226,6 @@ public class PendingTradesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void updateState() { Trade.State tradeState = model.tradeState.get(); log.trace("tradeState " + tradeState); diff --git a/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesModel.java b/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesModel.java index 442e7eb1ac..7c9d572529 100644 --- a/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesModel.java +++ b/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesModel.java @@ -49,10 +49,6 @@ class PreferencesModel extends UIModel { final StringProperty btcDenomination = new SimpleStringProperty(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject PreferencesModel(Preferences preferences) { this.preferences = preferences; @@ -64,10 +60,6 @@ class PreferencesModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { diff --git a/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesPM.java b/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesPM.java index eda37c41c3..7bdca5d8c5 100644 --- a/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesPM.java +++ b/src/main/java/io/bitsquare/gui/main/settings/application/PreferencesPM.java @@ -32,20 +32,12 @@ public class PreferencesPM extends PresentationModel { private static final Logger log = LoggerFactory.getLogger(PreferencesPM.class); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject PreferencesPM(PreferencesModel model) { super(model); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @SuppressWarnings("EmptyMethod") @Override public void initialize() { @@ -71,14 +63,6 @@ public class PreferencesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - - - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// public ObservableList getBtcDenominationItems() { return model.btcDenominations; @@ -97,9 +81,5 @@ public class PreferencesPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - } diff --git a/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferModel.java b/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferModel.java index bc87472349..c93df5dea0 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferModel.java +++ b/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferModel.java @@ -106,10 +106,6 @@ class CreateOfferModel extends UIModel { final ObservableList acceptedArbitrators = FXCollections.observableArrayList(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - // non private for testing @Inject public CreateOfferModel(TradeManager tradeManager, WalletService walletService, AccountSettings accountSettings, @@ -127,10 +123,6 @@ class CreateOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { // static data @@ -194,10 +186,6 @@ class CreateOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - void placeOffer() { // data validation is done in the trade domain tradeManager.requestPlaceOffer(offerId, @@ -264,10 +252,6 @@ class CreateOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setter/Getter - /////////////////////////////////////////////////////////////////////////////////////////// - @Nullable Direction getDirection() { return direction; @@ -297,10 +281,6 @@ class CreateOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void updateBalance(@NotNull Coin balance) { isWalletFunded.set(totalToPayAsCoin.get() != null && balance.compareTo(totalToPayAsCoin.get()) >= 0); diff --git a/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferPM.java b/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferPM.java index b4bfa2c655..762a481ba9 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferPM.java +++ b/src/main/java/io/bitsquare/gui/main/trade/createoffer/CreateOfferPM.java @@ -95,10 +95,6 @@ class CreateOfferPM extends PresentationModel { final ObjectProperty
address = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - // non private for testing @Inject CreateOfferPM(CreateOfferModel model, FiatValidator fiatValidator, BtcValidator btcValidator, @@ -111,10 +107,6 @@ class CreateOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -150,10 +142,6 @@ class CreateOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public API methods - /////////////////////////////////////////////////////////////////////////////////////////// - // setOfferBookFilter is a one time call void initWithData(Direction direction, Coin amount, Fiat price) { model.setDirection(direction); @@ -181,10 +169,6 @@ class CreateOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // UI actions - /////////////////////////////////////////////////////////////////////////////////////////// - void placeOffer() { model.requestPlaceOfferErrorMessage.set(null); model.requestPlaceOfferSuccess.set(false); @@ -196,10 +180,6 @@ class CreateOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // UI events - /////////////////////////////////////////////////////////////////////////////////////////// - void onShowPayFundsScreen() { isPlaceOfferButtonVisible.set(true); } @@ -294,10 +274,6 @@ class CreateOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - WalletService getWalletService() { return model.getWalletService(); } @@ -310,10 +286,6 @@ class CreateOfferPM extends PresentationModel { return model.displaySecurityDepositInfo(); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void setupListeners() { // Bidirectional bindings are used for all input fields: amount, price, volume and minAmount // We do volume/amount calculation during input, so user has immediate feedback diff --git a/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookModel.java b/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookModel.java index 28c3e50dc1..09418fbb0c 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookModel.java +++ b/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookModel.java @@ -77,10 +77,6 @@ class OfferBookModel extends UIModel { private Direction direction; - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject OfferBookModel(User user, TradeManager tradeManager, @@ -98,10 +94,6 @@ class OfferBookModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { bankAccountChangeListener = (observableValue, oldValue, newValue) -> setBankAccount(newValue); @@ -141,10 +133,6 @@ class OfferBookModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public methods - /////////////////////////////////////////////////////////////////////////////////////////// - void removeOffer(Offer offer) { tradeManager.removeOffer(offer); } @@ -216,10 +204,6 @@ class OfferBookModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setDirection(Direction direction) { this.direction = direction; } @@ -240,10 +224,6 @@ class OfferBookModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - SortedList getOfferList() { return sortedItems; } @@ -284,10 +264,6 @@ class OfferBookModel extends UIModel { return direction; } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private void setBankAccount(BankAccount bankAccount) { if (bankAccount != null) { fiatCode.set(bankAccount.getCurrency().getCurrencyCode()); diff --git a/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookPM.java b/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookPM.java index 782fad8da2..9a2699bc34 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookPM.java +++ b/src/main/java/io/bitsquare/gui/main/trade/offerbook/OfferBookPM.java @@ -53,10 +53,6 @@ class OfferBookPM extends PresentationModel { final StringProperty restrictionsInfo = new SimpleStringProperty(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject OfferBookPM(OfferBookModel model, OptionalFiatValidator optionalFiatValidator, @@ -70,10 +66,6 @@ class OfferBookPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -135,10 +127,6 @@ class OfferBookPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Public methods - /////////////////////////////////////////////////////////////////////////////////////////// - void removeOffer(Offer offer) { model.removeOffer(offer); } @@ -148,19 +136,11 @@ class OfferBookPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setters - /////////////////////////////////////////////////////////////////////////////////////////// - void setDirection(Direction direction) { model.setDirection(direction); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - SortedList getOfferList() { return model.getOfferList(); } @@ -207,10 +187,6 @@ class OfferBookPM extends PresentationModel { return model.getPriceAsFiat(); } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private methods - /////////////////////////////////////////////////////////////////////////////////////////// - private InputValidator.ValidationResult isBtcInputValid(String input) { return optionalBtcValidator.validate(input); } diff --git a/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferModel.java b/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferModel.java index 4ae02a6aeb..b455902f63 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferModel.java +++ b/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferModel.java @@ -78,10 +78,6 @@ class TakeOfferModel extends UIModel { final ObjectProperty networkFeeAsCoin = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - @Inject TakeOfferModel(TradeManager tradeManager, WalletService walletService, Preferences preferences, Persistence persistence) { @@ -92,10 +88,6 @@ class TakeOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -126,10 +118,6 @@ class TakeOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Methods - /////////////////////////////////////////////////////////////////////////////////////////// - void initWithData(Coin amount, Offer offer) { this.offer = offer; @@ -235,14 +223,6 @@ class TakeOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setter - /////////////////////////////////////////////////////////////////////////////////////////// - - - /////////////////////////////////////////////////////////////////////////////////////////// - // Getter - /////////////////////////////////////////////////////////////////////////////////////////// WalletService getWalletService() { return walletService; @@ -253,10 +233,6 @@ class TakeOfferModel extends UIModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void updateBalance(@NotNull Coin balance) { isWalletFunded.set(totalToPayAsCoin.get() != null && balance.compareTo(totalToPayAsCoin.get()) >= 0); } diff --git a/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferPM.java b/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferPM.java index 14d32314eb..97b54c3f6b 100644 --- a/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferPM.java +++ b/src/main/java/io/bitsquare/gui/main/trade/takeoffer/TakeOfferPM.java @@ -89,10 +89,6 @@ class TakeOfferPM extends PresentationModel { final ObjectProperty totalToPayAsCoin = new SimpleObjectProperty<>(); - /////////////////////////////////////////////////////////////////////////////////////////// - // Constructor - /////////////////////////////////////////////////////////////////////////////////////////// - // non private for testing @Inject TakeOfferPM(TakeOfferModel model, BtcValidator btcValidator, BSFormatter formatter) { @@ -103,10 +99,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Lifecycle - /////////////////////////////////////////////////////////////////////////////////////////// - @Override public void initialize() { super.initialize(); @@ -138,10 +130,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Setter - /////////////////////////////////////////////////////////////////////////////////////////// - // setOfferBookFilter is a one time call void initWithData(Direction direction, Coin amount, Offer offer) { model.initWithData(amount, offer); @@ -178,10 +166,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // UI actions - /////////////////////////////////////////////////////////////////////////////////////////// - void takeOffer() { model.requestTakeOfferErrorMessage.set(null); model.requestTakeOfferSuccess.set(false); @@ -197,10 +181,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // UI events - /////////////////////////////////////////////////////////////////////////////////////////// - void onShowPayFundsScreen() { isTakeOfferButtonVisible.set(true); } @@ -231,10 +211,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Getters - /////////////////////////////////////////////////////////////////////////////////////////// - WalletService getWalletService() { return model.getWalletService(); } @@ -308,10 +284,6 @@ class TakeOfferPM extends PresentationModel { } - /////////////////////////////////////////////////////////////////////////////////////////// - // Private - /////////////////////////////////////////////////////////////////////////////////////////// - private void setupListeners() { // Bidirectional bindings are used for all input fields: amount, price, volume and minAmount // We do volume/amount calculation during input, so user has immediate feedback