} else if (document.all){
item = document.all[id];
} else if (document.layers){
item = document.layers[id];
В этом нет необходимости. Браузеры без поддержки document.getElementById уже успешно вымерли. Можно написать так:
var flag = false
function imgchange(id) {
var item = document.getElementById(id);
if (!item) return;
if (item.src) {
item.src = flag ? "plus.png" : "minus.png";
} else {
item.src = "plus.png";
}
flag = !flag;
};