Project

General

Profile

Bug #4069

Import of sessions via XML doesn't work

Added by Julio Montoya almost 8 years ago. Updated about 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
24/11/2011
Due date:
% Done:

80%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

main/admin/session_import.php seems that doesn't create any sessions, any courses, nothing

Associated revisions

Revision d813a546 (diff)
Added by Julio Montoya almost 8 years ago

Fixing export/import session list from CSV/XML see #4069

Revision d28f5445 (diff)
Added by Julio Montoya about 7 years ago

Fixes bug while importing sessions via a XML file see #4069

Revision dbde98a3 (diff)
Added by Julio Montoya about 7 years ago

Minor - adding message see #4069

History

#1

Updated by Julio Montoya almost 8 years ago

  • Status changed from New to Needs testing
  • % Done changed from 0 to 50
#2

Updated by Yannick Warnier over 7 years ago

  • Target version set to 1.9 Beta
#3

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 1.9 Beta to 1.9 RC1
#4

Updated by Yannick Warnier about 7 years ago

Julio, it is not entirely clear to me what was the problem here... The title seems to say the fact that the session import created courses wasn't the right thing, then the description says that "the session import doesn't do that", but the that is not clear either. Then you say you fixed it by removing it, and I'm completely lost :-D

#5

Updated by Julio Montoya about 7 years ago

  • Subject changed from Session import XML create courses to Import of sessions via XML doesn't work

Yannick Warnier wrote:

Julio, it is not entirely clear to me what was the problem here... The title seems to say the fact that the session import created courses wasn't the right thing, then the description says that "the session import doesn't do that", but the that is not clear either. Then you say you fixed it by removing it, and I'm completely lost :-D

Sorry for that! :)

#6

Updated by Julio Montoya about 7 years ago

  • Status changed from Needs testing to Assigned
  • Assignee set to Julio Montoya
#7

Updated by Julio Montoya about 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 50 to 70

I sent some fixes, now it works. But the XML and CSV import have different behaviours: the XML create users and courses (if the username doesn't exist), while the CSV doesn't do that.

#8

Updated by Yannick Warnier about 7 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya

No reason to have different behaviours, so I'm re-assigning to you (soooorryyyy :-p). I personally think users and courses should be created if they don't exist.

#9

Updated by Julio Montoya about 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

The CSV will be very very complicated if I do that because I need firstname, email, etc etc same thing with courses the current CSV file has something like this:

SessionName;Coach;DateStart;DateEnd;Users;Courses
xxx1;xxx;xxx;xxx;username1|username2;course1[coach1][username1,username2,...]|course2[coach1][username1,username2,...]
xxx2;xxx;xxx;xxx;username1|username2;course1[coach1][username1,username2,...]|course2[coach1][username1,username2,...]

So the XML import will not create users/sessions in order to have the same behaviour OR I add a lot of parameters in the CSV file ...

#10

Updated by Yannick Warnier about 7 years ago

  • Target version changed from 1.9 RC1 to 1.9 Stable
#11

Updated by Yannick Warnier about 7 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya
OK, let's do the following then:
  1. Leave things just as they are
  2. Add a language variable that says "The XML import let's you add more info and create resources (courses, users). The CSV import will only create sessions and let you assign existing resources to them." (and mark that as a normal-message class)
#12

Updated by Julio Montoya about 7 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 70 to 80

message added to translate.chamilo

TheXMLImportLetYouAddMoreInfoAndCreateResources

#13

Updated by Yannick Warnier about 7 years ago

  • Status changed from Needs more info to Bug resolved

Good for me

Also available in: Atom PDF