peerlibrary / meteor-publish-context Goto Github PK

View Code? Open in Web Editor NEW

8.0 6.0 2.0 7 KB

Provide DDP._CurrentPublish with access to current publish context

Home Page: https://atmospherejs.com/peerlibrary/publish-context

License: BSD 3-Clause "New" or "Revised" License

JavaScript 53.74% CoffeeScript 46.26%

meteor meteor-package ddp ddp-publish

meteor-publish-context's Issues

Publish override might be incomplete

I just had a look at the Meteor.publish override code and it looks like it doesn't take the following format of Meteor.publish into account.

Meteor.publish({
  allPosts() {
    const userId = this.userId;
    return userId ? Posts.find({ userId }) : this.ready();
});

Just as with Meteor.methods, Meteor.publish allows the developer to give it an object.

It does not work for the automated publications for users

I20180109-18:04:02.592(2)? no invoke, CursorDescription {
I20180109-18:04:02.592(2)?   collectionName: 'users',
I20180109-18:04:02.592(2)?   selector: { _id: 'jzsdfPdbjBMdAedPg' },
I20180109-18:04:02.592(2)?   options:
I20180109-18:04:02.592(2)?    { transform: null,
I20180109-18:04:02.592(2)?      fields: { profile: 1, username: 1, emails: 1 } } }```

When that publication entry is created it does not put it inside DDP._CurrentPublish.get()

Any clues why ?

Recommend Projects