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.

Sign up for GitHub

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

Jump to bottom

Recursion error in getRecords() when evaluating header names for certain form schema #132

Closed
jamiewhths opened this issue Oct 3, 2024 · 0 comments
Closed

Recursion error in getRecords() when evaluating header names for certain form schema #132

jamiewhths opened this issue Oct 3, 2024 · 0 comments
Assignees
@nickdickinson
Milestone
Release 4.37

Comments

@jamiewhths
Copy link
Contributor

Error when using getTables() with certain form schmea.

stopifnot(style$column
Names %in% c("pretty", "label", "code", :
node stack overflow
Error during wrapup: node stack overflow
Error: no more error handlers available (recursive errors?); invoking 'abort' restart

See form "Finance Tracker" (https://www.activityinfo.org/app#form/coavb59m0ao1zha1qax/table) in database: https://www.activityinfo.org/app#database/cy64z55m0ao1zh91qav
@jamiewhths jamiewhths added this to the Release 4.37 milestone Oct 3, 2024
nickdickinson added a commit that referenced this issue Oct 10, 2024
@nickdickinson
Fix to #132; addressing recursion and cyclic structures in varNames()…
0e4ccb9 
… and adding maxDepth to styles; Some additional tests
@nickdickinson nickdickinson changed the title Recursion error in getTables() when evaluating header names for certain form schema Recursion error in getRecords() when evaluating header names for certain form schema Oct 10, 2024
@jamiewhths jamiewhths mentioned this issue Oct 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nickdickinson nickdickinson

Labels
None yet
Projects
None yet
Milestone
Release 4.37
Development

No branches or pull requests
2 participants
@nickdickinson @jamiewhths