Diffusion GCompris e3c54ed3b794

core, support for local wordset fix
e3c54ed3b794
Actions

Authored by bcoudoin on Jul 3 2016, 12:06 PM.

Description

core, support for local wordset fix

When the wordset is found locally we now force the wordset configuration.
If we don't do this the lang activity does not use it.

Removed a second call to checkWordSet uppon voice download. this was triggering
twice the same image dialog download box.

Details

Committed
bcoudoinJul 3 2016, 12:10 PM
Parents
R2:0453ee6206e9: add highContrast button theme
Branches
Unknown
Tags
Unknown
Restricted Application added a subscriber: jjazeix. ยท View Herald TranscriptJul 3 2016, 12:13 PM
jjazeix added inline comments.Jul 3 2016, 3:45 PM
/src/core/main.qml
117

why did you remove ApplicationSettings? If you delete the file in ~/.cache/KDE/gcompris-qt/data2/words, it causes the issue: qrc:/gcompris/src/core/main.qml:117: ReferenceError: wordset is not defined.
Same below, it causes the error in qrc:/gcompris/src/core/main.qml:140. (and probably everywhere wordset is used without ApplicationSettings.)

bcoudoin added a comment.Jul 3 2016, 10:14 PM

You are right, It misses the wordset var declaration. I don't understand how I get there, probably during my cleanup before the commit. I pushed the fix on master and 0.60.

Changes (1)

PathPackages
Msrc/core/main.qml

R2:e3c54ed3b794

View Options

src/core/main.qml

Loading...