In location_entity_field_info_alter, two properties are added to the "location" field type, which is defined in location_cck_field_info. Since the location_cck module is not required by the location_entity module, we should first check if the type exists.

CommentFileSizeAuthor
#3 location_entity-field-type-alter-fix-2496969-3.patch883 bytespeterpoe
#1 location_entity-field-type-alter-fix-2496969-1.patch887 bytespeterpoe

Comments

peterpoe’s picture

peterpoe commented
Status: Active » Needs review
StatusFileSize
new location_entity-field-type-alter-fix-2496969-1.patch887 bytes
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Needs review » Needs work 
+++ b/contrib/location_entity/location_entity.module
@@ -9,9 +9,11 @@
+    $field_info['location']['property_callbacks'][] = 'location_entity_field_item_location_property_callback';    ¶

Please, remove ending whitespaces, according to Drupal coding standards

peterpoe’s picture

peterpoe commented
Status: Needs work » Needs review
StatusFileSize
new location_entity-field-type-alter-fix-2496969-3.patch883 bytes

  • podarok committed e05e3de on 7.x-3.x authored by peterpoe 
    Issue #2496969 by peterpoe: Location field type altered when the type...
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
commented
Status: Needs review » Fixed

Thanks, pushed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.