refactor: remove unused variable, improve doc comment

Author: Aryan-R-Patel

.github/workflows/ci.yml

@@ -12,4 +12,4 @@ jobs:
     uses: digitalutsc/reusable_workflows/.github/workflows/islandora-module-ci.yml@main
     with:
       module_name: islandora_iiif
-      install_chromedriver: false
+      install_chromedriver: false

islandora_iiif.info.yml

@@ -4,4 +4,4 @@ description: 'IIIF support for Islandora'
 core_version_requirement: ^9 || ^10 || ^11
 package: Islandora
 dependencies:
-  - islandora_lite:termwithuri_condition
+  - islandora_lite:termwithuri_condition

src/Plugin/views/style/IIIFManifest.php

@@ -290,8 +290,7 @@ protected function getTileSourceFromRow(ResultRow $row, $iiif_address, $iiif_bas
       if (isset($entity->{$viewsField->definition['field_name']})) {
         /** @var \Drupal\Core\Field\FieldItemListInterface $images */
         $images = $entity->{$viewsField->definition['field_name']};
-        // phpcs:ignore -- Unused variable $i.
-        foreach ($images as $i => $image) {
+        foreach ($images as $image) {
           if (!$image->entity->access('view')) {
             // If the user does not have permission to view the file, skip it.
             continue;
@@ -491,14 +490,10 @@ protected function getOcrUrl(EntityInterface $entity, int $id) {
   /**
    * Gets nodes that a media belongs to.
    *
-   * // phpcs:ignore -- Doc comment for parameter $media does not match actual variable name $id
-   * @param \Drupal\media\MediaInterface $media
-   *   The Media whose node you are searching for.
+   * @param \Drupal\Core\Entity\EntityInterface $entity
+   *   The entity.
    * @param int $id
    *   The ID of the book node.
-   * // phpcs:ignore -- Doc comment for parameter $entity does not match actual variable name <undefined>
-   * @param EntityInterface $entity
-   *   The entity.
    *
    * @return \Drupal\node\NodeInterface
    *   Parent node.