search

tableau / document-api-python

Create and modify Tableau workbook and datasource files
https://tableau.github.io/document-api-python/
MIT License
326 stars 178 forks source link

Bug: v0.10 prints content with fields.values() #240

Closed bryceglarsen closed 1 year ago

bryceglarsen commented 1 year ago

Hello! It seems that a bug was introduce in the latest version of the Document API. When running something such as: for field in datasource.fields.values(): print(field.name.rstrip("]").lstrip("["))

we get content printed to terminal.

bryceglarsen commented 1 year ago

I believe it's: https://github.com/tableau/document-api-python/blob/master/tableaudocumentapi/field.py#L73

bcantoni commented 1 year ago

Yes that line was added in an earlier change and then fixed in #238. I'll leave this open until the next release comes out, then that problem should be resolved.

bryceglarsen commented 1 year ago

Ah, thanks @bcantoni! Guess I won't open a PR then. Heh.

bcantoni commented 1 year ago

@bryceglarsen this fix should be available in release 0.11 from this morning.

bryceglarsen commented 1 year ago

Thank you, @bcantoni!