Skip to content
Snippets Groups Projects

Modify participant CSV imports to support student number

Closes #310 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
136 */
137 private User findImportedUser(String identifier) {
138 User user;
139 int num = 0;
140
141 try {
142 num = Integer.parseInt(identifier);
143 user = userRepository.findByStudentNumber(num);
144 } catch (NumberFormatException nfe) {
145 user = userRepository.findByUsername(identifier + "@tudelft.nl");
146 }
147
131 148 if (user == null) {
132 user = new User(netid + "@tudelft.nl", "", netid, "",
133 DefaultRole.ROLE_STUDENT, 0);
149 user = new User(identifier + "@tudelft.nl", "", identifier, "",
  • added 1 commit

    • 45575035 - Modify participant csv imports to support student number

    Compare with previous version

  • So the problem here might be that you try to import a student and the system silently ignores an unknown entry if I read the code correctly. How hard would it be to at the end display a list of students that could not be imported?

  • mentioned in issue #120 (closed)

  • This will be another issue to pickup for Queue 2.0.0/Portal, since this is a slightly sketchy change to put in now and might be more relevant to solve within Portal instead.

  • closed

  • Please register or sign in to reply
    Loading