View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0015231 | phpList 3 application | Subscriber Import | public | 11-02-09 09:46 | 21-06-18 13:05 |
Reporter | OGJuiceLoc | ||||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | 2.10.8 | ||||
Target Version | next major | Fixed in Version | 2.11.11 | ||
Summary | 0015231: False warnings during test import with 'import emails with different values for attributes' | ||||
Description | The warning described in http://mantis.phplist.com/view.php?id=3603 still occurs after modifying the code as directed in .../lists/admin/commonlib/pages/importcsv.php. After attempting to import using Test Output a few times starting at .../lists/admin/?page=import2, the warning messages ONLY appear when "Show Warnings:" box is checked. I suppose this is what is meant in the description ------ If you check "Show Warnings", you will get warnings for invalid records. Warnings will only be shown if you check "Test Output". They will be ignored when actually importing." ------ This is being brought up again because this was supposedly resolved in 2005 but is not. It's just a matter of figuring out why (with or without the "-1" modification): (sizeof($values)-1) != (sizeof($_SESSION["import_attribute"]) + sizeof($_SESSION["system_attributes"]) See attached snapshot. Lastly, it still occurs in the latest version 2.10.9. | ||||
Tags | No tags attached. | ||||
related to | 0003603 | resolved | michiel | False warnings during test import with 'import emails with different values for attributes' |
related to | 0006275 | resolved | michiel | False warnings during test import with 'import emails with different values for attributes' |
related to | 0003430 | resolved | michiel | False warnings always reported during import |
related to | 0016537 | new | Import not coping with "text \r\n text" ? |
11-02-09 09:46
|
|
|
Related to http://mantis.phplist.com/view.php?id=3430 |
|
I guess the best thing to do is to check that there's actually data in the "chopped off" columns and not give the warning if there isn't any. |
|
Attached is a slightly modified version of importcsv.php which resolves this issue, and is more compatible with excel CSV format. |
|
You can view the modified importcsv.php code here: http://forums.phplist.com/viewtopic.php?f=7&t=5085&p=89890#p89890 (I don't have the permissions to upload files to mantis) Cheers, Silas. |
|
I tried merging the code with the existing one, but there are so many changes, it makes it hard to see what to keep and what to merge. I wonder whether the PHP function http://php.net/fgetcsv should be used, and whether it can handle the multi-line issue and double quotes as you outline. |
|
applied 0006275:0051173 and that seems to resolve it. Thanks! |
Date Modified | Username | Field | Change |
---|---|---|---|
11-02-09 09:46 | OGJuiceLoc | New Issue | |
11-02-09 09:46 | OGJuiceLoc | File Added: PHPList-Import2_TestOutputWarning_2009-02-10.gif | |
11-02-09 09:55 | OGJuiceLoc | Note Added: 0050427 | |
29-04-10 02:06 | h2b2 | Relationship added | related to 0003603 |
29-04-10 02:10 | h2b2 | Relationship added | related to 0006275 |
12-05-11 14:30 | michiel | Relationship added | related to 0003430 |
12-05-11 14:30 | michiel | Note Added: 0051324 | |
12-05-11 14:30 | michiel | Target Version | => 2.10.X |
15-12-11 23:58 | silaspalmer | Note Added: 0051423 | |
16-12-11 00:03 | silaspalmer | Note Added: 0051424 | |
02-02-12 01:59 | michiel | Note Added: 0051450 | |
02-02-12 02:01 | michiel | Target Version | 2.10.X => 4.0.x |
30-05-12 18:54 | michiel | Relationship added | related to 0016537 |
04-06-13 15:46 | michiel | Note Added: 0052089 | |
04-06-13 15:46 | michiel | Status | new => resolved |
04-06-13 15:46 | michiel | Fixed in Version | => 2.11.11 |
04-06-13 15:46 | michiel | Resolution | open => fixed |
21-06-18 13:05 | samtuke | Target Version | 4.0.x => next major |