Skip to content

Fix a failure to reveal a key when all section keys have dependancies - #26

Open
klangman wants to merge 1 commit into
linuxmint:masterfrom
klangman:revealing_fix
Open

klangman wants to merge 1 commit into
linuxmint:masterfrom
klangman:revealing_fix

Conversation

@klangman

@klangman klangman commented Oct 9, 2026

Copy link
Copy Markdown

Given the follownig scenario.

"mysection": {
"type": "section",
"title": "MyTitle",
"keys": [
"key1_revealed_on_condition_false",
"key2_revealed_on_condition_false",
"key3_revealed_on_condition_true",
"key4_revealed_on_condition_true"
]
},

If the condition is "true" then keys 3 & 4 should be shown, but the bug causes key4 not to get drawn. If the condition is "false" then keys 1 & 2 are shown correctly.

If a fifth key is added that does not define a dependancy then the problem does not occur.

Given the follownig scenario.

"mysection": {
    "type": "section",
    "title": "MyTitle",
    "keys": [
       "key1_revealed_on_condition_false",
       "key2_revealed_on_condition_false",
       "key3_revealed_on_condition_true",
       "key4_revealed_on_condition_true"
  ]
},

If the condition is "true" then keys 3 & 4 should be shown, but the bug causes key4 not to get drawn.
If the condition is "false" then keys 1 & 2 are shown correctly.

If a fifth key is added that does not define a dependancy then the problem does not occur.
@klangman

Copy link
Copy Markdown
Author

I had Claude write a test-case to demonstrate the problem.

python3 test_reveal_section.py

test_reveal_section.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant