While excluding a field from display the module dose not count the option and force exporting all fields

CommentFileSizeAuthor
#5 views_data_export-respect_views_execlude-the_export_operation-2722317-5.patch1.51 KBAhmadZain
#3 views_data_export-respect-exclude-2722317.patch941 bytesmariano.barcia
#2 fix_export_is_not_respecting_execlude_option.diff2.03 KBAhmadZain
fix_export_is_not_respecting_execlude_option.diff1.15 KBAhmadZain
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AhmadZain created an issue. See original summary.

AhmadZain’s picture

AhmadZain CreditAttribution: AhmadZain commented
FileSize
fix_export_is_not_respecting_execlude_option.diff2.03 KB
mariano.barcia’s picture

mariano.barcia CreditAttribution: mariano.barcia at ASM Web Services S.L. commented
Status: Patch (to be ported) » Needs review
FileSize
views_data_export-respect-exclude-2722317.patch941 bytes

Hi, thanks for the patch. I applied it manually and it seems to work.
Adding the same patch re-rolled, renamed. Changing issue status.

Note:
I've created the patch on 7.x-3.0-beta9 but it applies cleanly to dev (with offset 25 lines).

AhmadZain’s picture

AhmadZain CreditAttribution: AhmadZain commented

Hi Mariano,
Thanks and waiting to release it

AhmadZain’s picture

AhmadZain CreditAttribution: AhmadZain as a volunteer and commented
FileSize
views_data_export-respect_views_execlude-the_export_operation-2722317-5.patch1.51 KB

Hi, I found the patch missing to be reformated so I added it again after reformat.
#2722317: Export operation should respect "exclude from display" option is referenced.

- Tha batch was diff formated
- @mariano.barcia reformated a version
- @AhmadZain reformated again.

Thanks
Zain

AhmadZain’s picture

AhmadZain CreditAttribution: AhmadZain as a volunteer and commented
Status: Needs review » Reviewed & tested by the community

The last submitted patch, 2: fix_export_is_not_respecting_execlude_option.diff, failed testing.

The last submitted patch, 2: fix_export_is_not_respecting_execlude_option.diff, failed testing.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 at Bluespark commented
Title: The Export operation dose not respect the exclude from display option » Export operation should respect "exclude from display" option