Labels Cleanup (Part 1 of Many)
authorJoe Atzberger <joe.atzberger@liblime.com>
Wed, 3 Dec 2008 15:31:49 +0000 (09:31 -0600)
committerGalen Charlton <galen.charlton@liblime.com>
Wed, 31 Dec 2008 21:56:11 +0000 (15:56 -0600)
commit55c2320a23b03d94dd6b7de463e9842ec0c4ccfc
tree4420496e9f7ac927ce21f827eba42e3a25d4edc1
parentca7ae0ab41168c9891795fd4d20c87c00a51b076
Labels Cleanup (Part 1 of Many)

Consolidated error catching after evals.
Removed unnecessary $sth->finish calls and some unused variables.
Pulled query for itemtype mappings outside DrawSpineText and added
a class level caching variable to eliminate repeated queries for
*each piece of text* on *each label*!  This was a major performance
downside.

Note: this does not fix Unicode problems, but it does add some notes
on unsuccessful attempted workaround using utf8::encode.

C4::Labels should likely be broken up to separate out the pieces that
do not touch the database (wrappers of PDF::Reuse) and those that are
CRUD API for table data.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
C4/Labels.pm