Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 334193 - [client] double GETs when using favorites, folder links, or breadcrumb
Summary: [client] double GETs when using favorites, folder links, or breadcrumb
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.2   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 334195
Blocks:
  Show dependency tree
 
Reported: 2011-01-12 21:59 EST by Susan McCourt CLA
Modified: 2011-09-01 11:42 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2011-01-12 21:59:09 EST
We seem to have a general problem with fetching navigation twice.
you can see double GETS in the console when navigating around the tree.
I suspect it's some kind of hashchange duplication.
There's code to detect whether a hashchange is happening due to something we've just programmatically set, but it must be wrong.
Comment 1 Susan McCourt CLA 2011-01-12 22:02:07 EST
I think it's best to revisit this after addressing bug 334195.  I spent a few hours on this prior to M4, and each time I found a problem, it pointed to a follow-on issue.  I have a patch in my eclipse workspace with some experiments that can be revisited.
Comment 2 Susan McCourt CLA 2011-01-20 12:38:36 EST
I'll look at this for M5
Comment 3 Susan McCourt CLA 2011-01-26 21:40:42 EST
There are triple GETS now
Comment 4 Susan McCourt CLA 2011-01-27 14:35:08 EST
I've committed the first part of this change, which eliminates the double gets from the breadcrumb navigation, favorites, when you first land, browser history, and user updating of the hash.  These were problems in both the tree and table.  The fix for these was to ensure that the post-processing, including parent/child caching, that we do in fileClient.getchildren was also done when loading workspaces.  In the navigator load workspace code, we were calling a subsequent getChildren (presumably to cause the post-processing fito happen).  Now fileClient calls the same post-processing code when a workspace is loaded.

The remaining problem is specific to nav-table.  We still do a double GET when the user navigates to a folder using the folder link.  I'm looking at that now
Comment 5 Susan McCourt CLA 2011-01-27 15:00:58 EST
(In reply to comment #4) 
> The remaining problem is specific to nav-table.  We still do a double GET when
> the user navigates to a folder using the folder link.  I'm looking at that now

The problem here was that the "onclick" which expands the tree was attached to the column that contains the expando, rather than just the expando.  So we would do a first get based on the onclick in the expando (to show children) but then immediately navigate the link, which changed the hash and caused the second get.

Fixed >20110127.