feeds 7.x-2.0-beta5

Improvements for field validation issues; account switch to the feed author or user 1; better PHP 7.2+ support; feeds_alter() removed.

Major changes in this release

Imports are ran as the feed author or user 1

When running imports, Feeds now switches to a different user. This is done to reduce field validation issues and to minimize the difference in (validation) results between importing using the UI and importing using cron.

• When attaching the importer to a content type, Feeds switches to the author of the feed node. If the author is anonymous, Feeds switches to user 1 instead.
• When using the standalone import form, Feeds switches to user 1.
• When the "Authorize" option is checked (setting for the node processor), an extra account switch happens for every item that gets imported. Feeds then switches to the author of the item.

While test coverage exists to make sure Feeds switches back after it's done, this change could in theory have regressions.

Improvements for field validation issues

Some improvements have been made to fix common field validation issues (that were introduced in 7.x-2.0-beta4). The following improvements have been made:

• Feeds now only validates mapped fields.
• Less validation issues for Organic Groups reference fields.
• A hook called hook_feeds_prevalidate() has been added to workaround validation issues: with that hook you can fix up the entity being imported right before it is validated.

PHP 7.2+ support

Most issues have been fixed that caused PHP notices or warnings on PHP 7.2 and above. You do may still get PHP notices/warnings when using the Date module. These issues may be fixed in the Date 7.x-2.11-beta3 release however. If not, please help on #2867810: Plan for Date 7.x-2.11 release!

Function feeds_alter() is removed

This function was deprecated 6 years ago and was removed because it did not work properly on recent PHP versions.

The signature of FeedsProcessor::entityValidate() changed

The signature of the method FeedsProcessor::entityValidate() changed. It now has a second optional parameter. If you override this method in your module, you need to update the parameters so that it matches with FeedsProcessor::entityValidate().
See the change record for details.

Contributors (49)

MegaChriz, RavindraSingh, joelpittet, twistor, _Geertje, NWOM, DamienMcKenna, Spokje, ravikk-drupal, Rajat Charde, knyshuk.vova, Dave Reid, sdrycroft, devdesagar, cosmicdreams, ruyakhokhar, Karishma Rawat, pcambra, colan, febbraro, robert.laszlo, johnv, naught101, BarisW, yched, pbuyle, emptyvoid, VladimirAus, ansergeyg, bwood, mradcliffe, wylbur, blackbull77, jacob.embree, GuyPaddock, rsbecker, dpw, Toki, dat deaf drupaler, markusd1984, Jason Dean, sicher, donquixote, walkerasindave, rooby, Anas_maw, jesss, Cyrilovich, michel.g

Changelog

Issues: 25 issues resolved.

Changes since 7.x-2.0-beta4:

Bug

• #2342143 by MegaChriz, RavindraSingh, joelpittet, twistor, _Geertje, NWOM, DamienMcKenna: Fixed a lot of coding standard issues.
• #3164491 by joelpittet: PHP 7.3: test failures.
• #1139676 by Dave Reid, sdrycroft, devdesagar, cosmicdreams, Rajat Charde, ruyakhokhar, Karishma Rawat, pcambra, colan, febbraro, MegaChriz, twistor, robert.laszlo, johnv, naught101, BarisW, yched, pbuyle, emptyvoid: Removed feeds_alter().
• #3013743 by VladimirAus, ansergeyg, Karishma Rawat, MegaChriz, bwood, mradcliffe, wylbur, joelpittet, blackbull77: PHP 7.3: 'continue' used within 'switch' control structure.
• #3121039 by jacob.embree: Fixed PHP 7.4 deprecation: Replaced curly brace syntax for accessing string offset in http_request_create_absolute_url().
• #3091682 by GuyPaddock, MegaChriz: Fixed term autocreate caching when multiple term reference fields point to the same vocabulary.
• #2911711 by MegaChriz: Inform the user Feeds must be enabled in order to run database update 7213.
• #2922268 by MegaChriz, rsbecker, dpw: Fixed validation issues when importing values for an og_reference field.
• #2980096 by MegaChriz, sicher: Fixed EntityMalformedException that could occur when creating/editing feeds importer and the module i18n_node was enabled.
• #2917703 by MegaChriz: Fixed import dates in Zulu time zone, using the date format 'YYYYMMDDTHHMMSSZ' (failing on PHP < 5.5.10).
• #2943953 by MegaChriz: Fixed fatal error "No more accounts to revert to." when trying to unlock a feed.
• #2943805 by MegaChriz, NWOM: Fixed multiple warnings: Declaration Of ~ should be compatible with FeedsConfigurable.
• #2541944 by MegaChriz, walkerasindave: Switch to feed author or user 1 during imports (fixed taxonomy mapping does not work with cron).
• #2921027 by MegaChriz: fixed failing tests due to SimplePie download file missing.
• #2563865 by Anas_maw, jesss, MegaChriz: feeds_source return empty values when using views.
• #2918931 by MegaChriz, Cyrilovich: fixed basic authorization failed with 401 in http_request.inc.
• #2914119 by michel.g, MegaChriz: Added a check for existing content types in the FeedsWebTestCase class.

Feature

• #3046592 by MegaChriz: Added a hook to invoke before validation: hook_feeds_prevalidate().
• #2446307 by MegaChriz: Added rules feeds import action.
• #2939042 by rooby: Use actions wrapper for buttons on standalone import form.

Task

• #3104226 by Spokje, ravikk-drupal, Rajat Charde, knyshuk.vova: Make phpcs ignore safe_mode.
• #3113851 by MegaChriz: Fixed failing test FeedsMapperLinkTestCase::testLinkFieldValidation().
• #3046797 by MegaChriz: Switch to item author when the "Authorize" option is checked.
• #2918840 by MegaChriz, Toki, dat deaf drupaler, markusd1984, Jason Dean: Only validate mapped fields.
• #2490782 by MegaChriz, donquixote: Added parameter types for a few docblocks and some other code style fixes.