Manufacturer Module debug help

[lyonstudio]

I entered in manufacturer.php module under /catalog/includes/modules/manufacturers.php to create a manufacturer page. This is from "Deep Inside osCommerce: The Cookbook: Ready-to-use recipes to customize and extend your e-commerce website" by Monika Mathé which I purchased from Amazon.com

 

After hours of tweaking, guessing, and looking at similar code in the Contributions sections, I am asking for help.

 

I keep getting this error but cannot locate the problem with the code on that line as well as before or after it:

 

Parse error: syntax error, unexpected T_STRING in /mnt/w0008/d33/s28/b006a8d1/www/lyonstudiojewelry.com/store/catalog/includes/modules/manufacturers.php on line 23

 

Here is the code of the file: I marked what should be the offending code in red. If you count the lines here, it is different that what I see in my editor (notepad++)

 

<?php

/*

$Id: manufacturers.php,v 1.00 2009

 

Copyright © 2003 osCommerce

 

Released under the GNU General Public License

*/

 

?>

<!-- manufacturers //-->

<?php

$info_box_contents = array();

$info_box_contents[] = array('text' => TEXT_BOX_ALL_MANUFACTURERS);

 

new contentBoxHeading($info_box_contents);

 

$manufacturers_query = tep_db_query("select distinct m.manufacturers_id, m.manufacturers_name, m.manufacturers_image from " . TABLE_MANUFACTURERS . " m where manufacturers_name <> '' order by manufacturers_name);

$row = 0;

$col = 0;

$info_box_contents = array ();

while ($manufacturers = tep_db_fetch_array($manufacturers_query)) {

$info_box_contents[$row][$col] = array('align' => 'center','params' => 'class="smallText" width="33%" valign="middle"',

'text' => '<a href="' .

tep_href_link(FILENAME_DEFAULT, 'manufacturers_id=' .

$manufacturers['manufacturers_id']) . '">' .

tep_image(DIR_WS_IMAGES .

$manufacturers['manufacturers_image'],

$manufacturers['manufacturers_name'],

SMALL_IMAGE_WIDTH,

SMALL_IMAGE_HEIGHT) . '</a>'));

 

$col ++;

if ($col > 2) {

$col = 0;

$row ++;

}

}

 

new contentBox($info_box_contents);

?><!-- manufacturers_eof //-->

 

 

The book is from 2006. Is it a php compatibility issue or some change in osc coding or is it just a simple syntax error that I cannot see?

 

Any help would be greatly appreciated.

 

Sincerely,

Lyonstudio

[♥bruyndoncx]

possibly a single vs double quote issue at line 24 here after href ?

[♥Monika in Germany]

the " is missing at the end of your query ... you typed the code, right?

 

you have

$manufacturers_query = tep_db_query("select distinct m.manufacturers_id, m.manufacturers_name, m.manufacturers_image from " . TABLE_MANUFACTURERS . " m where manufacturers_name <> '' order by manufacturers_name);

 

 

should be

$manufacturers_query = tep_db_query("select distinct m.manufacturers_id, m.manufacturers_name, m.manufacturers_image from " . TABLE_MANUFACTURERS . " m where manufacturers_name <> '' order by manufacturers_name");