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

Filter "swivt:redirectsTo" in CompoundConditionBuilder #1477

Merged
merged 1 commit into from Mar 25, 2016

Conversation

Projects
None yet
1 participant
@mwjames
Contributor

mwjames commented Mar 25, 2016

  • Remove entities that contain a "swivt:redirectsTo" predicate to avoid results sets contain duplicate entries
  • TitleLookup to actual return the redirect source not the targets (only relevent for --redirects option of rebuildData.php)
Filter "swivt:redirectsTo" in CompoundConditionBuilder
- Remove entities that contain a "swivt:redirectsTo" predicate
  to avoid results sets contain duplicate entries
- TitleLookup to actual return the redirect source not the targets
 (only relevent for `--redirects` option of `rebuildData.php`)

@mwjames mwjames added the enhancement label Mar 25, 2016

@mwjames mwjames added this to the SMW 2.4 milestone Mar 25, 2016

@mwjames

This comment has been minimized.

Contributor

mwjames commented Mar 25, 2016

p-0904.json contains the use case that would return different result sets for SQLStore / SPARQLStore if the PR is not applied.

@mwjames mwjames merged commit 213d220 into master Mar 25, 2016

3 checks passed

Scrutinizer 9 new issues, 7 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@mwjames mwjames deleted the sparql branch Mar 25, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment