nicfit/eyeD3

allow to set multiple TCON frames

Open

#296 opened on 2018年12月15日

GitHub で見る
 (2 comments) (0 reactions) (0 assignees)Python (487 stars) (58 forks)batch import
featurehelp wanted

説明

Setting multiple TCON frames logs all modification, but then only the last frame is written

$ eyeD3 --text-frame=TCON:Rock --text-frame=TCON:Ballad \
     --text-frame=TCON:Pop /tmp/1.mp3

Setting b'TCON' text frame to 'Rock'
Setting b'TCON' text frame to 'Ballad'
Setting b'TCON' text frame to 'Pop'

The situation is different if the file had multiple TCON frames in origin. Only the first is modified while others frames are kept.

Further info in debian bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=887405

Tested with eyed3 0.8.7 and python 3.6.7.

コントリビューターガイド

技術スタック
python
領域
clitooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Understanding of ID3 tagsPython programming
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
55
調査方針
Investigate the code that handles writing multiple TCON frames. The issue states that when multiple TCON frames are set via command line, only the last one is written. Look at the relevant functions in eyeD3 that process and write text frames, possibly in the tag writing module. Check the linked Debian bug for additional context. Focus on how frames are collected and how duplicate frames are handled. Consider adding support for multiple frames of the same type or modifying the existing logic to write all provided frames.