The simple fix here is to always use the canonize function before
passing any word into the word_map.
With this fix, the test implemented in the previous commit now passes.
const word_maps = {};
for (let e of agreed_equivalencies) {
const word_maps = {};
for (let e of agreed_equivalencies) {
- let group = word_maps[e.words[0]];
+ const word0_canon = this.canonize(e.words[0]);
+ const word1_canon = this.canonize(e.words[1]);
+ let group = word_maps[word0_canon];
- group = word_maps[e.words[1]];
+ group = word_maps[word1_canon];
if (! group)
group = { words: [], players: new Set()};
if (! group)
group = { words: [], players: new Set()};
- if (! word_maps[e.words[0]]) {
- word_maps[e.words[0]] = group;
+ if (! word_maps[word0_canon]) {
+ word_maps[word0_canon] = group;
group.words.push(e.words[0]);
}
group.words.push(e.words[0]);
}
- if (! word_maps[e.words[1]]) {
- word_maps[e.words[1]] = group;
+ if (! word_maps[word1_canon]) {
+ word_maps[word1_canon] = group;
group.words.push(e.words[1]);
}
}
group.words.push(e.words[1]);
}
}
* to the set corresponding to each word group. */
for (let a of this.answers) {
for (let word of a.answers) {
* to the set corresponding to each word group. */
for (let a of this.answers) {
for (let word of a.answers) {
+ const word_canon = this.canonize(word);
/* If there's no group yet, this is a singleton word. */
/* If there's no group yet, this is a singleton word. */
- if (word_maps[word]) {
- word_maps[word].players.add(a.player);
+ if (word_maps[word_canon]) {
+ word_maps[word_canon].players.add(a.player);
} else {
const group = { words: [word], players: new Set() };
group.players.add(a.player);
} else {
const group = { words: [word], players: new Set() };
group.players.add(a.player);
- word_maps[word] = group;
+ word_maps[word_canon] = group;