otoole icon indicating copy to clipboard operation
otoole copied to clipboard

[FEATURE]: Refactor default_values

Open willu47 opened this issue 1 year ago • 0 comments

Feature Request

There is no reason to "read in" default values in the ReadStrategy subclasses as this information is now stored in config.

Is your feature related to a bug?

No response

Suggested Solution

This functionality can be removed, and the ReadStrategy.read() method can return just the data rather than the tuple of data and default values.

The WriteStrategy does require default values if the write_defaults flag is set to True. So we can move the get_defauylts method over to the write strategies.

Additional Info

No response

willu47 avatar Jun 21 '23 12:06 willu47