[chore]Make get_field_attr safe and improve control flow in ts_core.py #318
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Refactor get_field_attr in library.py: - Ensure safety using list slicing to avoid IndexError. - Use .get() for dictionary access to handle default cases and prevent KeyError. Improve control flow in ts_core.py: - Remove nesting by exiting early in match_conditions. - Eliminate the need for try-catch blocks for better readability and control flow. Note: Kept existing_fields case unchanged due to add_field_to_entry's procedural nature. this commit introduces changes to library.py get_field_attr
cleans whitespace from file
Fix get_field_attr in library.py to ensure proper handling of empty keys list: - Previously, slicing avoided IndexError but returned a list instead. - Ensure the value is read only if present, defaulting to -1 otherwise. Ideal scenario: Define this function to return an int, as expected by the software elsewhere.
|
There is a bug on the assumption for list slicing. Closing and opening a new one with the change fixed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refactor get_field_attr in library.py:
Improve control flow in ts_core.py:
Note: Kept existing_fields case unchanged due to add_field_to_entry's procedural nature.
this commit introduces changes to library.py get_field_attr