Vandelay record match sets: fix bug loading some trees user/senator/match-sets-load-error
authorLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Tue, 14 Feb 2012 16:18:27 +0000 (11:18 -0500)
committerLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Tue, 14 Feb 2012 16:18:27 +0000 (11:18 -0500)
commit135fc6a01174f266b254618ad7dc5017e336071e
tree7682c63b5e2dba23d7e024a697f511d716972fe0
parent7510d7ff71c6cf1f2fc8ba9210a2d7ef34bc52d3
Vandelay record match sets: fix bug loading some trees

Reported to me by Bill Erickson.  The Vandelay Record Match Sets
interface was failing to load some simple trees of match set points.

Example: http://pastesite.com/31643

The proximate symptom was this error message:

    Error: dojo.data.ItemFileReadStore: Invalid item argument.
    Source File: http://dev198.esilibrary.com/js/dojo/dojo/dojo.js
    Line: 119

but the root cause was a problem in dojoize_match_set_tree(), which
was failing to generate unique identifiers for each item in the data
store it's responsible for generating.

Since that function is only ever used to deal with trees where all the
nodes come from the database and have actual IDs, that function now uses
the database IDs as is rather than attempting to generate new ones.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Open-ILS/web/js/ui/default/conify/global/vandelay/match_set.js