Merge pull request #48 from lepidus/main

fix/use chapter pages field for page interval instead page count (OMP 3.4.0)

Author: thiagolepidus

classes/factories/ThothChapterFactory.inc.php

@@ -29,6 +29,8 @@ public function createFromChapter($chapter)
         $submission = Repo::submission()->get($publication->getData('submissionId'));
         $context = Application::getContextDAO()->getById($submission->getData('contextId'));
 
+        $pages = $this->extractPages($chapter);
+
         return new ThothWork([
             'workType' => ThothWork::WORK_TYPE_BOOK_CHAPTER,
             'workStatus' => $this->getWorkStatusByDatePublished($chapter, $publication),
@@ -37,7 +39,9 @@ public function createFromChapter($chapter)
             'subtitle' => $chapter->getLocalizedData('subtitle'),
             'longAbstract' => HtmlStripper::stripTags($chapter->getLocalizedData('abstract')),
             'doi' => $chapter->getData('doiObject')?->getResolvingUrl(),
-            'pageCount' => !empty($chapter->getPages()) ? (int) $chapter->getPages() : null,
+            'pageInterval' => $pages['pageInterval'] ?? null,
+            'firstPage' => $pages['firstPage'] ?? null,
+            'lastPage' => $pages['lastPage'] ?? null,
             'publicationDate' => $chapter->getDatePublished() ?? $publication->getData('datePublished'),
             'landingPage' => $request->getDispatcher()->url(
                 $request,
@@ -60,4 +64,26 @@ public function getWorkStatusByDatePublished($chapter, $publication)
 
         return ThothWork::WORK_STATUS_FORTHCOMING;
     }
+
+    private function extractPages($chapter): array
+    {
+        $pages = $chapter->getPages();
+
+        if (empty($pages)) {
+            return [];
+        }
+
+        if (strpos($pages, '-') === false) {
+            return [
+                'firstPage' => trim($pages),
+            ];
+        }
+
+        list($firstPage, $lastPage) = explode('-', $pages);
+        return [
+            'pageInterval' => trim($pages),
+            'firstPage' => trim($firstPage),
+            'lastPage' => trim($lastPage)
+        ];
+    }
 }

tests/classes/factories/ThothChapterFactoryTest.php

@@ -137,7 +137,7 @@ private function setUpMockEnvironment()
             )
             ->shouldReceive('getPages')
             ->withAnyArgs()
-            ->andReturn('31')
+            ->andReturn('31 - 50')
             ->getMock();
 
         $this->mocks = [];
@@ -161,7 +161,9 @@ public function testCreateThothChapterFromChapter()
             'longAbstract' => 'This is my chapter abstract',
             'publicationDate' => '2024-01-01',
             'doi' => 'https://doi.org/10.12345/11112222',
-            'pageCount' => 31,
+            'firstPage' => '31',
+            'lastPage' => '50',
+            'pageInterval' => '31 - 50',
             'landingPage' => 'https://omp.publicknowledgeproject.org/index.php/press/catalog/book/17'
         ]), $thothChapter);
     }

version.xml

@@ -3,8 +3,8 @@
 <version>
 	<application>thoth</application>
 	<type>plugins.generic</type>
-	<release>0.2.8.4</release>
-	<date>2025-08-11</date>
+	<release>0.2.8.5</release>
+	<date>2025-08-15</date>
 	<lazy-load>1</lazy-load>
 	<class>ThothPlugin</class>
 </version>