Skip to content

Conversation

@anvmn
Copy link
Contributor

@anvmn anvmn commented Nov 13, 2016

Derives from #1520

Need to review Dianes comments from #1494

@anvmn anvmn changed the title [CFM-861-tests] Behat tests for permissions. WIP: [CFM-861-tests] Behat tests for permissions. Nov 13, 2016
@ordavidil ordavidil changed the base branch from feature/CFM-861-new-pr to develop November 14, 2016 06:40
@anvmn anvmn changed the title WIP: [CFM-861-tests] Behat tests for permissions. [CFM-861-tests] Behat tests for permissions. Nov 14, 2016
@anvmn
Copy link
Contributor Author

anvmn commented Nov 14, 2016

Diane's comment on previous PR:

selection_137

Content type 'discussion' is not allowed at project.

// We allow access only for draft / published groups.
$group = node_load($context['gid']);
$bundle = explode('/', $item['path'])[2];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Break the steps with explanations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants