Merge lp:~vthompson/music-app/remix-fix-1379452 into lp:music-app/remix

Proposed by Victor Thompson
Status: Rejected
Rejected by: Victor Thompson
Proposed branch: lp:~vthompson/music-app/remix-fix-1379452
Merge into: lp:music-app/remix
Diff against target: 103 lines (+16/-16)
4 files modified
MusicStart.qml (+2/-2)
common/SongsPage.qml (+2/-2)
meta-database.js (+3/-3)
po/com.ubuntu.music.pot (+9/-9)
To merge this branch: bzr merge lp:~vthompson/music-app/remix-fix-1379452
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Music App Developers Pending
Review via email: mp+238085@code.launchpad.net

Commit message

Do not return unknown to the model

Description of the change

Do not return unknown to the model. When the translatable string is returned from the database, the query of the mediascanner models will not work. Instead, the "Unknown" strings should be added only when displayed.

To post a comment you must log in.
667. By Victor Thompson

Update translatable strings

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)

Unmerged revisions

667. By Victor Thompson

Update translatable strings

666. By Victor Thompson

Do not return unknown to the model

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'MusicStart.qml'
2--- MusicStart.qml 2014-09-20 15:41:33 +0000
3+++ MusicStart.qml 2014-10-12 19:48:56 +0000
4@@ -166,7 +166,7 @@
5 anchors.rightMargin: units.gu(1)
6 color: styleMusic.common.white
7 elide: Text.ElideRight
8- text: title
9+ text: title || i18n.tr("Unknown Album")
10 fontSize: "small"
11 font.weight: Font.DemiBold
12 }
13@@ -180,7 +180,7 @@
14 anchors.rightMargin: units.gu(1)
15 color: styleMusic.common.white
16 elide: Text.ElideRight
17- text: title2
18+ text: title2 || i18n.tr("Unknown Artist")
19 fontSize: "x-small"
20 }
21 MouseArea {
22
23=== modified file 'common/SongsPage.qml'
24--- common/SongsPage.qml 2014-10-05 06:08:44 +0000
25+++ common/SongsPage.qml 2014-10-12 19:48:56 +0000
26@@ -101,7 +101,7 @@
27 rightMargin: units.gu(2)
28 }
29 elide: Text.ElideRight
30- text: line2
31+ text: line2 || i18n.tr("Unknown Album")
32 }
33
34 Label {
35@@ -121,7 +121,7 @@
36 rightMargin: units.gu(2)
37 }
38 elide: Text.ElideRight
39- text: line1
40+ text: line1 || i18n.tr("Unknown Artist")
41 }
42
43 Label {
44
45=== modified file 'meta-database.js'
46--- meta-database.js 2014-09-12 02:57:30 +0000
47+++ meta-database.js 2014-10-12 19:48:56 +0000
48@@ -87,9 +87,9 @@
49 if (dbItem.type === "album")
50 {
51 res.push({time:dbItem.time,
52- title:dbItem.title || i18n.tr("Unknown Album"),
53- title2:dbItem.title2 || i18n.tr("Unknown Artist"),
54- key:dbItem.key || i18n.tr("Unknown Album"),
55+ title:dbItem.title,
56+ title2:dbItem.title2,
57+ key:dbItem.key,
58 art:dbItem.cover || undefined,
59 type:dbItem.type
60 });
61
62=== modified file 'po/com.ubuntu.music.pot'
63--- po/com.ubuntu.music.pot 2014-10-09 01:50:11 +0000
64+++ po/com.ubuntu.music.pot 2014-10-12 19:48:56 +0000
65@@ -8,7 +8,7 @@
66 msgstr ""
67 "Project-Id-Version: music-app\n"
68 "Report-Msgid-Bugs-To: \n"
69-"POT-Creation-Date: 2014-10-08 20:48-0500\n"
70+"POT-Creation-Date: 2014-10-12 14:48-0500\n"
71 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
72 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
73 "Language-Team: LANGUAGE <LL@li.org>\n"
74@@ -242,6 +242,14 @@
75 msgid "Clear History"
76 msgstr ""
77
78+#: ../MusicStart.qml:169 ../common/SongsPage.qml:104
79+msgid "Unknown Album"
80+msgstr ""
81+
82+#: ../MusicStart.qml:183 ../common/SongsPage.qml:124
83+msgid "Unknown Artist"
84+msgstr ""
85+
86 #: ../MusicStart.qml:227
87 msgid "Genres"
88 msgstr ""
89@@ -312,14 +320,6 @@
90 msgid "Clear"
91 msgstr ""
92
93-#: ../meta-database.js:90 ../meta-database.js:92
94-msgid "Unknown Album"
95-msgstr ""
96-
97-#: ../meta-database.js:91
98-msgid "Unknown Artist"
99-msgstr ""
100-
101 #: ../music-app.qml:135
102 msgid "Search Track"
103 msgstr ""

Subscribers

People subscribed via source and target branches