! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
--audio-composition opens also a data composition when multiple FILE paths sp...
Status:
RESOLVED: FIXED

Comments

Description itteffi 2016-12-23 00:20:29 CET
the manual page says you can specify --data-composition or --audio-composition from the command line. the --data-composition option works as expected, however, the --audio-composition option opens both a data composition and an audio composition.
Comment 1 Skunnyk editbugs 2017-11-11 19:49:13 CET
It works for me, -data-composition option open data composition, --audio-composition option open audio composition.
Can you reproduce ?
Comment 2 itteffi 2017-11-12 13:05:43 CET
the bug actually only occurs when you also specify multiple FILE paths, e.g.:
xfburn --audio-composition "/home/itteffi/Music/song1.mp3" "/home/itteffi/Music/song2.mp3"

if no FILE paths are specified, the bug will not reproduce; if a single FILE path is specified, it doesn't appear to be applicable, as you're brought immediately to the "Burn image" dialog; but if 2 or more FILE paths are specified, a data composition tab is opened in addition to an audio composition tab.
Comment 3 Skunnyk editbugs 2017-11-12 13:50:50 CET
Ah, yes, good catch, I can reproduce.
It's an expected behavior 'for now', see here : https://git.xfce.org/apps/xfburn/tree/xfburn/xfburn-main.c#n340 .
So if someone is willing to implement the detection of audio track when passing multiple arguments… :)
Comment 4 itteffi 2017-11-12 15:44:38 CET
@Skunnyk, the logic is wrong ...

line 346 should be changed to:
} else if (argc > 2 && !add_data_composition && !add_audio_composition) {

the todo, "auto-detect music files for audio compositions", is only supposed to apply if the --audio-composition option wasn't specified; if the --audio-composition option is specified, obviously you don't need to guess the user's intention....
Comment 5 itteffi 2017-11-12 16:04:41 CET
in addition to the above, which i'm pretty sure about, i'm not so sure that lines 343 and 346 should be comparing argc with 2 (as they are) instead of 1.
Comment 6 itteffi 2017-11-12 16:10:24 CET
please disregard comment 5; http://crasseux.com/books/ctutorial/argc-and-argv.html says the command itself is considered an argument, so I believe 2 is the correct point of comparison.
Comment 7 Skunnyk editbugs 2017-11-12 22:32:54 CET
Oh, yes, by changing line 346, the bug is fixed (it open only the audio compositor).
Can you confirm ?
Comment 8 itteffi 2017-11-12 23:31:42 CET
well, i'm not actually compiling (as i'm not doing any xfburn development), so I can't technically confirm the fix
Comment 9 Skunnyk editbugs 2017-11-13 18:35:26 CET
Ok, so I confirm that the bug is fixed by this "patch". Thank you. 
I'll commit it to master.
Comment 10 Git Bot editbugs 2017-11-20 18:38:37 CET
Romain B referenced this bugreport in commit 6df73ec15e6b629d4bfbb8de81acbd24da554e09

Do no open data-composition when running --audio-composition

https://git.xfce.org/apps/xfburn/commit?id=6df73ec15e6b629d4bfbb8de81acbd24da554e09

Bug #13251

Reported by:
itteffi
Reported on: 2016-12-23
Last modified on: 2017-12-14

People

Assignee:
Xfburn Bug Triage
CC List:
1 user

Version

Attachments

Additional information