Merge pull request #605 from spacious-team/task-tbank

Поддержка брокера ТБанк

README.md

@@ -104,7 +104,7 @@ Investbook в сравнении с [Intelinvest](https://intelinvest.ru) и [Sn
    при необходимости позволит безболезненно перенести накопленные данные в другое приложение учета инвестиций.
 
 ### Брокеры
-Приложение анализирует отчеты брокеров Тинькофф (xlsx), Сбербанк (xlsx), ВТБ (xls), Промсвязьбанк (xlsx, xml)
+Приложение анализирует отчеты брокеров ТБанк / Тинькофф (xlsx), Сбербанк (xlsx), ВТБ (xls), Промсвязьбанк (xlsx, xml)
 и Твой Брокер / Уралсиб (zip с xls). Если ваш счет открыт у другого брокера,
 напишите [нам](https://t.me/investbook_support). Также вы можете уже на вашей версии приложения воспользоваться
 [формами](src/main/asciidoc/investbook-forms.adoc) ввода информации или 

src/main/java/ru/investbook/parser/tinkoff/TinkoffBrokerReport.java

@@ -43,6 +43,8 @@ public class TinkoffBrokerReport extends AbstractExcelBrokerReport {
     private static final String PORTFOLIO_MARKER = "Инвестор:";
     private static final Predicate<Object> tinkoffReportPredicate = cell ->
             (cell instanceof String) && ((String) cell).contains("Тинькофф");
+    private static final Predicate<Object> tbankReportPredicate = cell ->
+            (cell instanceof String) && ((String) cell).contains("ТБанк");
     private static final Predicate<Object> dateMarkerPredicate = cell ->
             (cell instanceof String) && ((String) cell).contains("за период");
 
@@ -61,8 +63,9 @@ public TinkoffBrokerReport(String excelFileName, InputStream is, SecurityRegistr
     }
 
     public static void checkReportFormat(String excelFileName, ReportPage reportPage) {
-        if (reportPage.find(0, 2, tinkoffReportPredicate) == TableCellAddress.NOT_FOUND) {
-            throw new RuntimeException("В файле " + excelFileName + " не содержится отчет брокера Тинькофф");
+        if (reportPage.find(0, 2, tbankReportPredicate) == TableCellAddress.NOT_FOUND &&
+                reportPage.find(0, 2, tinkoffReportPredicate) == TableCellAddress.NOT_FOUND) {
+            throw new RuntimeException("В файле " + excelFileName + " не содержится отчет брокера ТБанк (Тинькофф)");
         }
     }
 

src/main/java/ru/investbook/parser/tinkoff/TinkoffSecurityTransactionTable.java

@@ -39,6 +39,7 @@
 import java.math.BigDecimal;
 import java.time.Instant;
 
+import static org.springframework.util.StringUtils.hasLength;
 import static ru.investbook.parser.tinkoff.TinkoffSecurityTransactionTable.TransactionTableHeader.*;
 
 @Slf4j
@@ -77,12 +78,20 @@ private TinkoffSecurityTransactionTable(String tableNamePrefix,
 
     @Override
     protected AbstractTransaction parseRow(TableRow row) {
-        long _tradeId = row.getLongCellValueOrDefault(TRADE_ID, -1);
-        if (_tradeId == -1) return null;
-        String tradeId = String.valueOf(_tradeId);
+        String tradeId = row.getStringCellValueOrDefault(TRADE_ID, "");
+        if (!hasLength(tradeId)) return null;
+        String operation = row.getStringCellValue(OPERATION).toLowerCase();
+        boolean isBuy = operation.contains("покупка");
+        boolean isRepo = operation.contains("репо");
+        if (isRepo) {
+            // Сделки переноса РЕПО "РЕПО 1 Покупка" и "РЕПО 2 Продажа" имеют одинаковый tradeId и timestamp.
+            // Это синтетические сделки, но эти сделки имеют комиссии, поэтому их нужно иметь возможность сохранить в БД.
+            // Требуется уникальный tradeId для обоих РЕПО сделок.
+            String tradeIdSuffix = isBuy ? "repobuy" : "reposell";
+            tradeId += tradeIdSuffix;
+        }
 
         int securityId = transactionTableHelper.getSecurityId(row, codeAndIsin, getReport().getSecurityRegistrar());
-        boolean isBuy = row.getStringCellValue(OPERATION).toLowerCase().contains("покупка");
         int count = Math.abs(row.getIntCellValue(COUNT));
         BigDecimal amount = row.getBigDecimalCellValue(AMOUNT).abs();
         amount = isBuy ? amount.negate() : amount;