fix(series import): don't truncate a price that contains a space.

Fix #1221

Author: php-coder

src/main/java/ru/mystamps/web/feature/series/importing/SeriesInfoExtractorServiceImpl.java

@@ -64,6 +64,9 @@ public class SeriesInfoExtractorServiceImpl implements SeriesInfoExtractorServic
 	private static final Pattern MICHEL_NUMBERS_REGEXP =
 		Pattern.compile("#[ ]?(?<begin>[1-9][0-9]{0,3})-(?<end>[1-9][0-9]{0,3})");
 
+	// Regular expression matches prices that have a space between digits.
+	private static final Pattern PRICE_WITH_SPACES = Pattern.compile("([0-9]) ([0-9])");
+	
 	// Regular expression that matches Rubles (Russian currency).
 	private static final Pattern RUB_CURRENCY_REGEXP = Pattern.compile("[0-9][ ]?руб");
 
@@ -367,14 +370,24 @@ public SeriesExtractedInfo extract(String pageUrl, RawParsedDataDto data) {
 		return url;
 	}
 
-	@SuppressWarnings("PMD.AvoidInstantiatingObjectsInLoops")
+	@SuppressWarnings({
+		"PMD.AvoidInstantiatingObjectsInLoops",
+		"PMD.AvoidReassigningParameters",
+		"checkstyle:parameterassignment"
+	})
 	/* default */ BigDecimal extractPrice(String fragment) {
 		if (StringUtils.isBlank(fragment)) {
 			return null;
 		}
 
 		log.debug("Determine price from '{}'", fragment);
 
+		// "1 000" -> "1000"
+		Matcher matcher = PRICE_WITH_SPACES.matcher(fragment);
+		if (matcher.find()) {
+			fragment = matcher.replaceAll("$1$2");
+		}
+		
 		String[] candidates = StringUtils.split(fragment, ' ');
 		for (String candidate : candidates) {
 			// replace comma with dot to handle 10,5 in the same way as 10.5

src/test/groovy/ru/mystamps/web/feature/series/importing/SeriesInfoExtractorServiceImplTest.groovy

@@ -618,6 +618,17 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 			'10 руб 16 коп' | _
 	}
 
+	@Unroll
+	def 'extractPrice() should ignore a space in "#fragment"'(String fragment, BigDecimal result) {
+		expect:
+			service.extractPrice(fragment) == result
+		where:
+			fragment         || result
+			'10 800,00 руб.' || new BigDecimal('10800')
+			'1 200'          || new BigDecimal('1200')
+			'1 000 000'      || new BigDecimal('1000000')
+	}
+	
 	//
 	// Tests for extractCurrency()
 	//