Improve reporting

dimaip · dimaip · commit e438f68efee6 · 2026-04-30T09:50:43.000+03:00
diff --git a/Packages/Application/Sfi.Umo/Classes/Controller/BackendController.php b/Packages/Application/Sfi.Umo/Classes/Controller/BackendController.php
@@ -68,6 +68,7 @@ function arrayDeepSet(&$array, $path, $value)
 
 class BackendController extends AbstractModuleController
 {
+    const INTERNAL_SIGNED_PDFS_RUN_MARKER = 'Генерация внутренних подписанных PDF запущена.';
 
     /**
      * @Flow\Inject
@@ -138,13 +139,10 @@ class BackendController extends AbstractModuleController
     /**
      * @return void
      */
-    public function indexAction()
+    public function indexAction(bool $showInternalSignedPdfsLog = false)
     {
-        $logPath = $this->getInternalSignedPdfsLogPath();
-
-        $this->view->assign('internalSignedPdfsBasePath', $this->getInternalSignedPdfsBasePath());
-        $this->view->assign('internalSignedPdfsLogPath', $logPath);
-        $this->view->assign('internalSignedPdfsLog', $this->readLogTail($logPath));
+        $this->view->assign('showInternalSignedPdfsLog', $showInternalSignedPdfsLog);
+        $this->view->assign('internalSignedPdfsLog', $showInternalSignedPdfsLog ? $this->readLastInternalSignedPdfsRunLog($this->getInternalSignedPdfsLogPath()) : '');
     }
 
     /**
@@ -166,73 +164,17 @@ public function generateSignedPdfsAction()
      */
     public function generateInternalSignedPdfsAction()
     {
-        $basePath = $this->getInternalSignedPdfsBasePath();
-        $logPath = $this->getInternalSignedPdfsLogPath();
-
-        if (!$this->appendInternalSignedPdfsLog(sprintf('Запуск генерации внутренних подписанных PDF из интерфейса. Папка: %s', $basePath))) {
-            $this->addFlashMessage(
-                'Не удалось записать лог внутренних подписанных PDF: %s',
-                'Ошибка записи лога',
-                \Neos\Error\Messages\Message::SEVERITY_WARNING,
-                [$logPath]
-            );
-        }
-
+        $this->appendInternalSignedPdfsLog(self::INTERNAL_SIGNED_PDFS_RUN_MARKER, true);
         Scripts::executeCommandAsync('sfi.sfi:signature:generateinternalsignedpdfs', $this->flowSettings, []);
-        $this->addFlashMessage(sprintf('Генерация внутренних подписанных PDF запущена в фоновом режиме. Результаты будут сохранены в %s/*/signed/. Лог: %s', $basePath, $logPath));
-        $this->redirect('index');
-    }
-
-    /**
-     * Check internal signed PDF input folders without generating files
-     *
-     * @return void
-     */
-    public function dryRunInternalSignedPdfsAction()
-    {
-        $basePath = $this->getInternalSignedPdfsBasePath();
-        $logPath = $this->getInternalSignedPdfsLogPath();
-
-        $this->appendInternalSignedPdfsLog(sprintf('Проверка внутренних PDF без генерации запущена из интерфейса. Папка: %s', $basePath));
-
-        $bufferLevel = ob_get_level();
-        ob_start();
-
-        try {
-            Scripts::executeCommand('sfi.sfi:signature:generateinternalsignedpdfs', $this->flowSettings, true, ['dry-run' => 'true']);
-            $output = trim($this->collectOutputBuffer($bufferLevel));
-
-            $this->appendInternalSignedPdfsLog($output !== '' ? "Результат проверки:\n" . $output : 'Результат проверки: команда не вернула вывода.');
-            $this->addFlashMessage(sprintf('Проверка внутренних PDF завершена. Результат записан в лог: %s', $logPath));
-        } catch (\Exception $exception) {
-            $output = trim($this->collectOutputBuffer($bufferLevel));
-            $message = 'Ошибка проверки внутренних PDF: ' . $exception->getMessage();
-            if ($output !== '') {
-                $message .= "\nВывод команды:\n" . $output;
-            }
-            $this->appendInternalSignedPdfsLog($message);
-            $this->addFlashMessage(
-                'Проверка внутренних PDF завершилась с ошибкой. Подробности записаны в лог: %s',
-                'Ошибка проверки',
-                \Neos\Error\Messages\Message::SEVERITY_ERROR,
-                [$logPath]
-            );
-        }
-
-        $this->redirect('index');
-    }
-
-    protected function getInternalSignedPdfsBasePath(): string
-    {
-        return rtrim(FLOW_PATH_WEB, '/') . '/umo/internal';
+        $this->redirect('index', null, null, ['showInternalSignedPdfsLog' => true]);
     }
 
     protected function getInternalSignedPdfsLogPath(): string
     {
         return rtrim(FLOW_PATH_DATA, '/') . '/Logs/InternalSignedPdfs/generation.log';
     }
 
-    protected function appendInternalSignedPdfsLog(string $message): bool
+    protected function appendInternalSignedPdfsLog(string $message, bool $truncate = false): bool
     {
         $message = trim($message);
         if ($message === '') {
@@ -250,58 +192,38 @@ protected function appendInternalSignedPdfsLog(string $message): bool
             $entry .= $timestamp . ' ' . $line . PHP_EOL;
         }
 
-        return @file_put_contents($this->getInternalSignedPdfsLogPath(), $entry, FILE_APPEND | LOCK_EX) !== false;
+        $flags = $truncate ? LOCK_EX : FILE_APPEND | LOCK_EX;
+        return @file_put_contents($this->getInternalSignedPdfsLogPath(), $entry, $flags) !== false;
     }
 
-    protected function readLogTail(string $logPath, int $maxBytes = 65536): string
+    protected function readLastInternalSignedPdfsRunLog(string $logPath): string
     {
         if (!is_file($logPath)) {
             return '';
         }
 
-        $size = filesize($logPath);
-        if ($size === false) {
+        $contents = file_get_contents($logPath);
+        if ($contents === false) {
             return '';
         }
 
-        $handle = fopen($logPath, 'rb');
-        if ($handle === false) {
+        $lines = preg_split('/\r\n|\r|\n/', trim($contents));
+        if (!is_array($lines)) {
             return '';
         }
 
-        if ($size > $maxBytes) {
-            fseek($handle, -$maxBytes, SEEK_END);
-        }
-
-        $contents = stream_get_contents($handle);
-        fclose($handle);
-
-        if ($contents === false) {
-            return '';
-        }
-
-        if ($size > $maxBytes) {
-            $firstLineBreak = strpos($contents, "\n");
-            if ($firstLineBreak !== false) {
-                $contents = substr($contents, $firstLineBreak + 1);
+        $startIndex = null;
+        foreach ($lines as $index => $line) {
+            if (strpos($line, self::INTERNAL_SIGNED_PDFS_RUN_MARKER) !== false) {
+                $startIndex = $index;
             }
-            $contents = "... показаны последние строки лога\n" . $contents;
         }
 
-        return $contents;
-    }
-
-    protected function collectOutputBuffer(int $bufferLevel): string
-    {
-        $output = '';
-        while (ob_get_level() > $bufferLevel) {
-            $buffer = ob_get_clean();
-            if ($buffer !== false) {
-                $output = $buffer . $output;
-            }
+        if ($startIndex === null) {
+            return '';
         }
 
-        return $output;
+        return implode(PHP_EOL, array_slice($lines, $startIndex));
     }
 
     protected $collectionByType = [
diff --git a/Packages/Application/Sfi.Umo/README.md b/Packages/Application/Sfi.Umo/README.md
@@ -52,17 +52,10 @@ PDF-файлы должны лежать прямо в той же папке, 
 ## Как запустить генерацию
 
 1. Откройте модуль импорта УМО в административной части сайта.
-2. Если нужно сначала проверить, какие файлы будут найдены, нажмите кнопку `Проверить внутренние PDF без генерации`.
-3. Для запуска генерации нажмите кнопку `Генерировать внутренние подписанные PDF`.
-4. Генерация запустится в фоновом режиме.
+2. Нажмите кнопку `Генерировать внутренние подписанные PDF`.
+3. Генерация запустится в фоновом режиме.
 
-Внизу страницы модуля показывается путь к рабочей папке, путь к логу и последние строки лога. После запуска генерации можно нажимать `Обновить лог`, чтобы увидеть текущий результат.
-
-Лог записывается в файл в папке логов приложения:
-
-```text
-Data/Logs/InternalSignedPdfs/generation.log
-```
+После запуска под кнопками появится журнал последней генерации. Если результат еще не обновился, обновите страницу через несколько секунд.
 
 ## Что появится в результате
 
diff --git a/Packages/Application/Sfi.Umo/Resources/Private/Templates/Backend/Index.html b/Packages/Application/Sfi.Umo/Resources/Private/Templates/Backend/Index.html
@@ -34,29 +34,19 @@
         <div><f:form.submit class="button" value="Генерировать внутренние подписанные PDF"/></div>
     </f:form>
 
-    <br/>
-
-    <f:form
-        method="post"
-        action="dryRunInternalSignedPdfs"
-        controller="Backend"
-        package="Sfi.Umo"
-    >
-        <div><f:form.submit class="button" value="Проверить внутренние PDF без генерации"/></div>
-    </f:form>
-
-    <h2>Внутренние подписанные PDF</h2>
-    <p>Папка с исходными файлами: <code>{internalSignedPdfsBasePath}</code></p>
-    <p>Лог генерации: <code>{internalSignedPdfsLogPath}</code></p>
-
-    <f:link.action action="index" class="button">Обновить лог</f:link.action>
-
-    <f:if condition="{internalSignedPdfsLog}">
-        <f:then>
-            <pre style="margin-top: 16px; padding: 12px; max-height: 480px; overflow: auto; background: #f7f7f7; border: 1px solid #ccc; white-space: pre-wrap;">{internalSignedPdfsLog}</pre>
-        </f:then>
-        <f:else>
-            <p>Лог пока пуст. Нажмите кнопку проверки или генерации, чтобы создать первую запись.</p>
-        </f:else>
+    <f:if condition="{showInternalSignedPdfsLog}">
+        <section style="margin-top: 24px; max-width: 980px; padding: 16px; color: #222; background: #fff; border: 1px solid #d6d6d6;">
+            <h2 style="margin: 0 0 8px; color: #222;">Внутренние подписанные PDF</h2>
+            <p style="margin: 0 0 12px; color: #222;">Генерация запущена. Если результат еще не появился, обновите страницу через несколько секунд.</p>
+
+            <f:if condition="{internalSignedPdfsLog}">
+                <f:then>
+                    <pre style="margin: 0; padding: 12px; max-height: 360px; overflow: auto; color: #222; background: #f7f7f7; border: 1px solid #ccc; white-space: pre-wrap;">{internalSignedPdfsLog}</pre>
+                </f:then>
+                <f:else>
+                    <p style="margin: 0; color: #222;">Журнал генерации появится здесь.</p>
+                </f:else>
+            </f:if>
+        </section>
     </f:if>
 </f:section>
diff --git a/Packages/Sites/Sfi.Sfi/Classes/Sfi/Sfi/Command/SignatureCommandController.php b/Packages/Sites/Sfi.Sfi/Classes/Sfi/Sfi/Command/SignatureCommandController.php
@@ -184,18 +184,18 @@ public function generateInternalSignedPdfsCommand(bool $force = false, bool $dry
         $logFile = $this->getInternalSignedPdfsLogPath();
 
         if (!is_dir($internalBasePath)) {
-            $this->log($logFile, $dryRun, 'Internal UMO folder does not exist: %s', [$internalBasePath]);
+            $this->log($logFile, $dryRun, 'Папка с внутренними PDF не найдена.');
             return;
         }
 
-        $this->log($logFile, $dryRun, 'Starting internal signed PDF generation. Base path: %s', [$internalBasePath]);
+        $this->log($logFile, $dryRun, 'Обработка файлов началась.');
 
         $items = $this->collectInternalSignatureItems($internalBasePath, $logFile, $dryRun);
         $total = count($items);
-        $this->log($logFile, $dryRun, 'Found %d internal PDF files to process.', [$total]);
+        $this->log($logFile, $dryRun, 'Найдено PDF-файлов: %d', [$total]);
 
         if ($total === 0) {
-            $this->log($logFile, $dryRun, 'Nothing to do.');
+            $this->log($logFile, $dryRun, 'Подходящих PDF-файлов не найдено.');
             return;
         }
 
@@ -209,61 +209,61 @@ public function generateInternalSignedPdfsCommand(bool $force = false, bool $dry
             $progress = sprintf('[%d/%d]', $index + 1, $total);
 
             if (!$force && file_exists($outputPath)) {
-                $this->log($logFile, $dryRun, '%s SKIP (exists): %s', [$progress, $displayPath]);
+                $this->log($logFile, $dryRun, '%s Уже готово: %s', [$progress, $displayPath]);
                 $skipped++;
                 continue;
             }
 
             if ($dryRun) {
-                $this->log($logFile, $dryRun, '%s DRY-RUN: would generate %s', [$progress, $displayPath]);
+                $this->log($logFile, $dryRun, '%s Будет создано: %s', [$progress, $displayPath]);
                 $processed++;
                 continue;
             }
 
             try {
                 $pdfContent = $this->generateSignedPdfContent($item, 120);
             } catch (\Exception $e) {
-                $this->log($logFile, $dryRun, '%s ERROR: %s -- %s', [$progress, $displayPath, $e->getMessage()]);
+                $this->log($logFile, $dryRun, '%s ОШИБКА: %s -- %s', [$progress, $displayPath, $e->getMessage()]);
                 $errors++;
                 continue;
             }
 
             if (strlen($pdfContent) < 5 || substr($pdfContent, 0, 5) !== '%PDF-') {
-                $this->log($logFile, $dryRun, '%s ERROR: %s -- Response is not a valid PDF (%d bytes)', [$progress, $displayPath, strlen($pdfContent)]);
+                $this->log($logFile, $dryRun, '%s ОШИБКА: сервис подписи вернул некорректный PDF для файла %s (%d байт)', [$progress, $displayPath, strlen($pdfContent)]);
                 $errors++;
                 continue;
             }
 
             $outputDir = dirname($outputPath);
             if (!is_dir($outputDir)) {
                 if (!mkdir($outputDir, 0775, true) && !is_dir($outputDir)) {
-                    $this->log($logFile, $dryRun, '%s ERROR: %s -- Could not create directory: %s', [$progress, $displayPath, $outputDir]);
+                    $this->log($logFile, $dryRun, '%s ОШИБКА: не удалось создать папку signed для файла %s', [$progress, $displayPath]);
                     $errors++;
                     continue;
                 }
             }
 
             $bytesWritten = file_put_contents($outputPath, $pdfContent);
             if ($bytesWritten === false) {
-                $this->log($logFile, $dryRun, '%s ERROR: %s -- Failed to write file', [$progress, $displayPath]);
+                $this->log($logFile, $dryRun, '%s ОШИБКА: не удалось сохранить подписанный PDF для файла %s', [$progress, $displayPath]);
                 $errors++;
                 continue;
             }
 
-            $this->log($logFile, $dryRun, '%s OK: %s (%s)', [$progress, $displayPath, $this->formatBytes($bytesWritten)]);
+            $this->log($logFile, $dryRun, '%s Готово: %s (%s)', [$progress, $displayPath, $this->formatBytes($bytesWritten)]);
             $processed++;
         }
 
         $this->log($logFile, $dryRun, '');
-        $this->log($logFile, $dryRun, '--- Summary ---');
-        $this->log($logFile, $dryRun, 'Total:     %d', [$total]);
-        $this->log($logFile, $dryRun, 'Generated: %d', [$processed]);
-        $this->log($logFile, $dryRun, 'Skipped:   %d', [$skipped]);
-        $this->log($logFile, $dryRun, 'Errors:    %d', [$errors]);
+        $this->log($logFile, $dryRun, 'Итог:');
+        $this->log($logFile, $dryRun, 'Всего:     %d', [$total]);
+        $this->log($logFile, $dryRun, 'Создано:   %d', [$processed]);
+        $this->log($logFile, $dryRun, 'Пропущено: %d', [$skipped]);
+        $this->log($logFile, $dryRun, 'Ошибок:    %d', [$errors]);
 
         if ($dryRun) {
             $this->log($logFile, $dryRun, '');
-            $this->log($logFile, $dryRun, 'This was a dry run. Remove --dry-run to actually generate files.');
+            $this->log($logFile, $dryRun, 'Это была проверка без создания файлов.');
         }
     }
 
@@ -398,7 +398,7 @@ protected function collectInternalSignatureItems(string $internalBasePath, strin
 
                 $items[] = [
                     'relativePath' => $relativePath,
-                    'displayPath' => $relativePath . ' -> internal/' . $relativeFolderPath . '/signed/' . $filename,
+                    'displayPath' => $relativeFolderPath . '/' . $filename,
                     'outputPath' => $outputPath,
                     'url' => self::encodeURI($sourceUrlPrefix . $sourceUrl),
                     'signDate' => $metadata['signDate']->format('d.m.Y'),
