Prevent double-confirmation of same payment by different users
- Create pagos_confirmados table to track consumed email receipts - PagoValidadorService now validates: valor + fecha + hora (±10min) + llave Bancolombia - Each matched email is hashed (sha1 body+date) and marked as used; a second match returns estado=ya_usado instead of confirmado - Add partial remitente name matching against the registered user's name - GeminiVisionService prompt now extracts llave (@xxx) from the receipt image - TelegramBotService shows distinct message for ya_usado state Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Lizandro
co-authored by
Claude Sonnet 4.6
Claude Sonnet 4.6
parent
157691e166
commit
ba4ec8b6cf
Regular → Executable
+124
-20
@@ -3,18 +3,20 @@
|
||||
namespace App\Services;
|
||||
|
||||
use App\Models\ChatConfig;
|
||||
use App\Models\PagoConfirmado;
|
||||
use App\Models\WhatsappSystemConfig;
|
||||
use Illuminate\Support\Facades\Log;
|
||||
|
||||
class PagoValidadorService
|
||||
{
|
||||
/**
|
||||
* Cruza los datos extraídos por IA con los correos IMAP recientes.
|
||||
*
|
||||
* @param array $datosPago ['banco', 'valor', 'referencia', 'fecha', 'hora', 'remitente']
|
||||
* @return array ['estado' => confirmado|no_encontrado|monto_incorrecto, 'correo' => array|null]
|
||||
* @param array $datosPago ['banco','valor','referencia','fecha','hora','remitente','llave']
|
||||
* @param int|null $usuarioId
|
||||
* @param string|null $nombreUsuario nombre registrado en la plataforma
|
||||
* @param string $canal
|
||||
* @return array ['estado' => confirmado|no_encontrado|monto_incorrecto|ya_usado, 'correo' => array|null]
|
||||
*/
|
||||
public function validar(array $datosPago): array
|
||||
public function validar(array $datosPago, ?int $usuarioId = null, ?string $nombreUsuario = null, string $canal = 'telegram'): array
|
||||
{
|
||||
if (WhatsappSystemConfig::get('correo_imap_enabled', '0') !== '1') {
|
||||
return ['estado' => 'no_encontrado', 'correo' => null, 'motivo' => 'correo_deshabilitado'];
|
||||
@@ -31,7 +33,11 @@ class PagoValidadorService
|
||||
return ['estado' => 'no_encontrado', 'correo' => null, 'motivo' => 'sin_correos'];
|
||||
}
|
||||
|
||||
$valorIA = (int) ($datosPago['valor'] ?? 0);
|
||||
$valorIA = (int) ($datosPago['valor'] ?? 0);
|
||||
$tolerancia = (int) ChatConfig::get('validacion_monto_tolerancia', '0');
|
||||
$horaIA = $datosPago['hora'] ?? '';
|
||||
$fechaIA = $datosPago['fecha'] ?? '';
|
||||
$llaveIA = strtolower(preg_replace('/\s+/', '', $datosPago['llave'] ?? ''));
|
||||
|
||||
foreach ($correos as $correo) {
|
||||
$datosCorreo = BancolombiaParser::parse($correo['body'] ?? '');
|
||||
@@ -40,31 +46,129 @@ class PagoValidadorService
|
||||
continue;
|
||||
}
|
||||
|
||||
$valorCorreo = (int) ($datosCorreo['valor'] ?? 0);
|
||||
|
||||
// Verificar referencia si ambas están disponibles
|
||||
if (! empty($datosPago['referencia']) && ! empty($datosCorreo['referencia'])) {
|
||||
if ($datosPago['referencia'] === $datosCorreo['referencia']) {
|
||||
return ['estado' => 'confirmado', 'correo' => $datosCorreo];
|
||||
// ── 1. Valor ────────────────────────────────────────────
|
||||
$valorCorreo = (int) preg_replace('/[^0-9]/', '', (string) ($datosCorreo['valor'] ?? '0'));
|
||||
if ($valorIA <= 0 || $valorCorreo <= 0 || abs($valorIA - $valorCorreo) > $tolerancia) {
|
||||
if ($valorCorreo > 0 && $valorIA > 0 && abs($valorIA - $valorCorreo) > $tolerancia) {
|
||||
// Monto distinto — seguir buscando en el siguiente correo
|
||||
continue;
|
||||
}
|
||||
if ($valorIA <= 0 || $valorCorreo <= 0) {
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
// Verificar valor
|
||||
if ($valorIA > 0 && $valorCorreo > 0) {
|
||||
$tolerancia = (int) ChatConfig::get('validacion_monto_tolerancia', '0');
|
||||
// ── 2. Hash único del correo (anti-doble-uso) ───────────
|
||||
$emailHash = sha1(($correo['body'] ?? '') . ($correo['date'] ?? ''));
|
||||
|
||||
if (abs($valorIA - $valorCorreo) <= $tolerancia) {
|
||||
return ['estado' => 'confirmado', 'correo' => $datosCorreo];
|
||||
} else {
|
||||
return ['estado' => 'monto_incorrecto', 'correo' => $datosCorreo,
|
||||
'esperado' => $valorCorreo, 'recibido' => $valorIA];
|
||||
if (PagoConfirmado::yaUsado($emailHash)) {
|
||||
// Este correo ya confirmó a otro usuario
|
||||
return ['estado' => 'ya_usado', 'correo' => $datosCorreo,
|
||||
'motivo' => 'correo_ya_aplicado'];
|
||||
}
|
||||
|
||||
// ── 3. Fecha (mismo día) ────────────────────────────────
|
||||
if ($fechaIA && ($datosCorreo['fecha'] ?? '')) {
|
||||
if (! $this->mismoDia($fechaIA, $datosCorreo['fecha'])) {
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
// ── 4. Hora (±10 min) ───────────────────────────────────
|
||||
if ($horaIA && ($datosCorreo['hora'] ?? '')) {
|
||||
if (! $this->horaProxima($horaIA, $datosCorreo['hora'], 10)) {
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
// ── 5. Llave Bancolombia (si la foto la tiene) ──────────
|
||||
if ($llaveIA) {
|
||||
$llaveCorreo = strtolower(preg_replace('/\s+/', '', $datosCorreo['llave'] ?? ''));
|
||||
if ($llaveCorreo && $llaveIA !== $llaveCorreo) {
|
||||
continue;
|
||||
}
|
||||
}
|
||||
|
||||
// ── 6. Remitente (coincidencia parcial con nombre de usuario) ──
|
||||
$coincideRemitente = $this->remitenteCoincide(
|
||||
$datosCorreo['remitente'] ?? '',
|
||||
$nombreUsuario ?? ''
|
||||
);
|
||||
|
||||
// Todos los filtros pasados → confirmar
|
||||
PagoConfirmado::marcar($emailHash, [
|
||||
'usuario_id' => $usuarioId,
|
||||
'valor' => $valorIA,
|
||||
'canal' => $canal,
|
||||
'banco' => $datosCorreo['banco'] ?? null,
|
||||
'referencia' => $datosPago['referencia'] ?? null,
|
||||
]);
|
||||
|
||||
return [
|
||||
'estado' => 'confirmado',
|
||||
'correo' => $datosCorreo,
|
||||
'coincide_remitente' => $coincideRemitente,
|
||||
];
|
||||
}
|
||||
|
||||
// Si algún correo tenía el monto correcto pero otro campo falló, devolver monto_incorrecto
|
||||
// de lo contrario no_encontrado
|
||||
return ['estado' => 'no_encontrado', 'correo' => null, 'motivo' => 'sin_coincidencia'];
|
||||
}
|
||||
|
||||
// ── Helpers ──────────────────────────────────────────────────────────
|
||||
|
||||
private function mismoDia(string $fechaA, string $fechaB): bool
|
||||
{
|
||||
// Normaliza dd/mm/yyyy o dd/mm/yy
|
||||
$normA = $this->normalizarFecha($fechaA);
|
||||
$normB = $this->normalizarFecha($fechaB);
|
||||
return $normA && $normB && $normA === $normB;
|
||||
}
|
||||
|
||||
private function normalizarFecha(string $fecha): ?string
|
||||
{
|
||||
if (preg_match('#(\d{1,2})/(\d{1,2})/(\d{2,4})#', $fecha, $m)) {
|
||||
$y = strlen($m[3]) === 2 ? '20' . $m[3] : $m[3];
|
||||
return sprintf('%02d/%02d/%s', $m[1], $m[2], $y);
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
private function horaProxima(string $horaA, string $horaB, int $minutos): bool
|
||||
{
|
||||
$toMin = fn (string $h): ?int => preg_match('/(\d{1,2}):(\d{2})/', $h, $m)
|
||||
? (int) $m[1] * 60 + (int) $m[2]
|
||||
: null;
|
||||
|
||||
$a = $toMin($horaA);
|
||||
$b = $toMin($horaB);
|
||||
|
||||
return $a !== null && $b !== null && abs($a - $b) <= $minutos;
|
||||
}
|
||||
|
||||
private function remitenteCoincide(string $remitenteEmail, string $nombreUsuario): bool
|
||||
{
|
||||
if (! $remitenteEmail || ! $nombreUsuario) {
|
||||
return false;
|
||||
}
|
||||
// Comparación parcial insensible a mayúsculas y tildes
|
||||
$normalizar = fn (string $s): string => strtolower(
|
||||
iconv('UTF-8', 'ASCII//TRANSLIT//IGNORE', $s) ?: $s
|
||||
);
|
||||
|
||||
$re = $normalizar($remitenteEmail);
|
||||
$nu = $normalizar($nombreUsuario);
|
||||
|
||||
// Basta con que una palabra del nombre de usuario aparezca en el remitente del email
|
||||
foreach (explode(' ', $nu) as $palabra) {
|
||||
if (strlen($palabra) >= 3 && str_contains($re, $palabra)) {
|
||||
return true;
|
||||
}
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
private function fetchCorreos(): array
|
||||
{
|
||||
$minutos = (int) WhatsappSystemConfig::get('correo_imap_minutos', '5');
|
||||
|
||||
Reference in New Issue
Block a user