[MARMOTTA-670] LDPath - ReversePropertySelector with wrong code

• Type: Wish
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 3.4.0
• Fix Version/s: 3.4.0
• Component/s: LDPath
• Labels: easyfix
• Assignee: Dietmar Glachs
• Reporter: Dietmar Glachs
• Created: 27/Sep/17 09:30
• Updated: 18/Jun/18 19:32
• Resolved: 01/Dec/17 20:03

Description

The ReversePropertySelector is not retrieving the requested subjects properly when, and only when the parameters "path" and "resultPahts" are provided.

Code snippet as currently implemented in ReversePropertySelector:

ReversePropertySelector.java - select

...
if(path != null && resultPaths != null) {
  Collection<Node> results = rdfBackend.listSubjects(context, property);
  ...
  return results;
} else {
    return rdfBackend.listSubjects(property, context);
}
...

The wrong if part of the method is only used in marmotta-search (previously lmf-search) when parsing LDPath search programs, the else functionality is frequently used when parsing LDPath queries.
however, the calls to listSubjects should be equal in if and else.

Issue Links

• is duplicated by: MARMOTTA-673 LDPath: Inverse selection won't work when provided a path