[PEAR-BUG] Bug #8967 [Com]: Excel XP crashes when generated worksheet have many cell merges

[email protected] ("[email protected]")
Newsgroups php.pear.bugs
Message-ID <[email protected]>
Edit report at http://pear.php.net/bugs/bug.php?id=8967&edit=1

 ID:               8967
 Comment by:       [email protected]
 Reported By:      paladin80 at gmail dot com
 Summary:          Excel XP crashes when generated worksheet have many
                    cell merges
 Status:           Verified
 Type:             Bug
 Package:          Spreadsheet_Excel_Writer
 Operating System: win XP
 Package Version:  0.9.1
 PHP Version:      5.0.4
 Roadmap Versions: 
 New Comment:

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


Previous Comments:
------------------------------------------------------------------------

[2011-08-05 11:25:23] traxonja

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.

------------------------------------------------------------------------

[2010-09-07 11:27:57] thinklinux

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

------------------------------------------------------------------------

[2010-04-06 12:58:30] progi1984

-Status: Open
+Status: Verified


------------------------------------------------------------------------

[2009-10-14 10:29:08] sunny

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

------------------------------------------------------------------------

[2008-11-06 09:04:36] mek7

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

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://pear.php.net/bugs/bug.php?id=8967

-- 
Edit this bug report at http://pear.php.net/bugs/bug.php?id=8967&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.