Export to CSV or PDF

[jo-ka]

Hello,

Sorry to bring this discussion to here, but I'm having a problem as this one being discussed on this thread:

http://fabrikar.com/forums/index.php?threads/csv-export-not-working.48612/#post-253681

The question is that I use a pre-filter on the list, based on an SQL query, which receives data from an input form, as suggested here:

http://fabrikar.com/forums/index.php?threads/unavailability-list.48876/

And so, my pre-filter is set like this:

View attachment 17106

The result, after the variables are passed by the form with the redirect plugin is this:

View attachment 17107

But when I try to export to CSV I get this error ( In English, There's no records available to export to CSV):

View attachment 17108

Also the PDF returns empty...

View attachment 17109

I was checking if I have this commit but it looks like it's OK.

https://github.com/Fabrik/fabrik/commit/4cbe63edec3bacefbd3c8baaa71729c8257a38c7

Am I doing something wrong?

Thank's in advance.

 

[cheesegrits]

For the CSV, have you tried adding the "Custom QS" (query string), under the List's Publishing -> CSV settings? The tooltip explains using it for exactly what you are doing (pre-filtering using query string args).

For PDF, I think I'd have to add a similar feature, to preserve the addition query string args in the PDF link.

The problem is of course that your additional query string args simply aren't automatically carried through to links within the list. Query strings are only intended for the page being loaded, and never automatically get added to links generated on that page.

-- hugh

 

[jo-ka]

Huuummm... I didn't knew about this. Thanks for the tip

So, I've tried like this:

&di={___di}&df={___df}

{___di} and {___df} are the element names of the form which I'm using to filter the list.

And it didn't worked, but if I force a date like this:

&di=2018-05-08&df=2018-05-08

It works.

Can this be related with the 00:00:00 part of the date, or am I doing something wrong here?

Thanks in advance.

 

[jo-ka]

My mistake!
The solution is to have the QS like this:

&di={di}&df={df}

Works perfectly. It would be great if we could get this working on PDF too.

Thanks Hugh.

 

[jo-ka]

It would be perfect if we can add the args on the file name too, like this:
View attachment 17111
Is this possible?
Thanks in advance.

 

[cheesegrits]

As it happens, I added this change locally a while ago, never got round to committing it. Adds a new layout for the CSV filename (so you can write your own code to generate it if you want), and includes placeholder replacement for the filename generation.

https://github.com/Fabrik/fabrik/commit/7aa7da9a001a0a3d0edae4b791134f055414fbe8

-- hugh

 

[jo-ka]

Dear Hugh,
I've tried with these new files and I'm getting this error:

View attachment 17132

I've double checked permissions and are ok.
Any clue?

 

[troester]

Did you a full GitHub update?
Did you clear Joomla and browser cache?

 

[jo-ka]

I did clear the browser and Joomla cache, yes, but didn't made a full Github update.
Shall I?

 

[cheesegrits]

Always best to do a full update rather than cherry picking files.

-- hugh

 

[jo-ka]

OK, So I've made a full update from Fabrik and the error has now gone.

I understand that If I want to change something I need to do it on the new file fabrik-csv-filename.php.

So, how can I add the variables {di} and {df} from the custom CS &di={di}&df={df}?

Thanks in advance.

 

[troester]

Did you try with {di} in the file name?
Are these URL params still in the PDF link?

 

[jo-ka]

Yes, like this:

View attachment 17148

About the PDF link, where shall I check?

Thanks in advance

 

[cheesegrits]

OK, looks like I need to do a little more work on this. The additional QS args are making it through in the Javascript to the initial calls to build the CSV, but not to the part that downloads the file.

I'll let you know when I've got it going.

-- hugh

 

[jo-ka]

OK, thanks a lot. Please let me know.

 

[cheesegrits]

OK, should be working as of this commit today:

https://github.com/Fabrik/fabrik/commit/c4ba720167eedbb839fc9c0c560d41ed33c9bbb8

-- hugh