The VDE PHPExcel is working properly, but I miss some options. I have added some:

Autosize Width

Indicates whether the width of the columns should be sized to fit the contents.

Must be an option because you can have situations where columns size are huge and don't need sizing.

By default is set to FALSE, because is the current behavior.

Wrap Text

Allows to define the cell property "wrap text". I have defined 3 modes:

  • Never: Does not wrap to any cell. Is the default behavior.
  • Always: Apply wrap to all cells.
  • Auto: If a value contains a PHP_EOF character, the wrap applies to the whole column; otherwise, no wrap is applied.

This last option is useful when you have fields with multiple lines and you want all data is viewed in the cell, but you don't need to be applied everywhere. For example, when rendering a Address Field, which will provide info in a multi-line output.

NOTE: In this case, remember to apply https://www.drupal.org/node/1493014 patch to introduce PHP_EOF before stripping tags.

Keep HTML

Indicates if HTML should be stripped or not.

Of course, you can do it through the View, selecting strip tags for all desired fields. But in many cases, an export is exactly with the same data than the Default view, so for maintanibility is easier allow the module to strip tags.

If you have dozens of sites, and at every one you have at least a dozen of views with export capabilities, the numbers to maintain things easy are low. So I would prefer to left as may export views as Default View to avoid a full review of every field and options every time I need to do a change.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dtamajon’s picture

Issue summary: View changes
dtamajon’s picture

If above is acceptable to include in the module, I will be back with some more options, like header style or others, which can solve simple styling situation.

mollux’s picture

Status: Active » Needs review
FileSize
5.61 KB

The attached patches work.
I made a single patch file with all the changes in and some Drupal coding standards cleanup.

KristineC’s picture

mollux, the patch is great!! I would love to see these changes get committed.

Zekvyrin’s picture

Status: Needs review » Reviewed & tested by the community

These options are helpful to me as well and I don't see why not to commit them.

+1 and marking as RTBC

feldmarv’s picture

+1 works great! should be committed

Cauliflower’s picture

+1 works great!

delacosta456’s picture

hi nice work ..
Please can option like Grouping could be added , like it was done here https://www.drupal.org/node/1187712