Problem with <first_page> and <last_pages>

Software Support
1

Dear colleagues,
I have a problem with the display of the first page and the last page in the search Crossref. We are seen some OJS the first page and the last page.

For example

My journal

We don’t have “… to …”. How i can change it?
Kind regards,
Artem

2

Try to change:
XMLCustomWriter::createChildWithText($doc, $pageNode, ‘other_pages’, $otherPages);
to:
XMLCustomWriter::createChildWithText($doc, $pageNode, ‘last_page’, $otherPages);
in: plugins/importexport/crossref/classes/CrossRefExportDom.inc.php

And then export DOI metadata again.
The ‘other_pages’ is apparently not understood by Crossref.

1 Like
3

This is the first I tried it and I got the error: [Error]: 2: 3824: cvc-complex-type.2.4.a: Invalid content was found starting with element ‘last_pages’. One of ‘{ “http://www.crossref.org/schema/4.3.6":last_page," http://www.crossref.org/schema/4.3.6”:other_pages}’ is expected.

4

You put “last_pages”, I wrote “last_page”.

5

Oh my God!!!
Thank you, its working! My inattention…

6

It does not get what we need …
An error has disappeared, but the range of pages did not appear after update…

7

See also this old forum post:
https://pkp.sfu.ca/support/forum/viewtopic.php?f=8&t=12529&p=52679#p52679
And this languishing issue report:

There is a larger problem with our compliance with CrossRef’s expectation for page numbering. It is made complex by the large number of plugins which all use the pages field.

8

this issue seems to have been fixed since v2.4.8-3:

https://github.com/pkp/ojs/pull/1126

  * @file plugins/importexport/crossref/classes/CrossRefExportDom.inc.php 

 		/* publisher_item is the article pages */
-		if ($article->getPages() != '') {
-			$pageNode =& XMLCustomWriter::createElement($doc, 'pages');
-			// extract the first page for the first_page element, store the remaining bits in otherPages,
-			// after removing any preceding non-numerical characters.
-			if (preg_match('/^[^\d]*(\d+)\D*(.*)$/', $article->getPages(), $matches)) {
-				$firstPage = $matches[1];
-				$otherPages = $matches[2];
+		// CrossRef requires first_page and last_page of any contiguous range, then any other ranges go in other_pages
+		$pages = $article->getPageArray();
+		if (!empty($pages)) {
+			$firstRange = array_shift($pages);
+			$firstPage = array_shift($firstRange);
+			if (count($firstRange)) {
+				// There is a first page and last page for the first range
+				$lastPage = array_shift($firstRange);
+			} else {
+				// There is not a range in the first segment
+				$lastPage = '';
+			}