View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0013765 | mantisbt | csv | public | 2012-01-18 09:39 | 2014-09-23 18:05 |
Reporter | ChristianLuef | Assigned To | dregad | ||
Priority | high | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.2.8 | ||||
Target Version | 1.2.9 | Fixed in Version | 1.2.9 | ||
Summary | 0013765: Excel Export ends in infinite loop if number of viewing issues mod 100 = 0 | ||||
Description | If the number of "Viewing Issues" equals a multiple of $t_per_page from excel_xml_export.php, which is 100 by default, and you try to do an "Excel Export", the request end up in an infinite loop. I think it has something to do with the following code snippet from excel_xml_export.php: ...
... If for example the $t_row_count == 100 and $t_per_page == 100, the filter_get_bug_rows method is called which I think will not return false in this case but tries infinitely to export the elements [0-99]. | ||||
Tags | No tags attached. | ||||
I confirm the behavior, should be easy enough to fix. |
|
I changed the condition to process next page. |
|
Marking as 'acknowledged' not resolved/closed to track that change gets ported to master-2.0.x branch |
|
MantisBT: master d95ce2f0 2012-01-18 21:26 Details Diff |
Fix 0013765: infinite loop in Excel export If the number of records in the View Issues page was a multiple of 100, the Excel export ended up in an infinite loop. Instead of testing on the number of rows returned vs the number of rows per page, the code now uses the current page number vs the total number of pages returned by filter_get_bug_rows() to set the loop's exit flag. |
Affected Issues 0013765 |
|
mod - excel_xml_export.php | Diff File | ||
MantisBT: master-1.2.x 88d7fa68 2012-01-18 21:26 Details Diff |
Fix 0013765: infinite loop in Excel export If the number of records in the View Issues page was a multiple of 100, the Excel export ended up in an infinite loop. Instead of testing on the number of rows returned vs the number of rows per page, the code now uses the current page number vs the total number of pages returned by filter_get_bug_rows() to set the loop's exit flag. |
Affected Issues 0013765 |
|
mod - excel_xml_export.php | Diff File |