After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 600513 - Generates invalid Brasero project
Generates invalid Brasero project
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Plugins (other)
HEAD
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-11-03 06:21 UTC by Robert Ancell
Modified: 2009-11-03 11:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Correctly generates <audio> tags for brasero project (1.66 KB, patch)
2009-11-03 06:21 UTC, Robert Ancell
none Details | Review

Description Robert Ancell 2009-11-03 06:21:31 UTC
Created attachment 146796 [details] [review]
Correctly generates <audio> tags for brasero project

To reproduce:
1. Open Rhythmbox
2. Make a new playlist
3. Copy some tracks to new playlist
4. Select Burn

Result:
Brasero opened with only one track

Expected:
Brasero shows all tracks in playlist

Cause:
The brasero project in /tmp has invalid <audio> tags, i.e.:

<?xml version="1.0" encoding="UTF8"?>
<braseroproject>
        <version>0.2</version>
        <label>burn test</label>
        <track>
                <audio>
                       
<uri>file%3A%2F%2F%2Fhome%2Fbob%2FMusic%2FPink%2520Floyd%2FThe%2520Wall%2520%28disc%25202%29%2F11%2520-%2520Stop.mp3</uri>
                        <start>0</start>
                        <title>Stop</title>
                        <artist>Pink%20Floyd</artist>
                       
<uri>file%3A%2F%2F%2Fhome%2Fbob%2FMusic%2FHarvey%2520Danger%2FLittle%2520by%2520Little%2520%28bonus%2520disc%29%2F09%2520-%2520hidden%2520track%3A%2520The%2520Piano%2520Lesson.m4a</uri>
                        <start>0</start>
                        <title>hidden%20track%3A%20The%20Piano%20Lesson</title>
                        <artist>Harvey%20Danger</artist>
                </audio>
        </track>
</braseroproject>

instead of:

<?xml version="1.0" encoding="UTF8"?>
<braseroproject>
        <version>0.2</version>
        <label>burn test</label>
        <track>
                <audio>
                       
<uri>file%3A%2F%2F%2Fhome%2Fbob%2FMusic%2FPink%2520Floyd%2FThe%2520Wall%2520%28disc%25202%29%2F11%2520-%2520Stop.mp3</uri>
                        <start>0</start>
                        <title>Stop</title>
                        <artist>Pink%20Floyd</artist>
                </audio>
                <audio>
                       
<uri>file%3A%2F%2F%2Fhome%2Fbob%2FMusic%2FHarvey%2520Danger%2FLittle%2520by%2520Little%2520%28bonus%2520disc%29%2F09%2520-%2520hidden%2520track%3A%2520The%2520Piano%2520Lesson.m4a</uri>
                        <start>0</start>
                        <title>hidden%20track%3A%20The%20Piano%20Lesson</title>
                        <artist>Harvey%20Danger</artist>
                </audio>
        </track>
</braseroproject>
Comment 1 Bastien Nocera 2009-11-03 10:17:57 UTC
Came to the same conclusion when looking at the code, before seeing the patch.

There's a lot of white-space changes which makes the patch bigger than it should be. Could you please remove the white-space changes, and attach the new patch?