lp:~unity-team/unity-lens-music/improved-result-quality

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:unity-lens-music at revision 86

Michal Hruby (community): Approve on 2012-07-11

Diff: 102 lines (+50/-25)

1 file modified

src/musicstore-collection.vala (+50/-25)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1021656: Low quality of search results

High

Fix Released

Link a bug report

Related blueprints

89. By Mikkel Kamstrup Erlandsen on 2012-07-11

Make updated musicstore-collection.vala work with Vala 0.14. We re-used variabel names in 2 for-loops inside the same method. Vala 0.14 no likey!

88. By Mikkel Kamstrup Erlandsen on 2012-07-09

Removed obsolete null checks when adding rows to the results model in the musicstore. DeeModel is null-safe wrt strings these days.

87. By Michal Hruby on 2012-07-09

Don't use magic constants

86. By Mikkel Kamstrup Erlandsen on 2012-07-06

Fix u1 search results to not seem too random

This was rooted in the fact that musicsearch.ubuntu.com returns a mix of artists, albums, and tracks in the results. We do not show artists, so if the results contains mainly that, then we're screwed. Luckily newer versions of the webservice contains some parameters to tweak this behaviour. This commit makes us use that.

In addition to using the new musicsearch API I also fixed memory churn quite considerably by removing the need to create a lot of Track/Album objects and do excessive string copying.

Also fixed a bug where we'd pass possibly-null strings into a DeeModel. This could have been the root cause of bug #917974.

85. By Michal Hruby on 2012-07-04

Add assertions.vapi to EXTRA_DIST

84. By Michal Hruby on 2012-07-02

Release 6.0.0

83. By Didier Roche-Tolomelli on 2012-07-02

adds local radio detection from rhythmbox. The category will
only be placed after the "for purchase one" to be unobstrusive (LP: #969697)
Also fix the non translated "Unknown" album filter appearing for non english
users (LP: #1009069)
Add some tests for the parser part (radios and songs). Fixes: https://bugs.launchpad.net/bugs/969697, https://bugs.launchpad.net/bugs/1009069. Approved by Pawel Stolowski.

82. By Michal Hruby on 2012-04-27

Release 5.12.0

81. By Michal Hruby on 2012-04-26

Rhythmbox scope: Implement proper album handling. Fixes: https://bugs.launchpad.net/bugs/970509. Approved by Gord Allott, Alex Launi.

80. By Michal Hruby on 2012-04-24

Added rhythmbox's TDB parsing so we'll have more sources of album arts in the lens. Fixes: https://bugs.launchpad.net/bugs/976067. Approved by Gord Allott.