NFT Dealers

Description: En collectUsdcFromSelling, la instrucción usdc.safeTransfer(address(this), fees) ejecuta una transferencia del contrato hacia sí mismo. En términos contables no mueve valor neto ni cambia el balance final del contrato.

Why this matters: Aunque no genera pérdida directa de fondos, introduce ruido en la lógica y puede confundir futuras revisiones, monitoreo on-chain y reconciliaciones financieras (parece que “se transfieren fees” cuando realmente no ocurre movimiento económico).

Security impact: Bajo/no explotable de forma aislada. No permite robo por sí solo.

Operational impact: Riesgo de errores de mantenimiento y lectura incorrecta del flujo de cobro.

Recommended mitigation: Eliminar esa línea y dejar el flujo explícito: actualizar contabilidad de fees y transferir únicamente el neto al seller.