Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Cannot reference twice the same collection from another collection #9

Open
paulbalomiri opened this issue Mar 10, 2016 · 5 comments
Open

Comments

@paulbalomiri
Copy link

Hi,

according to the Meteor Docs#meteor_publish the publish function may not return two cursors for the same collection

Thus this kitchen-cli definition snippet fails:

"collections": [
 {name: "referenced"},
 {name: "referencing", fields:[
    {name: "ref1", join_collection: "referenced"},
    {name: "ref2", join_collection: "referenced"},
    ]
 } 
]

Now trying Meteor.subscribe(referencing_query) fails server side with

Error: Publish function returned multiple cursors for collection referencing

In any case I attached my app as i don't have time right now to expand the example above into an app.
The contacts_all query/publish function triggers the error.

Anyways, i figure it's not trivial to fix, so i'm just leaving this here as a feature request ...
adsubmission.json.zip

@perak
Copy link
Owner

perak commented Mar 10, 2016

@paulbalomiri you are talent to find bugs! 😄

Thanks!

@paulbalomiri
Copy link
Author

The contacts_all query/publish function triggers the error.

What I meant to say is that I attached an app for reproducing, and that the "contacts_all" publish showcases the behaviour. But part of the sentence never made it from brain to keyboard. 😄

adsubmission.json.zip

EDIT: I really need to go to bed, as it was there all along So ignore this comment 😞

@paulbalomiri
Copy link
Author

I just realised that there is PR #6 . Have you looked into it?

@perak
Copy link
Owner

perak commented Jul 24, 2016

@paulbalomiri Fixed.

Better late than never! :D

@paulbalomiri
Copy link
Author

paulbalomiri commented Oct 18, 2016

👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants