Version 4

[StorytellerCZ]

• BREAKING: Synchronous functions are now available only on client side
• BREAKING: Migration functions for past versions are no longer available
• UI Helpers now use console.debug for debugging messages
• RolesCollection and RoleAssignmentsCollection can now be exported in addition to being accessed via Meteor.roles and Meteor.roleAssignment

[jankapunkt]

@StorytellerCZ is there anything that isn't already implemented by the latest release that contains the async counterparts?

[StorytellerCZ]

Not 100% sure, but I think we have everything. I will remove the sync MongoDB calls in this branch. Or maybe for the client we can leave the sync methods? Thoughts?

[tchax]

@StorytellerCZ I think it would make sense to leave sync methods for the client. This is consistent with other packages like Accounts isn't it ? Thanks

[dallman2]

Synchronous Mongo calls on the client side are alright in Meteor 3.0, IIRC. When is this PR expected to be merged in and published? I would love to test this in my application.

[StorytellerCZ]

Once this is merged: https://github.com/Meteor-Community-Packages/meteor-roles/pull/344

I will release a beta.

[dallman2]

I would suggest updating the userIsInRole function on like 658 of roles_common.js to:

  userIsInRole: async function (user, roles, options) {
    let id
    options = Roles._normalizeOptions(options)

    // ensure array to simplify code
    if (!Array.isArray(roles)) roles = [roles]

    roles = roles.filter(r => r != null)

    if (!roles.length) return false

    Roles._checkScopeName(options.scope)

    options = Object.assign({
      anyScope: false
    }, options)

    if (user && typeof user === 'object') {
      id = user._id
    } else {
      id = user
    }

    if (!id) return false
    if (typeof id !== 'string') return false

    const selector = { 'user._id': id }

    if (!options.anyScope) {
      selector.scope = { $in: [options.scope, null] }
    }

    const promises = roles.map((roleName) => {
      selector['inheritedRoles._id'] = roleName

      return Meteor.roleAssignment.find(selector, { limit: 1 }).countAsync() > 0
    })
    await Promise.all(promises)
    return promises.some(p => p)
  },

For obvious reasons, the method needs to become async to be able to count how many records are in the collection.

[StorytellerCZ]

@dallman2 PRs welcome.

[StorytellerCZ]

Published alanning:[email protected], please take things out for a spin. It is Meteor 3.0-beta.7 compatible!

[StorytellerCZ]

Tests fixed, just trying one little thing for better hopefully better performance.