Relationship Graph View Issue Dependency Graph
related to child of duplicate of

View Issue Details

IDProjectCategoryView StatusLast Update
0015231phpList 3 applicationSubscriber Importpublic21-06-18 14:05
ReporterOGJuiceLoc 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version2.10.8 
Target Versionnext majorFixed in Version2.11.11 
Summary0015231: False warnings during test import with 'import emails with different values for attributes'
DescriptionThe 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.
TagsNo tags attached.

Activities

11-02-09 09:46

 

OGJuiceLoc

11-02-09 09:55

reporter   ~0050427

Related to http://mantis.phplist.com/view.php?id=3430

michiel

12-05-11 15:30

manager   ~0051324


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.

silaspalmer

15-12-11 23:58

reporter   ~0051423

Attached is a slightly modified version of importcsv.php which resolves this issue, and is more compatible with excel CSV format.

silaspalmer

16-12-11 00:03

reporter   ~0051424

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.

michiel

02-02-12 01:59

manager   ~0051450

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.

michiel

04-06-13 16:46

manager   ~0052089

applied 0006275:0051173 and that seems to resolve it. Thanks!

Issue History

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 03:06 h2b2 Relationship added related to 0003603
29-04-10 03:10 h2b2 Relationship added related to 0006275
12-05-11 15:30 michiel Relationship added related to 0003430
12-05-11 15:30 michiel Note Added: 0051324
12-05-11 15: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 19:54 michiel Relationship added related to 0016537
04-06-13 16:46 michiel Note Added: 0052089
04-06-13 16:46 michiel Status new => resolved
04-06-13 16:46 michiel Fixed in Version => 2.11.11
04-06-13 16:46 michiel Resolution open => fixed
21-06-18 14:05 samtuke Target Version 4.0.x => next major