Skip to content

Capitalizing a phoneme with an allophone crashes #212

@nai888

Description

@nai888

Langua Version

v0.4.2 through current

Expected Behavior

All phonemes in the Frequen consonants and vowels boxes should be converted to lowercase and deduped before analysis if distinguish upper and lower case = false.

Actual Behavior

When a phoneme with an allophone is capital and/or if its allophone is capital, the system crashes and the console prints the following error twice:

TypeError: Cannot read property 'count' of undefined
    at e.value (FrequenService.js:249)
    at t.<anonymous> (index.js:66)
    at oa (react-dom.production.min.js:4271)
    at ra (react-dom.production.min.js:4257)
    at Ba (react-dom.production.min.js:4983)
    at qa (react-dom.production.min.js:5123)
    at react-dom.production.min.js:5975
    at Object.t.unstable_runWithPriority (scheduler.production.min.js:274)
    at ju (react-dom.production.min.js:5974)
    at Cu (react-dom.production.min.js:5958)

This is likely caused by converting all phonemes to lowercase but failing to dedupe the allophones. Check that first.

Steps to Reproduce

Change a phoneme with an allophone and/or an allophone to a capital version of itself, make sure distinguish upper and lower case is unchecked, then click analyze.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

Status

High priority

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions