Merge lp://staging/unity-scopes-api/staging into lp://staging/unity-scopes-api

Proposed by Paweł Stołowski
Status: Merged
Approved by: Michi Henning
Approved revision: 285
Merged at revision: 328
Proposed branch: lp://staging/unity-scopes-api/staging
Merge into: lp://staging/unity-scopes-api
Diff against target: 123 lines (+26/-10)
5 files modified
debian/libunity-scopes3.symbols (+2/-2)
include/unity/scopes/internal/SearchQueryBaseImpl.h (+4/-2)
src/scopes/internal/ScopeImpl.cpp (+1/-0)
src/scopes/internal/ScopeObject.cpp (+2/-1)
src/scopes/internal/SearchQueryBaseImpl.cpp (+17/-5)
To merge this branch: bzr merge lp://staging/unity-scopes-api/staging
Reviewer Review Type Date Requested Status
Unity Team Pending
Review via email: mp+260101@code.staging.launchpad.net

Commit message

Fix for over-zealous loop detection. A loop is now detected only if
a query loops back with the exact same details, that is, query string,
department, filter state, user data, and metadata must all be the same. Fixes: https://bugs.launchpad.net/bugs/1457698 (cherry-picked from devel).

Description of the change

Fix for over-zealous loop detection. A loop is now detected only if
a query loops back with the exact same details, that is, query string,
department, filter state, user data, and metadata must all be the same. Fixes: https://bugs.launchpad.net/bugs/1457698. (cherry-picked from devel).

To post a comment you must log in.
285. By Paweł Stołowski

Fix for over-zealous loop detection. A loop is now detected only if
a query loops back with the exact same details, that is, query string,
department, filter state, user data, and metadata must all be the same. Fixes: https://bugs.launchpad.net/bugs/1457698.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: