I think we should consider to always write an log message just like FeedsNodeProcessor does.

Messages could be in same format:

  • Created by FeedsCommerceProductProcessor
  • Updated by FeedsCommerceProductProcessor
  • Replaced by FeedsCommerceProductProcessor
CommentFileSizeAuthor
#2 commerce_feeds-1611420-1.patch905 bytesimiksu

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

Agreed :)

imiksu’s picture

imiksu
Location Finland
commented
Status: Active » Needs review
StatusFileSize
new commerce_feeds-1611420-1.patch905 bytes

Patch against 7.x-1.2.

I see that there is no settings of updating vs replacing.

I haven't tested the patch since the code seems trivial.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Fixed

Had to apply the patch manually, not sure why, also added "replace" status too :)

Many thanks!!

Status: Fixed » Closed (fixed)

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