Posted by sdrycroft on August 1, 2011 at 11:09am
6 followers
Jump to:
| Project: | Feeds Excel |
| Version: | 7.x-2.x-dev |
| Component: | Miscellaneous |
| Category: | support request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
I would be interested in helping to co-maintain this module, and particularly in helping to develop code for a Drupal 7 version.
I have written and tested a version of this module for Drupal 7 that uses the PHPExcel Library (http://phpexcel.codeplex.com/). It is a little simpler than the current version of this module, with only two options:
- No headers: exactly the same options as Feeds' CSV parser.
- All worksheets: Whether to import all worksheets, or just the first worksheet.
Comments
#1
Sure ;)
Do you have source code anywhere, e.g. on github?
#2
I certainly do.
#3
Any progress with this?
#4
subscribe, note to self: D7 version of Feeds Excel with actual code. check prototype if working.
#5
Simon,
sure wish your link was git clone-able. Hehe I am back to good old fashion copy and pasting.
#6
I try to have a look on this in London ;)
#7
It should be git clone-able. Try:
$ git clone --branch import https://git.scratchpads.eu/scratchpads-2.0.gitderhasi - I'm also in London/Croydon this week. May not be a bad idea to chat over a beer.
#8
I just looked in this issue again. Are you in town for the code sprints tomorrow? Maybe we then can meet there.
#9
sdrycroft, I will work on the D7 Version within this week. I first want to get some overview of your D7 work and the possibilites, and then will contact you again for further maintainance.
see #1191426: Drupal 7 version?
#10
Apologies derhasi, I gave up waiting - Feeds XLS module.
#11
sdrycroft, arg, ok, so we missed the join. But I very much recommend we join forces in providing a single module. As both module's would do exactly the same.
The next days I will spent some time on a direct port to D7 (with using the old PHPExcelReader library).
In the 7.x-2.x, I'd recommend to implement the PHPExcel Library (or maybe make it pluggable) and clean up the UI.
What do you think about it?
#12
#13
derhasi - I'm all for joining forces, I just felt that others would benefit from the code I had written. It definitely makes sense to only have one PHP Excel/XLS parser for the feeds module.
Giving users the ability to choose the library to use with Feeds Excel definitely sounds like a good idea, especially if other modules/projects are currently using the PHPExcelReader library.
Let me know what you'd like me to do!
#14
So, I finally got to port the module to D7: #1191426: Drupal 7 version?
So now we can start to improve the module ;)
I had in mind to
* make Libraries pluggable
* improve UI (a simple and expert version?)
* improve walking the excel file (for batch processing)
* improving token support (chaining of file, sheet, row/column, cell)
* build wrapper objects for workbook, sheet, range / row / column, cell
* drush support for downloading Excel libraries
sdrycroft, feel free to post some issues of new features or ported functionality from feeds_xls, that shall go into 7.x-2.x
Just made you con maintainer of this module ;)
How'd you like to process with feeds_xls?