Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 362474

Summary: NavigationHistoryService: Bookmark.getText() is different from values in Bookmark.getPath()
Product: z_Archived Reporter: Adrian Moser <trekking09>
Component: ScoutAssignee: Andreas Hoegger <Andreas.Hoegger>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Andreas.Hoegger, zimmermann
Version: unspecifiedFlags: zimmermann: indigo+
zimmermann: juno+
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Proposed patch. Andreas.Hoegger: iplog+, Andreas.Hoegger: review+

Description Adrian Moser CLA 2011-10-31 07:54:37 EDT
Build Identifier: M20110210-1200

The NavigationHistoryService creates a bookmark for each step in Navigation history. Currently, the navigation step bookmark is created before a search is executed. The created bookmark does contain the correct path and search data, but the display text refers to one step before the current step.

Solution: Create the navigation bookmark after the search is executed. 

No migration for existing scout projects necessary.

Reproducible: Always

Steps to Reproduce:
1. Perform a search with search text 'A'
2. Perform a search with search text 'B'
3. The navigation bookmark created for step 2 will contain search text 'B', but it's display text value referes to text 'A'.
Comment 1 Adrian Moser CLA 2011-10-31 07:55:09 EDT
Created attachment 206204 [details]
Proposed patch.
Comment 2 Andreas Hoegger CLA 2011-10-31 08:18:40 EDT
Patch applied; Thank you Adrian.
Comment 3 Matthias Zimmermann CLA 2012-02-29 07:03:23 EST
shipped with eclipse scout 3.7.2 (indigo sr2)