Notice: Undefined index: country_name in getdirections_other_load_locations() (line 1827 of getdirections/getdirections.module).

I get this notice intermittently.

Comments

hutch’s picture

It looks like the addressfield module does not always supply "country_name" as well as "country" so I have added a check to eliminate the notice.

blicarea’s picture

I am also having the problem on line 2086:

Notice: Undefined index: country_name in getlocations_other_load_locations() (line 2086 of /home/blicarea/public_html/pilgrm/sites/all/modules/getlocations/getlocations.module).
Notice: Undefined index: country_name in getlocations_other_load_locations() (line 2086 of /home/blicarea/public_html/pilgrm/sites/all/modules/getlocations/getlocations.module).

hutch’s picture

Status: Active » Needs review

I have added this to my patch queue, thanks for reporting this.

sumeet.pareek’s picture

I get this on the line 421.

Notice: Undefined index: use_country_dropdown in getdirections_settings_form() (line 421 of /var/www/tp/sites/all/modules/contrib/getdirections/getdirections.admin.inc).

Eh. Mind rolling the patch in? Since you are already on it and have marked this 'needs review'.

Many thanks. The module looks good and I am looking forward to using it and adding my use-case related changes as patches soon.

hutch’s picture

You are working with an older version of Getdirections, all testing should be done on current dev.

The notice suggests that you have had a much older version and need to save the admin/config/services/getdirections form, this will update the saved configuration to include "use_country_dropdown" and update the cached verion of the configuration which is what is causing this notice.

You can of course switch notices off on production sites in admin/config/development/logging.

hutch’s picture

Issue summary: View changes
Status: Needs review » Closed (fixed)