Comments

vaplas created an issue. See original summary.

vaplas’s picture

I copied the idea of checking from Drupal\rest\Plugin\rest\resource\EntityResource and Drupal\KernelTests\Core\Http\LinkRelationsTest. Also, please add a credit to @Wim Leers, who found this bug and provided a fix and a test in #2835767-46: Media + REST: comprehensive test coverage for Media + MediaType entity types.

Added tag 'blocker', because #2835767: Media + REST: comprehensive test coverage for Media + MediaType entity types.

The last submitted patch, 2: 2905720-2-test-only.patch, failed testing. View results

vaplas’s picture

Sorry, incorrect test-only patch (without "new file mode 100644"). Reupload both patches (to save NR status).

The last submitted patch, 4: 2905720-4-test-only.patch, failed testing. View results

seanB’s picture

Status: Needs review » Reviewed & tested by the community

Looks good! Thanks.

  • larowlan committed 8cec5c8 on 8.5.x
    Issue #2905720 by vaplas: Media specifies a add-page link relation, but...

  • larowlan committed 9237a9e on 8.4.x
    Issue #2905720 by vaplas: Media specifies a add-page link relation, but...

larowlan credited larowlan.

larowlan’s picture

Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed as 8cec5c8 and pushed to 8.5.x.
Cherry-picked as 9237a9e and pushed to 8.4.x.

Status: Fixed » Closed (fixed)

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