search

jpd236 / kotwords

Collection of crossword puzzle file format converters and other utilities, written in Kotlin.
Apache License 2.0
25 stars 6 forks source link

Clarify or relax requirement for lower-case characters in Acrostic solutions #26

Closed jpd236 closed 2 years ago

jpd236 commented 2 years ago

If lower-case characters are entered as an Acrostic solution, we fail to generate a grid key since letters are assumed to be upper-case in a few places at https://github.com/jpd236/kotwords/blob/master/src/commonMain/kotlin/com/jeffpdavidson/kotwords/model/Acrostic.kt. This results in a fairly confusing error message.

At a minimum, clarifying the error would be nice. We could probably just go ahead and convert to upper-case since that's the clear intent here, and perhaps audit other forms for the same strictness.

jpd236 commented 2 years ago

This should be fixed now (and pushed). Every form field which is used to enter solutions is now converted to upper case.