xfile - Issue opening a .twbx containing custom geocoding
まだ誰も着手していません。
評価
調査の方向性
Start at find_file_in_zip() and inspect how candidate_files handles .twb and .tds entries in a .twbx containing custom geocoding. Reproduce with a workbook archive containing both file types if possible. Done means .tds files are not evaluated as workbooks and the .twbx opens without TableauInvalidFileException.
索引モデルが issue の本文から書いたものです。
説明
Hello,
I was trying to open a 9.3 workbook with custom geocoding attached. find_file_in_zip() evaluates .tds files as well as .twb files when opening a .twbx. A .twbx can contain .tds files if it contains custom geocoding.
I'm not quite sure what determines the order of the filenames in candidate_files, but if a .tds happens to end up before a .tdw in the list, the .tds will be evaluated first as a workbook. This raises the TableauInvalidFileException: filename is not a valid 'workbook' file.
I cannot share the workbook (it contains private data), I can share some screenshots of its contents:


A suggested fix for this would be to only check for .twb's if the file is a .twb/.twbx.
Cheers,
Marco
- 主要言語
- Python
- スター
- 363
- フォーク
- 183
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
tableau/document-api-python のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
tableau/document-api-python#270 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
tableau/document-api-python#260 ·
-
docs
難易度 2/5 1〜3時間 初心者へのやさしさ 55/100
tableau/document-api-python#258 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
tableau/document-api-python#251 · コメント 2 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
tableau/document-api-python#250 · コメント 2 件 ·
tableau/document-api-python の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
-
🐛 Bug 🔔 Pending processing
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
jumpserver/jumpserver#17584 ·