Project

General

Profile

Actions

Bug #3767

closed

CSV import users processed when errors

Added by dominique de guchtenaere about 12 years ago. Updated over 11 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
-
Target version:
Start date:
01/08/2011
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal

Description

When I import a CSV file (action: A) to add users to the system, the CSV is processed, even when there are errors in the file (e.g. users without email when email is set to mandatory in the system). I get an error message concerning only 10 non processed users, even when there are more errors.
See CSV file attached.


Files

add_studenten.csv (876 Bytes) add_studenten.csv dominique de guchtenaere, 01/08/2011 08:27
add_studenten.csv (831 Bytes) add_studenten.csv dominique de guchtenaere, 03/08/2011 08:47
Actions #1

Updated by dominique de guchtenaere about 12 years ago

I should have posted this under Admin

Actions #2

Updated by Anonymous about 12 years ago

  • Status changed from New to Needs testing
  • Assignee set to Anonymous
  • % Done changed from 0 to 100

Only the users with valid properties are processed, the first 10 of non processed users are displayed in the error message.

Actions #3

Updated by dominique de guchtenaere about 12 years ago

  • Status changed from Needs testing to Needs more info

It should be the way it used to be: if there were users with unvalid properties, nothing was processed. I got then the full list of users with unvalid properties. Since only the first 10 of non processed users are displayed, it's rather difficult to manage a CSV with over 20.500 users. The admin has to split it up in users already in the system (or at least he has to change the action from A to U) and users that still have to be added. I'm afraid this will end up in forgetting some users.

Actions #4

Updated by Anonymous about 12 years ago

  • Status changed from Needs more info to Needs testing

Fixed on stable:
- CSV is no longer processed if there's at least one invalid record.
- 20 first invalid records are now shown. If there are more, it's displayed how many more there are.

Actions #5

Updated by dominique de guchtenaere about 12 years ago

are you sure this is already on the stable? I have the latest code and a CSV with invalid records (email is mandatory, only 1 user has an email address) is still processed and only the first 10 invalid records are shown.

Actions #6

Updated by Michael Hosdez about 12 years ago

  • Status changed from Needs more info to Bug resolved

I tested this issue on a linux system and it works okay for me. The process stops and no users were added.

Actions #7

Updated by dominique de guchtenaere about 12 years ago

  • Status changed from Bug resolved to Feature implemented
Actions #8

Updated by Stefaan Vanbillemont over 11 years ago

  • Target version set to 2.1.0
Actions

Also available in: Atom PDF