Import CSV works, Export CSV does nothing

[kdiamond]

This worked within the last few weeks, I don't think anything was updated since then, perhaps just the linux box itself. The Export CSV does nothing - it just sits. Import shows the new import window.

When hovering - the export URL just looks to take the current URL and add a # to the end. i.e (/my-reports)#

The import adds (my-reports)/import/13/csv to the url.

Any help appreciated-

 

[coullet]

i think it's the same problem i have a few day ago, fixed now:

http://fabrikar.com/forums/index.php?threads/export-csv.44211/

 

[kdiamond]

I saw that - it sounds slightly different, plus I just did an update from master

wget --no-check-certificate https://github.com/Fabrik/fabrik/tarball/master -O - | tar -zx --strip-components 1

Didn't seem to change anything.

 

[cheesegrits]

I can't test, as the Event Reports list seems to be empty, so it won't even try.

-- hugh

 

[kdiamond]

its under admin> open the "admin event reports" item

thank you

 

[cheesegrits]

Hmmm, it's opening, but off screen, and I can't immediately see why.

-- hugh

 

[cheesegrits]

Working on it

 

[kdiamond]

Thanks - I agree - I saw a file that looks like it was created by the export - it was in the /tmp folder. General users can't get to it. I am not aware of any changes that would have caused this and I have updated from the master branch.

 

[cheesegrits]

OK, should be fixed as of this commit:

https://github.com/Fabrik/fabrik/commit/19aacc944f9ea32f16775205a0fbd217f2a3a7a6

For reasons I don't understand (probably to do with your site template) your CSS position for the window is 'static' rather than 'fixed', and we were only testing for 'fixed' before applying the vertical scroll offset.

Update and let me know.

-- hugh

 

[kdiamond]

updating with this method - wget --no-check-certificate https://github.com/Fabrik/fabrik/tarball/master -O - | tar -zx --strip-components 1

I am not seeing a change regarding any output, but oddly enough when I click on the export now, the vertical browser scroll bar goes away? Otherwise nothing happens, as before.

 

[cheesegrits]

Ooops.

https://github.com/Fabrik/fabrik/commit/9ccf63d9fb4a441024e9ee6326ca765152ce6746

-- hugh

 

[kdiamond]

Sorry to pester on this. I am seeing some kind of reaction on the bottom of the screen now. I could swear this used to open up in a small pop-up/frame? It looks like it's trying to open at the bottom of the screen and then it just sits. Since the scroll bar is gone I can't scroll down to use it.

Thank you again-

 

[cheesegrits]

First thing, I committed a change which removes the feature that disables scrolling while the modal is open.

Second ... can you test this with the protostar template, see if it works?

I'm pretty sure there's something very funky going on with the Gantry template. Something is changing the CSS 'position' attribute from static to relative, between us opening the window in our window.js, and styling it in our list.js. Also, all of the modal CSS seems to be missing, so the window has no framing.

-- hugh

 

[kdiamond]

yes, it works fine with protostar. Not good....

 

[cheesegrits]

Yeah, pain in the backside. I do wish templates wouldn't mess with other people's DOM structures and inline styling.

I'm still working on it. For now, you could set that menu item to Protostar in the menu manage.

-- hugh

 

[cheesegrits]

What template is that?

I've started trying to debug this locally, but I only have the new Gantry v5 Hydrogen installed, which doesn't seem to suffer from this problem.

-- hugh

 

[kdiamond]

Any thoughts around why the import modal would work but the export would act differently?

 

[cheesegrits]

Yes. But I need to install the template you are using, so if you could answer my last post...

 

[kdiamond]

Sorry I didn't see the second post you did. It's Gantry v4 with the base theme - BUT if Gantry v5 is working, then please don't fix this for me. I will update to G5 as I was planning to anyway. Pause this and I will confirm G5 is working. Thanks-

 

[cheesegrits]

OK.

It works in Gantry 5, I've tested on the Hydrogen template, although the CSS styling is off - the content of the window overflows, but that's a much easier issue to fix.

-- hugh