Skip to content
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

Duplicates shown in user interface. #508

Closed
stefjoosten opened this issue Aug 20, 2016 · 4 comments
Closed

Duplicates shown in user interface. #508

stefjoosten opened this issue Aug 20, 2016 · 4 comments
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior priority:high

Comments

@stefjoosten
Copy link
Contributor

Consider the following interface specification

INTERFACE StudentScripts FOR Tutor : '_SESSION'[SESSION]#(I[Account]/\submittor~;submittor) cRud
  BOX<SCOLS>
        [ indiener : I/\submittor~;submittor cRud
        , scripts : submittor~ cRud
          BOX<SCOLS>
              [ opdracht  : assignment cRud
              , bestand   : content cRUd
              , ingediend : submitted cRud
              ]
        ]

It produces the following effect:

image

In field "indiener", it is unexpected to see the same atom 3 times.

Reproducing this bug.

tbd

@stefjoosten stefjoosten added the bug Indicates an unexpected problem or unintended behavior label Aug 20, 2016
@hanjoosten
Copy link
Member

It might be due to the fact that no select DISTINCT is being done, because of optimalisations. In this case, the outer select DISTINCT should be wrapped by the frontend. @Michiel-s , do you agree?

@Michiel-s
Copy link
Member

This indeed relates to #486, which is not completely resolved and fully tested (yet).

@hanjoosten
Copy link
Member

@stefjoosten I have fixed the related bug of #486. I have made a pull request of that fix. However, since you still promised a reproduction path (see the original post above), could you test that this fix solves your problem?

@hanjoosten hanjoosten assigned stefjoosten and unassigned Michiel-s Jan 2, 2017
@Michiel-s
Copy link
Member

PR is merged. Issue is solved. @stefjoosten please reopen this issue if your case is not solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior priority:high
Projects
None yet
Development

No branches or pull requests

4 participants