Problem/Motivation

If the `field_c_n_thumbnail` has been modified by a site to have more than `civictheme_image`, we need to check that the media entity is indeed an image entity before we add as the thumbnail for reference cards / components.

Steps to reproduce

Proposed resolution

1. Add check for the media entity.

Remaining tasks

None - PR is coming.

Comments

richardgaunt created an issue. See original summary.

richardgaunt’s picture

richardgaunt commented
Assigned: richardgaunt » joshua1234511

@joshua12345 could you please review:

PR: https://github.com/civictheme/monorepo-drupal/pull/1467

joshua1234511’s picture

joshua1234511
Primary language English
Location Goa
commented
Assigned: joshua1234511 » Unassigned
Status: Needs review » Reviewed & tested by the community

Reviewed the PR, LGTM

  • 845816b2 committed on 1.x 
    #3573925 Added media source check for thumbnail. (#1467)
richardgaunt’s picture

richardgaunt commented
Version: 1.x-dev » 1.13.0
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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