fix: marcar pago manual no toca estado ni fecha_vencimiento

MarcarContratoPagado siempre sobreescribía estado='activo' y renovaba
fecha_vencimiento, lo que causaba doble renovación si el admin ya había
ajustado el contrato manualmente. ConfirmarPagoManual solo limpia
pago_confirmado, fecha_pago y el enlace Bold, sin tocar nada más.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

rest/controllers/contrato_controller.go

@@ -444,13 +444,15 @@ func VerificarPagoBold(c *fiber.Ctx) error {
 }
 
 // MarcarPagoManual confirma manualmente el pago de un contrato sin pasar por Bold.
+// Solo limpia el enlace y marca pago_confirmado; NO toca estado ni fecha_vencimiento
+// porque el admin puede haberlos ajustado manualmente antes de confirmar el pago.
 // POST /api/contratos/:id/marcar-pagado
 func MarcarPagoManual(c *fiber.Ctx) error {
 	id, err := strconv.ParseUint(c.Params("id"), 10, 64)
 	if err != nil {
 		return c.Status(400).JSON(fiber.Map{"error": "ID inválido"})
 	}
-	ok, err := models.MarcarContratoPagado(uint(id))
+	ok, err := models.ConfirmarPagoManual(uint(id))
 	if err != nil {
 		return c.Status(500).JSON(fiber.Map{"error": err.Error()})
 	}