Options as Images for MS2 By Mark Stephens

[aliharis2004]

Hi I am trying to setup this contribution " Options as Images for MS2" by Mark ... if someone has used it, I would appreciate their help ... the link to the contribution is :

 

http://www.oscommerce.com/community/contri...ns,1467/page,16

 

To my understanding I followed the instruction to the letter, however there is one area that is not clear to me which requires updating of product_info.php,

 

According to the instructions:

 

Add 2 further lines at around lines 165 and 170 in product_info.php as demonstrated in the following code by the "//Options as Images" comments:-

 

 

    <td class="main"><?php echo $products_options_name['products_options_name'] . ':'; ?></td>

    <td class="main"><?php echo tep_draw_pull_down_menu('id[' . $products_options_name['products_options_id'] . ']', $products_options_array, $selected_attribute); ?></td>

    </tr>

<?php

      }

?>

          </table>

<?php

 

    }

 

//Options as Images. Add the curly bracket as shown on the next line

}

?>

 

<?php

//Options as Images. This whole php clause needs to be added

if (OPTIONS_AS_IMAGES_ENABLED == 'true') include ('options_images.php');

?>

 

From what I understand this is what I am doing, after

 

<td class="main"><?php echo $products_options_name['products_options_name'] . ':'; ?></td>

 

and before

 

<td class="main"><?php echo $tmp_html;  ?></td>

            </tr>

<?php

            break;

          case PRODUCTS_OPTIONS_TYPE_CHECKBOX:

 

I am adding the following lines :

 

<td class="main"><?php echo tep_draw_pull_down_menu('id[' . $products_options_name['products_options_id'] . ']', $products_options_array, $selected_attribute); ?></td>

    </tr>

<?php

      }

?>

          </table>

<?php

 

    }

 

//Options as Images. Add the curly bracket as shown on the next line

}

?>

 

<?php

//Options as Images. This whole php clause needs to be added

if (OPTIONS_AS_IMAGES_ENABLED == 'true') include ('options_images.php');

?>

 

But after which I end up with the following error :

 

Parse error: parse error in /hsphere/local/home/mosaic/mosaicartkeepsakes.com/catalog/product_info.php on line 208

 

And line 208 is :

 

case PRODUCTS_OPTIONS_TYPE_CHECKBOX:

 

If somebody have used the contribution and could tell me what am I doing wrong ....

 

Thanks

[♥ecartz]

At a guess, you are closing the switch statement early with Mark's changes. When integrating with the Option Type Feature, you need to be careful to keep all the changes in a single case. Any new { need to be matched with a } and vice versa.

 

Hth,

Matt

[aliharis2004]

Thanks Matt, you were right I was missing a closing brace ... But here is yet another question which I think you can only answer ... this is in reference to the same contribution "Options as Images"

 

http://www.oscommerce.com/community/contri...ns,1467/page,16

 

The thing is after I make changes to the product_info. php it messes up your 'upload option" contribution,

 

Here is what I am adding

 

if (OPTIONS_AS_IMAGES_ENABLED == 'false'){

 

before

 

$products_attributes_query = tep_db_query("select count(*) as total from " . TABLE_PRODUCTS_OPTIONS . " popt, " . TABLE_PRODUCTS_ATTRIBUTES . " patrib where patrib.products_id='" . (int)$HTTP_GET_VARS['products_id'] . "' and patrib.options_id = popt.products_options_id and popt.language_id = '" . (int)$languages_id . "'");

    $products_attributes = tep_db_fetch_array($products_attributes_query);

    if ($products_attributes['total'] > 0)

 

And

 

<?php

if (OPTIONS_AS_IMAGES_ENABLED == 'true') include ('options_images.php');

?>

 

Before

<td><?php echo tep_draw_separator('pixel_trans.gif', '100%', '10'); ?></td>

      </tr>

<?php

    $reviews_query = tep_db_query("select count(*) as count from " . TABLE_REVIEWS . " where products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "'");

    $reviews = tep_db_fetch_array($reviews_query);

    if ($reviews['count'] > 0) {

?>

 

And the Options_images.php looks something liek this :

 

<?php

/*

  $Id: options_images.php,v 1.0 2003/08/18

 

  osCommerce, Open Source E-Commerce Solutions

  http://www.oscommerce.com

 

  Copyright ? 2003 osCommerce

 

  Released under the GNU General Public License

*/

 

 

 

    $products_attributes_query = tep_db_query("select count(*) as total from " . TABLE_PRODUCTS_OPTIONS . " popt, " . TABLE_PRODUCTS_ATTRIBUTES . " patrib where patrib.products_id='" . (int)$HTTP_GET_VARS['products_id'] . "' and patrib.options_id = popt.products_options_id and popt.language_id = '" . (int)$languages_id . "'");

    $products_attributes = tep_db_fetch_array($products_attributes_query);

    if ($products_attributes['total'] > 0) {

?>

          <table border="0" cellspacing="0" cellpadding="2">

            <tr>

              <?php  echo '<td class="main" colspan="2">' . TEXT_PRODUCT_OPTIONS . '<br>Please select your desired option using the buttons provided'; ?>

      <?php if (OPTIONS_IMAGES_CLICK_ENLARGE == 'true') echo '<br>Click the images to enlarge';?>

      </td>

            </tr>

<?php

      $products_options_name_query = tep_db_query("select distinct popt.products_options_id, popt.products_options_name, popt.products_options_images_enabled from " . TABLE_PRODUCTS_OPTIONS . " popt, " . TABLE_PRODUCTS_ATTRIBUTES . " patrib where patrib.products_id='" . (int)$HTTP_GET_VARS['products_id'] . "' and patrib.options_id = popt.products_options_id and popt.language_id = '" . (int)$languages_id . "' order by popt.products_options_name");

      while ($products_options_name = tep_db_fetch_array($products_options_name_query)) {

        $products_options_array = array();

        $products_options_query = tep_db_query("select pov.products_options_values_id, pov.products_options_values_name, pov.products_options_values_thumbnail, pa.options_values_price, pa.price_prefix from " . TABLE_PRODUCTS_ATTRIBUTES . " pa, " . TABLE_PRODUCTS_OPTIONS_VALUES . " pov where pa.products_id = '" . (int)$HTTP_GET_VARS['products_id'] . "' and pa.options_id = '" . (int)$products_options_name['products_options_id'] . "' and pa.options_values_id = pov.products_options_values_id and pov.language_id = '" . (int)$languages_id . "'");

        while($products_options = tep_db_fetch_array($products_options_query)){

          $products_options_array[] = array('id' => $products_options['products_options_values_id'], 'text' => $products_options['products_options_values_name'], 'thumbnail' => $products_options['products_options_values_thumbnail']);

          if ($products_options['options_values_price'] != '0') {

            $products_options_array[sizeof($products_options_array)-1]['text'] .= ' (' . $products_options['price_prefix'] . $currencies->display_price($products_options['options_values_price'], tep_get_tax_rate($product_info['products_tax_class_id'])) .') ';

          }

        }

   

      if (isset($cart->contents[$HTTP_GET_VARS['products_id']]['attributes'][$products_options_name['products_options_id']])) {

          $selected_attribute = $cart->contents[$HTTP_GET_VARS['products_id']]['attributes'][$products_options_name['products_options_id']];

        } else {

          $selected_attribute = false;

        }

 

?>

 

            <tr>

              <td class="main" valign="top"><?php echo $products_options_name['products_options_name'] . ':'; ?></td>

   

             

     

<?php

      if ($products_options_name['products_options_images_enabled'] == 'false'){

        echo '<td class="main">' . tep_draw_pull_down_menu('id[' . $products_options_name['products_options_id'] . ']', $products_options_array, $selected_attribute) . '</td></tr>';

              }   

              else {

        $count=0;

        echo '<td class="main"><table><tr>';

                foreach ($products_options_array as $opti_array){

          echo '<td><table cellspacing="1" cellpadding="0" border="0">';

          if (OPTIONS_IMAGES_CLICK_ENLARGE == 'true')

            echo '<td align="center"><a href="java script:popupWindow(\'' . tep_href_link(FILENAME_OPTIONS_IMAGES_POPUP, 'oID=' . $opti_array['id']) .'\')">' . tep_image(DIR_WS_IMAGES . 'options/' . $opti_array['thumbnail'], $opti_array['text'], OPTIONS_IMAGES_WIDTH, OPTIONS_IMAGES_HEIGHT) . '</a></td></tr>';

            else echo '<tr><td align="center">' . tep_image(DIR_WS_IMAGES . 'options/' . $opti_array['thumbnail'], $opti_array['text'], OPTIONS_IMAGES_WIDTH, OPTIONS_IMAGES_HEIGHT) . '</td></tr>';

              echo '<tr><td class="main" align="center">' . $opti_array['text'] . '</td></tr>';

              echo '<tr><td align="center"><input type="radio" name ="id[' . $products_options_name['products_options_id'] . ']" value="' . $opti_array['id'] . '" checked></td></tr></table></td>';

              $count++;

                      if ($count%OPTIONS_IMAGES_NUMBER_PER_ROW == 0) {

                echo '</tr><tr>';

                $count = 0;

              }

            }

        echo '</table>';

        }

 

?>

        </td></tr>

<?php

      }

?>

    </table>

<?php

    }

?>

 

Do you think there is a way around it ....

 

Thanks

Edited February 25, 2004 by aliharis2004