Hi!

I made a improvement in the mappers, and now with these patch you can import content to "All affiliates" and Base Domain.

I hope that these can be a part of the module, because it's really handy when you have dynamic Domains.

cheers!

CommentFileSizeAuthor
#8 domain_feeds-publish_to_all-1217990-8.patch704 bytesrolfmeijer
domain_feeds-6.x-1.0.patch1.08 KBsebas5384

Comments

jim.m’s picture

jim.m commented
Assigned: Unassigned » jim.m

Hey this is good. I'll get it in the next release.

Thanks!

Jim

sebas5384’s picture

sebas5384 commented

Great! Glad to help!

Cheers,
Sebas.

colorado’s picture

colorado commented

I can confirm that this works, and is a critical feature. Thank you so much sebas5384!

sebas5384’s picture

sebas5384 commented
Status: Needs review » Reviewed & tested by the community

Glad to help! :)

Would be grate to have this in the module out of the box, if that happens i can make a port to D7

Cheers,
Sebas.

jim.m’s picture

jim.m commented

This feature has been added to the latest release.

jim.m’s picture

jim.m commented
Status: Reviewed & tested by the community » Closed (fixed)
Funksmaname’s picture

Funksmaname commented
Version: 6.x-1.0 » 7.x-1.1
Assigned: jim.m » Unassigned
Status: Closed (fixed) » Needs work

This feature isn't working correctly in the latest version... the sub domain ones are working, but not 'all'

rolfmeijer’s picture

rolfmeijer commented
StatusFileSize
new domain_feeds-publish_to_all-1217990-8.patch704 bytes

I’ve made a patch to get this function in D7 as well. Not sure if this is the correct way, but it seems to work.

rolfmeijer’s picture

rolfmeijer commented
Status: Needs work » Needs review
rolfmeijer’s picture

rolfmeijer commented

btw, when using the Publish to all, because it always needs one domain to publish the node to, it will be published to the default domain. Unfortunately the Publish to All mapping cannot be combined with a Publish to a specific other domain. If I really need that I might look into why this is.

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

ndf commented
Issue summary: View changes
Status: Needs review » Fixed

@ #10: i think its because both the 'domain_feeds_set_target' and the 'domain_feeds_set_target_all' set '$entity->domain_site'.
They basicly override each other. If you want to publish to all domains it must be set to TRUE.

I guess you can outsmart this behavior when you ensure the 'domain_feeds_set_target_all' is called last.

In your mapping you'll get

  • Domain One -> Publish to: domain one
  • Domain Two -> Publish to: domain two
  • Domain All -> Publish to: All

And in your input data the column order (critical) would be

Domain One|Domain Two|Domain All
1|1|1

What I expect is that when you edit the imported node under "Domain access options" you get:
- "Send to all affiliates" is checked. <-- all behaviour
- Under 'publish to' domain one and domain two are checked (they are the 'owners').

~I did not test this~

jim.m’s picture

jim.m commented

I just checked, you should set "Publish to: All" OR choose the domains an entity should be published to.

Domain One|Domain Two|Domain All
1| | 
 |1| 
1|1| 
 | |1

Status: Fixed » Closed (fixed)

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

bibo’s picture

bibo commented

I just checked, you should set "Publish to: All" OR choose the domains an entity should be published to.

Domain One|Domain Two|Domain All
1| |
|1|
1|1|
| |1

Please check again @Jim.M . I'm pretty sure your assumption is wrong and @ndf is right.

This should be patched so it works correctly, for now I did get it work as expected with the workaround of setting "Publish: All" after the invidual domains.