Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ошибка в фильтрации номера журнала и несколько предложений по коду #3

Open
vodingithub opened this issue Apr 14, 2023 · 1 comment

Comments

@vodingithub
Copy link

Здравствуйте.
Спасибо за плагин, отличная вещь!
Обнаружили в коде несколько ошибок. С гитхабом не дружу, приведу здесь.
В фильтре 563 строка и plugins\importexport\rsciexport\RSCIExportPlugin.inc.php 158 строка

Не верно указано название номера 'issueId' , а должно быть 'issueIds' . В результате hook не может понять что за номер и проверяет все!!! У нас это занимает много минут.
protected function _getPublications($issue)
{
$submissionsIterator = Services::get('submission')->getMany([
'contextId' => $this->_getContext()->getId(),
'issueIds' => [$issue->getId()] //Vodin
]);

Мы в журнале храним PDF в отдельном хранилище, т.к. по факту мы руками указываем URL на файл. А у вас он формируется.
Вероятно, это нужно как-то разрулить. Приведу наш вариант, но не для всех он будет подходить. 394 строка в фильтре

foreach ($galleys as $galley) {
$resourceURL = $galley->getRemoteURL();
//$file = $galley->getFile();
if (!$resourceURL) {
continue;
}

        $furlNode = $doc->createElement('furl', $resourceURL);
        $furlNode->setAttribute('location', 'publisher');
        $furlNode->setAttribute('version', 'published');
        $filesNode->appendChild($furlNode);

            $galleys = $publication->getData('galleys');    $file = basename($resourceURL);
        $fileNode = $doc->createElement('file', $file);
        $fileNode->setAttribute('desc', 'fullText');
        $filesNode->appendChild($fileNode);
    }   

В фильтре для online журналов неверно указан префикс essn. Правильный вариант ниже
$essn = $journal->getData('onlineIssn');
if ($essn != '')
$journalNode->appendChild($doc->createElement('eissn', $essn));

Инструментарий для разработчика на маркапе выдает ошибку

еще раз спасибо за плагин.

@rodionstepanov
Copy link

@vodingithub Спасибо за исправления! А у вас под верстей OJS 3.4 работает?

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

No branches or pull requests

2 participants