Closed (fixed)
Project:
Views data export
Version:
7.x-3.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Feb 2012 at 06:44 UTC
Updated:
15 Nov 2012 at 18:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
klausiComment #2
brooksbrown commentedI think ive found the offending code. Beta 6 added an else statement the following conditional at line 85:
Upon commenting out the else statement I was able to get data into the file. Hope this helps.
Comment #3
steven jones commentedThat shouldn't make it print to the screen.
Please re-open if you can still reproduce with more details of any error messages you see etc.
Comment #4
brephraim commentedSame problem for me. No error messages, just no file created and instead output is printed to screen.
Comment #5
finn lewisHi guys!
I have the same problem.
PHP 5.3.2
Drupal 7.15
Drush 5.2
views_data_export 7.x-3.0-beta6
Running the following drush command:
drush views-data-export profile_report views_data_export_1 test.xlsShows a good deal of table based views output, with lots of table html - and does not appear to generate a test.xls file.
Downgrading to views_data_export 7.x-3.0-beta5 gives more expected drush output:
And the test.xls file is saved.
I'm sticking with beta5 for now, but happy to test patches if anyone can offer a fix.
Comment #6
steven jones commentedAh, so this is because non-batched output fails to work with Drush, because instead of returning the exported values, we print them out.
Comment #7
steven jones commentedHere's a patch that uses output buffering to save the non-batched output,
Comment #8
brephraim commentedAppears to work for me. Thanks!
Comment #9
steven jones commentedAwesome, committed.
Comment #10
johnvSorry, need to re-open this.
I have a non-batched view working properly with version beta6 d.d. 2012-May-10 .
After installing the -dev version, there is no export file anymore.
When I re-install the vde.drush.inc file from beta6, the export file is OK again. The only difference is this patch.
This is the drush-command, including a file-move to another location:
drush views-data-export <view-id> <display-id> <file-id> --uri=www.example.com > /home/../domains/www.example.com/files/ftp/<file-id>The drush-log gives the following message:
This seemed OK, but the 65 spaces should show the file-name from $options['output_file'], which is now empty.
[EDIT] In fact, the file exists in its original directory, but has size 0 on the target directory.
Comment #11
steven jones commentedThe output isn't sent to stdout, so you can redirect the output to a file using
>at the moment. You would need to copy the file usingcpor similar.