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.

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

Update civicrm_group_roles.module #518

Open
wants to merge 3 commits into
base: 7.x-master
Choose a base branch
from
Open

Conversation

yurg
Copy link
Contributor

@yurg yurg commented Feb 9, 2018

Preventing Notice: Undefined index: #id in civicrm_group_roles_form_alter() (line 126 of sites\all\modules\civicrm\drupal\modules\civicrm_group_roles\civicrm_group_roles.module).

Preventing Notice: Undefined index: #id in civicrm_group_roles_form_alter() (line 126 of sites\all\modules\civicrm\drupal\modules\civicrm_group_roles\civicrm_group_roles.module).
@civicrm-builder
Copy link

Can one of the admins verify this patch?

@jackrabbithanna
Copy link
Contributor

Jenkins test this please

1 similar comment
@jackrabbithanna
Copy link
Contributor

Jenkins test this please

@seamuslee001
Copy link
Contributor

Jenkins ok to test

@seamuslee001
Copy link
Contributor

Hi @yurg thanks for the PR are you able to fix the style issues found by our test bot https://test.civicrm.org/job/CiviCRM-Drupal-PR/104/checkstyleResult/new/

@aiden-grau
Copy link

+1 on getting this merged...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants