phplist

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


View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015399phplist applicationSubscriber Importpublic05-01-10 21:1203-06-13 22:44
ReporterJeff DDG 
PrioritynormalSeveritytweakReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version2.10.10 
Target Version4.0.xFixed in Version2.11.11 
Summary0015399: include_once statement inside import loop in import1.php
DescriptionOn line 208 of import1.php, you have:

include_once dirname(__FILE__)."/commonlib/lib/userlib.php";

This is inside the main import loop and could slow things down for big imports. Also, on line 206, you have:

mt_srand((double)microtime()*1000000);

This is also inside the loop. Both of these could be moved prior to line 177, to make the import more efficient. The mt_srand() call isn't even necessary. The mt_rand() function doesn't need to be initialized with a salt value like rand() does.

Simple tweaks, nothing big. Still reading through the code... :)

--jeff
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
There are no notes attached to this issue.


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker