Merge pull request #14742 from IgniteUI/rivanova/fix-14737-17.2.x

fix(excel-export): exportData when columnExporting is canceled - 17.2.x

Author: kacheshmarova

projects/igniteui-angular/src/lib/services/excel/excel-exporter.spec.ts

@@ -1,6 +1,7 @@
 import { ExportUtilities } from '../exporter-common/export-utilities';
 import { IgxExcelExporterService } from './excel-exporter';
 import { IgxExcelExporterOptions } from './excel-exporter-options';
+import { IColumnExportingEventArgs } from '../exporter-common/base-export-service';
 import { ZipWrapper } from './zip-verification-wrapper.spec';
 import { FileContentData } from './test-data.service.spec';
 import { SampleTestData } from '../../test-utils/sample-test-data.spec';
@@ -117,6 +118,22 @@ describe('Excel Exporter', () => {
         }
     });
 
+    it('should export data successfully when \'columnExporting\' is canceled.', async () => {
+        options.columnWidth = 50;
+
+        exporter.columnExporting.subscribe((args: IColumnExportingEventArgs) => {
+            if (args.field === 'phone') {
+                args.cancel = true;
+            }
+        });
+
+        const wrapper = await getExportedData(SampleTestData.contactsData(), options);
+
+        wrapper.verifyStructure();
+        await wrapper.verifyTemplateFilesContent();
+        await wrapper.verifyDataFilesContent(actualData.contactsDataSkippedColumnContent);
+    });
+
     const getExportedData = (data: any[], exportOptions: IgxExcelExporterOptions) => {
         const result = new Promise<ZipWrapper>((resolve) => {
             exporter.exportEnded.pipe(first()).subscribe((value) => {

projects/igniteui-angular/src/lib/services/excel/test-data.service.spec.ts

@@ -435,6 +435,26 @@ export class FileContentData {
         return this.createData();
     }
 
+    public get contactsDataSkippedColumnContent() {
+        this._sharedStringsData = `count="6" uniqueCount="6"><si><t>name</t></si><si><t>Terrance Orta</t></si>` +
+            `<si><t>Richard Mahoney LongerName</t></si><si><t>Donna Price</t></si><si><t></t></si>` +
+            `<si><t>Dorothy H. Spencer</t></si>`;
+
+        this._tableData = `ref="A1:A6" totalsRowShown="0">
+        <autoFilter ref="A1:A6"/><tableColumns count="1"><tableColumn id="1" name="name"/></tableColumns>`;
+
+        this._worksheetData =
+            `<dimension ref="A1:A6"/><sheetViews><sheetView tabSelected="1" workbookViewId="0"></sheetView></sheetViews>` +
+            `<sheetFormatPr defaultRowHeight="15" x14ac:dyDescent="0.25"/><cols><col min="1" max="1" width="50" customWidth` +
+            `="1"/></cols><sheetData><row r="1"><c r="A1" t="s"><v>0</v>` +
+            `</c></row><row r="2"><c r="A2" t="s"><v>1</v></c></row><row r="3">` +
+            `<c r="A3" t="s"><v>2</v></c></row><row r="4"><c r="A4" t="s"><v>3</v></c>` +
+            `</row><row r="5"><c r="A5" t="s"><v>4</v></c></row><row r="6"><c r="A6" t="s"><v>5</v></c>` +
+            `</row></sheetData>`;
+
+        return this.createData();
+    }
+
     public get noHeadersStringDataContent() {
         this._sharedStringsData = `count="6" uniqueCount="6"><si><t>Column1</t></si><si><t>Terrance Orta</t></si>` +
             `<si><t>Richard Mahoney LongerName</t></si><si><t>Donna Price</t></si><si><t>Lisa Landers</t></si><si><t>` +

projects/igniteui-angular/src/lib/services/exporter-common/base-export-service.ts

@@ -424,7 +424,7 @@ export abstract class IgxBaseExporter {
             });
         }
 
-        const targetCol = mapRecord.columns.filter(c => column.columnGroupParent !== null && c.columnGroup === column.columnGroupParent)[0];
+        const targetCol = mapRecord.columns.filter(c => column.columnGroupParent !== null && column.columnGroupParent !== undefined && c.columnGroup === column.columnGroupParent)[0];
         if (targetCol !== undefined) {
             targetCol.columnSpan -= span;