Automated Drupal 9 compatibility fixes

================================================================================
Feeds Paragraphs, --
Scanned on Thu, 01/26/2023 - 11:44

FILE: web/modules/contrib/feeds_para_mapper/feeds_para_mapper.info.yml

STATUS         LINE                           MESSAGE                           
--------------------------------------------------------------------------------
Check manually 0    Add core_version_requirement: ^8 || ^9 to designate that the
                    extension is compatible with Drupal 9. See                  
                    https://drupal.org/node/3070687.                            
--------------------------------------------------------------------------------

FILE: web/modules/contrib/feeds_para_mapper/composer.json

STATUS         LINE                           MESSAGE                           
--------------------------------------------------------------------------------
Check manually 0    The drupal/core requirement is not compatible with the next 
                    major version of Drupal. Either remove it or update it to be
                    compatible. See                                             
                    https://drupal.org/node/2514612#s-drupal-9-compatibility.   
--------------------------------------------------------------------------------

This still needs work for D9!

To test the MR, kindly update your composer.json's repositories with the following:

    "repositories": [
        {
            "type": "composer",
            "url": "https://packages.drupal.org/8",
            "exclude": [
                "drupal/feeds_para_mapper"
            ]
        },
        {
            "type": "git",
            "url": "https://git.drupalcode.org/issue/feeds_para_mapper-3147462.git"
        }
    ],

After that execute, composer require 'drupal/feeds_para_mapper:dev-3147462-automated-drupal-9'. Ensure that the minimum-stability in your composer.json is set to dev.

I want co-maintainership and I promise to abide by the conditions of the Open Ownership Pledge as described. I have provided patches for one of the corresponding issues.

Hey Rishi,
Thanks for taking point. I am currently pulling in using your comment on https://www.drupal.org/project/feeds_para_mapper/issues/3147462#comment-... 📌 Automated Drupal 9 compatibility fixes Needs review

I see the original maintainer hasn't checked PR in 3 months! Hoping this breathes new life into this module!

Cheers.

is that the right format for adding the repo? I use this:

    "repositories": {
        "0": {
            "type": "composer",
            "url": "https://packages.drupal.org/8",
            "exclude": [
                "drupal/feeds_para_mapper"
            ]
        },
        "feeds_para_mapper": {
            "type": "git",
            "url": "https://git.drupalcode.org/issue/feeds_para_mapper-3147462.git"
        }
    }

doing this and the composer require listed in #7 and module installs.

Throws error on most Feed type setup pages though:

The website encountered an unexpected error. Please try again later.
TypeError: in_array(): Argument #2 ($haystack) must be of type array, null given in in_array() (line 64 of modules\contrib\feeds_para_mapper\src\Mapper.php).

in_array('entity_reference', NULL) (Line: 64)
Drupal\feeds_para_mapper\Mapper->getTargets('node', 'department') (Line: 92)
Drupal\feeds_para_mapper\Feeds\Target\WrapperTarget::targets(Array, Object, Array) (Line: 305)

Also noticed before adding this module that there is already something in Feeds core which seems to be trying to map multi paragraphs - but doesnt quite work. Wondering if this work is being repeated by the core module?

🐛 Argument #2 ($haystack) must be of type array, null given in in_array() Fixed

https://www.drupal.org/project/feeds_para_mapper/releases/8.x-1.2-rc1 →

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