Refactor getConversionChar2Met

[dczielinski]

Currently the function behaves strangely where it can accept a list of metabolite name strings, or a reaction, and in both cases will return either a corresponding metabolite object list or the list of metabolites involved in the reaction, respectively. It would seem more natural to create two separate functions with more clear use cases.