search

jaredcowing / wmsNewBooks

Small program to view list of new books by fund & date acquired using WorldShare Acquisitions API.
MIT License
2 stars 1 forks source link

Addressing database issues #4

Closed gjerdery closed 4 years ago

gjerdery commented 4 years ago

I encountered a couple of database issues in my install. in getOINfromOCN(), the query wasn't returning any results. Changing 'received' to uppercase in the query resolved this. Also, in updateItems(), my db was throwing errors on strings with apostrophes. Swapping in Code Ignitor's $this->db->escape() function in place of addslashes() fixed this.

jaredcowing commented 4 years ago

Thanks for looking up the CI function for escaping! That's a good catch on case sensitivity, I'll have to check back on how my queries were being made case-insensitive.