Description: ------------ Excel XP crashes when generated worsheet have many column merges. Merges are become bugous when spreadsheet have about 400 of them. When count of merges is more than 700, Excel XP crushes. Test script: --------------- <?php include_once "Spreadsheet/Excel/Writer.php"; $xls =& new Spreadsheet_Excel_Writer(); $xls->send("table.xls"); $sheet =& $xls->addWorksheet('table'); for($i=0; $i<40; $i++){//change to $i<100 to get excel crush for($j=0;$j<20;$j+=2){ $sheet->setMerge($i, $j, $i, $j+1); } } $xls->close(); ?> Expected result: ---------------- A blank table, each cell must have colspan=2 and rowspan=1. Actual result: -------------- A blank table, each cell have colspan=2, but many cells from column "M" have rowspan=2. When I extend the table up to 100 rows, I get the excel crush.

Just avoided problem by using mergeCells() instead of setMerge().

Spam

Err, no spam. Sorry.

Man, thank you, you have saved me a lot of headache. It was indeed sufficient to replace "setMerge" with "mergeCells". Very confusing bug...

Thanks Serghei! You have saved me a lot of time by giving the solution.

Thanks. You save me a lot of time really!!! Interesting bug :)

Thank you man! Although, this merge function does not merge properly on my Excel 2007. I have to re-merge cells in Excel when the document is open.

Ok, I fixed this bug. Well, actually it is not a fix of the original problem, but it does the trick: In Worksheet.php find the function: "function setMerge($first_row, $first_col, $last_row, $last_col)" and add this code: if(count($this->_merged_ranges) >= 255) { $this->_storeMergedCells(); $this->_merged_ranges = array(); } just above the line: $this->_merged_ranges[] = array($first_row, $first_col, $last_row, $last_col); ----- Regards, Muris

mergeCells may avoid the blatent Excel error on open when there are too many merges but as others have mentioned it does not render exactly the same - it can cause a document to look very different from one merged with setMerge. I used traxonja's solution above so I could still use setMerge - it's working great - thanks!