From 1b0bf6ba544ef79776135752433af47207cd2cdc Mon Sep 17 00:00:00 2001 From: Tim Steiner <tsteiner2@unl.edu> Date: Thu, 30 Sep 2010 16:38:11 +0000 Subject: [PATCH] More updating to drupal coding standards. git-svn-id: file:///tmp/wdn_thm_drupal/branches/drupal-7.x@212 20a16fea-79d4-4915-8869-1ea9d5ebf173 --- .../taxonomy_access_control/tac.module | 180 +++++++++--------- 1 file changed, 87 insertions(+), 93 deletions(-) diff --git a/sites/all/modules/taxonomy_access_control/tac.module b/sites/all/modules/taxonomy_access_control/tac.module index 3c3707cb..a8f74050 100644 --- a/sites/all/modules/taxonomy_access_control/tac.module +++ b/sites/all/modules/taxonomy_access_control/tac.module @@ -1,111 +1,105 @@ <?php -function tac_menu() -{ - $items = array(); - - $items['admin/people/permissions/tac'] = array( - 'title' => 'Taxonomy Access Controls', - 'page callback' => 'drupal_get_form', - 'page arguments' => array('tac_admin'), - 'access arguments' => array('administer tac'), - 'file' => 'tac.admin.php', - 'type' => MENU_LOCAL_TASK - ); - - return $items; +function tac_menu() { + $items = array(); + + $items['admin/people/permissions/tac'] = array( + 'title' => 'Taxonomy Access Controls', + 'page callback' => 'drupal_get_form', + 'page arguments' => array('tac_admin'), + 'access arguments' => array('administer tac'), + 'file' => 'tac.admin.php', + 'type' => MENU_LOCAL_TASK + ); + + return $items; } -function tac_theme() -{ - return array( - 'tac_term_list' => array( - 'render element' => 'form' - ) - ); +function tac_theme() { + return array( + 'tac_term_list' => array( + 'render element' => 'form' + ) + ); } -function tac_permission() -{ - return array( - 'administer tac' => array( - 'title' => t('Administer'), - 'description' => t('Administer the specific grants given to role/term pairs.') - ), - ); +function tac_permission() { + return array( + 'administer tac' => array( + 'title' => t('Administer'), + 'description' => t('Administer the specific grants given to role/term pairs.') + ) + ); } -function tac_node_access($node, $op, $account) -{ - return NODE_ACCESS_IGNORE; +function tac_node_access($node, $op, $account) { + return NODE_ACCESS_IGNORE; } -function tac_node_access_records($node) -{ - $tac_vid = variable_get('tac_vocabulary', -1); - $fields_to_check = array(); - - $fields = field_info_instances('node', $node->type); - foreach ($fields as $field) { - $fieldInfo = field_info_field($field['field_name']); - if ($fieldInfo['type'] != 'taxonomy_term_reference') { - continue; - } - $fields_to_check[] = $field['field_name']; +function tac_node_access_records($node) { + $tac_vid = variable_get('tac_vocabulary', -1); + $fields_to_check = array(); + + $fields = field_info_instances('node', $node->type); + foreach ($fields as $field) { + $fieldInfo = field_info_field($field['field_name']); + if ($fieldInfo['type'] != 'taxonomy_term_reference') { + continue; } - - $selected_tids = array(); - foreach ($fields_to_check as $field) { - $fieldData = $node->$field; - foreach ($fieldData['und'] as $tid_array) { - $selected_tids[] = $tid_array['tid']; - } + $fields_to_check[] = $field['field_name']; + } + + $selected_tids = array(); + foreach ($fields_to_check as $field) { + $fieldData = $node->$field; + foreach ($fieldData['und'] as $tid_array) { + $selected_tids[] = $tid_array['tid']; } - - $selected_terms = taxonomy_term_load_multiple($selected_tids); - $grant_ids = array(); - foreach ($selected_terms as $selected_term) { - if ($selected_term->vid != $tac_vid) { - continue; - } - $grant_ids[] = $selected_term->tid; + } + + $selected_terms = taxonomy_term_load_multiple($selected_tids); + $grant_ids = array(); + foreach ($selected_terms as $selected_term) { + if ($selected_term->vid != $tac_vid) { + continue; } - - $grants = array(); + $grant_ids[] = $selected_term->tid; + } + + $grants = array(); + $grants[] = array( + 'realm' => 'all', + 'gid' => 0, + 'grant_view' => 1, + 'grant_update' => 0, + 'grant_delete' => 0, + 'priority' => 0 + ); + + foreach ($grant_ids as $grant_id) { + //TODO: lookup which grant_x are supposed to be applied from the database. $grants[] = array( - 'realm' => 'all', - 'gid' => 0, - 'grant_view' => 1, - 'grant_update' => 0, - 'grant_delete' => 0, - 'priority' => 0, + 'realm' => 'tac', + 'gid' => $grant_id, + 'grant_view' => 1, + 'grant_update' => 1, + 'grant_delete' => 1, + 'priority' => 0 ); - - foreach ($grant_ids as $grant_id) { - //TODO: lookup which grant_x are supposed to be applied from the database. - $grants[] = array( - 'realm' => 'tac', - 'gid' => $grant_id, - 'grant_view' => 1, - 'grant_update' => 1, - 'grant_delete' => 1, - 'priority' => 0, - ); - } - - return $grants; + } + + return $grants; } -function tac_node_grants($account, $op) -{ - $query = db_select('tac_map', 'm'); - $query->addField('m', 'tid'); - $query->condition('m.grant_' . $op, 1); - $query->condition('m.rid', array_keys($account->roles)); - $tids = $query->execute()->fetchCol(); - - $grants = array(); - $grants['tac'] = $tids; - - return $grants; +function tac_node_grants($account, $op) { + $query = db_select('tac_map', 'm'); + $query->addField('m', 'tid'); + $query->condition('m.grant_' . $op, 1); + $query->condition('m.rid', array_keys($account->roles)); + $tids = $query->execute()->fetchCol(); + + $grants = array(); + $grants['tac'] = $tids; + + return $grants; } -- GitLab