From ea09631d4ce2ef476013d097ddf5485cc452cbab Mon Sep 17 00:00:00 2001
From: Yavolte <Yavolte@nobla.cn>
Date: Thu, 19 Jun 2025 11:19:58 +0800
Subject: [PATCH] fix bugs

---
 internal/logic/department.go | 2 +-
 internal/logic/menu.go       | 2 +-
 internal/logic/role.go       | 4 ++--
 internal/logic/user.go       | 2 +-
 pkg/dbcache/cache.go         | 8 ++++++--
 5 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/internal/logic/department.go b/internal/logic/department.go
index b0ef290..2423579 100644
--- a/internal/logic/department.go
+++ b/internal/logic/department.go
@@ -94,6 +94,6 @@ func NewDepartmentLogic(db *gorm.DB, ch cache.Cache) *Department {
 	return &Department{
 		db:            db,
 		cache:         ch,
-		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM departments"),
+		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM `departments`"),
 	}
 }
diff --git a/internal/logic/menu.go b/internal/logic/menu.go
index 5614bad..3c7800e 100644
--- a/internal/logic/menu.go
+++ b/internal/logic/menu.go
@@ -44,6 +44,6 @@ func NewMenuLogic(db *gorm.DB, ch cache.Cache) *Menu {
 	return &Menu{
 		db:            db,
 		cache:         ch,
-		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM menus"),
+		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM `menus`"),
 	}
 }
diff --git a/internal/logic/role.go b/internal/logic/role.go
index 6d66744..4df33ad 100644
--- a/internal/logic/role.go
+++ b/internal/logic/role.go
@@ -59,7 +59,7 @@ func NewRoleLogic(db *gorm.DB, ch cache.Cache) *Role {
 	return &Role{
 		db:                      db,
 		cache:                   ch,
-		sqlDependency:           dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM roles"),
-		permissionSqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM permissions"),
+		sqlDependency:           dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM `roles`"),
+		permissionSqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM `permissions`"),
 	}
 }
diff --git a/internal/logic/user.go b/internal/logic/user.go
index e33a61f..70bff08 100644
--- a/internal/logic/user.go
+++ b/internal/logic/user.go
@@ -67,6 +67,6 @@ func NewUserLogic(db *gorm.DB, ch cache.Cache) *User {
 	return &User{
 		db:            db,
 		cache:         ch,
-		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM users"),
+		sqlDependency: dbcache.NewSqlDependency("SELECT MAX(`updated_at`) FROM `users`"),
 	}
 }
diff --git a/pkg/dbcache/cache.go b/pkg/dbcache/cache.go
index 71919fd..b53c30c 100644
--- a/pkg/dbcache/cache.go
+++ b/pkg/dbcache/cache.go
@@ -37,13 +37,17 @@ func WithDB(db *gorm.DB) CacheOption {
 
 func WithCache(c cache.Cache) CacheOption {
 	return func(o *CacheOptions) {
-		o.cache = c
+		if o != nil {
+			o.cache = c
+		}
 	}
 }
 
 func WithCacheDuration(d time.Duration) CacheOption {
 	return func(o *CacheOptions) {
-		o.cacheDuration = d
+		if d > 0 {
+			o.cacheDuration = d
+		}
 	}
 }