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.
Comment | File | Size | Author |
---|---|---|---|
#3 | views_data_export_phpexcel-export_options-2479439-2.patch | 5.61 KB | mollux |
Comments
Comment #1
dtamajon CreditAttribution: dtamajon commentedComment #2
dtamajon CreditAttribution: dtamajon commentedIf 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.
Comment #3
mollux CreditAttribution: mollux commentedThe attached patches work.
I made a single patch file with all the changes in and some Drupal coding standards cleanup.
Comment #4
KristineC CreditAttribution: KristineC commentedmollux, the patch is great!! I would love to see these changes get committed.
Comment #5
Zekvyrin CreditAttribution: Zekvyrin commentedThese options are helpful to me as well and I don't see why not to commit them.
+1 and marking as RTBC
Comment #6
feldmarv CreditAttribution: feldmarv commented+1 works great! should be committed
Comment #7
Cauliflower CreditAttribution: Cauliflower commented+1 works great!
Comment #8
delacosta456 CreditAttribution: delacosta456 commentedhi nice work ..
Please can option like Grouping could be added , like it was done here https://www.drupal.org/node/1187712