Last modified: 2014-09-17 13:15:41 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T72938, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 70938 - Confusing/out of date comment in ep_users_per_course table SQL
Confusing/out of date comment in ep_users_per_course table SQL
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
EducationProgram (Other open bugs)
unspecified
All All
: High minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-17 12:19 UTC by Sam Reed (reedy)
Modified: 2014-09-17 13:15 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2014-09-17 12:19:15 UTC
-- Links the students with their courses.
CREATE TABLE IF NOT EXISTS /*_*/ep_users_per_course (
  upc_user_id                INT unsigned        NOT NULL, -- Foreign key on ep_user.user_id
  upc_course_id              INT unsigned        NOT NULL, -- Foreign key on ep_courses.course_id
  upc_role                   TINYINT unsigned    NOT NULL, -- The role the user has for the course
  upc_time                   varbinary(14)       NOT NULL -- Time at which the link was made
) /*$wgDBTableOptions*/;


ep_user.user_id doesn't exist. Is this actually the core user table user_id column?
Comment 1 Gerrit Notification Bot 2014-09-17 13:03:02 UTC
Change 160955 had a related patch set uploaded by Reedy:
ep_user -> user

https://gerrit.wikimedia.org/r/160955
Comment 2 Gerrit Notification Bot 2014-09-17 13:13:11 UTC
Change 160955 merged by jenkins-bot:
ep_user -> user

https://gerrit.wikimedia.org/r/160955
Comment 3 Andrew Green 2014-09-17 13:15:41 UTC
Yes, that's it.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links