commenting code

Author: Shae Smith

lib/modules/dosomething/dosomething_campaign/dosomething_campaign.module

@@ -1269,19 +1269,21 @@ function dosomething_campaign_node_update($node) {
  */
 function dosomething_campaign_node_validate($node, $form, &$form_state) {
   if ($node->type == 'campaign') {
+    // Get the language of the translation being submitted.
     $language = $form_state['entity_translation']['form_langcode'];
-    $values = $form_state['values'];
 
+    // Get the current run set in the db and the one the user submitted.
+    // so we can see if it has changed.
     $current_run_id = $form['#node']->field_current_run[$language][0]['target_id'];
-    $submitted_current_run = $values['field_current_run'][$language][0]['target_id'];
+    $submitted_current_run = $form_state['values']['field_current_run'][$language][0]['target_id'];
 
     if ($current_run_id !== $submitted_current_run) {
+      // Get the dates of the run.
       $new_run = entity_metadata_wrapper('node', $submitted_current_run);
-      // @TODO -- Extract into function.
       $start_date = new DateTime($new_run->language($language)->field_run_date->value()['value']);
       $end_date = new DateTime($new_run->language($language)->field_run_date->value()['value2']);
+      // Update the campaign status.
       $status = dosomething_helpers_get_campaign_status($start_date, $end_date);
-      // -- end function.
       $form_state['values']['field_campaign_status'][$language][0]['value'] = $status;
     }
   }

lib/modules/dosomething/dosomething_campaign_run/dosomething_campaign_run.module

@@ -299,15 +299,16 @@ function dosomething_campaign_run_form_campaign_run_node_form_alter(&$form, &$fo
  */
 function dosomething_campaign_run_node_submit($node, $form, &$form_state) {
   if ($node->type == 'campaign_run') {
+    // Get language of the translation being submittted.
     $language = $form_state['entity_translation']['form_langcode'];
-    // @TODO -- Extracted into function.
+    // Grab the run dates the user just entered;
     $start_date = new DateTime($form_state['values']['field_run_date'][$language][0]['value']);
     $end_date = (!$form_state['values']['field_run_date'][$language][0]['value2']) ? $start_date : new DateTime($form_state['values']['field_run_date'][$language][0]['value2']);
     $status = dosomething_helpers_get_campaign_status($start_date, $end_date);
-    // -- end extraction.
 
     // Just get the first campaign this is tied to. While we allow for multiple campaigns to be selected per campaign run, only one should be selected.
     $campaign = $form_state['values']['field_campaigns'][$language][0];
+    // Update the campaign's status.
     $campaign = entity_metadata_wrapper('node', $campaign['target_id']);
     $current_run = $campaign->language($language)->field_current_run->value();
     if ($current_run->nid == $form_state['values']['nid']) {