Better Together Contribution Support Thread

[PSP Queen]

It is, there are no blank lines after ?> The bracket I removed was at the end of your code above.

[swguy]

Better Together 2.2 is now available. This version features another tax setting (Re-Calculate Tax = VAT) for tax included shops. If you do not run with Display Prices with Tax = true, you may skip this upgrade.

An additional change was made for currency-based discounting for tax included shops. Now *all* discounts - not just percent off discounts - are grossed up for taxes if you use include_tax = yes. In some earlier versions, I tried not grossing up dollar based discounts. However, I believe uniform treatment is a more consistent and logical approach.

[swguy]

Better Together will work fine for non-VAT liable, non-tax-included shops in 1.3.9. I will release another version after 1.3.9 is finalized which is compliant with the new style of tax handling for tax-included shops in 1.3.9.

[swguy]

Better Together 2.3 has been released, with support for Cross Selling (undiscounted linkages). Cross sells are shown in a new centerbox on the product info page.

[thelawman]

Fantastic mod!

I was added a fieldset to the marketing text template (tpl_better_together_marketing_images.php) to put a label on it. However, the fieldset and label shows even when there are no "Better Together" discounts.

<!-- bof Better Together Marketing -->
<fieldset><legend><i>Our "Better Together" Deals:</i></legend>
<?php 
  // Better Together Discount Marketing
  $value = "ot_better_together.php";
  include_once(zen_get_file_directory(DIR_WS_LANGUAGES . $_SESSION['language'] .
          '/modules/order_total/', $value, 'false'));
  include_once(DIR_WS_MODULES . "order_total/" . $value);
  $discount = new ot_better_together();
  if ($discount->check() > 0) { 
     $resp = $discount->get_discount_info($_GET['products_id'], $current_category_id, true); 
     $rresp = $discount->get_reverse_discount_info($_GET['products_id'], $current_category_id, true); 
     if ( (count($resp) > 0) || (count($rresp) > 0) ) {
        $marketing_data = array_merge($resp, $rresp); 
        echo '<div class="content" id="betterTogetherDiscountPolicy">';
        for ($i=0, $n=count($marketing_data); $i<$n; $i++) {
             echo '<hr />';
             echo '<div class="discountText">'; 
             echo $marketing_data[$i]['data'];
             echo '</div>'; 
             echo '<div class="discountImages">'; 
             echo  
                $marketing_data[$i]['first_href'] .
                $marketing_data[$i]['first_image'] . '</a>' . 
                '<span class="product_plus">+</span>' .
                $marketing_data[$i]['disc_href'] .
                $marketing_data[$i]['second_image'] . '</a>'; 
                if (isset($marketing_data[$i]['bbn_string'])) { 
                   echo '<span class="bbn_button">' . $marketing_data[$i]['bbn_string'] . '</span">';
                }
             echo '</div>'; 
         }
         echo '</div>';
         echo '<br class="clearBoth" />'; 
     }

  }
?>
</fieldset>
<!-- eof Better Together Marketing -->

I tried inserting after the IF statement but doesn't seem to like that... where should it go so as not to show up when no products are linked?

Thanks!

[swguy]

You could move the fieldset stuff below

if ( (count($resp) > 0) || (count($rresp) > 0) ) {


Or you could just create some Better Together Discounts!

[thelawman]

Thanks for such a quick reply...

You could move the fieldset stuff below
if ( (count($resp) > 0) || (count($rresp) > 0) ) {

When I do that the "Better Together" and the "May We Also Recommend" box doesn't show at all.

Or you could just create some Better Together Discounts!

might just have to do that!