{"action":"created","issue":{"url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/1273","labels_url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/1273/labels{/name}","comments_url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/1273/comments","events_url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/1273/events","html_url":"https://github.com/eucalyptus/eucaconsole/pull/1273","id":57061945,"number":1273,"title":"GUI-1590 handle smart-refresh better with multiple getItems() callers","user":{"login":"dkavanagh","id":367566,"avatar_url":"https://avatars.githubusercontent.com/u/367566?v=3","gravatar_id":"","url":"https://api.github.com/users/dkavanagh","html_url":"https://github.com/dkavanagh","followers_url":"https://api.github.com/users/dkavanagh/followers","following_url":"https://api.github.com/users/dkavanagh/following{/other_user}","gists_url":"https://api.github.com/users/dkavanagh/gists{/gist_id}","starred_url":"https://api.github.com/users/dkavanagh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dkavanagh/subscriptions","organizations_url":"https://api.github.com/users/dkavanagh/orgs","repos_url":"https://api.github.com/users/dkavanagh/repos","events_url":"https://api.github.com/users/dkavanagh/events{/privacy}","received_events_url":"https://api.github.com/users/dkavanagh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-09T17:26:57Z","updated_at":"2015-02-09T19:18:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/eucalyptus/eucaconsole/pulls/1273","html_url":"https://github.com/eucalyptus/eucaconsole/pull/1273","diff_url":"https://github.com/eucalyptus/eucaconsole/pull/1273.diff","patch_url":"https://github.com/eucalyptus/eucaconsole/pull/1273.patch"},"body":"handle smart-refresh better with multiple getItems() callers\r\n\r\nhttps://eucalyptus.atlassian.net/browse/GUI-1590\r\n\r\nThe changes are these:\r\n - add state variable to indicate smart refresh is already happening, so we can avoid starting a new one.\r\n - reset that variable when no state requiring smart refresh is in the table\r\n - pass optional value into getItems() so that we can know when it was called from existing smart refresh timeout\r\n\r\nThis works well. It refreshes properly with no filters. As filters are added, the single refresh timeout is the only one running and the new filters are used. If the transient item is filtered out, smart refresh halts after the next cycle. If a transient item is brought back in by a filter change, smart refresh initiates properly again. This change fixes the mult-fetch problem seen prior."},"comment":{"url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/comments/73571111","html_url":"https://github.com/eucalyptus/eucaconsole/pull/1273#issuecomment-73571111","issue_url":"https://api.github.com/repos/eucalyptus/eucaconsole/issues/1273","id":73571111,"user":{"login":"dkavanagh","id":367566,"avatar_url":"https://avatars.githubusercontent.com/u/367566?v=3","gravatar_id":"","url":"https://api.github.com/users/dkavanagh","html_url":"https://github.com/dkavanagh","followers_url":"https://api.github.com/users/dkavanagh/followers","following_url":"https://api.github.com/users/dkavanagh/following{/other_user}","gists_url":"https://api.github.com/users/dkavanagh/gists{/gist_id}","starred_url":"https://api.github.com/users/dkavanagh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dkavanagh/subscriptions","organizations_url":"https://api.github.com/users/dkavanagh/orgs","repos_url":"https://api.github.com/users/dkavanagh/repos","events_url":"https://api.github.com/users/dkavanagh/events{/privacy}","received_events_url":"https://api.github.com/users/dkavanagh/received_events","type":"User","site_admin":false},"created_at":"2015-02-09T19:18:16Z","updated_at":"2015-02-09T19:18:16Z","body":"hal, retest this please"}}