PHPOffice /
PhpSpreadsheet
| Conditions | 2 |
| Paths | 2 |
| Total Lines | 125 |
| Code Lines | 105 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 1 | ||
| Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 16 | public function testHideWithMerge(): void |
||
| 17 | { |
||
| 18 | $spreadsheet = new Spreadsheet(); |
||
| 19 | $worksheet = $spreadsheet->getActiveSheet(); |
||
| 20 | // just some labels for better visualisation of the problem |
||
| 21 | $worksheet->setCellValue('A1', 'A'); |
||
| 22 | $worksheet->setCellValue('B1', 'B'); |
||
| 23 | $worksheet->setCellValue('C1', 'C'); |
||
| 24 | // setting the row height to better visualize the problem |
||
| 25 | for ($i = 1; $i <= 10; ++$i) { |
||
| 26 | $worksheet->getRowDimension($i)->setRowHeight(17); |
||
| 27 | } |
||
| 28 | // Headline - merged over two cells AND two rows |
||
| 29 | $worksheet->mergeCells('B2:C3'); |
||
| 30 | $worksheet->setCellValue('B2', 'Hello World Headline'); |
||
| 31 | $worksheet->getStyle('B2:C3')->getFont()->setBold(true); |
||
| 32 | $worksheet->getStyle('B2:C3') |
||
| 33 | ->getAlignment() |
||
| 34 | ->setHorizontal(Alignment::HORIZONTAL_CENTER); |
||
| 35 | $worksheet->getStyle('B2:C3') |
||
| 36 | ->getBorders() |
||
| 37 | ->getAllBorders() |
||
| 38 | ->setBorderStyle(Border::BORDER_THIN) |
||
| 39 | ->setColor(new Color(Color::COLOR_BLACK)); |
||
| 40 | |||
| 41 | // Content 1 - merge over two rows |
||
| 42 | $worksheet->mergeCells('B4:B5'); |
||
| 43 | $worksheet->mergeCells('C4:C5'); |
||
| 44 | $worksheet->setCellValue('B4', 'Label 1'); |
||
| 45 | $worksheet->setCellValue('C4', 'Text 1'); |
||
| 46 | $worksheet->getStyle('B4:B5')->getFont()->setBold(true); |
||
| 47 | $worksheet->getStyle('B4:C5') |
||
| 48 | ->getAlignment() |
||
| 49 | ->setVertical(Alignment::VERTICAL_CENTER); |
||
| 50 | $worksheet->getStyle('B4:B5') |
||
| 51 | ->getBorders() |
||
| 52 | ->getAllBorders() |
||
| 53 | ->setBorderStyle(Border::BORDER_THIN) |
||
| 54 | ->setColor(new Color(Color::COLOR_BLACK)); |
||
| 55 | $worksheet->getStyle('C4:C5') |
||
| 56 | ->getBorders() |
||
| 57 | ->getAllBorders() |
||
| 58 | ->setBorderStyle(Border::BORDER_THIN) |
||
| 59 | ->setColor(new Color(Color::COLOR_BLACK)); |
||
| 60 | |||
| 61 | // Content 2 - merge over two rows |
||
| 62 | $worksheet->mergeCells('B6:B7'); |
||
| 63 | $worksheet->mergeCells('C6:C7'); |
||
| 64 | $worksheet->setCellValue('B6', 'Label 2'); |
||
| 65 | $worksheet->setCellValue('C6', 'Text 2'); |
||
| 66 | $worksheet->getStyle('B6:B7')->getFont()->setBold(true); |
||
| 67 | $worksheet->getStyle('B6:C7') |
||
| 68 | ->getAlignment() |
||
| 69 | ->setVertical(Alignment::VERTICAL_CENTER); |
||
| 70 | $worksheet->getStyle('B6:B7') |
||
| 71 | ->getBorders() |
||
| 72 | ->getAllBorders() |
||
| 73 | ->setBorderStyle(Border::BORDER_THIN) |
||
| 74 | ->setColor(new Color(Color::COLOR_BLACK)); |
||
| 75 | $worksheet->getStyle('C6:C7') |
||
| 76 | ->getBorders() |
||
| 77 | ->getAllBorders() |
||
| 78 | ->setBorderStyle(Border::BORDER_THIN) |
||
| 79 | ->setColor(new Color(Color::COLOR_BLACK)); |
||
| 80 | |||
| 81 | // This is where the error was introduced (!!!) |
||
| 82 | $worksheet->getColumnDimension('A')->setVisible(false); |
||
| 83 | $Dompdf = new Dompdf($spreadsheet); |
||
| 84 | $html = $Dompdf->generateHtmlAll(); |
||
| 85 | $html = preg_replace('/^\\s+/m', '', $html) ?? $html; |
||
| 86 | $html = preg_replace('/[\\n\\r]/', '', $html) ?? $html; |
||
| 87 | self::assertStringContainsString( |
||
| 88 | 'table.sheet0 .column0 { display:none }', |
||
| 89 | $html |
||
| 90 | ); |
||
| 91 | self::assertStringContainsString( |
||
| 92 | '<tr class="row0">' |
||
| 93 | . '<td class="column0 style0 s" style="width:42pt; height:17pt">A</td>' |
||
| 94 | . '<td class="column1 style0 s" style="width:42pt; height:17pt">B</td>' |
||
| 95 | . '<td class="column2 style0 s" style="width:42pt; height:17pt">C</td>' |
||
| 96 | . '</tr>', |
||
| 97 | $html |
||
| 98 | ); |
||
| 99 | self::assertStringContainsString( |
||
| 100 | '<tr class="row1">' |
||
| 101 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 102 | . '<td class="column1 style1 s style1" style="width:84pt; height:17pt" colspan="2" rowspan="2">Hello World Headline</td>' |
||
| 103 | . '</tr>', |
||
| 104 | $html |
||
| 105 | ); |
||
| 106 | self::assertStringContainsString( |
||
| 107 | '<tr class="row2">' |
||
| 108 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 109 | . '</tr>', |
||
| 110 | $html |
||
| 111 | ); |
||
| 112 | self::assertStringContainsString( |
||
| 113 | '<tr class="row3">' |
||
| 114 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 115 | . '<td class="column1 style2 s style2" style="width:42pt; height:17pt" rowspan="2">Label 1</td>' |
||
| 116 | . '<td class="column2 style3 s style3" style="width:42pt; height:17pt" rowspan="2">Text 1</td>' |
||
| 117 | . '</tr>', |
||
| 118 | $html |
||
| 119 | ); |
||
| 120 | self::assertStringContainsString( |
||
| 121 | '<tr class="row4">' |
||
| 122 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 123 | . '</tr>', |
||
| 124 | $html |
||
| 125 | ); |
||
| 126 | self::assertStringContainsString( |
||
| 127 | '<tr class="row5">' |
||
| 128 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 129 | . '<td class="column1 style2 s style2" style="width:42pt; height:17pt" rowspan="2">Label 2</td>' |
||
| 130 | . '<td class="column2 style3 s style3" style="width:42pt; height:17pt" rowspan="2">Text 2</td>' |
||
| 131 | . '</tr>', |
||
| 132 | $html |
||
| 133 | ); |
||
| 134 | self::assertStringContainsString( |
||
| 135 | '<tr class="row6">' |
||
| 136 | . '<td class="column0 style0" style="width:42pt; height:17pt"> </td>' |
||
| 137 | . '</tr>', |
||
| 138 | $html |
||
| 139 | ); |
||
| 140 | $spreadsheet->disconnectWorksheets(); |
||
| 141 | } |
||
| 143 |
