Browse Source

Tripal Bulk Loader adheres to Drupal coding standards

Pubudu Basnayaka 12 years ago
parent
commit
48b2f067d1

File diff suppressed because it is too large
+ 249 - 224
tripal_bulk_loader/tripal_bulk_loader.admin.inc


+ 135 - 127
tripal_bulk_loader/tripal_bulk_loader.constants.inc

@@ -22,18 +22,18 @@
  *   On success it returns the object (with primary key if inserted);
  *   on failure it returns FALSE
  */
-function tripal_bulk_loader_update_constant ($nid, $group_id, $table, $field, $record_id, $field_id, $value) {
-  
+function tripal_bulk_loader_update_constant($nid, $group_id, $table, $field, $record_id, $field_id, $value) {
+
   $record = array(
-    'nid'=>$nid, 
+    'nid' => $nid,
     'group_id' => $group_id,
-    'chado_table'=>$table, 
-    'chado_field'=>$field, 
-    'record_id'=>$record_id, 
-    'field_id'=>$field_id, 
-    'value'=>$value
+    'chado_table' => $table,
+    'chado_field' => $field,
+    'record_id' => $record_id,
+    'field_id' => $field_id,
+    'value' => $value
   );
-  
+
   // Check to see if already exists
   $exists = db_fetch_object(db_query(
     "SELECT constant_id FROM {tripal_bulk_loader_constants} WHERE nid=%d AND record_id=%d AND field_id=%d AND group_id=%d",
@@ -44,45 +44,51 @@ function tripal_bulk_loader_update_constant ($nid, $group_id, $table, $field, $r
   ));
   if ($exists->constant_id) {
     $record['constant_id'] = $exists->constant_id;
-    $status = drupal_write_record('tripal_bulk_loader_constants',$record,'constant_id');
+    $status = drupal_write_record('tripal_bulk_loader_constants', $record, 'constant_id');
     if ($status) {
       return $record;
-    } else {
+    }
+    else {
       return FALSE;
     }
-  } else {
+  }
+  else {
 
-    $status = drupal_write_record('tripal_bulk_loader_constants',$record);
+    $status = drupal_write_record('tripal_bulk_loader_constants', $record);
     if ($status) {
       return $record;
-    } else {
+    }
+    else {
       return FALSE;
     }
   }
 }
 
-function tripal_bulk_loader_has_exposed_fields ($node) {
+function tripal_bulk_loader_has_exposed_fields($node) {
 
   // exposed fields isn't set
   if (!isset($node->exposed_fields)) {
     return FALSE;
   }
-  
+
   // exposed fields has at least one element
   if (sizeof($node->exposed_fields) == 1) {
     // need to check if single element is an empty array
     $element = reset($node->exposed_fields);
     if ($element) {
       return TRUE;
-    } else {
+    }
+    else {
       return FALSE;
     }
-  } elseif (sizeof($node->exposed_fields) > 1) {
+  }
+  elseif (sizeof($node->exposed_fields) > 1) {
     return TRUE;
-  } else {
+  }
+  else {
     return FALSE;
   }
-  
+
   return FALSE;
 }
 
@@ -93,7 +99,7 @@ function tripal_bulk_loader_has_exposed_fields ($node) {
 /**
  * Set constants (exposed fields in template)
  *
- * @param $form_state 
+ * @param $form_state
  *   The current state of the form
  * @param $node
  *   The node to set constants for
@@ -101,23 +107,23 @@ function tripal_bulk_loader_has_exposed_fields ($node) {
  * @return
  *   A form array to be rendered by drupal_get_form()
  */
-function tripal_bulk_loader_set_constants_form ($form_state, $node) {
+function tripal_bulk_loader_set_constants_form($form_state, $node) {
   $form = array();
-  
+
   $form['nid'] = array(
     '#type' => 'hidden',
     '#value' => $node->nid
   );
-  
+
   if (!tripal_bulk_loader_has_exposed_fields($node)) {
     return $form;
   }
-  
+
   $form['exposed_array'] = array(
     '#type' => 'hidden',
     '#value' => serialize($node->exposed_fields),
   );
-  
+
   $form['exposed_fields'] = array(
     '#type' => 'fieldset',
     '#title' => t('Constant Values'),
@@ -137,16 +143,16 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
         .'in the template to be loaded x number of times where there are x sets of '
         .'constants (rows in the following table).')
     );
-    
+
     $form['exposed_fields']['existing'] = array(
       '#tree' => TRUE,
     );
-    
+
     foreach ($node->constants as $set) {
 
       foreach ($set as $record) {
         foreach ($record as $field) {
-          $index = $field['record_id'].'-'.$field['field_id'];
+          $index = $field['record_id'] . '-' . $field['field_id'];
           $group = $field['group_id'];
           $form['exposed_fields']['existing'][$group][$index] = array(
             '#type' => 'markup',
@@ -157,40 +163,40 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
 
       $form['exposed_fields']['existing'][$group]['delete'] = array(
         '#type' => 'markup',
-        '#value' => l('Edit', 'node/'. $node->nid . '/constants/'.$group.'/edit') .'<br />'.
-          l('Delete', 'node/'. $node->nid . '/constants/'.$group.'/delete'),
+        '#value' => l('Edit', 'node/' . $node->nid . '/constants/' . $group . '/edit') . '<br />'  .
+          l('Delete', 'node/' . $node->nid . '/constants/' . $group . '/delete'),
       );
-                  
+
     }
   }
-  
+
   $form['exposed_fields']['new'] = array(
     '#type' => 'fieldset',
     '#title' => t('New set of Constants'),
   );
-    
+
   $form['exposed_fields']['new']['explanation-2'] = array(
     '#type' => 'item',
     '#value' => t('The following fields are constants in the selected template that you need to set values for.')
   );
-  
+
   // Add textifelds for exposed fields of the current template
   $exposed_fields = FALSE;
   $indexes = array();
   if (tripal_bulk_loader_has_exposed_fields($node)) {
     foreach ($node->exposed_fields as $exposed_index) {
-      
+
       $record_id = $exposed_index['record_id'];
       $field_id = $exposed_index['field_id'];
       $field = $node->template->template_array[$record_id]['fields'][$field_id];
-      
+
       if ($field['exposed']) {
         $exposed_fields = TRUE;
         $indexes[$record_id][] = $field_id;
-        
-        switch($field['type']) {
+
+        switch ($field['type']) {
           case 'table field':
-            $form['exposed_fields']['new'][$record_id.'-'.$field_id] = array(
+            $form['exposed_fields']['new'][$record_id . '-' . $field_id] = array(
               '#type' => 'textfield',
               '#title' => t($field['title']),
               '#description' => t($field['exposed_description']),
@@ -198,7 +204,7 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
             );
           break;
           case 'constant':
-            $form['exposed_fields']['new'][$record_id.'-'.$field_id] = array(
+            $form['exposed_fields']['new'][$record_id . '-' . $field_id] = array(
               '#type' => 'textfield',
               '#title' => t($field['title']),
               '#description' => t('Enter the case-sensitive value of this constant for your spreadsheet'),
@@ -206,16 +212,16 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
             );
           break;
         }
-        
-        $form['exposed_fields']['new'][$record_id.'-'.$field_id.'-table'] = array(
+
+        $form['exposed_fields']['new'][$record_id . '-' . $field_id . '-table'] = array(
           '#type' => 'hidden',
           '#value' => $node->template->template_array[$record_id]['table'],
         );
-        $form['exposed_fields']['new'][$record_id.'-'.$field_id.'-field'] = array(
+        $form['exposed_fields']['new'][$record_id . '-' . $field_id . '-field'] = array(
           '#type' => 'hidden',
           '#value' => $field['field'],
         );
-        $form['exposed_fields']['new'][$record_id.'-'.$field_id.'-type'] = array(
+        $form['exposed_fields']['new'][$record_id . '-' . $field_id . '-type'] = array(
           '#type' => 'hidden',
           '#value' => $field['type'],
         );
@@ -227,25 +233,25 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
     '#type' => 'hidden',
     '#value' => serialize($node->template->template_array)
   );
-  
+
   $form['exposed_fields']['new']['indexes'] = array(
     '#type' => 'hidden',
     '#value' => serialize($indexes),
   );
-  
+
   if (!$exposed_fields) {
     $form['exposed_fields']['new']['explanation'] = array(
       '#type' => 'item',
       '#value' => t('There are no exposed fields for this template.')
-    );    
+    );
   }
-  
+
   $form['exposed_fields']['new']['submit-2'] = array(
     '#type' => 'submit',
     '#name' => 'add_constant',
     '#value' => t('Add Constant Set')
   );
-  
+
   return $form;
 }
 
@@ -253,89 +259,91 @@ function tripal_bulk_loader_set_constants_form ($form_state, $node) {
  * Validate that the values entered exist in the database
  * if indicated in hte template array
  */
-function tripal_bulk_loader_set_constants_form_validate ($form, $form_state) {
+function tripal_bulk_loader_set_constants_form_validate($form, $form_state) {
 
   $template = unserialize($form_state['values']['template']);
   $indexes = unserialize($form_state['values']['indexes']);
 
   $op = $form_state['values'][ $form_state['clicked_button']['#name'] ];
-  if (strcmp('Add Constant Set',$op) == 0) {
+  if (strcmp('Add Constant Set', $op) == 0) {
       foreach ($indexes as $record_id => $array) {
-        foreach($array as $field_id) {
+        foreach ($array as $field_id) {
           if ($template[$record_id]['fields'][$field_id]['exposed_validate']) {
             $result = db_fetch_object(db_query(
               "SELECT 1 as valid FROM %s WHERE %s='%s'",
               $template[$record_id]['table'],
               $template[$record_id]['fields'][$field_id]['field'],
-              $form_state['values'][$record_id.'-'.$field_id]
+              $form_state['values'][$record_id . '-' . $field_id]
             ));
-    
+
             if (!$result->valid) {
-              $msg = 'A '.$form['exposed_fields']['new'][$record_id.'-'.$field_id]['#title'].' of "'.$form['exposed_fields']['new'][$record_id.'-'.$field_id]['#value'].'" must already exist!';
-              form_set_error($record_id.'-'.$field_id, $msg);
-            } else {
-              drupal_set_message('Confirmed a '.$form['exposed_fields']['new'][$record_id.'-'.$field_id]['#title'].' of "'.$form['exposed_fields']['new'][$record_id.'-'.$field_id]['#value'].'" already exists.');
+              $msg = 'A ' . $form['exposed_fields']['new'][$record_id . '-' . $field_id]['#title'] . ' of "' . $form['exposed_fields']['new'][$record_id . '-' . $field_id]['#value'] . '" must already exist!';
+              form_set_error($record_id . '-' . $field_id, $msg);
+            }
+            else {
+              drupal_set_message('Confirmed a '. $form['exposed_fields']['new'][$record_id . '-' . $field_id]['#title'] . ' of "'. $form['exposed_fields']['new'][$record_id . '-' . $field_id]['#value'] . '" already exists.');
             }
           }
         }
       }
   }
-  
+
 }
 
 /**
  * Insert/update the constants associated with this node
  */
-function tripal_bulk_loader_set_constants_form_submit ($form, $form_state) {
+function tripal_bulk_loader_set_constants_form_submit($form, $form_state) {
 
-	// Insert/Update constants
+  // Insert/Update constants
   $template = unserialize($form_state['values']['template']);
   $indexes = unserialize($form_state['values']['indexes']);
 
   $op = $form_state['values'][ $form_state['clicked_button']['#name'] ];
-  if (strcmp('Add Constant Set',$op) == 0) {
-      $max_group = db_fetch_object(db_query("SELECT max(group_id) as value FROM tripal_bulk_loader_constants WHERE nid=%d",$form_state['values']['nid']));
+  if (strcmp('Add Constant Set', $op) == 0) {
+      $max_group = db_fetch_object(db_query("SELECT max(group_id) as value FROM {tripal_bulk_loader_constants} WHERE nid=%d", $form_state['values']['nid']));
       foreach ($indexes as $record_id => $array) {
-        foreach($array as $field_id) {	
+        foreach ($array as $field_id) {
           tripal_bulk_loader_update_constant(
-            $form_state['values']['nid'], 
+            $form_state['values']['nid'],
             $max_group->value+1,
-            $form_state['values'][$record_id.'-'.$field_id.'-table'], 
-            $form_state['values'][$record_id.'-'.$field_id.'-field'],
+            $form_state['values'][$record_id . '-' . $field_id . '-table'],
+            $form_state['values'][$record_id . '-' . $field_id . '-field'],
             $record_id,
             $field_id,
-            $form_state['values'][$record_id.'-'.$field_id]
+            $form_state['values'][$record_id . '-' . $field_id]
           );
         }
       }
   }
-  
+
 }
 
-function theme_tripal_bulk_loader_set_constants_form ($form) {
+function theme_tripal_bulk_loader_set_constants_form($form) {
   $output = '';
-  
+
   $exposed_fields = unserialize($form['exposed_array']['#value']);
   // need to put in the context of a node so we can use the has_exposed_fields function
   if ($exposed_fields) {
     $node->exposed_fields = $exposed_fields;
-  } else {
+  }
+  else {
     $node->exposed_fields = array();
   }
-  
+
   // Add draggable table for constant sets
   if (tripal_bulk_loader_has_exposed_fields($node)) {
     $i=1;
     foreach (element_children($form['exposed_fields']['existing']) as $key) {
       $element = &$form['exposed_fields']['existing'][$key];
       $element['group']['#attributes']['class'] = 'weight-group';
-  
+
       $row = array();
       foreach ($exposed_fields as $exposed) {
         if ($i==1) {
           $header[] = $exposed['title'];
         }
-        $k = $exposed['record_id'].'-'.$exposed['field_id'];
+        $k = $exposed['record_id'] . '-' . $exposed['field_id'];
         $row[] = drupal_render($element[$k]);
       }
       $row[] = drupal_render($element['delete']);
@@ -349,7 +357,7 @@ function theme_tripal_bulk_loader_set_constants_form ($form) {
       '#value' => theme('table', $header, $rows, array('id' => 'mytable')) . '<br />'
     );
   }
-    
+
   $output .= drupal_render($form);
   return $output;
 }
@@ -361,7 +369,7 @@ function theme_tripal_bulk_loader_set_constants_form ($form) {
 /**
  * Edit a constant set (exposed fields in template)
  *
- * @param $form_state 
+ * @param $form_state
  *   The current state of the form
  * @param $node
  *   The node to set constants for
@@ -371,11 +379,11 @@ function theme_tripal_bulk_loader_set_constants_form ($form) {
  * @return
  *   A form array to be rendered by drupal_get_form()
  */
-function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_id) {
+function tripal_bulk_loader_edit_constant_set_form($form_state, $node, $group_id) {
   $form = array();
 
-  $form['#redirect'] = 'node/'.$node->nid;
-  
+  $form['#redirect'] = 'node/' . $node->nid;
+
   $form['nid'] = array(
     '#type' => 'hidden',
     '#value' => $node->nid,
@@ -386,18 +394,18 @@ function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_i
     '#value' => $group_id,
   );
 
-    
+
   $form['explanation'] = array(
     '#type' => 'item',
     '#value' => t('The following fields are constants in the selected template that you need to set values for.')
   );
-  
+
   // Add textifelds for exposed fields of the current template
   $exposed_fields = FALSE;
   $indexes = array();
   if (tripal_bulk_loader_has_exposed_fields($node)) {
     foreach ($node->exposed_fields as $exposed_index) {
-      
+
       $record_id = $exposed_index['record_id'];
       $field_id = $exposed_index['field_id'];
       $field = $node->template->template_array[$record_id]['fields'][$field_id];
@@ -405,10 +413,10 @@ function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_i
       if ($field['exposed']) {
         $exposed_fields = TRUE;
         $indexes[$record_id][] = $field_id;
-        
-        switch($field['type']) {
+
+        switch ($field['type']) {
           case 'table field':
-            $form[$record_id.'-'.$field_id] = array(
+            $form[$record_id . '-' . $field_id] = array(
               '#type' => 'textfield',
               '#title' => t($field['title']),
               '#description' => t($field['exposed_description']),
@@ -416,7 +424,7 @@ function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_i
             );
           break;
           case 'constant':
-            $form[$record_id.'-'.$field_id] = array(
+            $form[$record_id . '-' . $field_id] = array(
               '#type' => 'textfield',
               '#title' => t($field['title']),
               '#description' => t('Enter the case-sensitive value of this constant for your spreadsheet'),
@@ -424,16 +432,16 @@ function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_i
             );
           break;
         }
-        
-        $form[$record_id.'-'.$field_id.'-table'] = array(
+
+        $form[$record_id . '-' . $field_id . '-table'] = array(
           '#type' => 'hidden',
           '#value' => $record['table'],
         );
-        $form[$record_id.'-'.$field_id.'-field'] = array(
+        $form[$record_id . '-' . $field_id . '-field'] = array(
           '#type' => 'hidden',
           '#value' => $field['field'],
         );
-        $form[$record_id.'-'.$field_id.'-type'] = array(
+        $form[$record_id . '-' . $field_id . '-type'] = array(
           '#type' => 'hidden',
           '#value' => $field['type'],
         );
@@ -445,60 +453,60 @@ function tripal_bulk_loader_edit_constant_set_form ($form_state, $node, $group_i
     '#type' => 'hidden',
     '#value' => serialize($node->template->template_array)
   );
-  
+
   $form['indexes'] = array(
     '#type' => 'hidden',
     '#value' => serialize($indexes),
   );
-  
+
   $form['save'] = array(
     '#type' => 'submit',
     '#value' => 'Save',
   );
-  
+
   $form['cancel'] = array(
     '#type' => 'submit',
     '#value' => 'Cancel',
   );
-    
+
   return $form;
 }
 
 /**
  * Edit constants in the current constant set
  */
-function tripal_bulk_loader_edit_constant_set_form_submit ($form, $form_state) {
-  
+function tripal_bulk_loader_edit_constant_set_form_submit($form, $form_state) {
+
   // Update constants
   $template = unserialize($form_state['values']['template']);
   $indexes = unserialize($form_state['values']['indexes']);
-  
+
   $op = $form_state['values'][ $form_state['clicked_button']['#name'] ];
-  if (strcmp('Save',$op) == 0) {
+  if (strcmp('Save', $op) == 0) {
 
     foreach ($indexes as $record_id => $array) {
-      foreach($array as $field_id) {	
+      foreach ($array as $field_id) {
         tripal_bulk_loader_update_constant(
-          $form_state['values']['nid'], 
+          $form_state['values']['nid'],
           $form_state['values']['group_id'],
-          $form_state['values'][$record_id.'-'.$field_id.'-table'], 
-          $form_state['values'][$record_id.'-'.$field_id.'-field'],
+          $form_state['values'][$record_id . '-' . $field_id . '-table'],
+          $form_state['values'][$record_id . '-' . $field_id . '-field'],
           $record_id,
           $field_id,
-          $form_state['values'][$record_id.'-'.$field_id]
+          $form_state['values'][$record_id . '-' . $field_id]
         );
       }
     }
     drupal_set_message('The constant set was successfully updated.');
-      
+
   }
-  
+
 }
 
 /**
  * Delete a constant set (exposed fields in template)
  *
- * @param $form_state 
+ * @param $form_state
  *   The current state of the form
  * @param $node
  *   The node to set constants for
@@ -508,41 +516,41 @@ function tripal_bulk_loader_edit_constant_set_form_submit ($form, $form_state) {
  * @return
  *   A form array to be rendered by drupal_get_form()
  */
-function tripal_bulk_loader_delete_constant_set_form ($form_state, $node, $group_id) {
+function tripal_bulk_loader_delete_constant_set_form($form_state, $node, $group_id) {
   $form = array();
 
-  $form['#redirect'] = 'node/'.$node->nid;
-	
-	$form['nid'] = array(
-		'#type' => 'value',
-		'#value' => $node->nid,
-	);
+  $form['#redirect'] = 'node/' . $node->nid;
+
+  $form['nid'] = array(
+    '#type' => 'value',
+    '#value' => $node->nid,
+  );
 
   $form['group_id'] = array(
     '#type' => 'hidden',
     '#value' => $group_id,
   );
-   
-	return confirm_form($form,
-    	t('Are you sure you want to delete this constant set?'),
-    	'node/'.$node->nid,
-    	t('This action cannot be undone.'),
-    	t('Delete'),
-    	t('Cancel')
+
+  return confirm_form($form,
+      t('Are you sure you want to delete this constant set?'),
+      'node/' . $node->nid,
+      t('This action cannot be undone.'),
+      t('Delete'),
+      t('Cancel')
   );
-  
+
 }
 
 /**
  * Delete the current constant set
  */
-function tripal_bulk_loader_delete_constant_set_form_submit ($form, $form_state) {
+function tripal_bulk_loader_delete_constant_set_form_submit($form, $form_state) {
 
   $group_id = $form_state['values']['group_id'];
   $nid = $form_state['values']['nid'];
   if ($nid && $form_state['values']['confirm']) {
-    db_query("DELETE FROM {tripal_bulk_loader_constants} WHERE nid=%d AND group_id=%d",$nid, $group_id);
+    db_query("DELETE FROM {tripal_bulk_loader_constants} WHERE nid=%d AND group_id=%d", $nid, $group_id);
     drupal_set_message('Constant set successfully deleted.');
   }
-        
-}
+
+}

+ 81 - 81
tripal_bulk_loader/tripal_bulk_loader.install

@@ -3,14 +3,14 @@
 /**
  * Implements hook_install
  */
-function tripal_bulk_loader_install(){
+function tripal_bulk_loader_install() {
    drupal_install_schema('tripal_bulk_loader');
 }
 
 /**
  * Implements hook_uninstall
  */
-function tripal_bulk_loader_uninstall(){
+function tripal_bulk_loader_uninstall() {
    drupal_uninstall_schema('tripal_bulk_loader');
 }
 
@@ -23,93 +23,93 @@ function tripal_bulk_loader_uninstall(){
  *  - tripal_bulk_loader_inserted: Keeps track of all records inserted for a given bulk loading job
  */
 function tripal_bulk_loader_schema() {
-	$schema = array();
-	$schema['tripal_bulk_loader'] = array(
+  $schema = array();
+  $schema['tripal_bulk_loader'] = array(
       'fields' => array(
          'nid' => array(
             'type' => 'int',
             'unsigned' => TRUE,
             'not null' => TRUE,
-			),
+      ),
          'loader_name' => array(
             'type' => 'varchar',
-			),
+      ),
          'template_id' => array(
             'type' => 'varchar',
-			),
-			'file' => array(
+      ),
+      'file' => array(
         'type' => 'varchar',
         'not null' => TRUE
-			),
-			'job_id' => array(
-			  'type' => 'int',
-			),
-			'job_status' => array(
-			  'type' => 'varchar',
-			),
-			'file_has_header' => array(
-				'type' => 'int',
-				'size' => 'tiny',
-				'not null' => TRUE,
+      ),
+      'job_id' => array(
+        'type' => 'int',
+      ),
+      'job_status' => array(
+        'type' => 'varchar',
+      ),
+      'file_has_header' => array(
+        'type' => 'int',
+        'size' => 'tiny',
+        'not null' => TRUE,
         'default' => 0,
-			),
-		),
-  		'primary key' => array('nid'),
-  		'unique keys' => array(
-    		'name' => array('loader_name')
-		),
-	);
-	$schema['tripal_bulk_loader_template'] = array(
+      ),
+    ),
+      'primary key' => array('nid'),
+      'unique keys' => array(
+        'name' => array('loader_name')
+    ),
+  );
+  $schema['tripal_bulk_loader_template'] = array(
       'fields' => array(
          'template_id' => array(
             'type' => 'serial',
             'unsigned' => TRUE,
             'not null' => TRUE,
-			),
-			'name' => array(
+      ),
+      'name' => array(
             'type' => 'varchar',
-			),
+      ),
          'template_array' => array(
             'type' => 'varchar',
-			)
-		),
-  		'primary key' => array('template_id'),
-		'unique keys' => array(
-    		'name' => array('name')
-		),
-	);
-	$schema['tripal_bulk_loader_inserted'] = array(
-	  'fields' => array(
-	    'tripal_bulk_loader_inserted_id' => array(
-	      'type' => 'serial',
-	      'not null' => TRUE
-	    ),
+      )
+    ),
+      'primary key' => array('template_id'),
+    'unique keys' => array(
+        'name' => array('name')
+    ),
+  );
+  $schema['tripal_bulk_loader_inserted'] = array(
+    'fields' => array(
+      'tripal_bulk_loader_inserted_id' => array(
+        'type' => 'serial',
+        'not null' => TRUE
+      ),
       'nid' => array(
         'type' => 'int',
         'unsigned' => TRUE,
         'not null' => TRUE,
-			),
-			'table_inserted_into' => array(
-			  'type' => 'varchar',
-			  'not null' => TRUE,
-			),
-			'table_primary_key' => array(
-			  'type' => 'varchar',
-			  'not null' => TRUE,
-			),
-			'ids_inserted' => array(
-			  'type' => 'text',
-			  'not null' => TRUE
-			),
-	  ),
-	  'primary key' => array('tripal_bulk_loader_inserted_id'),
-	);
-	$schema['tripal_bulk_loader_constants'] = array(
-	  'fields' => array(
-	    'constant_id' => array(
-	      'type' => 'serial',
-	      'not null' => TRUE
-	    ),
+      ),
+      'table_inserted_into' => array(
+        'type' => 'varchar',
+        'not null' => TRUE,
+      ),
+      'table_primary_key' => array(
+        'type' => 'varchar',
+        'not null' => TRUE,
+      ),
+      'ids_inserted' => array(
+        'type' => 'text',
+        'not null' => TRUE
+      ),
+    ),
+    'primary key' => array('tripal_bulk_loader_inserted_id'),
+  );
+  $schema['tripal_bulk_loader_constants'] = array(
+    'fields' => array(
+      'constant_id' => array(
+        'type' => 'serial',
+        'not null' => TRUE
+      ),
       'nid' => array(
         'type' => 'int',
         'unsigned' => TRUE,
@@ -140,41 +140,41 @@ function tripal_bulk_loader_schema() {
       'value' => array(
         'type' => 'text',
       ),
-	  ),
-	  'primary key' => array('constant_id'),
-	);
-	
-	return $schema;
+    ),
+    'primary key' => array('constant_id'),
+  );
+
+  return $schema;
 }
 
 /**
  * Update schema for version 6.x-0.3.1b-1.5
  * - Add the tripal_bulk_loader_constants table
  */
-function tripal_bulk_loader_update_6150 () {
+function tripal_bulk_loader_update_6150() {
 
   // Create tripal_bulk_loader_constants table
   $schema = tripal_bulk_loader_schema();
   $ret = array();
   db_create_table($ret, 'tripal_bulk_loader_constants', $schema['tripal_bulk_loader_constants']);
-  
+
   return $ret;
-  
+
 }
 
 /**
  * Update schema for version 6.x-0.3.1b-1.5
- * - Add the tripal_bulk_loader_constants.group_id column 
+ * - Add the tripal_bulk_loader_constants.group_id column
  *   to allow multiple sets of constants per job
  */
-function tripal_bulk_loader_update_6151 () {
+function tripal_bulk_loader_update_6151() {
   $ret = array();
-  
+
   $schema = tripal_bulk_loader_schema();
   db_add_field(
-    $ret, 
-    'tripal_bulk_loader_constants', 
-    'group_id', 
+    $ret,
+    'tripal_bulk_loader_constants',
+    'group_id',
     array(
         'type' => 'int',
         'unsigned' => TRUE,
@@ -183,5 +183,5 @@ function tripal_bulk_loader_update_6151 () {
       )
     );
 
-  return $ret;  
-}
+  return $ret;
+}

+ 224 - 183
tripal_bulk_loader/tripal_bulk_loader.loader.inc

@@ -3,27 +3,27 @@
 /**
  * Add Loader Job Form
  *
- * This form is meant to be included on the node page to allow users to submit/re-submit 
+ * This form is meant to be included on the node page to allow users to submit/re-submit
  * loading jobs
  */
-function tripal_bulk_loader_add_loader_job_form ($form_state, $node) {
+function tripal_bulk_loader_add_loader_job_form($form_state, $node) {
   $form = array();
-  
+
   $form['nid'] = array(
     '#type' => 'hidden',
     '#value' => $node->nid,
-  ); 
-  
+  );
+
   $form['file'] = array(
-  	'#type' => 'hidden',
-  	'#value' => $node->file
+    '#type' => 'hidden',
+    '#value' => $node->file
   );
-  
+
   $form['job_id'] = array(
     '#type' => 'hidden',
     '#value' => $node->job_id,
   );
-  
+
   $form['submit'] = array(
     '#type' => 'submit',
     '#value' => ($node->job_id) ? 'Re-Submit Job' : 'Submit Job',
@@ -33,60 +33,65 @@ function tripal_bulk_loader_add_loader_job_form ($form_state, $node) {
     '#type' => ($node->job_id)? 'submit' : 'hidden',
     '#value' => 'Cancel Job',
   );
-  
+
   $form['submit-revert'] = array(
     '#type' => ($node->job_id) ? 'submit' : 'hidden',
     '#value' => 'Revert',
-  );  
-  
+  );
+
   return $form;
 }
 
 /**
  * Add Loader Job Form (Submit)
  */
-function tripal_bulk_loader_add_loader_job_form_submit ($form, $form_state) {
+function tripal_bulk_loader_add_loader_job_form_submit($form, $form_state) {
   global $user;
-  
+
   if (preg_match('/Submit Job/', $form_state['values']['op'])) {
     //Submit Tripal Job
-		$job_args[1] = $form_state['values']['nid'];
-		if (is_readable($form_state['values']['file'])) {
-			$fname = basename($form_state['values']['file']);
-			$job_id = tripal_add_job("Bulk Loading Job: $fname",'tripal_bulk_loader', 'tripal_bulk_loader_load_data', $job_args, $user->uid);
-			
-			// add job_id to bulk_loader node
+    $job_args[1] = $form_state['values']['nid'];
+    if (is_readable($form_state['values']['file'])) {
+      $fname = basename($form_state['values']['file']);
+      $job_id = tripal_add_job("Bulk Loading Job: $fname", 'tripal_bulk_loader', 'tripal_bulk_loader_load_data', $job_args, $user->uid);
+
+      // add job_id to bulk_loader node
       $success = db_query("UPDATE {tripal_bulk_loader} SET job_id=%d WHERE nid=%d", $job_id, $form_state['values']['nid']);
-      
+
       // change status
-      db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",'Submitted to Queue', $form_state['values']['nid']);
-		} else {
-			drupal_set_message("Can not open ".$form_state['values']['file'].". Job not scheduled.");
-		}    
-  } elseif (preg_match('/Re-Submit Job/', $form_state['values']['op'])) {
+      db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", 'Submitted to Queue', $form_state['values']['nid']);
+    }
+    else {
+      drupal_set_message("Can not open " . $form_state['values']['file'] . ". Job not scheduled.");
+    }
+  }
+  elseif (preg_match('/Re-Submit Job/', $form_state['values']['op'])) {
     tripal_jobs_rerun($form_state['values']['job_id']);
-    db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",'Submitted to Queue', $form_state['values']['nid']);
-  } elseif (preg_match('/Cancel Job/', $form_state['values']['op'])) {
-    db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",'Job Cancelled', $form_state['values']['nid']);
+    db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", 'Submitted to Queue', $form_state['values']['nid']);
+  }
+  elseif (preg_match('/Cancel Job/', $form_state['values']['op'])) {
+    db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", 'Job Cancelled', $form_state['values']['nid']);
     tripal_jobs_cancel($form_state['values']['job_id']);
-  } elseif (preg_match('/Revert/', $form_state['values']['op'])) {
-  
+  }
+  elseif (preg_match('/Revert/', $form_state['values']['op'])) {
+
     // Remove the records from the database that were already inserted
     $resource = db_query('SELECT * FROM {tripal_bulk_loader_inserted} WHERE nid=%d ORDER BY tripal_bulk_loader_inserted_id DESC', $form_state['values']['nid']);
     while ($r = db_fetch_object($resource)) {
-      $ids = preg_split('/,/',$r->ids_inserted);
-      db_query('DELETE FROM %s WHERE %s IN (%s)',$r->table_inserted_into, $r->table_primary_key, $r->ids_inserted);
+      $ids = preg_split('/,/', $r->ids_inserted);
+      db_query('DELETE FROM %s WHERE %s IN (%s)', $r->table_inserted_into, $r->table_primary_key, $r->ids_inserted);
       $result = db_fetch_object(db_query('SELECT true as present FROM %s WHERE %s IN (%s)', $r->table_inserted_into, $r->table_primary_key, $r->ids_inserted));
       if (!$result->present) {
-        drupal_set_message('Successfully Removed data Inserted into the '.$r->table_inserted_into.' table.');
-        db_query('DELETE FROM {tripal_bulk_loader_inserted} WHERE tripal_bulk_loader_inserted_id=%d',$r->tripal_bulk_loader_inserted_id);
-      } else {
-        drupal_set_message('Unable to remove data Inserted into the '.$r->table_inserted_into.' table!', 'error');
+        drupal_set_message('Successfully Removed data Inserted into the ' . $r->table_inserted_into . ' table.');
+        db_query('DELETE FROM {tripal_bulk_loader_inserted} WHERE tripal_bulk_loader_inserted_id=%d', $r->tripal_bulk_loader_inserted_id);
+      }
+      else {
+        drupal_set_message('Unable to remove data Inserted into the ' . $r->table_inserted_into . ' table!', 'error');
       }
     }
-    
+
     // reset status
-    db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",'Reverted -Data Deleted', $form_state['values']['nid']);
+    db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", 'Reverted -Data Deleted', $form_state['values']['nid']);
   }
 
 }
@@ -97,75 +102,80 @@ function tripal_bulk_loader_add_loader_job_form_submit ($form, $form_state) {
  * This is the function that's run by tripal_launch_jobs to bulk load chado data.
  *
  * @param $nid
- *   The Node ID of the bulk loading job node to be loaded. All other needed data is expected to be 
+ *   The Node ID of the bulk loading job node to be loaded. All other needed data is expected to be
  *   in the node (ie: template ID and file)
  *
  * Note: Instead of returning a value this function updates the tripal_bulk_loader.status.
  *   Errors are thrown through watchdog and can be viewed at admin/reports/dblog.
  */
 function tripal_bulk_loader_load_data($nid) {
-  
+
   // ensure no timeout
   set_time_limit(0);
-  
+
   // set the status of the job (in the node not the tripal jobs)
-  db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",'Loading...', $nid);
+  db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", 'Loading...', $nid);
+
 
-  
   $node = node_load($nid);
-  print "Template: ".$node->template->name." (".$node->template_id.")\n";
+  print "Template: " . $node->template->name . " (" . $node->template_id . ")\n";
 
   $total_lines = trim(`wc --lines < $node->file`);
-  print "File: ".$node->file." (".$total_lines." lines)\n";
-  
+  print "File: " . $node->file . " (" . $total_lines . " lines)\n";
+
   // Prep Work ==================================================================================
   $loaded_without_errors = TRUE;
-  
+
   // Generate default values array
   $default_data = array();
   $field2column = array();
   $record2priority = array();
 
   foreach ($node->template->template_array as $priority => $record_array) {
-    if (!is_array($record_array)) { continue; }
+    if (!is_array($record_array)) {
+      continue;
+    }
 
     //watchdog('T_bulk_loader','1)'.$record_array['record_id']." => \n<pre>".print_r($record_array,TRUE).'</pre>', array(), WATCHDOG_NOTICE);
-    
+
     foreach ($record_array['fields'] as $field_index => $field_array) {
-      
+
       $default_data[$priority]['table'] = $record_array['table'];
       $default_data[$priority]['mode'] = ($record_array['mode']) ? $record_array['mode'] : 'insert_unique';
       $default_data[$priority]['record_id'] = $record_array['record_id'];
       $record2priority[$record_array['record_id']] = $priority;
       $default_data[$priority]['required'][$field_array['field']] = $field_array['required'];
-      
+
       $one = $default_data[$priority];
       if (isset($field_array['regex'])) {
         $default_data[$priority]['regex_transform'][$field_array['field']] = $field_array['regex'];
       }
-      
+
       $two = $default_data[$priority];
-      
+
       if (preg_match('/table field/', $field_array['type'])) {
         $default_data[$priority]['values_array'][$field_array['field']] = '';
         $default_data[$priority]['need_further_processing'] = TRUE;
         $field2column[$priority][$field_array['field']] = $field_array['spreadsheet column'];
-        
-      } elseif (preg_match('/constant/', $field_array['type'])) {
+
+      }
+      elseif (preg_match('/constant/', $field_array['type'])) {
         $default_data[$priority]['values_array'][$field_array['field']] = $field_array['constant value'];
-        
-      } elseif (preg_match('/foreign key/', $field_array['type'])) {
+
+      }
+      elseif (preg_match('/foreign key/', $field_array['type'])) {
         $default_data[$priority]['values_array'][$field_array['field']] = array();
         $default_data[$priority]['values_array'][$field_array['field']]['foreign record'] = $field_array['foreign key'];
         $default_data[$priority]['need_further_processing'] = TRUE;
-        
-      } else {
-        print 'WARNING: Unsupported type: '. $field_array['type'] . ' for ' . $table . '.' . $field_array['field']."!\n";
-      }   
-      
+
+      }
+      else {
+        print 'WARNING: Unsupported type: ' . $field_array['type'] . ' for ' . $table . '.' . $field_array['field'] . "!\n";
+      }
+
       $three = $default_data[$priority];
       //watchdog('T_bulk_loader','A)'.$field_index.':<pre>Field Array =>'.print_r($field_array,TRUE)."Initial => \n".print_r($one, TRUE)."\nAfter Regex =>".print_r($two, TRUE)."Final =>\n".print_r($three,TRUE).'</pre>', array(), WATCHDOG_NOTICE);
-      
+
     } // end of foreach field
     //watchdog('T_bulk_loader','2)'.$record_array['record_id'].':<pre>'.print_r($default_data[$priority], TRUE).'</pre>', array(), WATCHDOG_NOTICE);
   } //end of foreach record
@@ -180,203 +190,220 @@ function tripal_bulk_loader_load_data($nid) {
     // revert default data array for next set of constants
     $default_data = $original_default_data;
     $group_index++;
-    
+
     // Add constants
     if (!empty($set)) {
       print "Constants:\n";
       foreach ($set as $priority => $record) {
         foreach ($record as $field_id => $field) {
-        
-          print "\t- ".$field['chado_table'].'.'.$field['chado_field'].' = '.$field['value']."\n";
-          
+
+          print "\t- " . $field['chado_table'] . '.' . $field['chado_field'] . ' = ' . $field['value'] . "\n";
+
           if ($default_data[$priority]['table'] == $field['chado_table']) {
             if (isset($default_data[$priority]['values_array'][$field['chado_field']])) {
               if (isset($field2column[$priority][$field['chado_field']])) {
                 $field2column[$priority][$field['chado_field']] = $field['value'];
-              } else {
+              }
+              else {
                 $default_data[$priority]['values_array'][$field['chado_field']] = $field['value'];
               }
-            } else {
+            }
+            else {
               print "ERROR: Template has changed after constants were assigned!\n";
-              watchdog('T_bulk_loader','Template has changed after constants were assigned', array(), WATCHDOG_NOTICE);
+              watchdog('T_bulk_loader', 'Template has changed after constants were assigned', array(), WATCHDOG_NOTICE);
               exit(1);
             }
-          } else {
+          }
+          else {
             print "ERROR: Template has changed after constants were assigned!\n";
-            watchdog('T_bulk_loader','Template has changed after constants were assigned', array(), WATCHDOG_NOTICE);
+            watchdog('T_bulk_loader', 'Template has changed after constants were assigned', array(), WATCHDOG_NOTICE);
             exit(1);
           }
         }
       }
     }
-    
+
     //print "Default Data:".print_r($default_data,TRUE)."\n";
     //watchdog('T_bulk_loader','Default Data:<pre>'.print_r($default_data, TRUE).'</pre>', array(), WATCHDOG_NOTICE);
-    
+
     //print "\nDefault Values Array: ".print_r($default_data, TRUE)."\n";
     //print "\nField to Column Mapping: ".print_r($field2column, TRUE)."\n";
-    
+
     // Parse File adding records as we go ========================================================
     $file_handle = fopen($node->file, 'r');
-    if (preg_match('/(t|true|1)/', $node->file_has_header)) { fgets($file_handle, 4096); }
+    if (preg_match('/(t|true|1)/', $node->file_has_header)) {
+      fgets($file_handle, 4096);
+    }
     $num_records = 0;
     $num_lines = 0;
     $num_errors = 0;
     $interval = intval($total_lines * 0.10);
-    if($interval == 0){ $interval = 1; }
+    if ($interval == 0) {
+       $interval = 1;
+     }
     while (!feof($file_handle)) {
-  
+
       // Clear variables
       // Was added to fix memory leak
       unset($line);                     unset($raw_line);
       unset($data);                     unset($data_keys);
       unset($priority);                 unset($sql);
-      unset($result);                   
-      
+      unset($result);
+
       $raw_line = fgets($file_handle, 4096);
       $raw_line = trim($raw_line);
-      if (empty($raw_line)) { continue; } // skips blank lines
+      if (empty($raw_line)) {
+        continue;
+      } // skips blank lines
       $line = explode("\t", $raw_line);
       $num_lines++;
-      
+
       // update the job status every 10% of lines processed for the current group
-      if($node->job_id and $num_lines % $interval == 0){
+      if ($node->job_id and $num_lines % $interval == 0) {
         // percentage of lines processed for the current group
         $group_progress = round(($num_lines/$total_lines)*100);
-        
+
         // percentage of lines processed for all groups
         // <previous group index> * 100 + <current group progress>
         // --------------------------------------------------------
         //               <total number of groups>
-        // For example, if you were in the third group of 3 constant sets 
+        // For example, if you were in the third group of 3 constant sets
         // and had a group percentage of 50% then the job progress would be
         // (2*100 + 50%) / 3 = 250%/3 = 83%
         $job_progress = round(((($group_index-1)*100)+$group_progress)/$total_num_groups);
-        
+
         /**
           print "\nProgress Update:\n"
             ."\t- ".$num_lines." lines have been processed for the current constant set.\n"
             ."\t- ".$group_progress."% of the lines in the file have been processed for the current constant set.\n"
             ."\t- ".$job_progress."% of the current job has been completed.\n";
         */
-         
-        tripal_job_set_progress($node->job_id,$job_progress);
-      }    
-  
+
+        tripal_job_set_progress($node->job_id, $job_progress);
+      }
+
       $data = $default_data;
-  
-      $data_keys = array_keys($data); 
+
+      $data_keys = array_keys($data);
       foreach ($data_keys as $priority) {
         $status = process_data_array_for_line($priority, $data, $default_data, $field2column, $record2priority, $line, $nid, $num_lines, $group_index);
-        if (!$status ) { $loaded_without_errors = FALSE; }
+        if (!$status ) {
+          $loaded_without_errors = FALSE;
+        }
       } // end of foreach table in default data array
-      
+
     } //end of foreach line of file
   } //end of foreach constant set
-    
+
   // check that data was inserted and update job_status
   $sql = 'SELECT count(*) as num_tables FROM {tripal_bulk_loader_inserted} WHERE nid=%d GROUP BY nid';
   $result = db_fetch_object(db_query($sql, $nid));
   if ($result->num_tables > 0) {
     $node->job_status = 'Data Inserted';
-    drupal_write_record('node',$node,'nid');
+    drupal_write_record('node', $node, 'nid');
   }
-  
+
   // set the status of the job (in the node not the tripal jobs)
-  if ($loaded_without_errors) { $status = 'Loading Completed Successfully'; } else { $status = 'Errors Encountered'; }
-  db_query("UPDATE tripal_bulk_loader SET job_status='%s' WHERE nid=%d",$status, $nid);
+  if ($loaded_without_errors) {
+      $status = 'Loading Completed Successfully';
+    }
+    else {
+      $status = 'Errors Encountered';
+      }
+  db_query("UPDATE {tripal_bulk_loader} SET job_status='%s' WHERE nid=%d", $status, $nid);
 
 }
 
 /**
- * 
+ *
  *
  */
-function process_data_array_for_line ($priority, &$data, &$default_data, $field2column, $record2priority, $line, $nid, $line_num, $group_index) {
+function process_data_array_for_line($priority, &$data, &$default_data, $field2column, $record2priority, $line, $nid, $line_num, $group_index) {
   $table_data = $data[$priority];
 
   $no_errors = TRUE;
 
   $table = $table_data['table'];
   $values = $table_data['values_array'];
-  
+
   //watchdog('T_bulk_loader','Original:<pre>'.print_r($table_data, TRUE).'</pre>', array(), WATCHDOG_NOTICE);
-  
+
   //print 'default values:'.print_r($values,TRUE)."\n";
   if ($table_data['need_further_processing']) {
-    $values = tripal_bulk_loader_add_spreadsheetdata_to_values ($values, $line, $field2column[$priority]);
+    $values = tripal_bulk_loader_add_spreadsheetdata_to_values($values, $line, $field2column[$priority]);
     if (!$values) {
-      watchdog('T_bulk_loader','Line '.$line_num.' Spreadsheet Added:'.print_r($values, TRUE), array(), WATCHDOG_NOTICE);
+      watchdog('T_bulk_loader', 'Line ' . $line_num . ' Spreadsheet Added:' . print_r($values, TRUE), array(), WATCHDOG_NOTICE);
     }
-    
+
     $values = tripal_bulk_loader_add_foreignkey_to_values($values, $data, $record2priority);
     if (!$values) {
-      watchdog('T_bulk_loader','Line '.$line_num.' FK Added:<pre>'.print_r($values, TRUE).print_r($data[$priority],TRUE).'</pre>', array(), WATCHDOG_NOTICE);
+      watchdog('T_bulk_loader', 'Line ' . $line_num . ' FK Added:<pre>' . print_r($values, TRUE) . print_r($data[$priority], TRUE) . '</pre>', array(), WATCHDOG_NOTICE);
     }
   }
   $values = tripal_bulk_loader_regex_tranform_values($values, $table_data, $line);
   if (!$values) {
-    watchdog('T_bulk_loader','Line '.$line_num.' Regex:<pre>'.print_r($values, TRUE).print_r($table_data, TRUE).'</pre>'.'</pre>', array(), WATCHDOG_NOTICE);
+    watchdog('T_bulk_loader', 'Line ' . $line_num . ' Regex:<pre>' . print_r($values, TRUE) . print_r($table_data, TRUE) . '</pre>' . '</pre>', array(), WATCHDOG_NOTICE);
   }
 
   if (!$values) {
-    $msg = 'Line '.$line_num.' '.$table_data['record_id'].' ('.$table_data['mode'].') Aborted due to error in previous record. Values of current record:'.print_r($table_data['values_array'],TRUE);
-    watchdog('T_bulk_loader', $msg, array(), WATCHDOG_WARNING); 
-    print "ERROR: ".$msg."\n";
+    $msg = 'Line ' . $line_num . ' ' . $table_data['record_id'] . ' (' . $table_data['mode'] . ') Aborted due to error in previous record. Values of current record:' . print_r($table_data['values_array'], TRUE);
+    watchdog('T_bulk_loader', $msg, array(), WATCHDOG_WARNING);
+    print "ERROR: " . $msg . "\n";
     $data[$priority]['error'] = TRUE;
     $no_errors = FALSE;
   }
-  
-  $table_desc = module_invoke_all('chado_'.$table.'_schema');
+
+  $table_desc = module_invoke_all('chado_' . $table . '_schema');
   if (preg_match('/optional/', $table_array['mode'])) {
     // Check all db required fields are set
     $fields = $table_desc['fields'];
-    foreach($fields as $field => $def){
+    foreach ($fields as $field => $def) {
       // a field is considered missing if it cannot be null and there is no default
       // value for it or it is of type 'serial'
-      if($def['not null'] == 1 and !array_key_exists($field,$insert_values) and !isset($def['default']) and strcmp($def['type'],serial)!=0){
-         $msg = 'Line '.$line_num.' '.$table_data['record_id'].' ('.$table_data['mode'].') Missing Database Required Value: '.$table.'.'.$field;
-         watchdog('T_bulk_loader', $msg, array(), WATCHDOG_NOTICE); 
+      if ($def['not null'] == 1 and !array_key_exists($field, $insert_values) and !isset($def['default']) and strcmp($def['type'], serial)!=0) {
+         $msg = 'Line ' . $line_num . ' ' . $table_data['record_id'] . ' (' . $table_data['mode'] . ') Missing Database Required Value: ' . $table . '.' . $field;
+         watchdog('T_bulk_loader', $msg, array(), WATCHDOG_NOTICE);
          $data[$priority]['error'] = TRUE;
       }
     }
   } //end of if optional record
-  
+
   // Check required fields are present
   foreach ($table_data['required'] as $field => $required) {
     if ($required) {
       if (!isset($values[$field])) {
-        $msg = 'Line '.$line_num.' '.$table_data['record_id'].' ('.$table_data['mode'].') Missing Template Required Value: '.$table.'.'.$field;
-        watchdog('T_bulk_loader', $msg, array(), WATCHDOG_NOTICE); 
-        $data[$priority]['error'] = TRUE;  
+        $msg = 'Line ' . $line_num . ' ' . $table_data['record_id'] . ' (' . $table_data['mode'] . ') Missing Template Required Value: ' . $table . '.' . $field;
+        watchdog('T_bulk_loader', $msg, array(), WATCHDOG_NOTICE);
+        $data[$priority]['error'] = TRUE;
       }
     }
   }
-  
+
   // add new values array into the data array
   $data[$priority]['values_array'] = $values;
-  
+
   // check if it is already inserted
   if ($table_data['inserted']) {
     //watchdog('T_bulk_loader','Already Inserted:'.print_r($values,TRUE),array(),WATCHDOG_NOTICE);
     return $no_errors;
   }
-  
+
   // if there was an error already -> don't insert
   if ($data[$priority]['error']) {
     return $no_errors;
   }
-  
+
   $header = '';
   if (isset($values['feature_id'])) {
-    $header = $values['feature_id']['uniquename'] .' '. $table_data['record_id'];
-  } else {
-    $header = $values['uniquename'] .' '. $table_data['record_id'];
+    $header = $values['feature_id']['uniquename'] . ' ' . $table_data['record_id'];
+  }
+  else {
+    $header = $values['uniquename'] . ' ' . $table_data['record_id'];
   }
-  
+
   // if insert unique then check to ensure unique
-  if (preg_match('/insert_unique/',$table_data['mode'])) {
-    $unique = tripal_core_chado_select($table, array_keys($table_desc['fields']), $values, array('has_record'=>TRUE));
+  if (preg_match('/insert_unique/', $table_data['mode'])) {
+    $unique = tripal_core_chado_select($table, array_keys($table_desc['fields']), $values, array('has_record' => TRUE));
     //print 'Unique?'.print_r(array('table' => $table, 'columns' => array_keys($table_desc['fields']), 'values' => $values),TRUE).' returns '.$unique."\n";
     if ($unique > 0) {
       //$default_data[$priority]['inserted'] = TRUE;
@@ -384,9 +411,9 @@ function process_data_array_for_line ($priority, &$data, &$default_data, $field2
       return $no_errors;
     }
   }
-  
-    
-  if (!preg_match('/select/',$table_data['mode'])) {
+
+
+  if (!preg_match('/select/', $table_data['mode'])) {
     //watchdog('T_bulk_loader',$header.': Inserting:'.print_r($values, TRUE), array(), WATCHDOG_NOTICE);
     $options = array('statement_name' => $priority);
     if ($line_num == 1 && $group_index == 1) {
@@ -394,21 +421,22 @@ function process_data_array_for_line ($priority, &$data, &$default_data, $field2
     }
     $record = tripal_core_chado_insert($table, $values, $options);
     if (!$record) {
-      $msg = 'Line '.$line_num.' '.$table_data['record_id'].' ('.$table_data['mode'].') Unable to insert record into '.$table.' where values:'.print_r($values,TRUE);
-      watchdog('T_bulk_loader', $msg, array(), WATCHDOG_ERROR); 
-      print "ERROR: ".$msg."\n";  
+      $msg = 'Line ' . $line_num . ' ' . $table_data['record_id'] . ' (' . $table_data['mode'] . ') Unable to insert record into ' . $table . ' where values:' . print_r($values, TRUE);
+      watchdog('T_bulk_loader', $msg, array(), WATCHDOG_ERROR);
+      print "ERROR: " . $msg . "\n";
       $data[$priority]['error'] = TRUE;
       $no_errors = FALSE;
-    } else {
+    }
+    else {
       //add changes back to values array
       $data[$priority]['values_array'] = $record;
       $values = $record;
-      
+
       // if mode=insert_once then ensure we only insert it once
-      if (preg_match('/insert_once/',$table_data['mode'])) {
+      if (preg_match('/insert_once/', $table_data['mode'])) {
         $default_data[$priority]['inserted'] = TRUE;
       }
-      
+
       // add to tripal_bulk_loader_inserted
       $insert_record = db_fetch_object(db_query(
         "SELECT * FROM {tripal_bulk_loader_inserted} WHERE table_inserted_into='%s' AND nid=%d",
@@ -420,7 +448,8 @@ function process_data_array_for_line ($priority, &$data, &$default_data, $field2
         drupal_write_record('tripal_bulk_loader_inserted', $insert_record, 'tripal_bulk_loader_inserted_id');
         //print 'Update: '.print_r($insert_record,TRUE)."\n";
         return $no_errors;
-      } else {
+      }
+      else {
         $insert_record = array(
           'nid' => $nid,
           'table_inserted_into' => $table,
@@ -431,15 +460,16 @@ function process_data_array_for_line ($priority, &$data, &$default_data, $field2
         $success = drupal_write_record('tripal_bulk_loader_inserted', $insert_record);
         return $no_errors;
       }//end of if insert record
-      
+
     } //end of if insert was successful
-  } else {
-    $exists = tripal_core_chado_select($table, array_keys($table_desc['fields']), $values, array('has_record'=>TRUE));
+  }
+  else {
+    $exists = tripal_core_chado_select($table, array_keys($table_desc['fields']), $values, array('has_record' => TRUE));
     if (!$exists) {
       // No record on select
-      $msg = 'Line '.$line_num.' '.$table_data['record_id'].' ('.$table_data['mode'].') No Matching record in '.$table.' where values:'.print_r($values,TRUE);
-      watchdog('T_bulk_loader', $msg, array(), WATCHDOG_WARNING); 
-      $data[$priority]['error'] = TRUE;  
+      $msg = 'Line ' . $line_num . ' ' . $table_data['record_id'] . ' (' . $table_data['mode'] . ') No Matching record in ' . $table . ' where values:' . print_r($values, TRUE);
+      watchdog('T_bulk_loader', $msg, array(), WATCHDOG_WARNING);
+      $data[$priority]['error'] = TRUE;
     }
   }
   return $no_errors;
@@ -454,24 +484,29 @@ function process_data_array_for_line ($priority, &$data, &$default_data, $field2
  *   An array of values for the current line
  * @param $field2column
  *   An array mapping values fields to line columns
- * @return 
+ * @return
  *   Supplemented values array
  */
-function tripal_bulk_loader_add_spreadsheetdata_to_values ($values, $line, $field2column) {
+function tripal_bulk_loader_add_spreadsheetdata_to_values($values, $line, $field2column) {
 
   foreach ($values as $field => $value) {
-    if (is_array($value)) { continue; }
-    
+    if (is_array($value)) {
+      continue;
+    }
+
     $column = $field2column[$field] - 1;
-    if ($column < 0) { continue; }
-    
-    if (preg_match('/\S+/',$line[$column])) {
+    if ($column < 0) {
+      continue;
+    }
+
+    if (preg_match('/\S+/', $line[$column])) {
       $values[$field] = $line[$column];
-    } else {
+    }
+    else {
       unset($values[$field]);
     }
   }
-  
+
   return $values;
 }
 
@@ -490,7 +525,7 @@ function tripal_bulk_loader_add_foreignkey_to_values($values, $data, $record2pri
       $foreign_record = $value['foreign record'];
       $foreign_priority = $record2priority[$foreign_record];
       $foreign_values = $data[$foreign_priority]['values_array'];
-      
+
       // add to current values array
       $values[$field] = $foreign_values;
     }
@@ -507,42 +542,46 @@ function tripal_bulk_loader_add_foreignkey_to_values($values, $data, $record2pri
  * @param $table_data
  *   The data array for the given table
  */
-function tripal_bulk_loader_regex_tranform_values ($values, $table_data, $line) {
+function tripal_bulk_loader_regex_tranform_values($values, $table_data, $line) {
+
+  if (empty($table_data['regex_transform']) OR !is_array($table_data['regex_transform'])) {
+    return $values;
+  }
 
-  if (empty($table_data['regex_transform']) OR !is_array($table_data['regex_transform'])) { return $values; }
-  
   //watchdog('T_bulk_loader','Regex Transformation:<pre>'.print_r($table_data['regex_transform'], TRUE).'</pre>', array(), WATCHDOG_NOTICE);
-  
+
   foreach ($table_data['regex_transform'] as $field => $regex_array) {
-    if (!is_array($regex_array['replace'])) { continue; }
-    
+    if (!is_array($regex_array['replace'])) {
+       continue;
+     }
+
     //print 'Match:'.print_r($regex_array['pattern'],TRUE)."\n";
     //print 'Replace:'.print_r($regex_array['replace'],TRUE)."\n";
     //print 'Was:'.$values[$field]."\n";
-    
+
     // Check for <#column:\d+#> notation
     // if present replace with that column in the current line
     foreach ($regex_array['replace'] as $key => $replace) {
       if (preg_match_all('/<#column:(\d+)#>/', $replace, $matches)) {
         foreach ($matches[1] as $k => $column_num) {
-          $replace = preg_replace('/'.$matches[0][$k].'/', $line[$column_num-1], $replace);
+          $replace = preg_replace('/' . $matches[0][$k] .'/', $line[$column_num-1], $replace);
         }
         $regex_array['replace'][$key] = $replace;
       }
     }
-    
+
     // do the full replacement
     $old_value = $values[$field];
     $new_value = preg_replace($regex_array['pattern'], $regex_array['replace'], $old_value);
     $values[$field] = $new_value;
-    
+
     if ($values[$field] === '') {
-      unset($values[$field]); 
+      unset($values[$field]);
     }
     //print 'Now:'.$values[$field]."\n";
   }
-  
-  
+
+
 
   return $values;
 }
@@ -551,25 +590,27 @@ function tripal_bulk_loader_regex_tranform_values ($values, $table_data, $line)
  * Flattens an array up to two levels
  * Used for printing of arrays without taking up much space
  */
-function tripal_bulk_loader_flatten_array ($values) {
+function tripal_bulk_loader_flatten_array($values) {
   $flattened_values = array();
-  
+
   foreach ($values as $k => $v) {
     if (is_array($v)) {
       $vstr = array();
       foreach ($v as $vk => $vv) {
         if (strlen($vv) > 20) {
-          $vstr[] = $vk .'=>'. substr($vv, 0, 20) . '...';
-        } else {
-          $vstr[] = $vk .'=>'. $vv;
+          $vstr[] = $vk . '=>' . substr($vv, 0, 20) . '...';
+        }
+        else {
+          $vstr[] = $vk . '=>' . $vv;
         }
       }
-      $v = '{'. implode(',',$vstr) .'}';
-    } elseif (strlen($v) > 20) {
+      $v = '{' . implode(',', $vstr) . '}';
+    }
+    elseif (strlen($v) > 20) {
       $v = substr($v, 0, 20) . '...';
     }
-    $flattened_values[] = $k .'=>'. $v;
+    $flattened_values[] = $k . '=>' . $v;
   }
-  
-  return implode(', ',$flattened_values);
-}
+
+  return implode(', ', $flattened_values);
+}

+ 173 - 172
tripal_bulk_loader/tripal_bulk_loader.module

@@ -7,10 +7,10 @@ include('tripal_bulk_loader.constants.inc');
  * Implements hook_init
  * Used to add stylesheets and javascript files to the header
  */
-function tripal_bulk_loader_init(){
-	// Add javascript and style sheet
-	drupal_add_css(drupal_get_path('theme', 'tripal').'/css/tripal_bulk_loader.css');
-	drupal_add_js(drupal_get_path('theme', 'tripal').'/js/tripal_bulk_loader.js');
+function tripal_bulk_loader_init() {
+  // Add javascript and style sheet
+  drupal_add_css(drupal_get_path('theme', 'tripal') . '/css/tripal_bulk_loader.css');
+  drupal_add_js(drupal_get_path('theme', 'tripal') . '/js/tripal_bulk_loader.js');
 }
 
 /**
@@ -30,39 +30,39 @@ function tripal_bulk_loader_menu() {
    $items['node/%node/constants/%/edit'] = array(
       'title' => 'Edit Constant Set',
       'description' => 'Edit a group of constants associated with the current bulk loader',
-    	'page callback' => 'drupal_get_form',
-    	'page arguments' => array('tripal_bulk_loader_edit_constant_set_form',1,3),
+      'page callback' => 'drupal_get_form',
+      'page arguments' => array('tripal_bulk_loader_edit_constant_set_form', 1, 3),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		  'file' => 'tripal_bulk_loader.constants.inc',
+      'type' => MENU_CALLBACK,
+      'file' => 'tripal_bulk_loader.constants.inc',
    );
    $items['node/%node/constants/%/delete'] = array(
       'title' => 'Delete Constant Set',
       'description' => 'Delete a group of constants associated with the current bulk loader',
-    	'page callback' => 'drupal_get_form',
-    	'page arguments' => array('tripal_bulk_loader_delete_constant_set_form',1,3),
+      'page callback' => 'drupal_get_form',
+      'page arguments' => array('tripal_bulk_loader_delete_constant_set_form', 1, 3),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		  'file' => 'tripal_bulk_loader.constants.inc',
+      'type' => MENU_CALLBACK,
+      'file' => 'tripal_bulk_loader.constants.inc',
    );
    // Admin page to create the template
    $items['admin/tripal/tripal_bulk_loader_template'] = array(
       'title' => 'Bulk Loader Template',
       'description' => 'Templates for loading tab-delimited data',
-    	'page callback' => 'tripal_bulk_loader_admin_template',
+      'page callback' => 'tripal_bulk_loader_admin_template',
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		  'file' => 'tripal_bulk_loader.admin.inc',
-	);
-	// Create/Edit Template -------
+      'type' => MENU_NORMAL_ITEM,
+      'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  // Create/Edit Template -------
   $items['admin/tripal/tripal_bulk_loader_template/create'] = array(
       'title' => 'Create Bulk Loader Template',
       'description' => 'Create loader template for loading tab-delimited data',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_modify_template_base_form', 'create'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		'file' => 'tripal_bulk_loader.admin.inc',
+      'type' => MENU_NORMAL_ITEM,
+    'file' => 'tripal_bulk_loader.admin.inc',
   );
   $items['admin/tripal/tripal_bulk_loader_template/edit'] = array(
       'title' => 'Edit Bulk Loader Template',
@@ -70,79 +70,79 @@ function tripal_bulk_loader_menu() {
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_modify_template_base_form', 'edit'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		  'file' => 'tripal_bulk_loader.admin.inc',  
+      'type' => MENU_NORMAL_ITEM,
+      'file' => 'tripal_bulk_loader.admin.inc',
   );
-	$items['admin/tripal/tripal_bulk_loader_template/edit_record'] = array(
+  $items['admin/tripal/tripal_bulk_loader_template/edit_record'] = array(
       'title' => 'Edit Template Record',
       'description' => 'Edit a record in an existing tripal bulk loader template.',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_edit_template_record_form'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		'file' => 'tripal_bulk_loader.admin.inc',
-	);
-	$items['admin/tripal/tripal_bulk_loader_template/add_field'] = array(
+      'type' => MENU_CALLBACK,
+    'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  $items['admin/tripal/tripal_bulk_loader_template/add_field'] = array(
       'title' => 'Add Template Field',
       'description' => 'Add a template field to an existing tripal bulk loader template.',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_add_template_field_form'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		'file' => 'tripal_bulk_loader.admin.inc',
-	);
-	$items['admin/tripal/tripal_bulk_loader_template/edit_field'] = array(
+      'type' => MENU_CALLBACK,
+    'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  $items['admin/tripal/tripal_bulk_loader_template/edit_field'] = array(
       'title' => 'Edit Template Field',
       'description' => 'Edit an existing field from a tripal bulk loader template.',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_edit_template_field_form'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		'file' => 'tripal_bulk_loader.admin.inc',
-	);
-	// Delete Template -----
-	$items['admin/tripal/tripal_bulk_loader_template/delete'] = array(
+      'type' => MENU_CALLBACK,
+    'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  // Delete Template -----
+  $items['admin/tripal/tripal_bulk_loader_template/delete'] = array(
       'title' => 'Delete Bulk Loader Template',
       'description' => 'Delete bulk loader template',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_delete_template_base_form'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		'file' => 'tripal_bulk_loader.admin.inc',
-	);
-	// Import/Export ---------
+      'type' => MENU_NORMAL_ITEM,
+    'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  // Import/Export ---------
   $items['admin/tripal/tripal_bulk_loader_template/import'] = array(
       'title' => 'Import Bulk Loader Template',
       'description' => 'Import Loaders',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_import_export_template_form', 'import'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		  'file' => 'tripal_bulk_loader.admin.inc',  
-  );	
+      'type' => MENU_NORMAL_ITEM,
+      'file' => 'tripal_bulk_loader.admin.inc',
+  );
   $items['admin/tripal/tripal_bulk_loader_template/export'] = array(
       'title' => 'Export Bulk Loader Template',
       'description' => 'Export Loaders',
       'page callback' => 'drupal_get_form',
       'page arguments' => array('tripal_bulk_loader_import_export_template_form', 'export'),
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_NORMAL_ITEM,   
-		  'file' => 'tripal_bulk_loader.admin.inc',  
-  );	
-	// AHAH ---------
-	$items['admin/tripal/tripal_bulk_loader_template/add_field_ahah'] = array(
+      'type' => MENU_NORMAL_ITEM,
+      'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  // AHAH ---------
+  $items['admin/tripal/tripal_bulk_loader_template/add_field_ahah'] = array(
       'page callback' => 'tripal_bulk_loader_add_field_ahah',
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		  'file' => 'tripal_bulk_loader.admin.inc',	
-	);
-	$items['admin/tripal/tripal_bulk_loader_template/edit_field_ahah'] = array(
+      'type' => MENU_CALLBACK,
+      'file' => 'tripal_bulk_loader.admin.inc',
+  );
+  $items['admin/tripal/tripal_bulk_loader_template/edit_field_ahah'] = array(
       'page callback' => 'tripal_bulk_loader_edit_field_ahah',
       'access arguments' => array('administer site configuration'),
-      'type' => MENU_CALLBACK,   
-		  'file' => 'tripal_bulk_loader.admin.inc',	
-	);
-	
+      'type' => MENU_CALLBACK,
+      'file' => 'tripal_bulk_loader.admin.inc',
+  );
+
   return $items;
 }
 
@@ -156,12 +156,12 @@ function tripal_bulk_loader_theme() {
       'arguments' => array('form' => NULL),
     ),
     'tripal_bulk_loader_template' => array(
-    	'arguments'=> array('template_id' => NULL),
-    	'template' => 'tripal_bulk_loader_template'
+      'arguments' => array('template_id' => NULL),
+      'template' => 'tripal_bulk_loader_template'
     ),
     'tripal_bulk_loader_modify_template_base_form' => array(
-    	'arguments' => array('form' => NULL),
-    	'template' => 'tripal_bulk_loader_modify_template_base_form',
+      'arguments' => array('form' => NULL),
+      'template' => 'tripal_bulk_loader_modify_template_base_form',
     ),
     'tripal_bulk_loader_edit_template_field_form' => array(
       'arguments' => array('form' => NULL),
@@ -177,69 +177,69 @@ function tripal_bulk_loader_theme() {
 /**
  *  Implements hook_access
  */
-function tripal_bulk_loader_access($op, $node, $account){
-	if ($op == 'create') {
-		if(!user_access('create tripal_bulk_loader', $account)){
+function tripal_bulk_loader_access($op, $node, $account) {
+  if ($op == 'create') {
+    if (!user_access('create tripal_bulk_loader', $account)) {
          return FALSE;
       }
-	}
-	if ($op == 'update') {
-		if (!user_access('edit tripal_bulk_loader', $account)) {
-			return FALSE;
-		}
-	}
-	if ($op == 'delete') {
-		if (!user_access('delete tripal_bulk_loader', $account)) {
-			return FALSE;
-		}
-	}
-	if ($op == 'view') {
-		if (!user_access('access tripal_bulk_loader', $account)) {
-			return FALSE;
-		}
-	}
-	return NULL;
+  }
+  if ($op == 'update') {
+    if (!user_access('edit tripal_bulk_loader', $account)) {
+      return FALSE;
+    }
+  }
+  if ($op == 'delete') {
+    if (!user_access('delete tripal_bulk_loader', $account)) {
+      return FALSE;
+    }
+  }
+  if ($op == 'view') {
+    if (!user_access('access tripal_bulk_loader', $account)) {
+      return FALSE;
+    }
+  }
+  return NULL;
 }
 
 /**
  * Implements hook_perm
  */
-function tripal_bulk_loader_perm(){
-	return array(
+function tripal_bulk_loader_perm() {
+  return array(
       'access tripal_bulk_loader',
       'create tripal_bulk_loader',
       'delete tripal_bulk_loader',
       'edit tripal_bulk_loader',
-	);
+  );
 }
 
 /**
  * Creates a listing page for all bulk loading jobs
  */
-function tripal_bulk_loader_list () {
+function tripal_bulk_loader_list() {
   $num_results_per_page = 50;
   $output = '';
-  
-  $header = array('','Status','Loader','File');
+
+  $header = array('', 'Status', 'Loader', 'File');
   $rows = array();
-  
+
   $query = 'SELECT * FROM {tripal_bulk_loader} l '
     .'LEFT JOIN {node} n ON n.nid = l.nid '
     .'LEFT JOIN {tripal_bulk_loader_template} t ON t.template_id = cast(l.template_id as integer)';
   $resource = pager_query($query, $num_results_per_page, 0, NULL);
   while ($r = db_fetch_object($resource)) {
     $row = array(
-      l($r->title, 'node/'.$r->nid),
+      l($r->title, 'node/' . $r->nid),
       $r->job_status,
       $r->name,
       $r->file
     );
     $rows[] = $row;
   }
-  
+
   $output .= theme('table', $header, $rows);
   return $output;
-  
+
 }
 
 //////////////////////////////////////////////////////////////////////////////////////////////
@@ -250,52 +250,52 @@ function tripal_bulk_loader_list () {
  * Implements hook_node_info
  */
 function tripal_bulk_loader_node_info() {
-	$nodes = array();
-	$nodes['tripal_bulk_loader'] = array(
+  $nodes = array();
+  $nodes['tripal_bulk_loader'] = array(
       'name' => t('Bulk Loading Job'),
       'module' => 'tripal_bulk_loader',
       'description' => t('A bulk loader for inserting tab-delimited data into chado database'),
       'has_title' => TRUE,
       'has_body' => FALSE,
       'locked' => TRUE
-	);
-	return $nodes;
+  );
+  return $nodes;
 }
 
 /**
  * Implements node_form
  * Used to gather the extra details stored with a Bulk Loading Job Node
  */
-function tripal_bulk_loader_form ($node, $form_state){
+function tripal_bulk_loader_form($node, $form_state) {
   $form = array();
-  
+
   if (isset($form_state['values'])) {
     $node = $form_state['values'] + (array)$node;
     $node = (object) $node;
   }
-  
+
   $sql = "SELECT * FROM {tripal_bulk_loader_template}";
   $results = db_query($sql);
-  $templates = array ();
+  $templates = array();
   while ($template = db_fetch_object ($results)) {
     $templates [$template->template_id] = $template->name;
   }
-  
+
   if (!$templates) {
     $form['label'] = array(
     '#type' => 'item',
       '#description' => t("Loader template needs to be created before any bulk loader can be added. Go to 'Tripal Management > Bulk Loader Template' to create the template."),
       '#weight'        => -10,
     );
-    
+
     return $form;
   }
-  
+
   $form['loader'] = array(
     '#type' => 'fieldset',
     '#title' => t('Basic Details'),
   );
-  
+
   $form['loader']['loader_name'] = array(
     '#type'          => 'textfield',
     '#title'         => t('Loading Job Name'),
@@ -303,7 +303,7 @@ function tripal_bulk_loader_form ($node, $form_state){
     '#required'      => TRUE,
     '#default_value' => $node->loader_name
   );
-  
+
   $form['loader']['template_id'] = array(
     '#type' => 'select',
     '#title' => t('Template'),
@@ -313,7 +313,7 @@ function tripal_bulk_loader_form ($node, $form_state){
     '#required'      => TRUE,
     '#default_value' => $node->template_id,
   );
-  
+
   $form['loader']['file']= array(
     '#type'          => 'textfield',
     '#title'         => t('Data File'),
@@ -321,7 +321,7 @@ function tripal_bulk_loader_form ($node, $form_state){
     '#weight'        => -8,
     '#default_value' => $node->file
   );
-  
+
   $form['loader']['has_header'] = array(
     '#type' => 'radios',
     '#title' => t('File has a Header'),
@@ -329,7 +329,7 @@ function tripal_bulk_loader_form ($node, $form_state){
     '#weight' => -7,
     '#default_value' => $node->file_has_header,
   );
-  
+
   return $form;
 }
 
@@ -337,24 +337,24 @@ function tripal_bulk_loader_form ($node, $form_state){
 /**
  * Implements node_load
  */
-function tripal_bulk_loader_load($node){
-	$sql = "SELECT * FROM {tripal_bulk_loader} WHERE nid = %d";
-	$node = db_fetch_object(db_query($sql, $node->nid));
-
-	$node->title = 'Bulk Loading Job: '.$node->loader_name;
-	
-	// Add the loader template
-	$sql = "SELECT * FROM {tripal_bulk_loader_template} WHERE template_id=%d";
-	$results = db_fetch_object(db_query($sql, $node->template_id));
-	$template = unserialize($results->template_array);
-	$node->template = $results;
-	$node->template->template_array = $template;
+function tripal_bulk_loader_load($node) {
+  $sql = "SELECT * FROM {tripal_bulk_loader} WHERE nid = %d";
+  $node = db_fetch_object(db_query($sql, $node->nid));
+
+  $node->title = 'Bulk Loading Job: ' . $node->loader_name;
+
+  // Add the loader template
+  $sql = "SELECT * FROM {tripal_bulk_loader_template} WHERE template_id=%d";
+  $results = db_fetch_object(db_query($sql, $node->template_id));
+  $template = unserialize($results->template_array);
+  $node->template = $results;
+  $node->template->template_array = $template;
 
   // Add inserted records
   $sql = 'SELECT * FROM {tripal_bulk_loader_inserted} WHERE nid=%d';
-  $resource = db_query($sql,$node->nid);
+  $resource = db_query($sql, $node->nid);
   while ($r = db_fetch_object($resource)) {
-    $r->num_inserted = sizeof(preg_split('/,/',$r->ids_inserted)); 
+    $r->num_inserted = sizeof(preg_split('/,/', $r->ids_inserted));
     $node->inserted_records->{$r->table_inserted_into} = $r;
   }
 
@@ -372,86 +372,87 @@ function tripal_bulk_loader_load($node){
         }
       }
     }
-    
+
     if (empty($node->exposed_fields)) {
       $node->exposed_fields[] = array();
     }
   }
 
-  
+
   // Add constants
   $sql = 'SELECT * FROM {tripal_bulk_loader_constants} WHERE nid=%d ORDER BY group_id, record_id, field_id';
-  $resource = db_query($sql,$node->nid);
+  $resource = db_query($sql, $node->nid);
   while ($r = db_fetch_object($resource)) {
     $node->constants[$r->group_id][$r->record_id][$r->field_id] = array(
       'constant_id' => $r->constant_id,
       'group_id' => $r->group_id,
-      'chado_table'=>$r->chado_table, 
-      'chado_field'=>$r->chado_field, 
-      'record_id'=>$r->record_id, 
-      'field_id'=>$r->field_id, 
-      'value'=>$r->value
+      'chado_table' => $r->chado_table,
+      'chado_field' => $r->chado_field,
+      'record_id' => $r->record_id,
+      'field_id' => $r->field_id,
+      'value' => $r->value
     );
   }
   if (!$node->constants) {
     $node->constants[] = array();
   }
-  
-	return $node;
+
+  return $node;
 }
 
 /**
  * Implements node_insert
  * Insert the data from the node form on Create content
  */
-function tripal_bulk_loader_insert ($node) {
-  
+function tripal_bulk_loader_insert($node) {
+
   // Insert into tripal_bulk_loader
   $sql = "INSERT INTO {tripal_bulk_loader} (nid, loader_name, template_id, file, file_has_header, job_status) VALUES (%d, '%s', %d, '%s', %d, '%s')";
   db_query($sql, $node->nid, $node->loader_name, $node->template_id, $node->file, $node->has_header, 'Initialized');
-	
-	// Update title
-	$node->title =$node->loader_name;
-	drupal_write_record('node',$node,'nid');
-	drupal_write_record('node_revision',$node,'nid');	
-  
+
+  // Update title
+  $node->title =$node->loader_name;
+  drupal_write_record('node', $node, 'nid');
+  drupal_write_record('node_revision', $node, 'nid');
+
   drupal_set_message('After reviewing the details, please Submit this Job (by clicking the "Submit Job" button below). No data will be loaded until the submitted job is reached in the queue.');
-  
+
 }
 
 /**
  * Implements node_delete
  * Deletes the data when the delete button on the node form is clicked
  */
-function tripal_bulk_loader_delete ($node) {
-	$sql = "DELETE FROM {tripal_bulk_loader} WHERE nid = %d";
-	db_query($sql, $node->nid);
+function tripal_bulk_loader_delete($node) {
+  $sql = "DELETE FROM {tripal_bulk_loader} WHERE nid = %d";
+  db_query($sql, $node->nid);
 }
 
 /**
  * Implements node_update
  * Updates the data submitted by the node form on edit
  */
-function tripal_bulk_loader_update ($node) {
-  
+function tripal_bulk_loader_update($node) {
+
   // Update tripal_bulk_loader
-	$sql = "UPDATE {tripal_bulk_loader} SET nid = %d, loader_name = '%s', template_id = %d, file = '%s', file_has_header = '%s' WHERE nid = %d";
-	db_query($sql, $node->nid, $node->loader_name, $node->template_id, $node->file, $node->has_header, $node->nid);
-	
-	// Add a job if the user want to load the data
-	global $user;
-	if($node->job) {
-		$job_args[0] =$node->loader_name;
-		$job_args[1] = $node->template_id;
-		$job_args[2] = $node->file;
-		if (is_readable($node->file)) {
-			$fname = preg_replace("/.*\/(.*)/", "$1", $node->file);
-			tripal_add_job("Bulk Load: $fname",'tripal_bulk_loader', 'tripal_bulk_loader_load_data', $job_args, $user->uid);
-		} else {
-			drupal_set_message("Can not open $node->file. Job not scheduled.");
-		}
-	}
-  
+  $sql = "UPDATE {tripal_bulk_loader} SET nid = %d, loader_name = '%s', template_id = %d, file = '%s', file_has_header = '%s' WHERE nid = %d";
+  db_query($sql, $node->nid, $node->loader_name, $node->template_id, $node->file, $node->has_header, $node->nid);
+
+  // Add a job if the user want to load the data
+  global $user;
+  if ($node->job) {
+    $job_args[0] =$node->loader_name;
+    $job_args[1] = $node->template_id;
+    $job_args[2] = $node->file;
+    if (is_readable($node->file)) {
+      $fname = preg_replace("/.*\/(.*)/", "$1", $node->file);
+      tripal_add_job("Bulk Load: $fname", 'tripal_bulk_loader', 'tripal_bulk_loader_load_data', $job_args, $user->uid);
+    }
+    else {
+      drupal_set_message("Can not open $node->file. Job not scheduled.");
+    }
+  }
+
 }
 
 ///////////////////////////////////////////////////////////
@@ -459,13 +460,13 @@ function tripal_bulk_loader_update ($node) {
 /**
  * Preprocessor function for the tripal_bulk_loader template
  */
-function tripal_bulk_loader_preprocess_tripal_bulk_loader_template (&$variables) {
+function tripal_bulk_loader_preprocess_tripal_bulk_loader_template(&$variables) {
+
+  $sql = "SELECT * FROM {tripal_bulk_loader_template} WHERE template_id=%d";
+  $template = db_fetch_object(db_query($sql, $variables['template_id']));
+  $template->template_array = unserialize($template->template_array);
+  $variables['template'] = $template;
 
-	$sql = "SELECT * FROM {tripal_bulk_loader_template} WHERE template_id=%d";
-	$template = db_fetch_object(db_query($sql, $variables['template_id']));
-	$template->template_array = unserialize($template->template_array);	
-	$variables['template'] = $template;
-	
 }
 
 /**
@@ -476,19 +477,19 @@ function tripal_bulk_loader_preprocess_tripal_bulk_loader_template (&$variables)
  *   The callback passed into tripal_add_job()
  * @param $args
  *   The arguements passed into tripal_add_job()
- * @return 
- *   An array where keys are the human readable headers describing each arguement 
+ * @return
+ *   An array where keys are the human readable headers describing each arguement
  *   and the value is the aguement passed in after formatting
  */
-function tripal_bulk_loader_job_describe_args($callback,$args){
+function tripal_bulk_loader_job_describe_args($callback, $args) {
 
   $new_args = array();
-  if($callback == 'tripal_bulk_loader_load_data'){
+  if ($callback == 'tripal_bulk_loader_load_data') {
     //1st arg is the nid for a bulk loader node
     $node = node_load($args[0]);
-    $new_args['Bulk Loading Job'] = l($node->title, 'node/'.$args[0]);
+    $new_args['Bulk Loading Job'] = l($node->title, 'node/' . $args[0]);
     return $new_args;
   }
-  
+
 }
 

Some files were not shown because too many files changed in this diff