bugfix: do not try to set items.cn_sort twice
authorGalen Charlton <galen.charlton@liblime.com>
Wed, 12 Dec 2007 21:27:46 +0000 (15:27 -0600)
committerJoshua Ferraro <jmf@liblime.com>
Thu, 13 Dec 2007 23:30:25 +0000 (17:30 -0600)
commitd46581b8f0499fe7353204d732f78e8465fa10c5
treeb789e309fea98f9e9e008e248a9a2a0c0d40f4f5
parent897c3a4eee42b1caecbbb0e848ed5bce32de2f66
bugfix: do not try to set items.cn_sort twice

The way this API is used by additem.pl, the $item hasref
already contains a 'items.cn_sort' key whose value is
undef.  Setting $item->{'cn_sort'} instead of $item->{'items.cn_sort'}
ends putting with items.cn_sort and cn_sort in the UPDATE
items statement, making the final value of cn_sort in the DB
dependent on the order produced by Perl's keys function.

Also added a comment explaining a subtle (perhaps too subtle)
point in the code.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
C4/Biblio.pm