? 685942-1_feeds_fixes.patch
Index: FeedsIcalDateParser.inc
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/parser_ical/FeedsIcalDateParser.inc,v
retrieving revision 1.4
diff -u -p -r1.4 FeedsIcalDateParser.inc
--- FeedsIcalDateParser.inc	15 Jan 2010 10:36:23 -0000	1.4
+++ FeedsIcalDateParser.inc	15 Jan 2010 18:17:03 -0000
@@ -2,7 +2,8 @@
 // $Id: FeedsIcalDateParser.inc,v 1.4 2010/01/15 10:36:23 ekes Exp $
 
 /**
- * Overridden version of FeedsDateTimeElement that supports iCal specific parsing and repetition.
+ * Overridden version of FeedsDateTimeElement that supports iCal specific
+ * parsing and repetition.
  *
  * @todo repeating date storage not iCal specific, could be moved to base class.
  */
@@ -47,7 +48,8 @@ class FeedsIcalDateTimeElement extends F
        * Make sure we don't end up with thousands of values with RRULES
        * that have no UNTIL or COUNT.
        * @todo could be adjusted or made configurable later.
-       * NOTE: This is not properly timezone converted; that's the least of its problems.
+       * NOTE: This is not properly timezone converted; that's the least of its
+       * problems.
        */
       $max = date_now();
       $max_repeats = 52;
@@ -63,7 +65,8 @@ class FeedsIcalDateTimeElement extends F
       elseif (empty($form_values['UNTIL'])) {
         $form_values['UNTIL'] = array('datetime' => $until, 'tz' => 'UTC');
       }
-      // Save these in the form_values format, which date can convert to an rrule with date_api_ical_build_rrule()
+      // Save these in the form_values format, which date can convert to an
+      // rrule with date_api_ical_build_rrule()
       $this->repeat_vals = $form_values;
     }
   }
Index: tests/feeds_mapper_ical.test
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/parser_ical/tests/feeds_mapper_ical.test,v
retrieving revision 1.1
diff -u -p -r1.1 feeds_mapper_ical.test
--- tests/feeds_mapper_ical.test	15 Jan 2010 10:36:23 -0000	1.1
+++ tests/feeds_mapper_ical.test	15 Jan 2010 18:17:03 -0000
@@ -18,9 +18,9 @@ class FeedsMapperiCalTestCase extends  F
 
   public static function getInfo() {
     return array(
-      'name' => t('iCal Parser'),
-      'description' => t('Test Feeds iCal parser support and date mapping.'),
-      'group' => t('Feeds'),
+      'name' => t('Feeds integration'),
+      'description' => t('Test Feeds iCal parser support and date mapping. <strong>Requires Feeds and Date module.</strong>'),
+      'group' => t('iCal Parser'),
     );
   }
 
@@ -102,7 +102,11 @@ class FeedsMapperiCalTestCase extends  F
     }
   }
 
-protected function getFormFieldsNames($field_name, $index) {
+  /**
+   * Return the form field name for a given CCK field. Special handling for date
+   * fields.
+   */
+  protected function getFormFieldsNames($field_name, $index) {
     if (in_array($field_name, array('date', 'datetime', 'datestamp', 'eventdate', 'eventdate2'))) {
       return array("field_{$field_name}[{$index}][value][date]");
     }