reference fieldmap type [#1897172]

Let me know if I'm going overboard with the tickets folks...

I have some Salesforce mappings set up and I want to map to some term reference fields but they aren't available in the admin form so I decided to add another type - reference. I would love to use relation for everything but that's just not going to work for my project as yet.

Patch attached. This patch also includes code to narrow the options lists down to those specific to the bundle chosen.

So far I've added to entity_wrapper->type() value cases to salesforce_mapping_reference_fieldmap_pull_value:

list<taxonomy_term>
taxonomy_term

Other entity reference fields will have to have their value cases added as necessary

Comment	File	Size	Author
#2	reference_fieldmap-1897172-2.patch	9.78 KB	markisatacomputer
	reference-fieldmap.patch	9.39 KB	markisatacomputer

Comments

Comment #1

kostajh commented 29 January 2013 at 00:02

Version:	7.x-3.0-alpha2	» 7.x-3.x-dev
Status:	Active	» Postponed

Thanks for the patch. Let's get #1896134: Properties option list specific to bundle when possible reviewed and then come back to this.

Comment #2

markisatacomputer commented 6 February 2013 at 00:25

Status	File	Size
new	reference_fieldmap-1897172-2.patch	9.78 KB

I've updated my local branch to avoid creating new terms unless the field instance has autocomplete/tagging widget. Here's the updated patch.

Comment #3

ohthehugemanatee commented 28 February 2014 at 19:14

Issue summary:	View changes
Status:	Postponed	» Needs work
Issue tags:		+Needs reroll

This needs a reroll, if not a complete rewrite. Unfortunately this patch also includes some preliminary work on restricting fields to the selected bundle (#1896134: Properties option list specific to bundle when possible), which makes it hard to see what code does what functionality. Please keep your patches restricted to one issue at a time!

The related entities functionality is so poorly documented I can't tell if this patch is even necessary anymore. Can someone please advise on how it's supposed to work, and whether it should work with term_reference fields?