Location's phone and fax contrib modules don't reveal their fields to views. What a shame! Is there a patch floating around?

I can probably give one in the next month if necessary... but for now I'm using CCK Phone field instead.

CommentFileSizeAuthor
#3 location_views-239658.patch1.3 KBles lim

Comments

andrew_mallis’s picture

I'd also like to see phone and fax exposed to views.

les lim’s picture

Title: implement phone and fax fields in views » expose phone and fax fields to views
Version: 5.x-2.7 » 6.x-3.x-dev

I suppose this is a bump of sorts. Views still can't pull Location Phone or Location Fax information as of Location 5.x-3.x-dev (July 7, 2008).

On a related note, neither phone nor fax information is getting output by default into an unthemed node. Both are available in the node object, though, so at least they can be themed pretty easily.

les lim’s picture

Assigned: Unassigned » les lim
Status: Active » Needs review
StatusFileSize
new1.3 KB

Okay, this gets the Location Phone and Fax fields working in Views for me. It's my first patch, so be nice.

les lim’s picture

Assigned: les lim » Unassigned
bdragon’s picture

Well formatted patch, two issues with it.

A) As I'm trying to reduce interdependencies between modules, I consider location_phone.module and location_fax.module adding their own hook_views_tables() to be the correct solution.
B) Joins need to go against the base location table, not location_instance_node.

Other than that, nice job.

I'll fix it up and commit it.

bdragon’s picture

Status: Needs review » Fixed

OK, committed. I added some filters as well.
http://drupal.org/cvs?commit=129348

Anonymous’s picture

Status: Fixed » Closed (fixed)

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

shushu’s picture

So, it is not clear to me - is it in the official release or just in the dev ?

helloari’s picture

i also would like to know if it's just in the dev and if that is close to being stable? i need this for a user location function and will probably just use 'profile' for now.

thanks for an amazing module! my client is blown away by the map function. now i just need to figure out how to do this zip code proximity search...

matt v.’s picture

I just made the same comment in a similar issue, but in case someone here misses it. . .

I believe you need to first add a "Relationship" for "Content: Location - (field_location)" because the phone is stored in a separate table. You should then be able to see phone and fax listed under the Fields section. However, I just did the same and when I try to actually add the phone number to the view, I'm getting the following error:

An error occurred at http://localhost/mysite/admin/build/views/ajax/add-item/daily_deals/defa...

edjay’s picture

have you desactive the theme developer module ?