> On Авг. 23, 2017, 3:59 п.п., Christoph Feck wrote: > > Thanks for the patch. Someone familiar with the code should merge this to > > 'kf5' branch. It is not trivial, because of the port of the parser to > > QObject. > > > > I locally tried an automatic merge. Not only did this fail, but it was also > > wrong because it reverted parts of those porting commits.
I'll try to do It. - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/130232/#review103636 ----------------------------------------------------------- On Авг. 23, 2017, 3:40 п.п., Sergey Ivanov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/130232/ > ----------------------------------------------------------- > > (Updated Авг. 23, 2017, 3:40 п.п.) > > > Review request for Amarok. > > > Bugs: 328359 > https://bugs.kde.org/show_bug.cgi?id=328359 > > > Repository: amarok > > > Description > ------- > > + Fixed search result appending process to prevent creation of invalid > indices. > + Added "Accept" header to musicbrainz request, because sometimes MusicBrainz > returns response in JSON format instead of XML. > > > Diffs > ----- > > src/musicbrainz/MusicBrainzFinder.h beb155166f > src/musicbrainz/MusicBrainzFinder.cpp 9a1b3bb14a > src/musicbrainz/MusicBrainzTagsItem.h 6852c9fd31 > src/musicbrainz/MusicBrainzTagsItem.cpp 0941a901bc > src/musicbrainz/MusicBrainzTagsModel.h eaee7f8e90 > src/musicbrainz/MusicBrainzTagsModel.cpp 0ffb284055 > src/musicbrainz/MusicBrainzXmlParser.h c7f9e72c0e > src/musicbrainz/MusicBrainzXmlParser.cpp 290a275aed > > Diff: https://git.reviewboard.kde.org/r/130232/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Ivanov > >
