Model List support

[thedjphat]

The other plugin was Corner Site Seal. Sorry about crossposting, but I thought it might be relevant.

[thedjphat]

I'm betting a lot of those errors are caused by the corner seal - the model_list errors start at line 178.

[gjh42]

The Model List errors are all caused by a snippet of link code I copied without examining deeply. It needs to have its ampersands escaped.
In /includes/modules/your_template/model_list.php, replace lines 36 and 47:

  $content .= '<h3><a href="index.php?main_page=advanced_search_result&search_in_description=1&keyword=' . $mfrs_name . '" title="' . MODEL_LIST_ALT_LINK . $mfrs_name . '">' . /*$mfrs_name*/ title_image_exists($mfrs_name, 'model_list_mfr' . $model_list[$mfr_key]['mfrs_id']) . '</a>' . MODEL_LIST_MFR_COMMENT . '</h3>' . "\n";
//should be
  $content .= '<h3><a href="index.php?main_page=advanced_search_result&amp;search_in_description=1&amp;keyword=' . $mfrs_name . '" title="' . MODEL_LIST_ALT_LINK . $mfrs_name . '">' . /*$mfrs_name*/ title_image_exists($mfrs_name, 'model_list_mfr' . $model_list[$mfr_key]['mfrs_id']) . '</a>' . MODEL_LIST_MFR_COMMENT . '</h3>' . "\n";

      $content .= '  <li><a href="index.php?main_page=advanced_search_result&search_in_description=1&keyword=' . $current_model . '" title="' . MODEL_LIST_ALT_LINK . $current_model . '">' . $current_model . '</a></li>' . "\n";
//should be
      $content .= '  <li><a href="index.php?main_page=advanced_search_result&amp;search_in_description=1&amp;keyword=' . $current_model . '" title="' . MODEL_LIST_ALT_LINK . $current_model . '">' . $current_model . '</a></li>' . "\n"; 

[thedjphat]

here's the repaired and modified for sidebox code, with a sidebox built in

[gjh42]

Since I didn't mention it before, Model List is available in Free Addons.
I will update it with the validation fix, and will consider including a readymade sidebox option.

[DaMixa]

here's the repaired and modified for sidebox code, with a sidebox built in

Hi,
Your site & sidebox looks a lot better than mine.
Can you tell me what you edited in css etc etc? or can I be cheeky and ask for a copy of your files?

Thanks
Mike

[thedjphat]

sure, here ya go.

[DaMixa]

sure, here ya go.

You are now officially a legend sir!
Thanks , saved me soooo much time!

[autoace]

I'm a bit confused, does this mod display only in a sidebox or can it be displayed in the center column as well?

[gjh42]

The official version in Free Addons defaults to displaying the model list in the center column of Define Page 4 (for demo purposes), but you can display it anywhere by putting the "include" statement described in the readme into the appropriate file. The zip attached above in posts 14 and 17 also has files to make a model list sidebox.