Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Capitalizing a phoneme with an allophone crashes #212

Open
nai888 opened this issue Mar 22, 2019 · 0 comments
Open

Capitalizing a phoneme with an allophone crashes #212

nai888 opened this issue Mar 22, 2019 · 0 comments
Assignees
Labels

Comments

@nai888
Copy link
Collaborator

nai888 commented Mar 22, 2019

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.

@nai888 nai888 added the bug label Mar 22, 2019
@nai888 nai888 self-assigned this Mar 22, 2019
@nai888 nai888 moved this to High priority in Bug and Feature Triage Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: High priority
Development

No branches or pull requests

1 participant