Мил человек! Ты в итерациях покопайся внимательнее - у тебя ж "i" прирастает от 1 до количества элементов формы, а нумерация в идентификаторах посередине разорвана и начинается сначала.
А вот если заменить brand1 на brand5 и т.д... ))
Кстати, здесь можно обойтись одной функцией вместо двух:
function selectCatAll(nn){
for(i=1; iif (document.formName.elements.id== nn +i){
if(document.formName.elements[nn].checked==true){
document.formName.elements.checked = true;
} else {
document.formName.elements.checked = false;
};
};
};
};
а вызывать ее так:
input id="cat" name="cat_all" type="checkbox" OnClick="selectCatAll(this.id)"
input id="brand" name="cat_all" type="checkbox" OnClick="selectCatAll(this.id)"