View Issue Details

IDProjectCategoryView StatusLast Update
0014185mantisbtplug-inspublic2014-09-23 18:05
Reportervincent_sels Assigned Todregad  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.2.8 
Target Version1.2.11Fixed in Version1.2.11 
Summary0014185: Only first 100 rows of plug-in columns loaded when exporting to Excel
Description

When you add plug-in columns to the 'bug view page' and export to Excel, only the values for the first 100 rows are displayed.

TagsNo tags attached.

Relationships

related to 0015721 closedgrangeway Functionality to consider porting to master-2.0.x 

Activities

vincent_sels

vincent_sels

2012-04-24 04:28

reporter   ~0031709

Can easily be fixed in file: excel_xml_export.php

Replace lines 48 - 50 by (equal to the csv_export.php):

$t_page_number = 1;
$t_per_page = -1;
$t_bug_count = null;
$t_page_count = null;

atrol

atrol

2012-04-24 05:18

developer   ~0031710

Last edited: 2012-04-24 05:19

IMO a workaround but not a fix.
Setting $p_per_page to -1 (get all bugs) might introduce memory issues.

I didn't have a deeper look, but maybe the real problem has been fixed.
Can you try 1.2.10 to check this or try the patch which is attached to 0013765 ?

dregad

dregad

2012-04-25 09:14

developer   ~0031720

I was not able to reproduce the behavior you described, either with 1.2.8 or latest git trunk. In both cases, the custom field comes fully populated for all records when I exported to Excel on my test environment.

If you are able to confirm this behavior, then please reopen this issue with detailed steps to reproduce it.

vincent_sels

vincent_sels

2012-05-29 11:10

reporter   ~0031919

Dregad - the problem occurs with plugin columns, nog custom field columns. That's probably the reason why you can't reproduce it.

I did find that I fixed it incorrectly. The correct fix is even easier and very straightforward.

The code that caches plugin columns based on the resultset, must be placed inside the do-while loop:

pre-cache custom column data

columns_plugin_cache_issue_data( $result );

I'll see if I can create a pull request for this.

dregad

dregad

2012-05-29 18:31

developer   ~0031921

Sorry for misreading your initial post Vincent - indeed with plugin columns I can reproduce the problem. I'll commit a fix shortly.

grangeway

grangeway

2013-04-05 17:57

reporter   ~0036277

Marking as 'acknowledged' not resolved/closed to track that change gets ported to master-2.0.x branch

Related Changesets

MantisBT: master 30e3d692

2012-05-29 11:20

dregad


Details Diff
Fix Excel export of plug-in columns limited to first 100 rows

Due to an incorrect location of columns_plugin_cache_issue_data()
function call outside of the main loop, only the first 100 rows of
plug-in columns were loaded when exporting issues to Excel.

Thanks to vincent_sels for spotting the issue and identifying the root
cause.

Fixes 0014185
Affected Issues
0014185
mod - excel_xml_export.php Diff File

MantisBT: master-1.2.x e48ce931

2012-05-29 11:20

dregad


Details Diff
Fix Excel export of plug-in columns limited to first 100 rows

Due to an incorrect location of columns_plugin_cache_issue_data()
function call outside of the main loop, only the first 100 rows of
plug-in columns were loaded when exporting issues to Excel.

Thanks to vincent_sels for spotting the issue and identifying the root
cause.

Fixes 0014185
Affected Issues
0014185
mod - excel_xml_export.php Diff File