Computed field will make your module complete. :D

CommentFileSizeAuthor
#4 simple_gmap.patch469 bytestlogan

Comments

jhodgdon’s picture

Status: Active » Postponed (maintainer needs more info)

I have no idea what you are asking for here... can you provide more details?

espurnes’s picture

I think Rebekaloz is talking about Display Suite (http://drupal.org/project/ds).
DS alows you to create custom fields (code field, dynamic field, block field and preprocess field).

For instance you can have three fields:
- adress
- zip/postal code
- city

and merge them to a new code field (adress_map_field) like [adress],[zip/postal code],[city].
Would be great if you can use this new code field with the simple map.

jhodgdon’s picture

It could also be the Computed Field module (http://drupal.org/project/computed_field). I would like to know for sure before I do any work to support either of these in the Simple Google Maps module.

tlogan’s picture

Issue summary: View changes
StatusFileSize
new469 bytes

I came upon this post while looking for a solution to this module not working for computed fields. Since I couldn't find a solution I started poking at the code. Here's what I found to work...simply add "computed" to the field_types array. Patch is attached.

jhodgdon’s picture

Title: COMPUTED FIELD » Support Computed Field fields
Version: 7.x-1.0-rc1 » 7.x-1.x-dev
Status: Postponed (maintainer needs more info) » Reviewed & tested by the community

I suppose that would be fine to add to the module... the original poster never did get back on his/her intention, so let's say it was Computed Field module.

  • jhodgdon committed 03f94b6 on 7.x-1.x authored by tlogan
    Issue #1792398 by tlogan: Support Computed Field fields
    

  • jhodgdon committed 9f470dd on 8.x-1.x
    Issue #1792398 by tlogan, jhodgdon: Support Computed Field fields
    
jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

I don't see a problem with adding this to the module, since it will not harm what the module already does, and it has been tested to work. Sorry for the long delay.

Added something similar to Drupal 8 version also, where the machine name of the field is 'computed_string' for computed text fields in version 8.x-2.x, and 'computed' for 8.x-1.x. I added both, since it doesn't really hurt anything.

I tested both using the usual manual testing plan on #2543032: Steps to manually test the module.. No problems observed. I didn't test functionality with Computed Field itself.

Status: Fixed » Closed (fixed)

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