Steps:

  1. Prepare .csv file with lots of users (ie: > Maximum Users/Process setting)
  2. Create import
  3. Run cron.php
  4. While cron.php is running, click Continue Processing

Result: several import errors will be logged (duplicate email)

Seems to me that the simplest fix would be to check to see if another process is working on the import before starting. Perhaps use a variable as some sort of a mutex?

CommentFileSizeAuthor
#1 user_import_run_mutex-1244298-1.patch957 bytesryan_courtnage

Comments

ryan_courtnage’s picture

ryan_courtnage commented
StatusFileSize
new user_import_run_mutex-1244298-1.patch957 bytes

Something like the attached perhaps?

ryan_courtnage’s picture

ryan_courtnage commented
Status: Active » Needs review
robert castelo’s picture

robert castelo commented
Status: Needs review » Needs work

Agreed that this could be a problem, but not sure about this solution - there's the possibility of the mutex not being deleted and the import being stuck forever.