|
| 1 | +<?php |
| 2 | + |
| 3 | +declare(strict_types=1); |
| 4 | + |
| 5 | +namespace PhpOffice\PhpSpreadsheetTests\Writer\Xlsx; |
| 6 | + |
| 7 | +use PhpOffice\PhpSpreadsheet\Spreadsheet; |
| 8 | +use PhpOffice\PhpSpreadsheet\Style\Alignment; |
| 9 | +use PhpOffice\PhpSpreadsheet\Style\Conditional; |
| 10 | +use PhpOffice\PhpSpreadsheet\Style\Fill; |
| 11 | +use PhpOffice\PhpSpreadsheet\Worksheet\Worksheet; |
| 12 | +use PhpOffice\PhpSpreadsheet\Writer\Xlsx as XlsxWriter; |
| 13 | +use PHPUnit\Framework\TestCase; |
| 14 | + |
| 15 | +class Issue4025Test extends TestCase |
| 16 | +{ |
| 17 | + private static function merge(Worksheet $sheet, string $start, string $end): void |
| 18 | + { |
| 19 | + $sheet->mergeCells("$start:$end"); |
| 20 | + $sheet->getStyle($start)->getAlignment()->setHorizontal(Alignment::HORIZONTAL_CENTER); |
| 21 | + $sheet->getStyle($start)->getAlignment()->setVertical(Alignment::VERTICAL_CENTER); |
| 22 | + } |
| 23 | + |
| 24 | + public function testIssue4025(): void |
| 25 | + { |
| 26 | + // Writing alignment in conditional style caused weirdness |
| 27 | + $spreadsheet = new Spreadsheet(); |
| 28 | + $sheet = $spreadsheet->getActiveSheet(); |
| 29 | + $sheet->setRightToLeft(true); |
| 30 | + $sheet->setCellValue('E1', 'אושר עד'); |
| 31 | + $sheet->setCellValue('M1', 'ויקטורי'); |
| 32 | + $sheet->setCellValue('E2', '2024-05-01'); |
| 33 | + $sheet->setCellValue('H2', '2024-05-12'); |
| 34 | + $sheet->setCellValue('M2', '2024-05-01'); |
| 35 | + $sheet->setCellValue('P2', '2024-05-12'); |
| 36 | + $sheet->setCellValue('A3', 'ברקוד'); |
| 37 | + $sheet->setCellValue('B3', 'תיאור מוצר'); |
| 38 | + $sheet->setCellValue('C3', 'קטגוריה'); |
| 39 | + $sheet->setCellValue('D3', 'יצרן'); |
| 40 | + $sheet->setCellValue('E3', 'מחיר'); |
| 41 | + $sheet->setCellValue('F3', 'מבצע'); |
| 42 | + $sheet->setCellValue('G3', 'קובע'); |
| 43 | + $sheet->setCellValue('H3', 'מחיר'); |
| 44 | + $sheet->setCellValue('I3', 'מבצע'); |
| 45 | + $sheet->setCellValue('J3', 'קובע'); |
| 46 | + $sheet->setCellValue('K3', 'הפרש ב-₪'); |
| 47 | + $sheet->setCellValue('L3', 'הפרש ב-₪'); |
| 48 | + $sheet->setCellValue('M3', 'מחיר'); |
| 49 | + $sheet->setCellValue('N3', 'מבצע'); |
| 50 | + $sheet->setCellValue('O3', 'קובע'); |
| 51 | + $sheet->setCellValue('P3', 'מחיר'); |
| 52 | + $sheet->setCellValue('Q3', 'מבצע'); |
| 53 | + $sheet->setCellValue('R3', 'קובע'); |
| 54 | + $sheet->setCellValue('S3', 'הפרש ב-₪'); |
| 55 | + $sheet->setCellValue('T3', 'הפרש ב-₪'); |
| 56 | + $sheet->setCellValue('A4', '7290010945306'); |
| 57 | + $sheet->setCellValue('B4', 'חלב טרי בקרטון 3% בד"צ טרה 2 ליטר'); |
| 58 | + $sheet->setCellValue('C4', 'חלב טרי'); |
| 59 | + $sheet->setCellValue('D4', 'החברה המרכזית ל.מ.ק - טרה'); |
| 60 | + |
| 61 | + $sheet->setCellValue('E4', 13.50); |
| 62 | + $sheet->setCellValue('F4', '-'); |
| 63 | + //$sheet->setCellValue('G4', 13.50); |
| 64 | + $sheet->setCellValue('G4', 14.70); |
| 65 | + $sheet->setCellValue('H4', 14.20); |
| 66 | + $sheet->setCellValue('I4', '-'); |
| 67 | + $sheet->setCellValue('J4', 14.20); |
| 68 | + $sheet->setCellValue('K4', 0.70); |
| 69 | + $sheet->setCellValue('L4', 0.0519); |
| 70 | + $sheet->setCellValue('M4', 13.62); |
| 71 | + $sheet->setCellValue('N4', '-'); |
| 72 | + $sheet->setCellValue('O4', 13.62); |
| 73 | + $sheet->setCellValue('P4', 14.22); |
| 74 | + $sheet->setCellValue('Q4', '-'); |
| 75 | + $sheet->setCellValue('R4', 14.22); |
| 76 | + $sheet->setCellValue('S4', 0.60); |
| 77 | + $sheet->setCellValue('T4', 0.0441); |
| 78 | + |
| 79 | + $sheet->setCellValue('E10', '=SUM(G4:G10)'); |
| 80 | + $sheet->setCellValue('H10', '=SUM(J4:J10)'); |
| 81 | + $sheet->setCellValue('K10', '=COUNTIF(K4:K9,">0")'); |
| 82 | + $sheet->setCellValue('M10', '=SUM(O4:O10)'); |
| 83 | + $sheet->setCellValue('P10', '=SUM(R4:R10)'); |
| 84 | + $sheet->setCellValue('S10', '=COUNTIF(S4:S9,">0")'); |
| 85 | + $sheet->setCellValue('H11', '=(SUM(J4:J10) / SUM(G4:G10) - 1)'); |
| 86 | + $sheet->setCellValue('K11', '=COUNTIF(K4:K9,"<0")'); |
| 87 | + $sheet->setCellValue('P11', '=(SUM(R4:R10) / SUM(O4:O10) - 1)'); |
| 88 | + $sheet->setCellValue('S11', '=COUNTIF(S4:S9,"<0")'); |
| 89 | + |
| 90 | + $sheet->setAutoFilter('A3:T3'); |
| 91 | + self::merge($sheet, 'E1', 'L1'); |
| 92 | + self::merge($sheet, 'E2', 'G2'); |
| 93 | + self::merge($sheet, 'H2', 'J2'); |
| 94 | + self::merge($sheet, 'K2', 'L2'); |
| 95 | + self::merge($sheet, 'M1', 'T1'); |
| 96 | + self::merge($sheet, 'M2', 'O2'); |
| 97 | + self::merge($sheet, 'P2', 'R2'); |
| 98 | + self::merge($sheet, 'S2', 'T2'); |
| 99 | + self::merge($sheet, 'E10', 'G10'); |
| 100 | + self::merge($sheet, 'H10', 'J10'); |
| 101 | + self::merge($sheet, 'H11', 'J11'); |
| 102 | + self::merge($sheet, 'M10', 'O10'); |
| 103 | + self::merge($sheet, 'P10', 'R10'); |
| 104 | + self::merge($sheet, 'P11', 'R11'); |
| 105 | + self::merge($sheet, 'K10', 'L10'); |
| 106 | + self::merge($sheet, 'K11', 'L11'); |
| 107 | + self::merge($sheet, 'S10', 'T10'); |
| 108 | + self::merge($sheet, 'S11', 'T11'); |
| 109 | + |
| 110 | + $sheet->getStyle('E4:K8')->getNumberFormat()->setFormatCode('0.00'); |
| 111 | + $sheet->getStyle('L4:L8')->getNumberFormat()->setFormatCode('0.00%'); |
| 112 | + $sheet->getStyle('M4:S8')->getNumberFormat()->setFormatCode('0.00'); |
| 113 | + $sheet->getStyle('T4:T8')->getNumberFormat()->setFormatCode('0.00%'); |
| 114 | + $sheet->getStyle('P10')->getNumberFormat()->setFormatCode('0.00'); |
| 115 | + $sheet->getStyle('P11')->getNumberFormat()->setFormatCode('0.00%'); |
| 116 | + $sheet->getStyle('H1')->getNumberFormat()->setFormatCode('0.00'); |
| 117 | + $sheet->getStyle('H11')->getNumberFormat()->setFormatCode('0.00%'); |
| 118 | + |
| 119 | + $center = [ |
| 120 | + 'alignment' => ['horizontal' => Alignment::HORIZONTAL_CENTER], |
| 121 | + ]; |
| 122 | + $pink = [ |
| 123 | + 'alignment' => ['horizontal' => Alignment::HORIZONTAL_CENTER, 'vertical' => Alignment::VERTICAL_CENTER], |
| 124 | + 'font' => ['color' => ['rgb' => 'FF0000']], |
| 125 | + 'fill' => ['fillType' => Fill::FILL_SOLID, 'endColor' => ['argb' => 'FFFFE5E8'], 'startColor' => ['argb' => 'FFFFE5E8']], |
| 126 | + ]; |
| 127 | + $green = [ |
| 128 | + 'alignment' => ['horizontal' => Alignment::HORIZONTAL_CENTER, 'vertical' => Alignment::VERTICAL_CENTER], |
| 129 | + 'font' => ['color' => ['rgb' => '339966']], |
| 130 | + 'fill' => ['fillType' => Fill::FILL_SOLID, 'endColor' => ['argb' => 'ffccffcc'], 'startColor' => ['argb' => 'ffccffcc']], |
| 131 | + ]; |
| 132 | + //$sheet->getStyle('E4:T8')->getAlignment()->setHorizontal(Alignment::HORIZONTAL_CENTER); |
| 133 | + $sheet->getStyle('E4:T8')->applyFromArray($center); |
| 134 | + $sheet->getStyle('H4')->applyFromArray($pink); |
| 135 | + $sheet->getStyle('J4:L4')->applyFromArray($pink); |
| 136 | + $sheet->getStyle('P4')->applyFromArray($pink); |
| 137 | + $sheet->getStyle('R4:T4')->applyFromArray($pink); |
| 138 | + $sheet->getStyle('K10')->applyFromArray($pink); |
| 139 | + $sheet->getStyle('S10')->applyFromArray($pink); |
| 140 | + $sheet->getStyle('K11')->applyFromArray($green); |
| 141 | + $sheet->getStyle('S11')->applyFromArray($green); |
| 142 | + |
| 143 | + $condition1a = new Conditional(); |
| 144 | + $condition1a->setConditionType(Conditional::CONDITION_EXPRESSION); |
| 145 | + $condition1a->addCondition('$H$10 < $E$10'); |
| 146 | + $condition1a->getStyle()->applyFromArray($green); |
| 147 | + $condition1b = new Conditional(); |
| 148 | + $condition1b->setConditionType(Conditional::CONDITION_EXPRESSION); |
| 149 | + $condition1b->addCondition('$H$10 > $E$10'); |
| 150 | + $condition1b->getStyle()->applyFromArray($pink); |
| 151 | + $conditionalStyles = [$condition1a, $condition1b]; |
| 152 | + $sheet->getStyle('H10:H11')->setConditionalStyles($conditionalStyles); |
| 153 | + |
| 154 | + $condition2a = new Conditional(); |
| 155 | + $condition2a->setConditionType(Conditional::CONDITION_EXPRESSION); |
| 156 | + $condition2a->addCondition('$P$10 < $M$10'); |
| 157 | + $condition2a->getStyle()->applyFromArray($green); |
| 158 | + $condition2b = new Conditional(); |
| 159 | + $condition2b->setConditionType(Conditional::CONDITION_EXPRESSION); |
| 160 | + $condition2b->addCondition('$P$10 > $M$10'); |
| 161 | + $condition2b->getStyle()->applyFromArray($pink); |
| 162 | + $conditionalStyles2 = [$condition2a, $condition2b]; |
| 163 | + $sheet->getStyle('P10:P11')->setConditionalStyles($conditionalStyles2); |
| 164 | + |
| 165 | + $sheet->setSelectedCells('A1'); |
| 166 | + $writer = new XlsxWriter($spreadsheet); |
| 167 | + |
| 168 | + $writer->getStylesConditionalHashTable()->addFromSource($writer->getWriterPartStyle()->allConditionalStyles($spreadsheet)); |
| 169 | + $writerStyle = new XlsxWriter\Style($writer); |
| 170 | + $data = $writerStyle->writeStyles($spreadsheet); |
| 171 | + $spreadsheet->disconnectWorksheets(); |
| 172 | + |
| 173 | + $preg = preg_match('~<dxfs.*</dxfs>~ms', $data, $matches); |
| 174 | + if ($preg !== 1) { |
| 175 | + self::fail('preg failed'); |
| 176 | + } else { |
| 177 | + self::assertStringContainsString('<font', $matches[0]); |
| 178 | + self::assertStringContainsString('<fill', $matches[0]); |
| 179 | + self::assertStringNotContainsString('<align', $matches[0]); |
| 180 | + } |
| 181 | + } |
| 182 | +} |
0 commit comments