minor refactor to use callback selectItems

Did some minor refactoring so that Ovid uses callback instead of synchronous selectItems. Didn't want to refactor too much since there were no test cases.
`Zotero.selectItems` only returns a value for compatibility reasons and is likely to not return anything meaningful in the future.