Skip to content

Commit d881843

Browse files
ShkarinShkarin
committed
Guess Country
1 parent 0abdb81 commit d881843

File tree

14 files changed

+185
-7
lines changed

14 files changed

+185
-7
lines changed

‎src/main/java/ru/mystamps/web/Url.java‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ public final class Url {
5252
public static final String ADD_IMAGE_SERIES_PAGE = "/series/{id}/image";
5353
public static final String SEARCH_SERIES_BY_CATALOG = "/series/search/by_catalog";
5454

55+
public static final String SUGGEST_SERIES_COUNTRY = "/suggest/series_country";
56+
5557
public static final String ADD_CATEGORY_PAGE = "/category/add";
5658
public static final String LIST_CATEGORIES_PAGE = "/category/list";
5759
public static final String INFO_CATEGORY_PAGE = "/category/{slug}";
@@ -77,7 +79,7 @@ public final class Url {
7779
public static final String ADD_SERIES_WITH_COUNTRY_PAGE = "/series/add/country/{slug}";
7880

7981
// MUST be updated when any of our resources were modified
80-
public static final String RESOURCES_VERSION = "v0.3.0";
82+
public static final String RESOURCES_VERSION = "v0.3.1";
8183

8284
// CheckStyle: ignore LineLength for next 4 lines
8385
public static final String MAIN_CSS = "/static/" + RESOURCES_VERSION + "/styles/main.min.css";
@@ -125,6 +127,7 @@ public static Map<String, String> asMap(boolean serveContentFromSingleHost) {
125127
map.put("INFO_SERIES_PAGE", INFO_SERIES_PAGE);
126128
map.put("ADD_IMAGE_SERIES_PAGE", ADD_IMAGE_SERIES_PAGE);
127129
map.put("SEARCH_SERIES_BY_CATALOG", SEARCH_SERIES_BY_CATALOG);
130+
map.put("SUGGEST_SERIES_COUNTRY", SUGGEST_SERIES_COUNTRY);
128131
map.put("ADD_CATEGORY_PAGE", ADD_CATEGORY_PAGE);
129132
map.put("INFO_CATEGORY_PAGE", INFO_CATEGORY_PAGE);
130133
map.put("LIST_CATEGORIES_PAGE", LIST_CATEGORIES_PAGE);

‎src/main/java/ru/mystamps/web/config/ControllersConfig.java‎

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,5 +110,12 @@ public SiteController getSiteController() {
110110
public SitemapController getSitemapController() {
111111
return new SitemapController(servicesConfig.getSeriesService());
112112
}
113-
113+
114+
@Bean
115+
public SuggestionController getSuggestionController() {
116+
return new SuggestionController(
117+
servicesConfig.getCountryService()
118+
);
119+
}
120+
114121
}

‎src/main/java/ru/mystamps/web/controller/SuggestionController.java‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
/*
2+
* Copyright (C) 2009-2017 Slava Semushin <[email protected]>
3+
*
4+
* This program is free software; you can redistribute it and/or modify
5+
* it under the terms of the GNU General Public License as published by
6+
* the Free Software Foundation; either version 2 of the License, or
7+
* (at your option) any later version.
8+
*
9+
* This program is distributed in the hope that it will be useful,
10+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
11+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12+
* GNU General Public License for more details.
13+
*
14+
* You should have received a copy of the GNU General Public License
15+
* along with this program; if not, write to the Free Software
16+
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17+
*/
18+
package ru.mystamps.web.controller;
19+
20+
import org.springframework.stereotype.Controller;
21+
import org.springframework.web.bind.annotation.GetMapping;
22+
import org.springframework.web.bind.annotation.ResponseBody;
23+
24+
import lombok.RequiredArgsConstructor;
25+
26+
import ru.mystamps.web.Url;
27+
import ru.mystamps.web.controller.converter.annotation.CurrentUser;
28+
import ru.mystamps.web.service.CountryService;
29+
30+
@Controller
31+
@RequiredArgsConstructor
32+
public class SuggestionController {
33+
34+
private final CountryService countryService;
35+
36+
/**
37+
* @author John Shkarin
38+
*/
39+
@ResponseBody
40+
@GetMapping(Url.SUGGEST_SERIES_COUNTRY)
41+
public String suggestCountryForUser(@CurrentUser Integer currentUserId) {
42+
return countryService.suggestCountryForUser(currentUserId);
43+
}
44+
45+
}
46+

‎src/main/java/ru/mystamps/web/dao/CountryDao.java‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import ru.mystamps.web.dao.dto.AddCountryDbDto;
2424
import ru.mystamps.web.dao.dto.LinkEntityDto;
2525

26+
@SuppressWarnings("PMD.TooManyMethods")
2627
public interface CountryDao {
2728
Integer add(AddCountryDbDto country);
2829
long countAll();
@@ -35,4 +36,6 @@ public interface CountryDao {
3536
List<Object[]> getStatisticsOf(Integer collectionId, String lang);
3637
List<LinkEntityDto> findAllAsLinkEntities(String lang);
3738
LinkEntityDto findOneAsLinkEntity(String slug, String lang);
39+
String findLastCreatedByUser(Integer userId);
40+
String findPopularCountryInCollection(Integer userId);
3841
}

‎src/main/java/ru/mystamps/web/dao/impl/JdbcCountryDao.java‎

Lines changed: 40 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
import ru.mystamps.web.dao.dto.LinkEntityDto;
4040

4141
@RequiredArgsConstructor
42-
@SuppressWarnings("PMD.AvoidDuplicateLiterals")
42+
@SuppressWarnings({"PMD.AvoidDuplicateLiterals", "PMD.TooManyMethods"})
4343
public class JdbcCountryDao implements CountryDao {
4444

4545
private final NamedParameterJdbcTemplate jdbcTemplate;
@@ -76,6 +76,13 @@ public class JdbcCountryDao implements CountryDao {
7676

7777
@Value("${country.find_country_link_info_by_slug}")
7878
private String findCountryLinkEntityBySlugSql;
79+
80+
@Value("${country.find_last_created_by_user}")
81+
private String findLastCreatedByUserSql;
82+
83+
@SuppressWarnings("PMD.LongVariable")
84+
@Value("${country.find_popular_country_from_user_collection}")
85+
private String findPopularCountryInCollectionSql;
7986

8087
@Override
8188
public Integer add(AddCountryDbDto country) {
@@ -206,5 +213,37 @@ public LinkEntityDto findOneAsLinkEntity(String slug, String lang) {
206213
return null;
207214
}
208215
}
216+
217+
/**
218+
* @author Shkarin John
219+
*/
220+
@Override
221+
public String findLastCreatedByUser(Integer userId) {
222+
try {
223+
return jdbcTemplate.queryForObject(
224+
findLastCreatedByUserSql,
225+
Collections.singletonMap("created_by", userId),
226+
String.class
227+
);
228+
} catch (EmptyResultDataAccessException ignored) {
229+
return null;
230+
}
231+
}
232+
233+
/**
234+
* @author Shkarin John
235+
*/
236+
@Override
237+
public String findPopularCountryInCollection(Integer userId) {
238+
try {
239+
return jdbcTemplate.queryForObject(
240+
findPopularCountryInCollectionSql,
241+
Collections.singletonMap("user_id", userId),
242+
String.class
243+
);
244+
} catch (EmptyResultDataAccessException ignored) {
245+
return null;
246+
}
247+
}
209248

210249
}

‎src/main/java/ru/mystamps/web/service/CountryService.java‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import ru.mystamps.web.dao.dto.LinkEntityDto;
2424
import ru.mystamps.web.service.dto.AddCountryDto;
2525

26+
@SuppressWarnings("PMD.TooManyMethods")
2627
public interface CountryService {
2728
String add(AddCountryDto dto, Integer userId);
2829
List<LinkEntityDto> findAllAsLinkEntities(String lang);
@@ -35,4 +36,5 @@ public interface CountryService {
3536
long countAddedSince(Date date);
3637
long countUntranslatedNamesSince(Date date);
3738
List<Object[]> getStatisticsOf(Integer collectionId, String lang);
39+
String suggestCountryForUser(Integer userId);
3840
}

‎src/main/java/ru/mystamps/web/service/CountryServiceImpl.java‎

Lines changed: 30 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import ru.mystamps.web.util.SlugUtils;
4343

4444
@RequiredArgsConstructor
45+
@SuppressWarnings("PMD.TooManyMethods")
4546
public class CountryServiceImpl implements CountryService {
4647
private static final Logger LOG = LoggerFactory.getLogger(CountryServiceImpl.class);
4748

@@ -162,5 +163,33 @@ public List<Object[]> getStatisticsOf(Integer collectionId, String lang) {
162163

163164
return countryDao.getStatisticsOf(collectionId, lang);
164165
}
165-
166+
167+
/**
168+
* @author Shkarin John
169+
*/
170+
@Override
171+
@Transactional(readOnly = true)
172+
@PreAuthorize(HasAuthority.CREATE_SERIES)
173+
public String suggestCountryForUser(Integer userId) {
174+
Validate.isTrue(userId != null, "User id must be non null");
175+
176+
String slug = countryDao.findLastCreatedByUser(userId);
177+
if (slug != null) {
178+
LOG.debug(
179+
"Country {} has been suggested to user #{} as a recently created",
180+
slug,
181+
userId
182+
);
183+
return slug;
184+
}
185+
186+
slug = countryDao.findPopularCountryInCollection(userId);
187+
LOG.debug(
188+
"Country {} has been suggested to user #{} as popular in his collection",
189+
slug,
190+
userId
191+
);
192+
193+
return slug;
194+
}
166195
}

‎src/main/java/ru/mystamps/web/support/spring/security/SecurityConfig.java‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ protected void configure(HttpSecurity http) throws Exception {
7575
.mvcMatchers(Url.ADD_COUNTRY_PAGE).hasAuthority(StringAuthority.CREATE_COUNTRY)
7676
.mvcMatchers(Url.ADD_SERIES_PAGE).hasAuthority(StringAuthority.CREATE_SERIES)
7777
.mvcMatchers(Url.SITE_EVENTS_PAGE).hasAuthority(StringAuthority.VIEW_SITE_EVENTS)
78+
.mvcMatchers(Url.SUGGEST_SERIES_COUNTRY).hasAuthority(StringAuthority.CREATE_SERIES)
7879
.regexMatchers(HttpMethod.POST, "/series/[0-9]+")
7980
.hasAnyAuthority(
8081
StringAuthority.UPDATE_COLLECTION,

‎src/main/java/ru/mystamps/web/support/togglz/Features.java‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,11 @@ public enum Features implements Feature {
7474

7575
@Label("View site events")
7676
@EnabledByDefault
77-
VIEW_SITE_EVENTS;
77+
VIEW_SITE_EVENTS,
78+
79+
@Label("/series/add: show link with auto-suggestions")
80+
@EnabledByDefault
81+
INFO_COUNTRY_SERIES;
7882

7983
public boolean isActive() {
8084
return FeatureContext.getFeatureManager().isActive(this);

‎src/main/javascript/series/add.js‎

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// IMPORTANT:
33
// You have to update Url.RESOURCES_VERSION each time whenever you're modified this file!
44
//
5-
function initPage() {
5+
function initPage(suggestCountryUrl) {
66
$('#country').selectize();
77

88
$('.js-catalog-numbers').on('blur', function() {
@@ -21,4 +21,22 @@ function initPage() {
2121
$('.js-with-tooltip').tooltip({
2222
'placement': 'right'
2323
});
24+
25+
if (suggestCountryUrl != null) {
26+
$.get(suggestCountryUrl, function handleSuggestedCountry(slug) {
27+
if (slug == "") {
28+
return;
29+
}
30+
31+
var country = $("#js-guess-country-link");
32+
country.show();
33+
country.click(function chooseSuggestedCountry() {
34+
$(this).hide();
35+
36+
var select_country = $("#country").selectize();
37+
var selectize = select_country[0].selectize;
38+
selectize.setValue(slug);
39+
});
40+
});
41+
}
2442
}

0 commit comments

Comments
 (0)