Add record label metadata support

Author: Matthew Davey

res/schema.xml

@@ -593,4 +593,12 @@ reapplying those migrations.
       ALTER TABLE library ADD COLUMN tuning_frequency_hz FLOAT DEFAULT 0.0;
     </sql>
   </revision>
-</schema>
+  <revision version="41" min_compatible="3">
+    <description>
+      Add record_label column to library table.
+    </description>
+    <sql>
+      ALTER TABLE library ADD COLUMN record_label TEXT DEFAULT '';
+    </sql>
+  </revision>
+</schema>

src/database/mixxxdb.cpp

@@ -14,7 +14,7 @@
 const QString MixxxDb::kDefaultSchemaFile(":/schema.xml");
 
 //static
-const int MixxxDb::kRequiredSchemaVersion = 40;
+const int MixxxDb::kRequiredSchemaVersion = 41;
 
 namespace {
 

src/library/basetrackcache.cpp

@@ -315,6 +315,9 @@ QVariant BaseTrackCache::getTrackValueForColumn(TrackPointer pTrack,
     if (fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_ALBUMARTIST) == column) {
         return QVariant{pTrack->getAlbumArtist()};
     }
+    if (fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_RECORDLABEL) == column) {
+    return QVariant{pTrack->getRecordLabel()};
+    }
     if (fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_YEAR) == column) {
         return QVariant{pTrack->getYear()};
     }

src/library/basetracktablemodel.cpp

@@ -322,6 +322,7 @@ QList<int> BaseTrackTableModel::pasteTracks(const QModelIndex& insertionIndex) {
 bool BaseTrackTableModel::isColumnHiddenByDefault(
         int column) {
     return column == fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_ALBUMARTIST) ||
+            column == fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_RECORDLABEL) ||
             column == fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_BPM_LOCK) ||
             column == fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_BITRATE) ||
             column == fieldIndex(ColumnCache::COLUMN_LIBRARYTABLE_CHANNELS) ||

src/library/columncache.cpp

@@ -52,6 +52,9 @@ constexpr ColumnProperties kColumnPropertiesByEnum[] = {
         DI(ColumnCache::COLUMN_LIBRARYTABLE_ALBUMARTIST){&LIBRARYTABLE_ALBUMARTIST,
                 QT_TRANSLATE_NOOP("BaseTrackTableModel", "Album Artist"),
                 kDefaultColumnWidth * 4},
+        DI(ColumnCache::COLUMN_LIBRARYTABLE_RECORDLABEL){&LIBRARYTABLE_RECORDLABEL,
+                QT_TRANSLATE_NOOP("BaseTrackTableModel", "Record Label"),
+                kDefaultColumnWidth * 4},
         DI(ColumnCache::COLUMN_LIBRARYTABLE_YEAR){&LIBRARYTABLE_YEAR,
                 QT_TRANSLATE_NOOP("BaseTrackTableModel", "Year"),
                 kDefaultColumnWidth},
@@ -234,6 +237,7 @@ void ColumnCache::setColumns(QStringList columns) {
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_TITLE, kSortNoCaseLex);
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_ALBUM, kSortNoCaseLex);
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_ALBUMARTIST, kSortNoCaseLex);
+    insertColumnSortByEnum(COLUMN_LIBRARYTABLE_RECORDLABEL, kSortNoCaseLex);
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_YEAR, kSortNoCase);
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_GENRE, kSortNoCaseLex);
     insertColumnSortByEnum(COLUMN_LIBRARYTABLE_COMPOSER, kSortNoCaseLex);

src/library/columncache.h

@@ -21,6 +21,7 @@ class ColumnCache : public QObject {
         COLUMN_LIBRARYTABLE_TITLE,
         COLUMN_LIBRARYTABLE_ALBUM,
         COLUMN_LIBRARYTABLE_ALBUMARTIST,
+        COLUMN_LIBRARYTABLE_RECORDLABEL,
         COLUMN_LIBRARYTABLE_YEAR,
         COLUMN_LIBRARYTABLE_GENRE,
         COLUMN_LIBRARYTABLE_COMPOSER,

src/library/dao/trackdao.cpp

@@ -442,6 +442,7 @@ void TrackDAO::addTracksPrepare() {
             "title,"
             "album,"
             "album_artist,"
+            "record_label,"
             "year,"
             "genre,"
             "tracknumber,"
@@ -491,6 +492,7 @@ void TrackDAO::addTracksPrepare() {
             ":title,"
             ":album,"
             ":album_artist,"
+            ":record_label,"
             ":year,"
             ":genre,"
             ":tracknumber,"
@@ -597,6 +599,9 @@ void bindTrackLibraryValues(
     pTrackLibraryQuery->bindValue(":title", trackInfo.getTitle());
     pTrackLibraryQuery->bindValue(":album", albumInfo.getTitle());
     pTrackLibraryQuery->bindValue(":album_artist", albumInfo.getArtist());
+    pTrackLibraryQuery->bindValue(
+        ":record_label",
+        albumInfo.getRecordLabel());
     pTrackLibraryQuery->bindValue(":year", trackInfo.getYear());
     pTrackLibraryQuery->bindValue(":genre", trackInfo.getGenre());
     pTrackLibraryQuery->bindValue(":composer", trackInfo.getComposer());
@@ -1174,6 +1179,10 @@ void setTrackAlbumArtist(const QSqlRecord& record, const int column, Track* pTra
     pTrack->setAlbumArtist(record.value(column).toString());
 }
 
+void setTrackRecordLabel(const QSqlRecord& record, const int column, Track* pTrack) {
+    pTrack->setRecordLabel(record.value(column).toString());
+}
+
 void setTrackYear(const QSqlRecord& record, const int column, Track* pTrack) {
     pTrack->setYear(record.value(column).toString());
 }
@@ -1391,6 +1400,7 @@ TrackPointer TrackDAO::getTrackById(TrackId trackId) const {
             {"title", setTrackTitle},
             {"album", setTrackAlbum},
             {"album_artist", setTrackAlbumArtist},
+            {"record_label", setTrackRecordLabel},
             {"year", setTrackYear},
             {"genre", setTrackGenre},
             {"composer", setTrackComposer},
@@ -1686,6 +1696,7 @@ bool TrackDAO::updateTrack(const Track& track) const {
                     << trackId
                     << track.getLocation();
 
+
     SqlTransaction transaction(m_database);
     // PerformanceTimer time;
     // time.start();
@@ -1699,6 +1710,7 @@ bool TrackDAO::updateTrack(const Track& track) const {
             "title=:title,"
             "album=:album,"
             "album_artist=:album_artist,"
+            "record_label=:record_label,"
             "year=:year,"
             "genre=:genre,"
             "composer=:composer,"

src/library/dao/trackschema.h

@@ -15,6 +15,7 @@ const QString LIBRARYTABLE_ARTIST = QStringLiteral("artist");
 const QString LIBRARYTABLE_TITLE = QStringLiteral("title");
 const QString LIBRARYTABLE_ALBUM = QStringLiteral("album");
 const QString LIBRARYTABLE_ALBUMARTIST = QStringLiteral("album_artist");
+const QString LIBRARYTABLE_RECORDLABEL = QStringLiteral("record_label");
 const QString LIBRARYTABLE_YEAR = QStringLiteral("year");
 const QString LIBRARYTABLE_GENRE = QStringLiteral("genre");
 const QString LIBRARYTABLE_COMPOSER = QStringLiteral("composer");

src/library/dlgtrackinfo.cpp

@@ -74,6 +74,7 @@ void DlgTrackInfo::init() {
     m_propertyWidgets.insert("titleInfo", txtTitle);
     m_propertyWidgets.insert("album", txtAlbum);
     m_propertyWidgets.insert("album_artist", txtAlbumArtist);
+    m_propertyWidgets.insert("record_label", txtRecordLabel);
     m_propertyWidgets.insert("composer", txtComposer);
     m_propertyWidgets.insert("genre", txtGenre);
     m_propertyWidgets.insert("year", txtYear);
@@ -224,6 +225,19 @@ void DlgTrackInfo::init() {
                 m_trackRecord.refMetadata().refAlbumInfo().setArtist(
                         txtAlbumArtist->text());
             });
+    connect(txtRecordLabel,
+        &QLineEdit::editingFinished,
+        this,
+        [this]() {
+            txtRecordLabel->setText(
+                    txtRecordLabel->text().trimmed());
+
+            m_trackRecord
+                    .refMetadata()
+                    .refAlbumInfo()
+                    .setRecordLabel(
+                            txtRecordLabel->text());
+        });
     connect(txtGenre,
             &QLineEdit::editingFinished,
             this,
@@ -636,6 +650,7 @@ void DlgTrackInfo::trackColorDialogSetColor(const mixxx::RgbColor::optional_t& n
 }
 
 void DlgTrackInfo::saveTrack() {
+    qDebug() << "DlgTrackInfo::saveTrack() called";
     if (!m_pLoadedTrack) {
         return;
     }
@@ -657,6 +672,12 @@ void DlgTrackInfo::saveTrack() {
     // updated by the editingFinished signal.
     m_trackRecord.refMetadata().refTrackInfo().setComment(txtComment->toPlainText());
 
+    m_trackRecord
+        .refMetadata()
+        .refAlbumInfo()
+        .setRecordLabel(
+                txtRecordLabel->text().trimmed());
+
     // First, disconnect the track changed signal. Otherwise we signal ourselves
     // and repopulate all these fields.
     const QSignalBlocker signalBlocker(this);