Hi

At first I would like to thank you for this great module and for active maintaining.

I've installed it (v1.0) on fresh D7.12, Getlocations Fields enabled, but noticed few bugs:
- according to documentation at admin/config/services/getlocations should be some configuration, but without taxonomy or views isn't
- when I tried to install views there was an error at line 1496, maybe try to add checking for taxonomy module one line above or add it as a dependency

With views and taxonomy it's much better ;)
thanks

Comments

altavis’s picture

altavis commented

I think all we need is:

function getlocations_get_vocabularies() {
  if (module_exists('getlocations_fields') && module_exists('taxonomy')) {
    $vocabularies = taxonomy_get_vocabularies();
hutch’s picture

hutch commented
Version: 7.x-1.0 » 7.x-1.x-dev
Status: Active » Closed (fixed)

Seems sensible, it's easy to forget that taxonomy is optional :-)
I've added the taxonomy module check to getlocations_get_vocabularies() and commited along with a few other bits and pieces.