fix Boards.uniqueTitle not working as expected #4400

[Ben0it-T]

fix #4400

[Ben0it-T]

Pervert side effect !!
I understand why this was changed.
If you import a board this will had an unnecessary [1] !
Sorry.
Fell free to revert.
Need to dig a little more... or make two functions, one for import, one for duplicate board

[Ben0it-T]

Adding a "first check" seems to bring something better

Boards.uniqueTitle = title => {
  const m = title.match(
    new RegExp('^(?<title>.*?)\\s*(\\[(?<num>\\d+)]\\s*$|\\s*$)'),
  );
  const base = escapeForRegex(m.groups.title);
  boards = Boards.find({ title: new RegExp(`^${base}\\s*(\\[(?<num>\\d+)]\\s*$|\\s*$)`) });
  if (boards.count() > 0) {
    let num = 0;
    Boards.find({ title: new RegExp(`^${base}\\s*\\[\\d+]\\s*$`) }).forEach(
      board => {
        const m = board.title.match(
          new RegExp('^(?<title>.*?)\\s*\\[(?<num>\\d+)]\\s*$'),
        );
        if (m) {
          const n = parseInt(m.groups.num, 10);
          num = num < n ? n : num;
        }
      },
    );
    return `${m.groups.title} [${num + 1}]`;
  }
  return title;
};

I will redo some tests tomorrow. Then I'll send a PR