-
Notifications
You must be signed in to change notification settings - Fork 38
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
Fix fetchn on queries with zero results #355
Conversation
Update release log
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.
Looks great!
@@ -289,7 +289,8 @@ function clip(self) | |||
for iArg=1:length(specs) | |||
% if renamed, use the renamed attribute | |||
name = regexp(specs{iArg}, '(\w+)\s*$', 'tokens'); | |||
if isnumeric(s(1).(name{1}{1})) && length(s(1).(name{1}{1})) == 1 | |||
sel = cellfun(@(x) strcmp(x, name{1}{1}), {self.header.attributes.name}); | |||
if self.tableHeader.attributes(sel).isNumeric |
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.
this may have introduced a regression - received a report elsewhere:
To fix the bug I changed:
In dj.internal.GeneralRelvar (line 293) I changed
if self.tableHeader.attributes(sel).isNumeric
to
if self.header.attributes(sel).isNumeric
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.
looks like this has been filed as an issue in: #364; disregard discussion here.
Depends on datajoint/mysql-docker#29
Fix #353
Fix #352