openCloseImport incorrectly fills data into tables (W)

[afig]

The following issues have been identified:

• [x] After importing several terms, the same instructor is inserted several times. This results in multiple entries for each instructor in the Instructor table. This seems to be an issue with no longer checking for uniqueness in names after they were split into fname, mname, and lname.

• [x] Only the initial title of each course is stored. Some courses have their names updated between terms. For example, even after importing terms up to Sp2017, The entry for CS170 is titled Computer Sci I: Language.

• [x] Sections that have at least one instructor who does not have a middle name registered are not added to the Section table.

The course title issue brings up the question as to whether we want to store these historical titles. (We may choose not to do so currently.)

[afig]

Due to the middle name issue, the three sections that we need for testing roster and attendance parts of the system are not populated.

The following script will manually insert these three sections: https://gist.github.com/afig/4e11d6a60a006c0e6b9e539e32ca8851

The above script should be run after importOpenClose, but before importRoster.

[wildtayne]

Changing the RETURNING statement to this: RETURNING id, FName || ' ' || COALESCE(MName || ' ', '') || LName as FullName should fix the middle name problem. This issue was that if the instructor had no middle name, this statement inserted two spaces between FName and LName, which caused the LIKE predicate to fail later. This updated statement should only insert a space after MName if it is not null.