Skip to content

Fix IIIF manifest output for IIIF Image API v3#1102

Merged
aOelschlager merged 3 commits into
2.xfrom
fix/iiif-v3-manifest-service-metadata
May 13, 2026
Merged

Fix IIIF manifest output for IIIF Image API v3#1102
aOelschlager merged 3 commits into
2.xfrom
fix/iiif-v3-manifest-service-metadata

Conversation

@joecorall
Copy link
Copy Markdown
Member

@joecorall joecorall commented May 3, 2026

What does this Pull Request do?

This updates islandora_iiif manifest generation to emit consistent Image API metadata for both IIIF Image API 2 and 3 services.

What's new?

  • Detect Image API version from the IIIF service URL
  • Emit the correct service @context / profile for:
    • /iiif/2/
    • /iiif/3/
  • Use the correct painted image URL for v3:
    • v2: /full/full/0/default.jpg
    • v3: /full/max/0/default.jpg
  • Add explicit canvas thumbnails
  • Keep painted image format aligned with the declared JPEG derivative URL
  • Extract URL/service helper logic for testability
  • Add regression tests covering v2/v3 service metadata and URL generation

How should this be tested?

A description of what steps someone could take to:

  • Reproduce the problem you are fixing (if applicable)
  • Test that the Pull Request does what is intended.
  • Please be as detailed as possible.
  • Good testing instructions help get your PR completed faster.

Documentation Status

  • Does this change existing behaviour that's currently documented?
  • Does this change require new pages or sections of documentation?
  • Who does this need to be documented for?
  • Associated documentation pull request(s): ___ or documentation issue ___

Additional Notes:

Any additional information that you think would be helpful when reviewing this
PR.

Interested parties

@joecorall joecorall marked this pull request as ready for review May 4, 2026 11:53
@joecorall joecorall requested a review from a team May 4, 2026 11:53
Comment thread modules/islandora_iiif/src/Plugin/views/style/IIIFManifest.php Outdated
@aOelschlager aOelschlager self-requested a review May 6, 2026 17:08
Copy link
Copy Markdown

@aOelschlager aOelschlager left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this and this works.

@aOelschlager aOelschlager merged commit 285ba5b into 2.x May 13, 2026
32 checks passed
@aOelschlager aOelschlager deleted the fix/iiif-v3-manifest-service-metadata branch May 13, 2026 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants