Added comment updates to reflect new naming behaviour
This commit is contained in:
parent
f17ac2c98b
commit
09280166c7
|
@ -3,16 +3,17 @@
|
||||||
* e.g.
|
* e.g.
|
||||||
* name all names result
|
* name all names result
|
||||||
* ------ ----------- --------
|
* ------ ----------- --------
|
||||||
* ("foo") ["foo"] "foo (1)"
|
* ("foo") ["foo"] "foo 1"
|
||||||
* ("foo") ["foo", "foo (1)"] "foo (2)"
|
* ("foo") ["foo", "foo 1"] "foo 2"
|
||||||
* ("foo (1)") ["foo", "foo (1)"] "foo (2)"
|
* ("foo 1") ["foo", "foo 1"] "foo 2"
|
||||||
* ("foo") ["foo", "foo (2)"] "foo (1)"
|
* ("foo") ["foo", "foo 2"] "foo 1"
|
||||||
*
|
*
|
||||||
* Repl
|
* Repl
|
||||||
*/
|
*/
|
||||||
export const duplicateName = (name, allNames) => {
|
export const duplicateName = (name, allNames) => {
|
||||||
const baseName = name.split(" (")[0]
|
const duplicatePattern = new RegExp(`\\s(\\d+)$`)
|
||||||
const isDuplicate = new RegExp(`${baseName}\\s\\((\\d+)\\)$`)
|
const baseName = name.split(duplicatePattern)[0]
|
||||||
|
const isDuplicate = new RegExp(`${baseName}\\s(\\d+)$`)
|
||||||
|
|
||||||
// get the sequence from matched names
|
// get the sequence from matched names
|
||||||
const sequence = []
|
const sequence = []
|
||||||
|
@ -28,7 +29,6 @@ export const duplicateName = (name, allNames) => {
|
||||||
return false
|
return false
|
||||||
})
|
})
|
||||||
sequence.sort((a, b) => a - b)
|
sequence.sort((a, b) => a - b)
|
||||||
|
|
||||||
// get the next number in the sequence
|
// get the next number in the sequence
|
||||||
let number
|
let number
|
||||||
if (sequence.length === 0) {
|
if (sequence.length === 0) {
|
||||||
|
@ -46,5 +46,5 @@ export const duplicateName = (name, allNames) => {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return `${baseName} (${number})`
|
return `${baseName} ${number}`
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue