Skip to content

Commit

Permalink
JDF-374 Fix to ensure bookings view is refreshed on delete
Browse files Browse the repository at this point in the history
This required a server-side change to return HTTP 204 No Content responses instead of HTTP 200 OK. This is primarily due to a change in jQuery 1.9 that treats empty 200 OK responses as errors which cascaded to Backbone and eventually resulted in no updates to the view. The server-side change now results in a 204 response preventing this hidden failure.
  • Loading branch information
VineetReynolds committed Jun 20, 2013
1 parent 82ae51d commit d6824a0
Showing 1 changed file with 2 additions and 2 deletions.
Expand Up @@ -71,7 +71,7 @@ public Response deleteAllBookings() {
for (Booking booking : bookings) {
deleteBooking(booking.getId());
}
return Response.ok().build();
return Response.noContent().build();
}

/**
Expand Down Expand Up @@ -105,7 +105,7 @@ public Response deleteBooking(@PathParam("id") Long id) {
booking.getPerformance(), sectionListEntry.getValue());
}
cancelledBookingEvent.fire(booking);
return Response.ok().build();
return Response.noContent().build();
}

/**
Expand Down

0 comments on commit d6824a0

Please sign in to comment.