After playing around a bit with the Import module I have found a couple of bugs and some odd behaviour, I am a bit unclear as to what is supposed to happen so it may be by design.
Note
These comments are only for Companies, the same thing probably applies to Contacts as well
since they are very similar.
Bugs
[list:3i7n283i]
[*:3i7n283i]Creating new groups:
If box checked new groups should be created, which they are,
however these are never assigned to the company (missing line).
(There is also a issue with creating empty groups, which lead
to some sort of loop and maxes out the memory.)[/*:3i7n283i]
[*:3i7n283i]Country required:
Country is a required field (even if not marked as such).
Country select box should not be allowed to be empty, but should instead default to a country (probably main companys) so that rows with empty country fields get assigned a country and
not skipped as they are now.[/*:3i7n283i]
[*:3i7n283i]csv / xls:
Module description says it imports data from csv files,
which it does, but it also imports xls-files. Should say
so in description.[/*:3i7n283i]
[/list:u:3i7n283i]
Annoyances
As with many places in administration there it is sometimes hard to guess
what the modules does, and it would be nice with a little more descriptive texts.
[list:3i7n283i]
[*:3i7n283i]When you run import for the first time, you get a almost empty screen that
says "Please define column mapping". This is rather confusing as it isn't directly
obvious that this is a link you should click on, couldn't you automatically go to
column mapping instead of having to click the link.[/*:3i7n283i]
[*:3i7n283i]column mapping (view) should maybe be presented as a table instead of a list, easier to read.[/*:3i7n283i]
[*:3i7n283i]a longer description of how import works, see attached patch for example.[/*:3i7n283i]
[/list:u:3i7n283i]
Patch
The attached patch consists of me trying to fix the mentioned bugs and then also give
an example of what I mean with a bit longer more descriptive text. If you would
like I can try to make a separate patch for just the bugs but there are not many lines changed
and I guess you have better solutions for these anyway.
