не в даваясь в суть

function gebi (id) {
	return document.getElementById(id);
}

function obsch() {
	var plan<?php echo $id; ?> = gebi ('plan<?php echo $id; ?>').value;
    var cenaclick<?php echo $id; ?> = <?php echo $cena_click; ?>;
    var color<?php echo $id; ?> = '<?php echo $color; ?>';
    var standartcolor<?php echo $id; ?> = '<?php echo $standart_color; ?>';
    var planbase<?php echo $id; ?> = <?php echo $plan_base; ?>;
    var members<?php echo $id; ?> = <?php echo $members; ?>;
    var active<?php echo $id; ?> = <?php echo $active; ?>;
    var vip<?php echo $id; ?> = <?php echo $vip; ?>;

    // активное окно
    if(active<?php echo $id; ?> == 0) { var cenaactive<?php echo $id; ?> = 0; } else { var cenaactive<?php echo $id; ?> = parseFloat(<?php echo $cena_active; ?>); };
    // если цвет стандартный - стоимость 0. Если нестандартный и ссылка остановлена cenacolor = cenacolor
    if(color<?php echo $id; ?> == standartcolor<?php echo $id; ?>) { var cenacolor<?php echo $id; ?> = 0; } else { if(planbase<?php echo $id; ?> <= members<?php echo $id; ?>) {var cenacolor<?php echo $id; ?> = <?php echo $hldyn; ?>;}; };

    if(vip<?php echo $id; ?> == 0)
    {
       var vipprice<?php echo $id; ?> = 0;
    }
    else
    {
       if(planbase<?php echo $id; ?> <= members<?php echo $id; ?>)
       {
          var vipprice<?php echo $id; ?> = <?php echo $vipprice; ?>;
       };
    };
           
	gebi ('ads_price_<?php echo $id; ?>').innerHTML = Math.round(((plan<?php echo $id; ?> * cenaactive<?php echo $id; ?>) + (plan<?php echo $id; ?> * cenaclick<?php echo $id; ?>) + (cenacolor<?php echo $id; ?> + vipprice<?php echo $id; ?>))*10000)/10000 + ' руб.';
}

будет точно так же работать как и твой вариант
думается если повыкидывать все эти таскания $id за идентификаторами, то ошибка всплывет сама собой визуально. а так - это адъ! :)

vagon
1 покажите ,что выводит в ХТМЛ ,а то прочитать невозможно.
2 делайте хотя бы так

var a=<?php echo $a ?>
var b=<?php echo $b ?>
var c=<?php echo $c ?>

а потом нормальный код без вставок

В общем, убрал приклеенные id приклеенные к названиям переменных от безысходности, - стало попроще. У названий функций оставил,
т.к. они все же должны быть уникальными. Результат тот же...

function gebi<?php echo $id; ?>(id)
{
	return document.getElementById(id)
}
function obsch_<?php echo $id; ?>()
{
	var plan = gebi<?php echo $id; ?>('plan').value;
    var cenaclick = <?php echo $cena_click; ?>;
    var color = '<?php echo $color; ?>';
    var standartcolor = '<?php echo $standart_color; ?>';
    var planbase = <?php echo $plan_base; ?>;
    var members = <?php echo $members; ?>;
    var active = <?php echo $active; ?>;
    var vip = <?php echo $vip; ?>;

    if(active == 0) { var cenaactive = 0; } else { var cenaactive = parseFloat(<?php echo $cena_active; ?>); };
    if(color == standartcolor) { var cenacolor = 0; } else { if(planbase <= members) {var cenacolor = <?php echo $hldyn; ?>;}; };
    if(vip == 0)
    {
       var vipprice = 0;
    }
    else
    {
       if(planbase <= members)
       {
          var vipprice = <?php echo $vipprice; ?>;
       };
    };
	gebi<?php echo $id; ?>('ads_price<?php echo $id; ?>').innerHTML = Math.round(((plan * cenaactive) + (plan * cenaclick) + 
(cenacolor + vipprice))*10000)/10000 + ' руб.';
}

HTML код:

function gebi1341950(id)
{
	return document.getElementById(id)
}
function obsch_1341950()
{
    var plan = gebi1341950('plan').value;
    var cenaclick = 0.01;
    var color = '#0094FF';
    var standartcolor = '#0000FF';
    var planbase = 3012;
    var members = 2361;
    var active = 0;
    var vip = 0;

    if(active == 0) { var cenaactive = 0; } else { var cenaactive = parseFloat(0.01); };
    if(color == standartcolor) { var cenacolor = 0; } else { if(planbase <= members) {var cenacolor = 1;}; };
    if(vip == 0)
    {
       var vipprice = 0;
    }
    else
    {
       if(planbase <= members)
       {
          var vipprice = 1;
       };
    };
	gebi1341950('ads_price1341950').innerHTML = Math.round(((plan * cenaactive) + (plan * cenaclick) + 
(cenacolor + vipprice))*10000)/10000 + ' руб.';

}

vagon
Тихий ужасть,да и сами знаете ,что тихий :D
Будете для 100 товаров -100 функций генерить? :rolleyes:
Делаем так,вот это генерим.

var product=[];
product[0]=['ид1','артикул','цена','хрень','еще хрень'];
product[1]=['ид2','артикул','цена','хрень','еще хрень'];
product[2]=['ид3','артикул','цена','хрень','еще хрень'];
product[3]=['ид4','артикул','цена','хрень','еще хрень'];
product[4]=['ид5','артикул','цена','хрень','еще хрень'];

Потом ничего не генерим.Определяем размер массива и крутим в цикле,передавая счетчик аргументом функции.

for(var i=0;i<product.length;i++){
fun(i);
}

Ловим в функции и делаем что нам нужно.и располагаем в DOM

function fun(i) {
//product[i][0]-здесь ид
//product[i][1]-здесь артикул
//product[i][2]-здесь цена
document.getElementById('content').innerHTML=document.getElementById('content').innerHTML+'<div id='+product[i][0]+'>'+product[i][1]+' '+product[i][2]+'</div>';
}

Приведите примерно к такому виду,запустите без ПХП,
если не получится - выкладываем,если получится но не получится генерить массив- это не проблемы яваскрипта.