piwigo/PiwigoDirectoryUploader
2
0
Fork 0
Code Issues 3 Pull Requests Releases Wiki Activity

Fixed issue with duplicated category index on PiwigoId when Id is not yet known

Browse Source
This commit is contained in:
Philipp Häfelfinger 2019-04-07 22:01:58 +02:00
parent 321c18b6bd
commit 355d2a5f51
2 changed files with 29 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
internal/pkg/datastore/datastore.go
View File

@ -488,7 +488,7 @@ func (d *LocalDataStore) createTablesIfNeeded(db *sql.DB) error {
return err return err
} }
_, err = db.Exec("CREATE UNIQUE INDEX IF NOT EXISTS UX_Category_PiwigoId ON category (piwigoId);") _, err = db.Exec("CREATE UNIQUE INDEX IF NOT EXISTS UX_Category_PiwigoId ON category (piwigoId) WHERE piwigoId > 0;")
if err != nil { if err != nil {
return err return err
} }

28
internal/pkg/datastore/datastore_test.go
View File

@ -306,6 +306,34 @@ func Test_saveCategory_should_store_records(t *testing.T) {
} }
} }
func Test_saveCategory_should_store_multiple_records_with_piwigoid_zero(t *testing.T) {
if !dbinitOk {
t.Skip("Skipping test as TestDataStoreInitialize failed!")
}
dataStore := setupDatabase(t)
defer cleanupDatabase(t)
category1 := getExampleCategoryData("2019")
category1.PiwigoId = 0
category2 := getExampleCategoryData("2020")
category2.PiwigoId = 0
saveCategoryShouldNotFail("addcategory", dataStore, category1, t)
category1.CategoryId = 1
saveCategoryShouldNotFail("addcategory", dataStore, category2, t)
category2.CategoryId = 2
_, err := dataStore.GetCategoryByKey(category1.Key)
if err != nil {
t.Fatalf("Could not query category! %s", err)
}
_, err = dataStore.GetCategoryByKey(category2.Key)
if err != nil {
t.Fatalf("Could not query category! %s", err)
}
}
func Test_saveCategory_should_update_records(t *testing.T) { func Test_saveCategory_should_update_records(t *testing.T) {
if !dbinitOk { if !dbinitOk {
t.Skip("Skipping test as TestDataStoreInitialize failed!") t.Skip("Skipping test as TestDataStoreInitialize failed!")