moodle-an-hochschulen / moodle-local_bulkenrol

Moodle plugin which provides the possibility to bulk enrol a list of users who are identified by their e-mail adresses into a course.
5 stars 17 forks source link

#7739 Plugin local_bulkenrol #1

Closed soonsystems closed 6 years ago

soonsystems commented 6 years ago

Hallo Alex,

hier die erste Version des Plugins.

Einige Anmerkungen und Fragen habe ich dazu noch.

  1. $CFG->allowaccountssameemail == true wird aktuell nicht ausgewertet, sondern es wird generell bei mehreren Usern mit der gleichen Email diese Email beim Import ignoriert. Wenn diese Einstellung nicht gesetzt ist, kann das doch gar nicht vorkommen, dass zwei User dieselbe E-Mail haben, oder?

  2. Sollen evtl Fehler/Exceptions angezeigt werden? -> Wenn ja: wo/wie?

  3. Sollen Kurs-Gruppen, zu denen (nach Aussortieren von nicht vorhandenen E-Mails oder Fehler) keine User/Mitglieder mehr vorhanden sind, überhaupt angelegt werden? -> Aktuell werden die Kurs-Gruppen aus dem Textfeld angelegt, auch wenn es dann keine Mitglieder mehr gibt

  4. Es fehlen noch englische Übersetzungen und der "Hinweistext mit Nutzungshinweisen zum Plugin" -> Sollen wir das machen oder übernehmt ihr das?

LG Michael

abias commented 6 years ago

Code was merged manually in https://github.com/moodleuulm/moodle-local_bulkenrol/commit/541ba79b57137dca7e10dfde8ccf0d208752e44b.