From 1c99baf944c30f637d62e6f5e7bdff7954ff2f60 Mon Sep 17 00:00:00 2001 From: Robert Goldmann <deadlocker@gmx.de> Date: Sat, 2 Mar 2019 16:47:06 +0100 Subject: [PATCH] big refactoring: classes related to repeating --- .../{repositories => repeating}/RepeatingOptionRepository.java | 2 +- .../budgetmaster/repeating/RepeatingTransactionUpdater.java | 1 - .../{controller => settings}/SettingsController.java | 3 ++- .../deadlocker8/budgetmaster/settings/SettingsRepository.java | 1 - .../de/deadlocker8/budgetmaster/settings/SettingsService.java | 2 -- .../budgetmaster/transactions/TransactionController.java | 2 +- .../budgetmaster/transactions/TransactionService.java | 2 +- .../de/deadlocker8/budgetmaster/TransactionRepositoryTest.java | 2 +- 8 files changed, 6 insertions(+), 9 deletions(-) rename src/main/java/de/deadlocker8/budgetmaster/{repositories => repeating}/RepeatingOptionRepository.java (87%) rename src/main/java/de/deadlocker8/budgetmaster/{controller => settings}/SettingsController.java (98%) diff --git a/src/main/java/de/deadlocker8/budgetmaster/repositories/RepeatingOptionRepository.java b/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingOptionRepository.java similarity index 87% rename from src/main/java/de/deadlocker8/budgetmaster/repositories/RepeatingOptionRepository.java rename to src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingOptionRepository.java index 2da609239..25e9a63fc 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/repositories/RepeatingOptionRepository.java +++ b/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingOptionRepository.java @@ -1,4 +1,4 @@ -package de.deadlocker8.budgetmaster.repositories; +package de.deadlocker8.budgetmaster.repeating; import de.deadlocker8.budgetmaster.repeating.RepeatingOption; import org.springframework.data.jpa.repository.JpaRepository; diff --git a/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingTransactionUpdater.java b/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingTransactionUpdater.java index 4cf89932b..71e2a47b1 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingTransactionUpdater.java +++ b/src/main/java/de/deadlocker8/budgetmaster/repeating/RepeatingTransactionUpdater.java @@ -2,7 +2,6 @@ package de.deadlocker8.budgetmaster.repeating; import de.deadlocker8.budgetmaster.transactions.Transaction; import de.deadlocker8.budgetmaster.transactions.TransactionRepository; -import de.deadlocker8.budgetmaster.repositories.RepeatingOptionRepository; import de.deadlocker8.budgetmaster.services.HelpersService; import de.deadlocker8.budgetmaster.transactions.TransactionService; import org.joda.time.DateTime; diff --git a/src/main/java/de/deadlocker8/budgetmaster/controller/SettingsController.java b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsController.java similarity index 98% rename from src/main/java/de/deadlocker8/budgetmaster/controller/SettingsController.java rename to src/main/java/de/deadlocker8/budgetmaster/settings/SettingsController.java index ec76d674c..37b54eeb9 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/controller/SettingsController.java +++ b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsController.java @@ -1,10 +1,11 @@ -package de.deadlocker8.budgetmaster.controller; +package de.deadlocker8.budgetmaster.settings; import de.deadlocker8.budgetmaster.Build; import de.deadlocker8.budgetmaster.accounts.AccountService; import de.deadlocker8.budgetmaster.authentication.User; import de.deadlocker8.budgetmaster.authentication.UserRepository; import de.deadlocker8.budgetmaster.categories.CategoryService; +import de.deadlocker8.budgetmaster.controller.BaseController; import de.deadlocker8.budgetmaster.database.Database; import de.deadlocker8.budgetmaster.database.DatabaseParser; import de.deadlocker8.budgetmaster.database.accountmatches.AccountMatchList; diff --git a/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsRepository.java b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsRepository.java index 3bd6098af..5fa2d0482 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsRepository.java +++ b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsRepository.java @@ -1,6 +1,5 @@ package de.deadlocker8.budgetmaster.settings; -import de.deadlocker8.budgetmaster.settings.Settings; import org.springframework.data.jpa.repository.JpaRepository; diff --git a/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsService.java b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsService.java index a5dbcaabd..5af519809 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsService.java +++ b/src/main/java/de/deadlocker8/budgetmaster/settings/SettingsService.java @@ -1,7 +1,5 @@ package de.deadlocker8.budgetmaster.settings; -import de.deadlocker8.budgetmaster.settings.Settings; -import de.deadlocker8.budgetmaster.settings.SettingsRepository; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.beans.factory.annotation.Autowired; diff --git a/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionController.java b/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionController.java index f10eb3e16..8494dc9ad 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionController.java +++ b/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionController.java @@ -2,6 +2,7 @@ package de.deadlocker8.budgetmaster.transactions; import de.deadlocker8.budgetmaster.categories.CategoryRepository; import de.deadlocker8.budgetmaster.controller.BaseController; +import de.deadlocker8.budgetmaster.repeating.RepeatingOptionRepository; import de.deadlocker8.budgetmaster.settings.Settings; import de.deadlocker8.budgetmaster.settings.SettingsRepository; import de.deadlocker8.budgetmaster.tags.Tag; @@ -11,7 +12,6 @@ import de.deadlocker8.budgetmaster.repeating.RepeatingTransactionUpdater; import de.deadlocker8.budgetmaster.repeating.endoption.*; import de.deadlocker8.budgetmaster.repeating.modifier.RepeatingModifier; import de.deadlocker8.budgetmaster.repeating.modifier.RepeatingModifierType; -import de.deadlocker8.budgetmaster.repositories.*; import de.deadlocker8.budgetmaster.accounts.AccountService; import de.deadlocker8.budgetmaster.services.FilterHelpersService; import de.deadlocker8.budgetmaster.services.HelpersService; diff --git a/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionService.java b/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionService.java index 9009297c2..31d3e4234 100644 --- a/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionService.java +++ b/src/main/java/de/deadlocker8/budgetmaster/transactions/TransactionService.java @@ -6,7 +6,7 @@ import de.deadlocker8.budgetmaster.categories.CategoryType; import de.deadlocker8.budgetmaster.services.Resetable; import de.deadlocker8.budgetmaster.filter.FilterConfiguration; import de.deadlocker8.budgetmaster.categories.CategoryRepository; -import de.deadlocker8.budgetmaster.repositories.RepeatingOptionRepository; +import de.deadlocker8.budgetmaster.repeating.RepeatingOptionRepository; import de.deadlocker8.budgetmaster.utils.Strings; import de.thecodelabs.utils.util.Localization; import org.joda.time.DateTime; diff --git a/src/test/java/de/deadlocker8/budgetmaster/TransactionRepositoryTest.java b/src/test/java/de/deadlocker8/budgetmaster/TransactionRepositoryTest.java index 5f48b9b76..0c702d24e 100644 --- a/src/test/java/de/deadlocker8/budgetmaster/TransactionRepositoryTest.java +++ b/src/test/java/de/deadlocker8/budgetmaster/TransactionRepositoryTest.java @@ -6,6 +6,7 @@ import de.deadlocker8.budgetmaster.accounts.Account; import de.deadlocker8.budgetmaster.accounts.AccountType; import de.deadlocker8.budgetmaster.categories.Category; import de.deadlocker8.budgetmaster.categories.CategoryType; +import de.deadlocker8.budgetmaster.repeating.RepeatingOptionRepository; import de.deadlocker8.budgetmaster.tags.Tag; import de.deadlocker8.budgetmaster.tags.TagRepository; import de.deadlocker8.budgetmaster.transactions.Transaction; @@ -14,7 +15,6 @@ import de.deadlocker8.budgetmaster.transactions.TransactionSpecifications; import de.deadlocker8.budgetmaster.repeating.RepeatingOption; import de.deadlocker8.budgetmaster.repeating.endoption.RepeatingEndAfterXTimes; import de.deadlocker8.budgetmaster.repeating.modifier.RepeatingModifierDays; -import de.deadlocker8.budgetmaster.repositories.*; import org.joda.time.DateTime; import org.joda.time.format.DateTimeFormat; import org.junit.Before; -- GitLab