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
=== modified file 'MusicStart.qml'
--- MusicStart.qml 2014-09-20 15:41:33 +0000
+++ MusicStart.qml 2014-10-12 19:48:56 +0000
@@ -166,7 +166,7 @@
166 anchors.rightMargin: units.gu(1)166 anchors.rightMargin: units.gu(1)
167 color: styleMusic.common.white167 color: styleMusic.common.white
168 elide: Text.ElideRight168 elide: Text.ElideRight
169 text: title169 text: title || i18n.tr("Unknown Album")
170 fontSize: "small"170 fontSize: "small"
171 font.weight: Font.DemiBold171 font.weight: Font.DemiBold
172 }172 }
@@ -180,7 +180,7 @@
180 anchors.rightMargin: units.gu(1)180 anchors.rightMargin: units.gu(1)
181 color: styleMusic.common.white181 color: styleMusic.common.white
182 elide: Text.ElideRight182 elide: Text.ElideRight
183 text: title2183 text: title2 || i18n.tr("Unknown Artist")
184 fontSize: "x-small"184 fontSize: "x-small"
185 }185 }
186 MouseArea {186 MouseArea {
187187
=== modified file 'common/SongsPage.qml'
--- common/SongsPage.qml 2014-10-05 06:08:44 +0000
+++ common/SongsPage.qml 2014-10-12 19:48:56 +0000
@@ -101,7 +101,7 @@
101 rightMargin: units.gu(2)101 rightMargin: units.gu(2)
102 }102 }
103 elide: Text.ElideRight103 elide: Text.ElideRight
104 text: line2104 text: line2 || i18n.tr("Unknown Album")
105 }105 }
106106
107 Label {107 Label {
@@ -121,7 +121,7 @@
121 rightMargin: units.gu(2)121 rightMargin: units.gu(2)
122 }122 }
123 elide: Text.ElideRight123 elide: Text.ElideRight
124 text: line1124 text: line1 || i18n.tr("Unknown Artist")
125 }125 }
126126
127 Label {127 Label {
128128
=== modified file 'meta-database.js'
--- meta-database.js 2014-09-12 02:57:30 +0000
+++ meta-database.js 2014-10-12 19:48:56 +0000
@@ -87,9 +87,9 @@
87 if (dbItem.type === "album")87 if (dbItem.type === "album")
88 {88 {
89 res.push({time:dbItem.time,89 res.push({time:dbItem.time,
90 title:dbItem.title || i18n.tr("Unknown Album"),90 title:dbItem.title,
91 title2:dbItem.title2 || i18n.tr("Unknown Artist"),91 title2:dbItem.title2,
92 key:dbItem.key || i18n.tr("Unknown Album"),92 key:dbItem.key,
93 art:dbItem.cover || undefined,93 art:dbItem.cover || undefined,
94 type:dbItem.type94 type:dbItem.type
95 });95 });
9696
=== modified file 'po/com.ubuntu.music.pot'
--- po/com.ubuntu.music.pot 2014-10-09 01:50:11 +0000
+++ po/com.ubuntu.music.pot 2014-10-12 19:48:56 +0000
@@ -8,7 +8,7 @@
8msgstr ""8msgstr ""
9"Project-Id-Version: music-app\n"9"Project-Id-Version: music-app\n"
10"Report-Msgid-Bugs-To: \n"10"Report-Msgid-Bugs-To: \n"
11"POT-Creation-Date: 2014-10-08 20:48-0500\n"11"POT-Creation-Date: 2014-10-12 14:48-0500\n"
12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"12"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"13"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
14"Language-Team: LANGUAGE <LL@li.org>\n"14"Language-Team: LANGUAGE <LL@li.org>\n"
@@ -242,6 +242,14 @@
242msgid "Clear History"242msgid "Clear History"
243msgstr ""243msgstr ""
244244
245#: ../MusicStart.qml:169 ../common/SongsPage.qml:104
246msgid "Unknown Album"
247msgstr ""
248
249#: ../MusicStart.qml:183 ../common/SongsPage.qml:124
250msgid "Unknown Artist"
251msgstr ""
252
245#: ../MusicStart.qml:227253#: ../MusicStart.qml:227
246msgid "Genres"254msgid "Genres"
247msgstr ""255msgstr ""
@@ -312,14 +320,6 @@
312msgid "Clear"320msgid "Clear"
313msgstr ""321msgstr ""
314322
315#: ../meta-database.js:90 ../meta-database.js:92
316msgid "Unknown Album"
317msgstr ""
318
319#: ../meta-database.js:91
320msgid "Unknown Artist"
321msgstr ""
322
323#: ../music-app.qml:135323#: ../music-app.qml:135
324msgid "Search Track"324msgid "Search Track"
325msgstr ""325msgstr ""

Subscribers

People subscribed via source and target branches