Skip to content

Commit eae504e

Browse files
committed
refactor: move inclusion of series_sales_dao_queries.properties to the appropriate package config
We also switched to using Environment.getRequiredProperty() because @value doesn't work without PropertySourcesPlaceholderConfigurer bean (that we are going to remove soon). Relate to #927
1 parent aad47c1 commit eae504e

File tree

3 files changed

+14
-10
lines changed

3 files changed

+14
-10
lines changed

src/main/java/ru/mystamps/web/config/ApplicationContext.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ public static PropertySourcesPlaceholderConfigurer getPropertySourcesPlaceholder
4444
new PropertySourcesPlaceholderConfigurer();
4545
configurer.setLocations(
4646
new ClassPathResource("sql/series_dao_queries.properties"),
47-
new ClassPathResource("sql/series_sales_dao_queries.properties"),
4847
new ClassPathResource("sql/transaction_participants_dao_queries.properties")
4948
);
5049
return configurer;

src/main/java/ru/mystamps/web/feature/series/sale/JdbcSeriesSalesDao.java

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,26 +17,27 @@
1717
*/
1818
package ru.mystamps.web.feature.series.sale;
1919

20-
import lombok.RequiredArgsConstructor;
2120
import org.apache.commons.lang3.Validate;
22-
import org.springframework.beans.factory.annotation.Value;
21+
import org.springframework.core.env.Environment;
2322
import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate;
2423

2524
import java.util.Collections;
2625
import java.util.HashMap;
2726
import java.util.List;
2827
import java.util.Map;
2928

30-
@RequiredArgsConstructor
3129
public class JdbcSeriesSalesDao implements SeriesSalesDao {
3230

3331
private final NamedParameterJdbcTemplate jdbcTemplate;
32+
private final String addSeriesSaleSql;
33+
private final String findSeriesSalesBySeriesIdSql;
3434

35-
@Value("${series_sales.add}")
36-
private String addSeriesSaleSql;
37-
38-
@Value("${series_sales.find_sales_by_series_id}")
39-
private String findSeriesSalesBySeriesIdSql;
35+
@SuppressWarnings("checkstyle:linelength")
36+
public JdbcSeriesSalesDao(Environment env, NamedParameterJdbcTemplate jdbcTemplate) {
37+
this.jdbcTemplate = jdbcTemplate;
38+
this.addSeriesSaleSql = env.getRequiredProperty("series_sales.add");
39+
this.findSeriesSalesBySeriesIdSql = env.getRequiredProperty("series_sales.find_sales_by_series_id");
40+
}
4041

4142
@Override
4243
public void add(AddSeriesSalesDbDto sale) {

src/main/java/ru/mystamps/web/feature/series/sale/SeriesSalesConfig.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
import org.slf4j.LoggerFactory;
2222
import org.springframework.context.annotation.Bean;
2323
import org.springframework.context.annotation.Configuration;
24+
import org.springframework.context.annotation.PropertySource;
25+
import org.springframework.core.env.Environment;
2426
import org.springframework.jdbc.core.namedparam.NamedParameterJdbcTemplate;
2527

2628
/**
@@ -30,8 +32,10 @@
3032
public class SeriesSalesConfig {
3133

3234
@RequiredArgsConstructor
35+
@PropertySource("classpath:sql/series_sales_dao_queries.properties")
3336
public static class Services {
3437

38+
private final Environment env;
3539
private final NamedParameterJdbcTemplate jdbcTemplate;
3640

3741
@Bean
@@ -44,7 +48,7 @@ public SeriesSalesService seriesSalesService(SeriesSalesDao seriesSalesDao) {
4448

4549
@Bean
4650
public SeriesSalesDao seriesSalesDao() {
47-
return new JdbcSeriesSalesDao(jdbcTemplate);
51+
return new JdbcSeriesSalesDao(env, jdbcTemplate);
4852
}
4953

5054
}

0 commit comments

Comments
 (0)