Problem/Motivation

Getting Warnings while using JSON:API endpoint

Warning: Undefined array key "missing" in Drupal\project_browser\Plugin\ProjectBrowserSource\DrupalDotOrgJsonApi->getProjects() (line 372 of project_browser/src/Plugin/ProjectBrowserSource/DrupalDotOrgJsonApi.php)

Warning: Trying to access array offset on null in Drupal\project_browser\Plugin\ProjectBrowserSource\DrupalDotOrgJsonApi->getProjects() (line 372 of modules/contrib/project_browser/src/Plugin/ProjectBrowserSource/DrupalDotOrgJsonApi.php)

Steps to reproduce

  • Go to /admin/modules/browse/drupalorg_jsonapi
  • Search Editoria11y Accessibility Checker in search box.
  • Visit /admin/reports/dblog

Proposed resolution

Issue fork project_browser-3506377

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

narendrar created an issue. See original summary.

arunsahijpal’s picture

arunsahijpal commented
Assigned: Unassigned » arunsahijpal

Working on it.

arunsahijpal opened merge request !740

arunsahijpal’s picture

arunsahijpal commented
Assigned: arunsahijpal » Unassigned
Status: Active » Needs review

@narendrar, pls check

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs review » Needs work

This probably needs to be rebased/adjusted after #3498835: Feature: add plugin or option to show only locally installed/available modules. Even in 2.0.x HEAD right now, I no longer see any mention of a missing key in DrupalDotOrgJsonApi or its parent. So I'm not really clear on what's actually causing this. I'm guessing we need @fjgarlin's wisdom here.

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Status: Needs work » Needs review
fjgarlin’s picture

fjgarlin commented
Status: Needs review » Reviewed & tested by the community

The "missing" value is coming from jsonapi. It seems that the "media" referenced in the "node" is somehow not migrated. We'll need to review the data, but the fix here won't hurt in any case.

chrisfromredfin made their first commit to this issue’s fork.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Reviewed & tested by the community » Fixed

easy

Status: Fixed » Closed (fixed)

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