Problem/Motivation

It's been a long time since the last release (7.x-2.0-alpha8, release in April 2013), so it's time for a new release. However, there are some experimental things in dev that are either not yet properly working or haven't been tested well enough. These are:

There are also some issues that are near ready, but not necessary blockers for the next release:

Proposed resolution

A plan to fix issues with the generic entity processor must be written. Already existing issues about the generic entity processor in the queue must be collected. For clearing out issues automated tests must be written to ensure expected behavior for each mapping target that Feeds provides.

Remaining tasks

  • Write a plan to complete/fix the generic entity processor
  • Fix at least the most important issues of the generic entity processor
  • Write automated tests for the generic entity processor
  • Write automated tests for "Clear out values when source is empty"

Original report by AdelaideTech Jesse

Hi,

This is probably the wrong place to ask, but I read the guidelines and couldn't see where to put it, apologies.

I hit this issue : https://www.drupal.org/node/2153379

It was fixed in the current dev version ages ago. I just want to ask if a new recommended release could include this fix please. I have resolved my issue by upgrading to dev but I would rather stick with recommended (currently 7.x-2.0-alpha8) so that others can avoid this issue.

Regards
Jesse

Comments

megachriz’s picture

Status: Active » Postponed

There are some experimental things in dev as well, like:

These still need some finetuning/testing. I think that must be done first to not scare off the majority of users.

Then there are also some issues that are follow-ups of things that get in. I don't know these out the top of my head.

twistor’s picture

Title: Create new release with existing fixes » META: Create alpha 9 release.
Component: Code » Miscellaneous
Assigned: Unassigned » twistor
Category: Feature request » Task
Status: Postponed » Active

We should do this. As MegaChriz noted we have some things to sort out. Those issues should be link to this one.

megachriz’s picture

Title: META: Create alpha 9 release. » [Meta] Create alpha 9 release
Issue summary: View changes

I've updated the issue summary.

sonicthoughts’s picture

Hi - was wondering if #1107522: Framework for expected behavior when importing empty/blank values + text field fix should still be a blocker. It seems the patch was rolled into dev and there is some ambiguity on the functional spec of how missing values should be handled rather than faulty code. I realize there are many wanting to move deeds to stable but just wondering if the blockers/roadmap might be revisited ...

Frankly, I'm also wondering if I should use dev rather than alpha08 given so much work on the branch ....

megachriz’s picture

Marked #2177965: Release a stable version as a duplicate.

@sonicthoughts
It's true that a patch for #1107522: Framework for expected behavior when importing empty/blank values + text field fix was committed, but it is still lacking tests. At this point, it's not certain if emptying values works as expected for every mapper that Feeds provides.
There are indeed many fixes in dev, you should use it if you need this fixes. Keep in mind that if you use the generic entity processor, configuration could get broken when you update Feeds later.

sonicthoughts’s picture

Hi Thanks @MegaChriz - appreciate the response/update.

I can see lot's of activity by folks to move feeds to a stable release. Please consider folks like myself who are system builders, not developers. It can get scary juggling patches and later facing config changes with updates of dev branch. I guess this is a broader issue with drupal module support but please have us mortals in mind when making choices to patch dev vs. making incremental releases as the former can require a lot of dev skill to support. Appreciate the efforts and responsiveness - and of course such a powerful module.

scotwith1t’s picture

It's somewhat understandable, but really, folks understand that when they DL an alpha release that it still has issues and even experimental code in it. To go over 2 years without a new release is crazy and worse for site-builders who, in most cases, shy away from using -dev versions on any live site. Alphas are risky too, but I rarely use a dev version in prod. I may have to make an exception here because 2 years of fixes is a lot to miss out on...what about pulling out the experimental stuff if that's the primary concern?

sonicthoughts’s picture

just a note that ical dates will only work with alpha 8+ patch and NOT dev. See this comment: #2237177: Apply this patch to Feeds 2.0-alpha8 to fix broken date import

rob230’s picture

I agree that this puts site builders in a difficult position. They have the choice of using an alpha that is over 2 years old, or rolling the dice on the latest dev version, and then deciding whether to update it every time a commit happens. This is a very popular module, there are over 70,000 websites using alpha 8.

Is there anything I can test to expedite a stable release?

twistor’s picture

Title: [Meta] Create alpha 9 release » [Meta] Create beta 1 release
Assigned: twistor » Unassigned
Status: Active » Fixed

I've decided to release a beta1.

Things are very close to a stable release, API-wise I think we're pretty much there. Since we already keep backwards compatibility, there's no problem just biting the bullet.

megachriz’s picture

@twistor
Good idea to make a new release! If it were me, I would have called alpha10 though. This is because when #1183440: Multilingual Feeds - Make field import language-aware lands, modules that provide field mappers need to be adjusted to provide language support.

sonicthoughts’s picture

@MegaChriz - perhaps yes, but given 91K installs it's pretty much in production already....IMO it is far more stable than many beta builds...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.