]> git.evergreen-ils.org Git - working/Evergreen.git/commit
LP#1379824 Make PermaCrud.js disconnect() actually disconnect user/dbwells/lp1379824_fix_pcrud_disconnects
authorDan Wells <dbw2@calvin.edu>
Thu, 9 Oct 2014 21:32:42 +0000 (17:32 -0400)
committerDan Wells <dbw2@calvin.edu>
Fri, 10 Oct 2014 15:17:44 +0000 (11:17 -0400)
commit2b380f7666175575ac7d25f939755a5f35825510
tree0baa1f37670109ad26bf4fe1bda857d88c6e658a
parentf7edee62ba832ffd5cd3a8c1946dd6a10bf33d1c
LP#1379824 Make PermaCrud.js disconnect() actually disconnect

The disconnect() method in PermaCrud.js was meant to wrap the
underlying session disconnect, but it never actually disconnected
the session.  This could lead to problems in long-living PermaCrud
objects, as they may think they are still connected when they are not
(the session remote_id is never cleared).

Also, remove a couple (now redundant) manual session.disconnect() calls.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/web/js/dojo/openils/PermaCrud.js