Notice: Undefined variable: node in domain_feeds_set_target() on line 60 of domains_feeds.module

The return variable is $node but it looks like it should be $entity. I made the change locally and everything seems to be working correctly.

CommentFileSizeAuthor
#3 domain_feeds-set_target-2009404-3.patch417 bytesgeneralredneck
#1 domain_feeds-set-target.patch495 bytesjohn.money

Comments

john.money’s picture

john.money commented
Status: Active » Needs review
StatusFileSize
new domain_feeds-set-target.patch495 bytes

Works for me, patch attached.

hanksterr7’s picture

hanksterr7 commented

I concur. Great module, but definitely needs this patch applied to make it work. Someone should update the primary version.

generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
commented
Issue summary: View changes
StatusFileSize
new domain_feeds-set_target-2009404-3.patch417 bytes

I had to reroll this patch as it was created inside a drupal project and not inside the domain_feeds project.

  • Commit f2d460d on 7.x-1.x by Jim.M: 
    #1217990 fixed, #2009404 fixed
ndf’s picture

ndf commented
Status: Needs review » Fixed
jim.m’s picture

jim.m commented

This issue is fixed at the 7.x-1.2 release.

Status: Fixed » Closed (fixed)

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