mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-14 03:56:40 +00:00
oleibman
f3f09bc9a6
Fix #4773. Mpdf is applying cell style only to the first line of a multi-line string. Explanation - Html Writer replaces newlines with `<br />` by calling `nl2br`, at least that was the intention. However, `nl2br` doesn't replace - it prepends. This doesn't do any harm in Html, Dompdf, or Tcpdf. However, Mpdf is known to be subject to occasional regexp backtracking errors when parsing large blocks of html. To avoid this, we chunk the html by splitting it at newlines, but this causes the styling to be lost for rich text elements following a newline. The solution to this problem seems easy enough - replace `nl2br` with a routine that replaces rather than prepends. Unfortunately, Html Reader needs the newlines intact in order to parse things correctly. So a solution where the `nl2br` calls are used for non-Mpdf and using a substitute routine for Mpdf ought to work. However, doing my testing uncovered another bug - rich text with newlines can result in multiple `<br />` tags (see new Html test). The solution to this is a bit kludgey, but it does seem to fix both the Mpdf problem and the newly discovered Html problem. It will also tend to avoid the very minor problem of generating different line endings on Windows systems.
56 lines
2.3 KiB
PHP
56 lines
2.3 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Writer\Mpdf;
|
|
|
|
use PhpOffice\PhpSpreadsheet\RichText\RichText;
|
|
use PhpOffice\PhpSpreadsheet\Spreadsheet;
|
|
use PhpOffice\PhpSpreadsheet\Style\Alignment;
|
|
use PhpOffice\PhpSpreadsheet\Writer\Pdf\Mpdf as MpdfWriter;
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
class Issue4773Test extends TestCase
|
|
{
|
|
public static function testLineBreaks(): void
|
|
{
|
|
$spreadsheet = new Spreadsheet();
|
|
$sheet = $spreadsheet->getActiveSheet();
|
|
|
|
$sheet->getPageSetup()->setFitToWidth(1);
|
|
$sheet->getPageSetup()->setFitToHeight(0);
|
|
$sheet->getPageMargins()->setTop(0.2);
|
|
$sheet->getPageMargins()->setRight(0.2);
|
|
$sheet->getPageMargins()->setLeft(0.2);
|
|
$sheet->getPageMargins()->setBottom(0.5);
|
|
|
|
$sheet->getColumnDimension('A')->setWidth(20);
|
|
$sheet->getColumnDimension('B')->setWidth(20);
|
|
|
|
$sheet->getStyle('A1')->getFont()
|
|
->setSize(12)
|
|
->setItalic(true);
|
|
$sheet->getStyle('A1')->getAlignment()
|
|
->setWrapText(true);
|
|
$sheet->setCellValue('A1', "ABC\nDEF\nGHI");
|
|
|
|
$richText = new RichText();
|
|
$run1 = $richText->createTextRun("bold\n");
|
|
$run1->getFont()?->setBold(true);
|
|
$run3 = $richText->createTextRun('italic');
|
|
$run3->getFont()?->setItalic(true);
|
|
$sheet->getStyle('B1')->getAlignment()
|
|
->setVertical(Alignment::VERTICAL_CENTER)
|
|
->setWrapText(true);
|
|
$sheet->setCellValue('B1', $richText);
|
|
|
|
$writer = new MpdfWriter($spreadsheet);
|
|
$content = $writer->generateHtmlAll();
|
|
$expected1 = '<td class="column0 style1 s" style="width:105pt">ABC<br />DEF<br />GHI</td>';
|
|
self::assertStringContainsString($expected1, $content, 'br tags without newline in normal text');
|
|
$expected2 = '<td class="column1 style2 inlineStr" style="width:105pt"><span style="font-weight:bold; text-decoration:normal; font-style:normal; color:#000000; font-family:\'Calibri\'; font-size:11pt">bold<br /></span><span style="font-weight:normal; text-decoration:normal; font-style:italic; color:#000000; font-family:\'Calibri\'; font-size:11pt">italic</span></td>';
|
|
self::assertStringContainsString($expected2, $content, 'br tag without newline in rich text');
|
|
$spreadsheet->disconnectWorksheets();
|
|
}
|
|
}
|