yorl1n / ext.ExportableGrid

Wrapper over ExtJs's grid to make grid exportable.
MIT License
13 stars 7 forks source link

Error with Extjs 6.5 #8

Open vredevoogda opened 6 years ago

vredevoogda commented 6 years ago

Thank you for creating and sharing this functionality. I have used this exportable grid for about a year now. I use it in several grids. When I upgraded to extjs 6.5.0 it immediately had an issue. It appears to export the file. When you open the file you get an error in a popup box: "We found a problem with some content in the filename.xlsx. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes". When you choose yes, and the file opens, there is an extra column before the data columns. All rows in this column appear to contain: "span class="x-grid-checkcolumn" role="presentation"></span"

Thank you for your time

abenhamdine commented 6 years ago

FWIW, I use it with ExtJS 6.5.2 (since several months) and I have no errors.

vredevoogda commented 6 years ago

That is good news @abenhamdine. I was using it in 5.1.0 with no issues but when I converted to 6.5 and a universal app this problem started. I am using it in several grids and they all exhibit the same problem. I have re read the directions and see a few differences. I am not requiring ExportableGrid in my app.js. i am doing that in classic/src/view/main. Due to the fact that my ux directory is under classic and not in app. Moving the ux dir causes the grids using it, to not find it. Are you using exportableGrid in a universal app? If so, where is your ux dir? Are you requiring it in app.js? What does your ux path look like? Thanks for any help you can provide.

abenhamdine commented 6 years ago

Are you using exportableGrid in a universal app?

no, just classic

If so, where is your ux dir?

paths: {
                Ext: '.',
                'Ext.ux': '../ux',

Are you requiring it in app.js?

No, I only extend my grid class :

 extend: 'Ext.ux.ExportableGrid',
abenhamdine commented 2 years ago

Update : we use this library with ExtJS 7.1 without any issue.