learn-co-curriculum / dsc-database-admin-101-lab Goto Github PK

A student reported this to me and said there seems to be a disconnect between the solution and lab.

Link to Canvas

https://learning.flatironschool.com/courses/5329/assignments/187391?module_item_id=417321

Issue Subtype

• Master branch code
• [ x] Solution branch code
• Code tests
• Layout/rendering issue
• Instructions unclear
• Other (explain below)

Describe the Issue

Source

# Delete the duplicate entry
cur.execute("""DELETE FROM contactInfo 
               WHERE telephone = 3259909290;""")

Concern

The goal for this snippit of code is to delete a duplicate entry, but the solution deletes all entries.

(Optional) Proposed Solution

cur.execute("""SELECT firstName, lastName, telephone, COUNT()
FROM contactInfo
GROUP BY firstName, lastName, telephone
HAVING COUNT() > 1;""").fetchall()

cur.execute("""SELECT userID
FROM contactInfo
WHERE tel_num = 3259909290;""").fetchall()

cur.execute("""DELETE FROM contactInfo
WHERE userID = 6""")

The code finding the duplicate entry can still be used. Then using the telephone number find the primary keys of all the entries containing that number. Using the primary key delete the duplicate. In the case of this example the primary keys were 5 and 6. Code can be adjusted if there are multiple duplicate entries by changing the operator to > lowest_userID.

What OS Are You Using?

• OS X
• [X ] Windows
• WSL
• Linux
• IllumiDesk

Any Additional Context?

The file "school.sqlite" - exists in the forked repo (master branch), causes error due to pre-existing tables. Suggest removing to avoid errors. Otherwise, students need to delete it or use another workaround to move through the lab. Thanks!

I believe it should only be in the solution branch