Доработка проверки периодов отчёта

CHANGELOG.md

@@ -1,5 +1,11 @@
 # CHANGELOG
 
+### [0.6.3] - 2020-09-22
+
+- Доработка проверки периодов. Если формат периода отчета отличается от описанного формата в документации. Происходит попытка конвертации его допустимый. Механизм конвертации должен полностью повторять аналогичный в оффициальной программе от РосСтата.
+- Вернул на место проверку формата значений ячеек. Удалил по невнимательности.
+
+
 ### [0.6.2] - 2020-09-21
 
 - Доработка проверок формата строк и значений в графах:

README.md

@@ -1,10 +1,10 @@
 # РосСтат ФЛК
 
-Форматно-логический контроль отчетности РосСтата.
+Инструмент для форматно-логического контроля отчетности, отправляемой в РосСтат.
 
 Документация описывающая формат отчетности и контроли - [Приказ РосСтата от 28.10.2010 №372](http://www.consultant.ru/document/cons_doc_LAW_115689/)
 
-**Не готово для использования в продакшене! [CHANGELOG](CHANGELOG.md)**
+Список изменений - [CHANGELOG](CHANGELOG.md)
 
 ## Установка
 ```bash

rosstat/report.py

@@ -5,6 +5,20 @@
 from .validator.schema import str_int
 
 
+def max_divider(num, terms):
+    def _calc(a, b):
+        mn = min(a, b)
+        mx = max(a, b)
+        for i in range(mn, 1, -1):
+            if mn % i == 0 and mx % i == 0:
+                return i
+        return 1
+
+    for term_id in terms:
+        num = _calc(num, int(term_id))
+    return num
+
+
 @dataclass
 class Row:
     code: str
@@ -73,8 +87,9 @@ class Report:
     _year: str = None
     _title: Dict[str, str] = None
     _data: Dict[str, Section] = None
+    _period_raw: str = None
     _period_type: Optional[str] = None
-    _period_code: str = None
+    _period_code: Optional[str] = None
     _row_counters: defdict = f(default_factory=lambda: defdict(int))
 
     def __repr__(self):
@@ -148,14 +163,41 @@ def _read_data(self, xml):
     def _read_row_specs(self, row):
         return (row.attrib.get(f's{i}') for i in range(1, 4))
 
-    def _get_periods(self, xml):
-        period = xml.xpath('/report/@period')[0]
-        if len(period) != 4:
-            self._period_code = str_int(period)
-        else:
-            period_type, period_code = period[:2], period[2:]
-            self._period_type = str_int(period_type)
-            self._period_code = str_int(period_code)
-
     def _get_year(self, xml):
         self._year = xml.xpath('/report/@year')[0]
+
+    def _get_periods(self, xml):
+        self._period_raw = xml.xpath('/report/@period')[0]
+        if len(self._period_raw) == 4:
+            self._period_type = str_int(self._period_raw[:2])
+            self._period_code = str_int(self._period_raw[2:])
+
+    def set_periods(self, schema):
+        try:
+            periods_id = self._get_periods_id(schema.dics)
+
+            if int(self._period_raw) not in periods_id:
+                return False
+
+            max_code = max(periods_id)
+
+            if max_code <= int(schema.idp):
+                self._period_type = schema.idp
+                self._period_code = self._period_raw
+                return True
+
+            max_div = max_divider(max_code, periods_id)
+
+            if max_code <= int(schema.idp) * max_div:
+                self._period_type = schema.idp
+                self._period_code = str(int(int(self._period_raw) / max_div))
+                return True
+            return False
+        except Exception as ex:
+            return False
+
+    def _get_periods_id(self, dics):
+        try:
+            return [int(term_id) for term_id in dics['s_time'].keys()]
+        except KeyError:
+            return [int(term_id) for term_id in dics['s_mes'].keys()]

rosstat/validator/checkers/format.py

@@ -49,7 +49,7 @@ def check(self, obj, spec=None, specs_map=None):
             self._check_row(obj, spec, specs_map)
 
     def _check_cell(self, cell):
-        '''Метод вызова проверки формата значения'''
+        '''Метод проверки значения'''
         self.__check_format(cell)
         if self.vld_type == '1':
             self.__check_value_dic(cell)
@@ -59,6 +59,7 @@ def _check_cell(self, cell):
             self.__check_value_list(cell)
 
     def _check_row(self, row, spec, specs_map):
+        '''Метод проверки специфик строки'''
         if self.vld_type == '4':
             self.__check_value_dic_add(row, spec)
         elif self.vld_type == '5':
@@ -88,11 +89,12 @@ def __check_value_list(self, value):
             raise OutOfList()
 
     def __check_value_dic_add(self, row, spec):
-        '''Проверка на вхождение в справочник приложение'''
+        '''Проверка на вхождение в справочник'''
         if getattr(row, spec) not in self._dics[self.vld_param]:
             raise OutOfAdditionDict()
 
     def __check_value_dic_coord(self, row, spec, specs_map):
+        '''Проверка на вхождение в справочник и связь с главной спецификой'''
         dic, coords = self.vld_param.split('=#')
         *_, c_idx = coords.split(',')
         spec_value = getattr(row, spec)

rosstat/validator/schema.py

@@ -77,7 +77,7 @@ def _prepare_format(self):
         return form
 
     def _get_adds(self, section, section_code):
-        '''Получение дополнительных данных для проверки: черекеры по умолчанию,
+        '''Получение дополнительных данных для проверки: чекеры по умолчанию,
            заполнение словаря размерности секций отчёта, определение имен
            колонок специфик
         '''
@@ -134,11 +134,14 @@ def _check(self, report):
                 break
 
     def _check_attributes(self, report):
-        '''Проверка соответствия атрибутов шаблона: периода и года'''
+        '''Проверка соответствия атрибутов шаблона: периодов и года'''
         if report.period_type is not None and report.period_type != self.idp:
             self._add_error('Тип периодичности отчёта не соответствует '
                             'типу периодичности шаблона')
 
+        if report.period_code is None and not report.set_periods(self):
+            self._add_error('Неверное значение периода отчёта')
+
         if not year_pattern.match(report.year):
             self._add_error('Указан недопустимый год')
 
@@ -206,6 +209,7 @@ def _check_format(self, report):
             for r_idx, rows in section.items():
                 for row in rows:
                     self.__check_row(row, s_idx, r_idx)
+                    self.__check_cells(row, s_idx, r_idx)
 
     def __check_row(self, row, s_idx, r_idx):
         '''Итерация по ожидаемым спецификам с их последующей проверкой'''
@@ -233,7 +237,7 @@ def __check_fmt(self, s_idx, r_idx, c_idx, *args, err_msg=None):
             print('Unexpected Error', traceback.format_exc())
 
     def __get_format_checker(self, s_idx, r_idx, c_idx):
-        '''Получение чекера для поля'''
+        '''Получение чекера для поля/строки'''
         try:
             return self.format[s_idx][r_idx][c_idx]
         except KeyError:

setup.py

@@ -2,7 +2,7 @@
 
 setup(
     name='rosstat-flc',
-    version='0.6.2',
+    version='0.6.3',
     description='Tool for format-logistic control of reports sent to RosStat',
     long_description=open('README.md', 'r').read(),
     long_description_content_type="text/markdown",