Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #8338 (closed)

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

setInterval cancel on AJAX failure

Reported by: wmoore Owned by: wmoore
Priority: major Milestone: OMERO-4.4
Component: Web Version: n.a.
Keywords: n.a. Cc: atarkowska, cxallan
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2012-04-10 (12)

Description

If we are making repeated AJAX calls to the server, this should be cancelled if any calls fail, otherwise we get repeated error handling dialogs etc.

The only one we have currently is /activities/ I think.

Change History (4)

comment:1 Changed 12 years ago by wmoore

  • Status changed from new to accepted

comment:2 Changed 12 years ago by wmoore

  • Resolution set to fixed
  • Status changed from accepted to closed

This is fixed by commit https://github.com/will-moore/openmicroscopy/commit/1fa3f5e7ac25affd3b85627fa5badf6301e28ee5
pushed to a copy of 8118_share_connection on my github. NB: This commit should be cherry-picked since the branch also
contains a merge of 7105_js_lib_update. This is because the above commit requires jQuery 1.5 or above.

Closing...

comment:3 Changed 12 years ago by Will Moore <will@…>

  • Remaining Time set to 0

(In [f279a0bdb88b51b0c5b0712e10ba5b0e2eaa6dcb/ome.git] on branch develop) clearInterval() if AJAX request is error. Closes #8338

comment:4 Changed 12 years ago by Aleksandra Tarkowska <A.Tarkowska@…>

(In [b48a44eba2d334e2c920b8ada299547e8463fa0c/ome.git] on branch develop) clearInterval() if AJAX request is error. Closes #8338

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.70742 sec.)

We're Hiring!