Moodle

Evaluate usage of idnumber fileld in Moodle database tables

Details

  • Type: Task Task
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 2.0
  • Fix Version/s: None
  • Component/s: Administration
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE

Description

While looking at MDL-15187 (assigning roles via CSV upload), I noticed that we are making an assumption that the idnumber field can be used as a unique record identifier for the user and course tables. I also noticed that we do not have an idnumber field for the course_category or groups tables which I would advocate for adding to Moodle 2.0 for increased interaction between Moodle and other student information systems (SISs) which may have unique internal identifiers for course categories and groups. I do not see a need for an idnumber field for the course_modules and block_instance tables. If the idnumber fields are assumed to be unique identifiers and are used as such for importing records, I wonder if we ought to enforce that either via an index or some checking similar to how we handle checking for duplicate usernames. I figured now is a good time to take a look at these issues and see if we wanted to add the idnumber field to course_categories and groups tables. Peace - Anthony

Issue Links

Activity

Hide
Anthony Borrow added a comment -

If we have an idnumber field in course_categories and groups it will help make resolving MDL-15187 consistent in that the context name or idnumber can be used as is the case for users and courses.

Show
Anthony Borrow added a comment - If we have an idnumber field in course_categories and groups it will help make resolving MDL-15187 consistent in that the context name or idnumber can be used as is the case for users and courses.
Hide
Anthony Borrow added a comment -

correction, idnumber is already in the course_modules table but not block_instance.

Show
Anthony Borrow added a comment - correction, idnumber is already in the course_modules table but not block_instance.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated: