Differential D12586

starts storing lyricists in their own table to prepare for more
ClosedPublic
Actions

Authored by mgallien on Apr 28 2018, 4:12 PM.

Details

Reviewers
None
Group Reviewers
Elisa
Maniphest Tasks
T7883: genre browser
Commits
R255:b5a0f6b32f95: starts storing lyricists in their own table to prepare for more
Summary

starts storing composers in their own table to prepare for more

Test Plan

automatic tests are OK

Diff Detail

Repository
R255 Elisa
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mgallien requested review of this revision.Apr 28 2018, 4:12 PM
mgallien created this revision.
mgallien updated this revision to Diff 33383.May 1 2018, 11:56 AM

rebase on top of master

mgallien updated this revision to Diff 33418.May 1 2018, 6:26 PM

rebase on master

astippich added a subscriber: astippich.May 3 2018, 7:10 PM

same story here again, cannot apply. Imho you can go ahead and push the remaining patches in this series. I haven't noticed any issues anyway until now and if there will be, I will report back

This revision was not accepted when it landed; it landed in state Needs Review.May 3 2018, 9:33 PM
Closed by commit R255:b5a0f6b32f95: starts storing lyricists in their own table to prepare for more (authored by mgallien). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/databaseinterface.h (6 lines)
Msrc/databaseinterface.cpp (204 lines)

Diff 33581

View Options

src/databaseinterface.h

Loading...
View Options

src/databaseinterface.cpp

Loading...