Legacy code is

  • modules/crm_core_activity/legacy/CRMFeedsActivityProcessor.inc
  • modules/crm_core_contact/legacy/CRMFeedsContactProcessor.inc

New plugins can be written by extending EntityProcessorBase and the legacy code functionality is mostly already implemented by this base class.

CommentFileSizeAuthor
#10 2753603-10.patch2.13 KBmikran
#10 interdiff.txt1.8 KBmikran
#7 crm_core_feed-2753603-7.patch2.06 KBmikran
#7 interdiff.txt2.37 KBmikran
#3 crm_core_feed-2753603-3.patch2.14 KBDropa

Issue fork crm_core-2753603

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikran created an issue. See original summary.

Dropa’s picture

Dropa CreditAttribution: Dropa at Mediamaisteri Oy commented
Assigned: Unassigned » Dropa
Dropa’s picture

Dropa CreditAttribution: Dropa at Mediamaisteri Oy commented
FileSize
crm_core_feed-2753603-3.patch2.14 KB

Added feed processor for Individuals and Organizations.

Dropa’s picture

Dropa CreditAttribution: Dropa at Mediamaisteri Oy commented
Component: Code » CRM Core Contact
Status: Active » Needs review
Dropa’s picture

Dropa CreditAttribution: Dropa at Mediamaisteri Oy commented
Assigned: Dropa » Unassigned
mikran’s picture

mikran CreditAttribution: mikran at Mediamaisteri Oy commented
Assigned: Unassigned » mikran
Status: Needs review » Needs work

Needs work as individual name is not mappable.

mikran’s picture

mikran CreditAttribution: mikran at Mediamaisteri Oy commented
FileSize
interdiff.txt2.37 KB
crm_core_feed-2753603-7.patch2.06 KB

Oh, turns out that name module is actually responsible of providing feeds target for the name field. But I added configuration forms to these processors so here is slightly improved patch.

mikran’s picture

mikran CreditAttribution: mikran at Mediamaisteri Oy commented
Status: Needs work » Needs review
mikran’s picture

mikran CreditAttribution: mikran at Mediamaisteri Oy commented
Related issues: +#2785491: Add a new FeedsTarget plugin to make name fields mappable in feeds importers.
mikran’s picture

mikran CreditAttribution: mikran at Mediamaisteri Oy commented
FileSize
interdiff.txt1.8 KB
2753603-10.patch2.13 KB

Updated patch to work with current version of feeds.

grahl’s picture

grahl
he/him
Location Zürich
CreditAttribution: grahl as a volunteer commented
Version: 8.x-1.x-dev » 8.x-3.x-dev
Assigned: mikran » Unassigned
Status: Needs review » Postponed

Setting to postponed for now since Migrate is available.

bluegeek9 made their first commit to this issue’s fork.

bluegeek9 opened merge request !18

  • bluegeek9 committed f0060b2 on 8.x-3.x 
    Issue #2753603 by mikran, bluegeek9, Dropa: Port FeedsProcessor plugins
bluegeek9’s picture

bluegeek9 CreditAttribution: bluegeek9 as a volunteer commented
Title: Port FeedsProcessor plugins » Remove legacy FeedsProcessor plugins
Status: Postponed » Fixed

Removes the legacy Feeds Processor code.

Status: Fixed » Closed (fixed)

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