-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store the original variant description as the source of the submission #561
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few changes are needed; please see my comments (by accident in Dutch, but oh well). Could you take care of the needed changes?
Additionally, to actually test the code written in inc-upgrade.php
and to be able to use this new field, you'll have to increase the version of the code to 3.5-pre-03
as well. You can do this in inc-init.php
. Please make that change as well and add it to this branch.
Thanks!
cd4c8b4
to
49bf4a1
Compare
src/variants.php
Outdated
// Get the ID by its column suffix and give this as the source. | ||
$sID = $_DB->query( | ||
'SELECT id FROM ' . TABLE_GENOME_BUILDS . | ||
'WHERE column_suffix = ?', array($sColumnSuffix))->fetchColumn(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how this is possible, but there is another space missing. I thought we tested this code - this should have caused a query error. Anyway, I'll fix it.
- SQL is prettier and easier to read when aligned. - Since I already forgot an implementation detail, better document it.
Added the
source
column to the VOG table.Closes #560
Related to #550