Merge pull request #10002 from bozana/9946-3_4_0

#9946 Do not remove numbers from the beginning of each cit…
pkp · Jun 1, 2024 · facf529 · facf529
2 parents 4a0a09e + 7bc8876
commit facf529
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/classes/citation/CitationListTokenizerFilter.php b/classes/citation/CitationListTokenizerFilter.php
@@ -56,10 +56,8 @@ public function &process(&$input)
         } else {
             $citations = explode("\n", $input);
         }
-        // 4) Remove numbers from the beginning of each citation.
-        foreach ($citations as $index => $citation) {
-            $citations[$index] = PKPString::regexp_replace('/^\s*[\[#]?[0-9]+[.)\]]?\s*/', '', $citation);
-        }
+        // 4) Remove whitespace from the beginning and the end of each citation.
+        $citations = array_map('trim', $citations);
 
         return $citations;
     }

diff --git a/tests/classes/citation/CitationListTokenizerFilterTest.php b/tests/classes/citation/CitationListTokenizerFilterTest.php
@@ -31,10 +31,10 @@ public function testCitationListTokenizerFilter()
         $tokenizer = new CitationListTokenizerFilter();
         $rawCitationList = "\t1. citation1\n\n2 citation2\r\n 3) citation3\n[4]citation4";
         $expectedResult = [
-            'citation1',
-            'citation2',
-            'citation3',
-            'citation4'
+            '1. citation1',
+            '2 citation2',
+            '3) citation3',
+            '[4]citation4'
         ];
         self::assertEquals($expectedResult, $tokenizer->process($rawCitationList));