Never Pass FeedsDateTime objects into date_create

Works for me and at least one other person.

+1
Works for me as well.

Feeds wasn't able to import or map dates from a CiviCrm ical event feed (version 4.2.2). This fixed that.

The code change appears solid and contained.

I think I've fixed this. I just went ahead and made the check for a string, since that's what date_create() expects.
Also fixed multivalue handling while I was in there.

http://drupalcode.org/project/feeds.git/commit/7e6679d

I realize this patch is for 7.x-2.x-dev, but I applied the patch in #4 to 7.x-2.0-alpha8 and it worked for me. Thanks!

Marked #1988562: Begin and end date not beeing parsed (iCal from google calendar) and #1669964: Multiple Values and Empty Values in Date Fields as duplicates of this issue.

Please forgive me if I'm just reading this wrong, but the updated function confuses me:

function date_feeds_set_target($source, $entity, $target, $feed_element) {
  list($field_name, $sub_field) = explode(':', $target, 2);

  if (!is_array($feed_element)) {
    $feed_element = array($feed_element);
  }

  $delta = 0;
  foreach ($feed_element as $f) {

    if (!($feed_element instanceof FeedsDateTimeElement)) {

      if (empty($f) || !is_numeric($f) && is_string($f) && !date_create($f)) {
        $f = new FeedsDateTimeElement(NULL, NULL);
      }
      elseif ($sub_field == 'end') {
        $f = new FeedsDateTimeElement(NULL, $f);
      }
      else {
        $f = new FeedsDateTimeElement($f, NULL);
      }
    }

    $f->buildDateField($entity, $field_name, $delta);
    $delta++;
  }
}

Right near the top, it checks if $feed_element is an array, and makes it an array if it's not. And with the new change, it later checks if $feed_element is a FeedsDateTimeElement, which it can never be because of the above array code.

Shouldn't that second if be checking $f instead of $feed_element?
if (!($f instanceof FeedsDateTimeElement)) {

Same here: not sure about how to understand this piece of code, but testing in the enumeration loop an impossible property of a parent element seems - at least - a bit strange…
I've tried here with the change proposed in #9, it works too…

Here's the patch I've been using to fix this issue. It's essentially the same as the commit linked in #4, but with the $feed_element -> $f fix applied.

Oops. The patch in #11 is for alpha8, not the current dev.

Just a note, the bug reported in #9, and patched in #11 was fixed a while ago, but this issue is left open until we get a new release since the patch is for alpha-8.

now beta4, I close this according to #13