Skip to main content
4 de 4
Actualización de condición que no se cumple; estaba como si se cumple.
RobertoLeOr
  • 1.6k
  • 1
  • 6
  • 7

En tu codigo (a continuación); mi sugerencia:

Siendo tú código

  if (in_array($discount_role, $user_roles) && $eligible_products_count > 0) {
    // Cambiar las condiciones para aplicar el descuento solo en múltiplos de 10
    if ($eligible_products_count % 10 == 0) {
      $discount_percent = 5;
    }
  }

Aquí estas usando if ($eligible_products_count % 10 == 0); recuerda que 11 o 14 (por decir unos números) no da como residuo 0; es aquí donde esta el detalle. Lo que debes verificar es cuantas veces es divisible el $eligible_products_count con (en tu caso) el 10. Lo que necesitas es el cociente el cual claro debe ser un numero entero para ello te sugiero lo siguiente:

  $cociente = (($eligible_products_count - ($eligible_products_count % 10)) / 10);
  if (in_array($discount_role, $user_roles) && $eligible_products_count > 0) {
    // Cambiar las condiciones para aplicar el descuento solo en múltiplos de 10
    if ($cociente>0) {
      $discount_percent = 5;
    }
  }

El $cociente te indica a cuando productos acorde a su precio debes aplicar el porcentaje de descuento; es decir si seleccionan 14; te dara 1; por tanto seria multiplicar ese 1 por 10 veces el precio del producto y aplicar el % que corresponda; si fuesen 21; te dará igual y misma operación.

De lo que leí de tu código; abajo te coloco un comentario, por que el descuento tendrías que hacer cuando es la N cantidad (quiero pensar) del mismo producto... si es el caso, tendrás que realizar los ajustes respectivos a tu código.

Comentario:

Su estructura del código abajo; me da la percepción que no está contabilizando por producto; en tu ejemplo colocas que si ingresa 10 televisores tiene derecho a descuento; pero tu codigo daria descuento si por ejemplo pongo (por decir) 5 televisores (y asumiendo radios tienen descuento) 5 radios; en total serian 10 unidades de productos con descuento.

// recorrer los productos en el carrito para contar la cantidad de productos elegibles
  foreach ($cart->cart_contents as $product) {
    if (in_array($product['product_id'], $discount_products)) {
      $eligible_products_count += $product['quantity'];
    }
  }

ACTUALIZACIÓN

Para efectos prácticos no profundizare mucho, será sugerencia de flujo y lo adaptes a tu código. Para agilidad de explicación partiré tomando de lo que tiene el carrito dos elementos estos serán Producto y Cantidad (lo que "nos interesa"); de su segundo código de aplicar_descuento_por_cantidad sería dentro del mismo realizar lo siguiente (pasos; no código):

function aplicar_descuento_por_cantidad($cart)
{
    /* lo anterior o necesario */
    // Tu código
    /*
    // recorrer los productos en el carrito para contar la cantidad de productos elegibles
    foreach ($cart->cart_contents as $product) {
       if (in_array($product['product_id'], $discount_products)) {
          $eligible_products_count += $product['quantity'];
       }
    }
    */

    /*
    Realiza una estructura que te permita agrupar los productos elegidos y la cantidad(ver nota 1).
    Lo anterior con tal de obtener un dato donde este el producto con el total de unidades solicitadas.
    Ejemplo
    */
    $productos=new array();
    foreach ($cart->cart_contents as $product) {
       /* Obtener de cada producto el total de unidades solicitadas */
       if (!in_array('product_id', producto)) {
          $productos[$product['product_id']] = new array();
          $productos[$product['product_id']]['totalUnidades'] =0; // Unidades en total
          $productos[$product['product_id']]['montoTotal'] =0;
          $productos[$product['product_id']]['precioUnidad'] =0;
          $productos[$product['product_id']]['montoDescuento'] =0;
          $productos[$product['product_id']]['porcentajeDescuento'] =0; // este debes obtenerlo y asignarlo o si es fijo ponerlo
          $productos[$product['product_id']]['ameritaDescuento'] =in_array($product['product_id'], $discount_products);
       }
       $productos[$product['product_id']]['totalUnidades'] += $product['quantity'];
    }

    /* Realizar proceso para obtener el precio por unidad de tu catálogo o donde corresponda */

    if (in_array($discount_role, $user_roles)) {
       // Cambiar las condiciones para aplicar el descuento solo en múltiplos de 10
           foreach ($productos as $producto) {
              // Cálculo del monto total (sin descuentos)
              $producto['montoTotal'] =$producto['totalUnidades'] * $producto['precioUnidad'];

              if ($producto['ameritaDescuento']) {
                 // Cálculo del descuento
                 $cociente = (($producto['totalUnidades'] - ($producto['totalUnidades'] % 10)) / 10); // si cada producto tiene descuento por unidades diferentes considera agregar igual cuantas unidades generan el descuento y reemplaza el 10
                 $producto['montoDescuento']= ($producto['precioUnidad']*10) * ($producto['porcentajeDescuento']/100) * $cociente;
              }

           }
    }

    /* si nada me falta y te es ilustrativo; luego de la interacción anterior tendrás en cada elemento el monto a descontarle */

} // aplicar_descuento_por_cantidad

(1).- Asumiendo que el usuario puede ("exagerando") 10 veces el televisor (o cualquier otro producto) con cantidad igual a 1. Realizar la sumatoria de lo que el mismo producto.

RobertoLeOr
  • 1.6k
  • 1
  • 6
  • 7