Problem/Motivation

When running tests on 11.x, I see:

The update to convert "numeric" arguments to "entity_target_id" for entity reference fields for view "dashboard_content" is deprecated in drupal:10.3.0 and is removed from drupal:12.0.0. Profile, module and theme provided configuration should be updated. See https://www.drupal.org/node/3441945

As we can support > 10.3 and 11.x, let's fix this.

Steps to reproduce

ddev exec -d /var/www/html "./vendor/bin/phpunit -c ./core/phpunit.xml.dist ./modules/contrib/dashboard/tests/src/Functional/Form/DashboardFormTest.php"

Proposed resolution

Fix.

Remaining tasks

Fix.

User interface changes

None.

API changes

None. We might want to change core constraints in info.yml and composer.json to be explicit about it.

Data model changes

None.

Issue fork dashboard-3490589

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented

Apparently we fixed this in #3474151: Resolve warning in tests for numeric argument, but actually not.
DrupalCI didn't complain.

penyaskito opened merge request !41

  • penyaskito committed e9f5e541 on 2.x 
    Issue #3490589 by penyaskito: Fix views warnings about deprecations
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Active » Fixed

Verified no warnings locally when running the tests.

The diff is tricky because of encoding issues?, but I verified with git diff 6cdb4a9..c45354c --word-diff

Status: Fixed » Closed (fixed)

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