Monitor Reply

Support

Should PHP switch work inside a loop?

Permalink Browser Info Environment March 31, 2016 at 2:09 PM

I'm trying to use a switch statement inside a foreach loop, but in the output, it seems to just skip over the switch without doing anything. I'm not a PHP expert, but from what I've read, it seems like this should work. I'm using the select field with "build PHP switch and I'll code the rest" and I've made it repeatable. Can you tell if it's something I'm doing wrong, or is this a level of complexity that Block Designer Pro can't quite handle?

<?php  if (!empty($product_items)) { ?>
   <?php $n = 1; ?>
    <?php  foreach ($product_items as $product_item_key => $product_item) { ?>
   <tr>
      <td class="name"><?php  if (isset($product_item["nameDisplayed"]) && trim($product_item["nameDisplayed"]) != "") { ?>
         <p class="product-name"><?php  echo h($product_item["nameDisplayed"]); ?></p><?php  } ?><?php  if (isset($product_item["subtext"]) && trim($product_item["subtext"]) != "") { ?>
         <p class="product-subtext"><?php  echo h($product_item["subtext"]); ?></p><?php  } ?>
      </td>
      <td class="price"><?php  if (isset($product_item["price"]) && trim($product_item["price"]) != "") { ?>
         <p class="product-price">$<?php  echo h($product_item["price"]); ?></p><?php  } ?>
      </td>
<?php  if (trim($availability) != "") { ?>
<?php  switch($availability) { ?>
<?php case "1": ?>
   <td class="td-quantity"><?php  if (isset($product_item["name"]) && trim($product_item["name"]) != "") { ?>

Viewing 15 lines of 36 lines. View entire code block.

 
<?php  if (!empty($product_items)) { ?>
   <?php $n = 1; ?>
    <?php  foreach ($product_items as $product_item_key => $product_item) { ?>
   <tr>
      <td class="name"><?php  if (isset($product_item["nameDisplayed"]) && trim($product_item["nameDisplayed"]) != "") { ?>
         <p class="product-name"><?php  echo h($product_item["nameDisplayed"]); ?></p><?php  } ?><?php  if (isset($product_item["subtext"]) && trim($product_item["subtext"]) != "") { ?>
         <p class="product-subtext"><?php  echo h($product_item["subtext"]); ?></p><?php  } ?>
      </td>
      <td class="price"><?php  if (isset($product_item["price"]) && trim($product_item["price"]) != "") { ?>
         <p class="product-price">$<?php  echo h($product_item["price"]); ?></p><?php  } ?>
      </td>
<?php  if (trim($availability) != "") { ?>
<?php  switch($availability) { ?>
<?php case "1": ?>
   <td class="td-quantity"><?php  if (isset($product_item["name"]) && trim($product_item["name"]) != "") { ?>
      <input type="hidden" value="<?php  echo h($product_item["name"]); ?>" name="<?php echo $n; ?>:name">
      <?php  } ?><?php  if (isset($product_item["price"]) && trim($product_item["price"]) != "") { ?>
      <input type="hidden" value="<?php  echo h($product_item["price"]); ?>" name="<?php echo $n; ?>:price">
      <?php  } ?><?php  if (isset($product_item["code"]) && trim($product_item["code"]) != "") { ?>
      <input type="hidden" value="<?php  echo h($product_item["code"]); ?>" name="<?php echo $n; ?>:code">
      <?php  } ?><?php  if (isset($product_item["category"]) && trim($product_item["category"]) != "" && array_key_exists($product_item["category"], $product["category_options"])) { ?>
      <input type="hidden" value="<?php  echo $product["category_options"][$product_item["category"]]; ?>" name="<?php echo $n; ?>:category">
      <?php  } ?><?php  if (isset($product_item["weight"]) && trim($product_item["weight"]) != "") { ?>
      <input type="hidden" value="<?php  echo h($product_item["weight"]); ?>" name="<?php echo $n; ?>:weight">
      <?php  } ?><?php  if (isset($product_item["noFirstClass"]) && trim($product_item["noFirstClass"]) != "" && array_key_exists($product_item["noFirstClass"], $product["noFirstClass_options"])) { ?>
      <input type="hidden" value="<?php  echo $product["noFirstClass_options"][$product_item["noFirstClass"]]; ?>" name="<?php echo $n; ?>:no_first_class"><?php  } ?>
      <input class="quantity-box" type="text" value="0" name="<?php echo $n; ?>:quantity">
   </td>
    <?php break; ?>
<?php case "2": ?>
   <td class="out-of-stock">Temporarily out of stock</td>
    <?php break; ?>
<?php } ?><?php  } ?>
</tr>
<?php $n++; ?>
<?php  } ?><?php  } ?>

I hoped this would output three table cells per row, with the third cell changing depending on the case. But it only outputs the first two cells and then goes on to the next row. Any help would be greatly appreciated!

Type:	Pre-Sale
Status:	Resolved

DustyStrings replied on Mar 31, 2016 at 3:21 pm Permalink Reply

Sorry, this was supposed to be a support ticket, not a pre-sale question!

ramonleenders replied on Mar 31, 2016 at 4:48 pm Permalink Reply

Hi there,

Is the "availability" value being saved when you actually save the form? If not, try out the first thing, noted on the Q&A page here -http://www.concrete5.org/marketplace/addons/block-designer/question...

Seeing the switch, there's a key "1" and "2". I am assuming you did not set the keys yourselves or wanted these keys, that's fine. Perhaps the value you actually save is none of the above? I do not know your Block setup/configuration, so I'd have to guess here.

As far as I can see, the code is looking fine. Availability equals 1, output some hidden values, availablitly equals 2, it will say out of stock. I do assume you have this availability field set on required? If not and left blank, it will not do anything. So for those products without an availability set, it will not do a thing.

Hope this helped?

Kind regards,

Ramon

DustyStrings replied on Mar 31, 2016 at 7:17 pm Permalink Reply

Thanks for the response! I just refreshed the block, but even when I create a new instance of the block on a page, it's still the same problem. How would I check if the availability value is being saved?

You are correct, I did not set the keys. I have a select field and I set the available values to "In Stock" and "Out of Stock." The field is required and defaults to "In Stock." The switch statement is then created by Block Designer, showing the "1" and "2" keys.

ramonleenders replied on Apr 1, 2016 at 3:13 am Permalink Reply

Oh wait, maybe I see the error in all this. You can enter the availability per product, yes? Then it shouldn't be $availability, but $product_item["availability"].

<?php if (trim($product_item["availability"]) != "") { ?>
<?php switch($product_item["availability"]) { ?>

Let me know if that did the trick.

ramonleenders replied on Apr 1, 2016 at 5:49 am Permalink Reply

I can confirm this was a bug within Block Designer upon building the switch. It assumes it's not repeating. You can fix it with the code I gave you above.

If you want the updated version, update both Block Designer and Block Designer Pro to version 2.0.0. Both versions have A LOT of changes and bugfixes. A lot of code has been changed to improve everything yet again (with the latest concrete5 standards). This makes it a .0 version release, so I hope I got all bugs out.

Kind regards,

Ramon

DustyStrings replied on Apr 1, 2016 at 12:54 pm Permalink Reply

Thank you, that fixes it! I appreciate the help!

I will update both blocks to the new version.

Block Designer Pro

Support

Should PHP switch work inside a loop?

Issue Reply

concrete5 Environment Information

Browser User-Agent String

Hide Post Content

Request Refund

Code

Delete Post