fix: check if slice is nil when calling BeforeAppendModel

internal/dbtest/orm_test.go

@@ -462,6 +462,12 @@ func (a Author) String() string {
 	return fmt.Sprintf("Author<ID=%d Name=%q>", a.ID, a.Name)
 }
 
+var _ bun.BeforeAppendModelHook = (*Author)(nil)
+
+func (*Author) BeforeAppendModel(ctx context.Context, query bun.Query) error {
+	return nil
+}
+
 type BookGenre struct {
 	bun.BaseModel `bun:"alias:bg"` // custom table alias
 
@@ -492,6 +498,12 @@ func (b Book) String() string {
 	return fmt.Sprintf("Book<Id=%d Title=%q>", b.ID, b.Title)
 }
 
+var _ bun.BeforeAppendModelHook = (*Book)(nil)
+
+func (*Book) BeforeAppendModel(ctx context.Context, query bun.Query) error {
+	return nil
+}
+
 // BookWithCommentCount is like Book model, but has additional CommentCount
 // field that is used to select data into it. The use of `bun:",inherit"` tag
 // is essential here so it inherits internal model properties such as table name.

model_table_slice.go

@@ -94,7 +94,7 @@ func (m *sliceTableModel) ScanRows(ctx context.Context, rows *sql.Rows) (int, er
 var _ schema.BeforeAppendModelHook = (*sliceTableModel)(nil)
 
 func (m *sliceTableModel) BeforeAppendModel(ctx context.Context, query Query) error {
-	if !m.table.HasBeforeAppendModelHook() {
+	if !m.table.HasBeforeAppendModelHook() || !m.slice.IsValid() {
 		return nil
 	}