Comment 4 for bug 1549393

Revision history for this message
Josh Stompro (u-launchpad-stompro-org) wrote :

Thanks for the feedback Mike.

I just tried your suggestion and did some more testing. I kept saying empty strings, but really the issue is with undef values in the arrays.

I added a pair of greps to remove the undef's from @isbns and @issns and now it works great, tested on our production system.

Content Cafe doesn't support ISSNs, but they have the same behavior of setting invalid numbers as undef, so I think it makes sense to clean them up also.

Working branch coming soon.

Josh