a bug in the article export?

[elicohen]

I believe that I found a bug in ExportArticle. While testing, I created a test article with two authors, "eli cohen", and "Author2 lastname". I made the second author the primary contact. (See http://journals.isjournals.org/index.php/testjournal/article/view/3 and http://find.isjournals.org/viewrecord.php?id=3). But the XML file that comes out lists the second author twice and does not list the primary author at all:
<article>
<title>A third Test paper</title>
-
<abstract>
Studies in gender have offered many reasons for the differing attitudes and ...
</abstract>
<date_published>2005-10-14</date_published>
-
<author primary_contact="true">
<firstname>Author2</firstname>
<lastname>LastName</lastname>
<email>test@1is.us</email>
<biography>second author bio</biography>
</author>
-
<author primary_contact="true">
<firstname>Author2</firstname>
<lastname>LastName</lastname>
<email>test@1is.us</email>
<biography>second author bio</biography>
</author>
-
<indexing>
<discipline>junk science;</discipline>
</indexing>
<pages>1-20</pages>
-
<galley>
...
...

[asmecher]

Hello Eli,

This appears to be a reference oddity that affects certain versions of PHP; see http://pkp.sfu.ca/bugzilla/show_bug.cgi?id=1899 for the bug report and http://research2.csci.educ.ubc.ca/cgi-bin/cvsweb/ojs2/plugins/importexport/native/NativeExportDom.inc.php.diff?r1=1.6.2.1;r2=1.6.2.2 for a patch to correct.

Regards,
Alec Smecher
Open Journal Systems Team

[leo]

Where/how should the patch be applied?

[asmecher]

Hello Leo,

Since this is literally a single-character change, it's easy enough to apply by hand. Edit plugins/importexport/native/NativeExportDom.inc.php, go to approx. line 147. You should fine a line that reads:

Code: Select all

$authorNode = NativeExportDom::generateAuthorDom($doc, $journal, $issue, $article, $author);

Insert an ampersand after the equals sign to make it read:

Code: Select all

$authorNode =& NativeExportDom::generateAuthorDom($doc, $journal, $issue, $article, $author);

This should correct the problem.

Regards,
Alec Smecher
Open Journal Systems Team