264a265,266
>   global $user;
> 
279a282
>   ksort($roles_array);
280a284,297
> 
>   if ($user->uid != 1) {
>     $show = false;
>     foreach ($user->roles as $roleid => $role) {
>       if(variable_get('node_privacy_byrole_show_panel_'.$roleid, FALSE) == true) {
>         $show = true;
>       }
>     }
> 
>     if (!$show) {
>       return '';
>     }
>   }
> 
313,314c330,331
<   $formgroup_view = form_group('View Permissions', $formitems_view, 'Select which users can view your post based on their role.');
<   $formgroup_edit = form_group('Edit Permissions', $formitems_edit, 'Select which users can edit your post based on their role.');
---
>   $formgroup_view = form_group(t('View Permissions'), $formitems_view, t('Select which users can view your post based on their role.'));
>   $formgroup_edit = form_group(t('Edit Permissions'), $formitems_edit, t('Select which users can edit your post based on their role.'));
323c340,343
<   $formitems = '';
---
> 
> 
>   $status_items = $output = '';
> 
325,327c345,370
<   $formitems .= form_hidden('node_privacy_byrole_settings_changed', 1);
<   $formitems .= form_radio(t('Enable the node_privacy_byrole module.'), 'node_privacy_byrole_enabled', 1,variable_get('node_privacy_byrole_enabled', FALSE) );
<   $formitems .= form_radio(t('Disable the node_privacy_byrole module.'), 'node_privacy_byrole_enabled', 0, !variable_get('node_privacy_byrole_enabled', FALSE) );
---
>   $status_items .= form_hidden('node_privacy_byrole_settings_changed', 1);
>   $status_items .= form_radio(t('Enable the node_privacy_byrole module.'), 'node_privacy_byrole_enabled', 1, $node_privacy_byrole_enabled);
>   $status_items .= form_radio(t('Disable the node_privacy_byrole module.'), 'node_privacy_byrole_enabled', 0, !$node_privacy_byrole_enabled);
> 
>   $output .= form_group(t('Module status'), $status_items);
> 
> 
>   if ($node_privacy_byrole_enabled) {
>     $display_form_items = form_hidden('node_privacy_byrole_show_panel_changed', 1);
> 
>     $formitems_roles = '';
>     $roles_array = user_roles();
>     ksort($roles_array);
>     $roleids_array = array_keys($roles_array);
> 
>     foreach ($roleids_array as $roleid) {
>       if (!($node_privacy_byrole_show_panel = variable_get('node_privacy_byrole_show_panel_'. $roleid, FALSE))) {
>         variable_set('node_privacy_byrole_show_panel_'.$roleid, false);
>         $node_privacy_byrole_show_panel = false;
>       }
>       $formitems_roles .= form_checkbox(t($roles_array[$roleid]), 'node_privacy_byrole_show_panel_'. $roleid, 1, $node_privacy_byrole_show_panel);
>     }
> 
>     $output .= form_group(t('Show node permissions panel'), $formitems_roles, t('Select which roles can see the node permissions panel.'));
> 
>   }
329c372,374
<   return $formitems; 
---
> 
> 
>   return $output;
384a430,503
> 
> 
> 
> /**
>  * Implementation of a Drupal action.
>  * Alters node level role permissions for a node.
>  *
>  */
> function action_node_assign_role_perms($op, &$edit, &$node) {
>   $roles_array = user_roles();
>   ksort($roles_array);
>   $roleids_array = array_keys($roles_array);
> 
>   switch($op) {
>     case 'metadata':
>       return array(
>         'description' => t('Change node permissions'),
>         'type' => t('Node'),
>         'batchable' => false,
>         'configurable' => true,
>       );
>       break;
> 
>     case 'do':
> 
> 
>       //echo "do-action";
> 
>       //print_r($edit);
>       foreach ($edit['node_privacy_byrole_forview'] as $key => $rid) {
>           $node->{'node_privacy_byrole_forview_'.$key} = $rid;
>       }
> 
>       foreach ($edit['node_privacy_byrole_foredit'] as $key => $rid) {
>           $node->{'node_privacy_byrole_foredit_'.$key} = $rid;
>       }
> 
>       node_privacy_byrole_update_permissions($node);
>       break;
> 
>     // build the form to configure the action
>     case 'form':
>       $formitems_view = '';
>       $formitems_edit = '';
> 
>       foreach ($roleids_array as $roleid) {
>         if (!isset($edit['node_privacy_byrole_forview'][$roleid])) {
>             $edit['node_privacy_byrole_forview'][$roleid] = false;
>         }
>         $formitems_view .= form_checkbox(t($roles_array[$roleid]), 'node_privacy_byrole_forview]['. $roleid, 1, $edit['node_privacy_byrole_forview'][$roleid]);
> 
>         if (!isset($edit['node_privacy_byrole_foredit'][$roleid])) {
>             $edit['node_privacy_byrole_foredit'][$roleid] = false;
>         }
>         $formitems_edit .= form_checkbox(t($roles_array[$roleid]), 'node_privacy_byrole_foredit]['. $roleid, 1, $edit['node_privacy_byrole_foredit'][$roleid]);
>       }
> 
>       $formgroup_view = form_group(t('View Permissions'), $formitems_view, t('Select which users can view your post based on their role.'));
>       $formgroup_edit = form_group(t('Edit Permissions'), $formitems_edit, t('Select which users can edit your post based on their role.'));
> 
>       return $formgroup_view . $formgroup_edit;
>       break;
> 
>     case 'validate':
>       return true;
>       break;
> 
>     case 'submit':
>       return array('node_privacy_byrole_forview' => $edit['node_privacy_byrole_forview'], 'node_privacy_byrole_foredit' => $edit['node_privacy_byrole_foredit']);
>       break;
>     }
> }
> 
>