Forráskód Böngészése

♻️ refactor(core): 清理未使用的函数参数和变量

- 在创建系统配置路由中移除未使用的 `user` 变量
- 在角色服务中将未使用的 `dataSource`、`userId` 参数重命名为带下划线前缀
- 在通用CRUD服务中将未使用的 `userId`、`adminRole` 参数重命名为带下划线前缀
yourname 5 napja
szülő
commit
b098a3b2cf

+ 0 - 1
packages/core-module/system-config-module/src/routes/custom/create-system-config.ts

@@ -42,7 +42,6 @@ const createSystemConfigRoutes = new OpenAPIHono<AuthContext>()
   // 创建系统配置路由
   .openapi(createSystemConfigRoute, async (c) => {
     const data = c.req.valid('json');
-    const user = c.get('user');
 
     try {
       const systemConfigService = new SystemConfigService(AppDataSource);

+ 3 - 3
packages/core-module/user-module/src/services/role.service.ts

@@ -3,7 +3,7 @@ import { Role } from '../entities/role.entity';
 import { ConcreteCrudService } from '@d8d/shared-crud';
 
 export class RoleService extends ConcreteCrudService<Role> {
-  constructor(dataSource: DataSource) {
+  constructor(_dataSource: DataSource) {
     super(Role, {
       userTracking: {
         // 禁用用户追踪
@@ -72,7 +72,7 @@ export class RoleService extends ConcreteCrudService<Role> {
   /**
    * 更新角色
    */
-  override async update(id: number, data: Partial<Role>, userId?: string | number): Promise<Role | null> {
+  override async update(id: number, data: Partial<Role>, _userId?: string | number): Promise<Role | null> {
     try {
       // 首先验证角色是否存在
       const existingRole = await this.getById(id);
@@ -92,7 +92,7 @@ export class RoleService extends ConcreteCrudService<Role> {
   /**
    * 删除角色
    */
-  override async delete(id: number, userId?: string | number): Promise<boolean> {
+  override async delete(id: number, _userId?: string | number): Promise<boolean> {
     try {
       // 首先验证角色是否存在
       const existingRole = await this.getById(id);

+ 2 - 2
packages/shared-crud/src/services/generic-crud.service.ts

@@ -251,7 +251,7 @@ export abstract class GenericCrudService<T extends ObjectLiteral> {
    * 检查用户是否为管理员
    * TODO: 需要集成实际的用户角色检查
    */
-  private async checkAdminRole(userId: string | number, adminRole: string): Promise<boolean> {
+  private async checkAdminRole(_userId: string | number, _adminRole: string): Promise<boolean> {
     // 这里需要从认证系统获取用户角色信息
     // 暂时返回false,实际实现中需要集成用户角色检查
     return false;
@@ -261,7 +261,7 @@ export abstract class GenericCrudService<T extends ObjectLiteral> {
    * 提取租户ID
    * 从用户对象或认证上下文中提取租户ID
    */
-  private async extractTenantId(userId?: string | number): Promise<string | number | undefined> {
+  private async extractTenantId(_userId?: string | number): Promise<string | number | undefined> {
     // 首先检查是否有存储的租户上下文
     if ((this as any)._tenantId !== undefined) {
       console.debug('从存储的租户上下文中获取租户ID:', (this as any)._tenantId);