InnoPeak/organizrr
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 🐛 use replaceAll to remove spaces instead of replace
All checks were successful
continuous-integration/drone/push Build is passing
Details

Browse Source
This commit is contained in:
RaviAnand Mohabir 2025-04-09 09:50:51 +02:00
parent 1bab67d36c
commit 1e283d8946
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/Organizrr.tsx
View File

@ -98,10 +98,10 @@ function Organizrr() {
?.split(",") ?.split(",")
.some( .some(
(val) => (val) =>
!/^\d+$/.test(val.replace(" ", "")) && !/^\d+$/.test(val.replaceAll(" ", "")) &&
!/^\d+\-\d+$/.test(val.replace(" ", "")) && !/^\d+\-\d+$/.test(val.replaceAll(" ", "")) &&
val.replace(" ", "") !== "odd" && val.replaceAll(" ", "") !== "odd" &&
val.replace(" ", "") !== "even" val.replaceAll(" ", "") !== "even"
) )
) { ) {
return "Page numbers must be given as a comma separated list"; return "Page numbers must be given as a comma separated list";
@ -227,7 +227,7 @@ function Organizrr() {
documents: f.documents.map((d) => { documents: f.documents.map((d) => {
const selectedPages = d.selectedPages const selectedPages = d.selectedPages
?.split(",") ?.split(",")
.map((sp) => sp.replace(" ", "")); .map((sp) => sp.replaceAll(" ", ""));
return { return {
...d, ...d,