phplist

NOTE:: Before reporting an issue, make sure you are running the latest version, currently 3.3.1

My View | View Issues | Change Log | Roadmap
  

View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015231phplist applicationSubscriber Importpublic11-02-09 09:4604-06-13 16:46
ReporterOGJuiceLoc 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version2.10.8 
Target Version4.0.xFixed 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.
Attached Filesgif file icon PHPList-Import2_TestOutputWarning_2009-02-10.gif [^] (29,524 bytes) 11-02-09 09:46

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to 0003603resolvedmichiel False warnings during test import with 'import emails with different values for attributes' 
related to 0006275resolvedmichiel False warnings during test import with 'import emails with different values for attributes' 
related to 0003430resolvedmichiel False warnings always reported during import 
related to 0016537new Import not coping with "text \r\n text" ? 

-  Notes
(0050427)
OGJuiceLoc (reporter)
11-02-09 09:55

 Related to http://mantis.phplist.com/view.php?id=3430 [^]
(0051324)
michiel (manager)
12-05-11 15:30
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.
(0051423)
silaspalmer (reporter)
15-12-11 23:58

 Attached is a slightly modified version of importcsv.php which resolves this issue, and is more compatible with excel CSV format.
(0051424)
silaspalmer (reporter)
16-12-11 00:03

 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.
(0051450)
michiel (manager)
02-02-12 01:59

 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.
(0052089)
michiel (manager)
04-06-13 16:46

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

Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker