Add MessageType filter to FindInbox to exclude legacy VERSION_UPDATE notifications from inbox queries. This resolves the issue where users saw notification counts but no items displayed, as VERSION_UPDATE entries cannot be rendered in the new UserNotification API. Changes: - Add MessageType field to FindInbox struct for database-level filtering - Implement JSON extraction filters in SQLite, MySQL, and PostgreSQL drivers - Update ListUserNotifications to filter MEMO_COMMENT type at store level This approach improves performance by filtering at the database rather than in application code, reducing unnecessary data transfer for users with many legacy inbox entries. Fixes #5278 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
137 lines
3.9 KiB
Go
137 lines
3.9 KiB
Go
package sqlite
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"strings"
|
|
|
|
"github.com/pkg/errors"
|
|
"google.golang.org/protobuf/encoding/protojson"
|
|
|
|
storepb "github.com/usememos/memos/proto/gen/store"
|
|
"github.com/usememos/memos/store"
|
|
)
|
|
|
|
func (d *DB) CreateInbox(ctx context.Context, create *store.Inbox) (*store.Inbox, error) {
|
|
messageString := "{}"
|
|
if create.Message != nil {
|
|
bytes, err := protojson.Marshal(create.Message)
|
|
if err != nil {
|
|
return nil, errors.Wrap(err, "failed to marshal inbox message")
|
|
}
|
|
messageString = string(bytes)
|
|
}
|
|
|
|
fields := []string{"`sender_id`", "`receiver_id`", "`status`", "`message`"}
|
|
placeholder := []string{"?", "?", "?", "?"}
|
|
args := []any{create.SenderID, create.ReceiverID, create.Status, messageString}
|
|
|
|
stmt := "INSERT INTO `inbox` (" + strings.Join(fields, ", ") + ") VALUES (" + strings.Join(placeholder, ", ") + ") RETURNING `id`, `created_ts`"
|
|
if err := d.db.QueryRowContext(ctx, stmt, args...).Scan(
|
|
&create.ID,
|
|
&create.CreatedTs,
|
|
); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
return create, nil
|
|
}
|
|
|
|
func (d *DB) ListInboxes(ctx context.Context, find *store.FindInbox) ([]*store.Inbox, error) {
|
|
where, args := []string{"1 = 1"}, []any{}
|
|
|
|
if find.ID != nil {
|
|
where, args = append(where, "`id` = ?"), append(args, *find.ID)
|
|
}
|
|
if find.SenderID != nil {
|
|
where, args = append(where, "`sender_id` = ?"), append(args, *find.SenderID)
|
|
}
|
|
if find.ReceiverID != nil {
|
|
where, args = append(where, "`receiver_id` = ?"), append(args, *find.ReceiverID)
|
|
}
|
|
if find.Status != nil {
|
|
where, args = append(where, "`status` = ?"), append(args, *find.Status)
|
|
}
|
|
if find.MessageType != nil {
|
|
// Filter by message type using JSON extraction
|
|
// Note: The type field in JSON is stored as string representation of the enum name
|
|
where, args = append(where, "JSON_EXTRACT(`message`, '$.type') = ?"), append(args, find.MessageType.String())
|
|
}
|
|
|
|
query := "SELECT `id`, `created_ts`, `sender_id`, `receiver_id`, `status`, `message` FROM `inbox` WHERE " + strings.Join(where, " AND ") + " ORDER BY `created_ts` DESC"
|
|
if find.Limit != nil {
|
|
query = fmt.Sprintf("%s LIMIT %d", query, *find.Limit)
|
|
if find.Offset != nil {
|
|
query = fmt.Sprintf("%s OFFSET %d", query, *find.Offset)
|
|
}
|
|
}
|
|
rows, err := d.db.QueryContext(ctx, query, args...)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
defer rows.Close()
|
|
|
|
list := []*store.Inbox{}
|
|
for rows.Next() {
|
|
inbox := &store.Inbox{}
|
|
var messageBytes []byte
|
|
if err := rows.Scan(
|
|
&inbox.ID,
|
|
&inbox.CreatedTs,
|
|
&inbox.SenderID,
|
|
&inbox.ReceiverID,
|
|
&inbox.Status,
|
|
&messageBytes,
|
|
); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
message := &storepb.InboxMessage{}
|
|
if err := protojsonUnmarshaler.Unmarshal(messageBytes, message); err != nil {
|
|
return nil, err
|
|
}
|
|
inbox.Message = message
|
|
list = append(list, inbox)
|
|
}
|
|
|
|
if err := rows.Err(); err != nil {
|
|
return nil, err
|
|
}
|
|
|
|
return list, nil
|
|
}
|
|
|
|
func (d *DB) UpdateInbox(ctx context.Context, update *store.UpdateInbox) (*store.Inbox, error) {
|
|
set, args := []string{"`status` = ?"}, []any{update.Status.String()}
|
|
args = append(args, update.ID)
|
|
query := "UPDATE `inbox` SET " + strings.Join(set, ", ") + " WHERE `id` = ? RETURNING `id`, `created_ts`, `sender_id`, `receiver_id`, `status`, `message`"
|
|
inbox := &store.Inbox{}
|
|
var messageBytes []byte
|
|
if err := d.db.QueryRowContext(ctx, query, args...).Scan(
|
|
&inbox.ID,
|
|
&inbox.CreatedTs,
|
|
&inbox.SenderID,
|
|
&inbox.ReceiverID,
|
|
&inbox.Status,
|
|
&messageBytes,
|
|
); err != nil {
|
|
return nil, err
|
|
}
|
|
message := &storepb.InboxMessage{}
|
|
if err := protojsonUnmarshaler.Unmarshal(messageBytes, message); err != nil {
|
|
return nil, err
|
|
}
|
|
inbox.Message = message
|
|
return inbox, nil
|
|
}
|
|
|
|
func (d *DB) DeleteInbox(ctx context.Context, delete *store.DeleteInbox) error {
|
|
result, err := d.db.ExecContext(ctx, "DELETE FROM `inbox` WHERE `id` = ?", delete.ID)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if _, err := result.RowsAffected(); err != nil {
|
|
return err
|
|
}
|
|
return nil
|
|
}
|