Fails to play a file with a # (hash symbol) in the path
Bug #1449790 reported by
Neil McPhail
This bug affects 14 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
High
|
John McAleely | ||
media-hub (Ubuntu) |
Fix Released
|
High
|
Jim Hodapp | ||
media-hub (Ubuntu RTM) |
Fix Released
|
High
|
Jim Hodapp | ||
qtubuntu-media (Ubuntu) |
Fix Released
|
High
|
Jim Hodapp | ||
qtubuntu-media (Ubuntu RTM) |
Fix Released
|
High
|
Jim Hodapp |
Bug Description
Playing a song from the music app will fail if the path contains a # symbol. The failure appears to be at the media-hub level.
Example: Try to play track under "Music/Lots of #1 hits/annoying_
What should happen: Song should play
What actually happens: Song info loads in music player screen but track does not play and duration is set to zero.
This bug has been confirmed by ahayzen on #ubuntu-touch who submitted this example log: http://
The file plays normally if the # is removed from the path.
Related branches
lp:~phablet-team/media-hub/fix-1449790
- Alfonso Sanchez-Beato: Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 780 lines (+429/-67)13 files modifiedinclude/core/media/player.h (+5/-0)
src/core/media/gstreamer/engine.cpp (+6/-3)
src/core/media/gstreamer/playbin.cpp (+129/-28)
src/core/media/gstreamer/playbin.h (+3/-0)
src/core/media/mpris/player.h (+8/-0)
src/core/media/player.cpp (+6/-0)
src/core/media/player_implementation.cpp (+2/-2)
src/core/media/player_skeleton.cpp (+57/-22)
src/core/media/player_stub.cpp (+2/-0)
src/core/media/track_list_implementation.cpp (+0/-1)
src/core/media/track_list_skeleton.cpp (+46/-11)
src/core/media/track_list_stub.cpp (+5/-0)
src/core/media/util/uri_check.h (+160/-0)
lp:~phablet-team/qtubuntu-media/fix-1449790
- Alfonso Sanchez-Beato: Approve
- PS Jenkins bot: Needs Fixing (continuous-integration)
-
Diff: 2525 lines (+1300/-1040)14 files modifiedREADME (+7/-0)
src/aal/aalmediaplayerservice.cpp (+10/-2)
src/aal/aalmediaplaylistprovider.cpp (+24/-9)
tests/integration/integration.pro (+6/-23)
tests/integration/playlists/playlists.pro (+25/-0)
tests/integration/playlists/tst_mediaplaylist.cpp (+896/-0)
tests/integration/playlists/tst_mediaplaylist.h (+110/-0)
tests/integration/tst_mediaplaylist.cpp (+0/-896)
tests/integration/tst_mediaplaylist.h (+0/-110)
tests/integration/uris/generate_test_files.sh (+25/-0)
tests/integration/uris/remove_test_files.sh (+7/-0)
tests/integration/uris/tst_mediauris.cpp (+116/-0)
tests/integration/uris/tst_mediauris.h (+51/-0)
tests/integration/uris/uris.pro (+23/-0)
Changed in media-hub: | |
status: | Triaged → Confirmed |
Changed in media-hub: | |
importance: | Undecided → High |
Changed in media-hub (Ubuntu): | |
status: | New → Triaged |
Changed in media-hub (Ubuntu RTM): | |
status: | New → Triaged |
no longer affects: | media-hub |
Changed in media-hub (Ubuntu): | |
importance: | Undecided → Low |
Changed in media-hub (Ubuntu RTM): | |
importance: | Undecided → Low |
Changed in qtubuntu-media (Ubuntu): | |
status: | New → Triaged |
Changed in qtubuntu-media (Ubuntu RTM): | |
status: | New → Triaged |
Changed in qtubuntu-media (Ubuntu): | |
importance: | Undecided → Low |
Changed in qtubuntu-media (Ubuntu RTM): | |
importance: | Undecided → Low |
Changed in canonical-devices-system-image: | |
importance: | Undecided → High |
assignee: | nobody → John McAleely (john.mcaleely) |
milestone: | none → ww08-2016 |
Changed in canonical-devices-system-image: | |
status: | Confirmed → In Progress |
Changed in media-hub (Ubuntu): | |
status: | Triaged → In Progress |
Changed in media-hub (Ubuntu RTM): | |
status: | Triaged → In Progress |
Changed in qtubuntu-media (Ubuntu): | |
status: | Triaged → In Progress |
Changed in qtubuntu-media (Ubuntu RTM): | |
status: | Triaged → In Progress |
Changed in media-hub (Ubuntu): | |
assignee: | nobody → Jim Hodapp (jhodapp) |
Changed in media-hub (Ubuntu RTM): | |
assignee: | nobody → Jim Hodapp (jhodapp) |
Changed in qtubuntu-media (Ubuntu): | |
assignee: | nobody → Jim Hodapp (jhodapp) |
Changed in qtubuntu-media (Ubuntu RTM): | |
assignee: | nobody → Jim Hodapp (jhodapp) |
Changed in qtubuntu-media (Ubuntu RTM): | |
status: | In Progress → Fix Committed |
Changed in qtubuntu-media (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in media-hub (Ubuntu RTM): | |
status: | In Progress → Fix Committed |
Changed in media-hub (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in media-hub (Ubuntu): | |
status: | Fix Committed → In Progress |
Changed in media-hub (Ubuntu RTM): | |
status: | Fix Committed → In Progress |
Changed in qtubuntu-media (Ubuntu): | |
status: | Fix Committed → In Progress |
Changed in qtubuntu-media (Ubuntu RTM): | |
status: | Fix Committed → In Progress |
Changed in media-hub (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in media-hub (Ubuntu RTM): | |
status: | In Progress → Fix Committed |
Changed in qtubuntu-media (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in qtubuntu-media (Ubuntu RTM): | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
krillin RTM release 21