Merge branch 'rollbackRelease' into 'main'

Rollback release

See merge request softwares-pkp/plugins_ojs/doiscielo!28

Author: YvesLepidus

ScieloScreeningPlugin.php

@@ -41,8 +41,6 @@ public function register($category, $path, $mainContextId = null)
 
         if ($success && $this->getEnabled($mainContextId)) {
             Hook::add('Form::config::after', [$this, 'editFormComponents']);
-            Hook::add('preprintgalleyform::display', [$this, 'modifyGalleyForm']);
-            Hook::add('preprintgalleyform::validate', [$this, 'modifyGalleyFormValidation']);
             Hook::add('TemplateManager::display', [$this, 'modifySubmissionSteps']);
             Hook::add('Submission::validateSubmit', [$this, 'validateSubmissionFields']);
             Hook::add('Template::SubmissionWizard::Section::Review', [$this, 'modifyReviewSections']);
@@ -166,52 +164,6 @@ private function removePrefixAndSubtitleFields($formConfig)
         return $formConfig;
     }
 
-    public function modifyGalleyForm($hookName, $params)
-    {
-        $request = Application::get()->getRequest();
-        $templateMgr = TemplateManager::getManager($request);
-
-        $templateMgr->registerFilter("output", [$this, 'removeFieldsFromGalleyFormFilter']);
-    }
-
-    public function removeFieldsFromGalleyFormFilter($output, $templateMgr)
-    {
-        if (preg_match('/id="preprintGalleyForm"/', $output)) {
-            preg_match('/<\/form>/', $output, $matches, PREG_OFFSET_CAPTURE);
-
-            $posMatch = $matches[0][1];
-            $removeGalleyFields = $templateMgr->fetch($this->getTemplateResource('removeGalleyFields.tpl'));
-            $output = substr_replace($output, $removeGalleyFields, $posMatch, 0);
-
-            $templateMgr->unregisterFilter('output', array($this, 'removeFieldsFromGalleyFormFilter'));
-        }
-
-        return $output;
-    }
-
-    public function modifyGalleyFormValidation($hookName, $params)
-    {
-        $form = &$params[0];
-        $submission = $form->_submission;
-
-        if (!empty($submission->getData('submissionProgress'))) {
-            return Hook::CONTINUE;
-        }
-
-        $checker = new ScreeningChecker();
-        $galleys = $submission->getGalleys();
-        $galleysFiletypes = array_map(function ($galley) {
-            return ($galley->getFileType());
-        }, $galleys);
-
-        if ($checker->checkNumberPdfs($galleysFiletypes)[0]) {
-            $form->addErrorField('preprintGalleyFormNotification');
-            $form->addError('preprintGalleyFormNotification', __("plugins.generic.scieloScreening.screeningRules.numPdfs"));
-        }
-
-        return Hook::CONTINUE;
-    }
-
     public function modifySubmissionSteps($hookName, $params)
     {
         $request = Application::get()->getRequest();

cypress/tests/Test1_submissionWizard.cy.js

@@ -194,15 +194,6 @@ describe('SciELO Screening Plugin - Submission wizard tests', function() {
         cy.contains('It was not possible to verify the scientific production of the ORCID records, since no PDF document was sent');
         cy.get('.pkpSteps__step button:contains("Upload Files")').click();
 
-        cy.get('a:contains("Add File")').click();
-        cy.get('.pkp_modal_panel').within(() => {
-            cy.contains('Add File');
-            cy.get('label:contains("This galley will be available at a separate website")').should('not.exist');
-            cy.get('input[name="remotelyHostedContent"]').should('not.exist');
-            cy.get('input[name="urlPath"]').should('not.exist');
-            cy.get('a.pkpModalCloseButton').click();
-        });
-
         cy.addSubmissionGalleys([files[0], files[0]]);
         cy.contains('button', 'Continue').click();
         cy.contains('button', 'Continue').click();

cypress/tests/Test2_workflowFeatures.cy.js

@@ -86,23 +86,6 @@ describe('SciELO Screening Plugin - WorkFlow features tests', function() {
         cy.contains('.pkpFormFieldLabel', 'Prefix').should('not.exist');
         cy.contains('.pkpFormFieldLabel', 'Subtitle').should('not.exist');
     });
-    it("Authors can not send multiple PDFs", function () {
-        cy.login('dphillips', null, 'publicknowledge');
-        cy.findSubmission('myQueue', screenedSubmissionTitle);
-
-        cy.contains('button', 'Preprint').click();
-        cy.contains('button', 'Galleys').click();
-
-        cy.contains('a', 'Add File').click();
-        cy.wait(200);
-        cy.get('#preprintGalleyForm').within(() => {
-            cy.get('input[name="label"]').type('PDF', {delay: 0});
-            cy.contains('.submitFormButton', 'Save').click();
-        });
-        cy.reload();
-
-        cy.contains("Only one PDF document should be sent");
-    });
     it("Disables plugin temporarily", function () {
         cy.login('dbarnes', null, 'publicknowledge');
 		cy.contains('a', 'Website').click();

templates/removeGalleyFields.tpl

@@ -8,8 +8,8 @@
 <version>
 	<application>scieloScreening</application>
 	<type>plugins.generic</type>
-	<release>2.1.4.0</release>
-	<date>2025-10-19</date>
+	<release>2.1.4.1</release>
+	<date>2025-10-22</date>
 	<lazy-load>1</lazy-load>
 	<class>ScieloScreeningPlugin</class>
 </version>