[Console] Fix computing column width containing multibyte chars

cay89 · nicolas-grekas · commit f32179ee12df · 2023-04-18T15:29:40.000+02:00
diff --git a/src/Symfony/Component/Console/Helper/Table.php b/src/Symfony/Component/Console/Helper/Table.php
@@ -805,7 +805,7 @@ private function calculateColumnsWidth(iterable $groups)
                             $textContent = Helper::removeDecoration($this->output->getFormatter(), $cell);
                             $textLength = Helper::width($textContent);
                             if ($textLength > 0) {
-                                $contentColumns = str_split($textContent, ceil($textLength / $cell->getColspan()));
+                                $contentColumns = mb_str_split($textContent, ceil($textLength / $cell->getColspan()));
                                 foreach ($contentColumns as $position => $content) {
                                     $row[$i + $position] = $content;
                                 }
diff --git a/src/Symfony/Component/Console/Tests/Helper/TableTest.php b/src/Symfony/Component/Console/Tests/Helper/TableTest.php
@@ -316,7 +316,7 @@ public function renderProvider()
                     ],
                     new TableSeparator(),
                     [
-                        new TableCell('Cupiditate dicta atque porro, tempora exercitationem modi animi nulla nemo vel nihil!', ['colspan' => 3]),
+                        new TableCell('Cupìdĭtâte díctá âtquè pôrrò, tèmpórà exercitátìónèm mòdí ânìmí núllà nèmò vèl níhìl!', ['colspan' => 3]),
                     ],
                 ],
                 'default',
@@ -333,7 +333,7 @@ public function renderProvider()
 | 9971-5-0210-0                 | A Tale of                                                   |
 |                               | Two Cities                                                  |
 +-------------------------------+-------------------------------+-----------------------------+
-| Cupiditate dicta atque porro, tempora exercitationem modi animi nulla nemo vel nihil!       |
+| Cupìdĭtâte díctá âtquè pôrrò, tèmpórà exercitátìónèm mòdí ânìmí núllà nèmò vèl níhìl!       |
 +-------------------------------+-------------------------------+-----------------------------+
 
 TABLE

Original file line number	Diff line number	Diff line change
`@@ -805,7 +805,7 @@ private function calculateColumnsWidth(iterable $groups)`
`805`	`805`	`$textContent = Helper::removeDecoration($this->output->getFormatter(), $cell);`
`806`	`806`	`$textLength = Helper::width($textContent);`
`807`	`807`	`if ($textLength > 0) {`
`808`		`- $contentColumns = str_split($textContent, ceil($textLength / $cell->getColspan()));`
	`808`	`+ $contentColumns = mb_str_split($textContent, ceil($textLength / $cell->getColspan()));`
`809`	`809`	`foreach ($contentColumns as $position => $content) {`
`810`	`810`	`$row[$i + $position] = $content;`
`811`	`811`	`}`