Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Added option in og_context_plugin_argument_default_group_context.inc to accompany the permission rewrite in og_context(). #131

Merged
merged 4 commits into from Jul 11, 2017
Merged

Added option in og_context_plugin_argument_default_group_context.inc to accompany the permission rewrite in og_context(). #131

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f9e77de
Merge pull request #1 from Gizra/7.x-2.x
alesrebec Jun 23, 2016
4982107
Issue #121: Added option in og_context_plugin_argument_default_group_…
alesrebec Jun 23, 2016
0e33e39
Issue #121: Added option_definition for ['check_access'].
alesrebec Jun 23, 2016
16be973
Issue #121: Changed form element type for check_access from select to…
alesrebec Jun 23, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 9 additions & 1 deletion og_context/includes/views/handlers/og_context_plugin_argument_default_group_context.inc
View file
Open in desktop
Expand Up @@ -32,13 +32,21 @@ class og_context_plugin_argument_default_group_context extends views_plugin_argu
'#default_value' => $this->options['group_type'],
'#description' => t('Determine what entity type that group should be of.')
);
$form['check_access'] = array(
Copy link
Member

Choose a reason for hiding this comment

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

you need to define it also in option_definition

'#type' => 'select',
Copy link
Member

Choose a reason for hiding this comment

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

this should be a checkbox

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Solved.

'#options' => array(t('No'), t('Yes')),
'#title' => t('Check access'),
'#default_value' => $this->options['check_access'],
'#description' => t('Determines if access to the group should be done. Defaults to "Yes".')
);
}

/**
* Return the group context argument.
*/
function get_argument() {
if ($group = og_context($this->options['group_type'])) {
$check_access = (isset($this->options['check_access']) && $this->options['check_access'] == 0) ? FALSE : TRUE;
Copy link
Member

Choose a reason for hiding this comment

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

and then you can do group = og_context($this->options['group_type'], NULL, NULL, $this->options['check_access'])

if ($group = og_context($this->options['group_type'], NULL, NULL, $check_access)) {
return $group['gid'];
}
return FALSE;
Expand Down