Upload filter.. :/

<SCRIPT LANGUAGE="JavaScript">
<!--
var submitcount=0;
extArray = new Array(".dcd", ".dsx");
function LimitAttach(form, file) {
allowSubmit = false;
if (!file) return;
while (file.indexOf("\\") != -1)
file = file.slice(file.indexOf("\\") + 1);
ext = file.slice(file.indexOf(".")).toLowerCase();
for (var i = 0; i < extArray.length; i++) {
if (extArray[i] == ext) { allowSubmit = true; break; }
}
if (submitcount == 0)
      {
      submitcount++;
      return true;
      }
   else 
      {
      alert("du har redan postat detta...");
      return false;
}
if (allowSubmit) form.submit(); 
else
alert("du kan inte ladda upp det där!!");
return false;
}

//-->
</script>

lite längre ned:
<input type="submit" value="Ladda upp.." onclick="LimitAttach(this.form, this.form.upload.value)">

Lite problem bara..
ett# saken laddas upp ändå även fast man får upp ngt av varningsmeddelandena..
Samt har det blivit lite problem med submitt knappen för när man får upp ett error och klickar ingen så får man upp du har redan postat varningsmeddelandet..

Någon som vet hur jag ska göra?

Lite problem bara..
ett# saken laddas upp ändå även fast man får upp ngt av varningsmeddelandena..

Dels returnerar du inte något från onClick. onClick="return LimitAttach(...osv..." Dels har du några lösa trådar i scriptet. I början har du en lös return hänger utan värde (return false). Nära slutet på scriptet har du en else-sats utan måsvingar { }. Loopar och if-satser utan dessa påverkar bara statementet efter. return false hamnar alltså utanför else-satsen här. Sedan gör dels scriptet en submit via raden form.submit(), dels har du en submit-knapp... Verkar lite överarbetat. Antingen så ändrar du type="submit" till type="button", och då behöver du inte bry dig om att returnera något från onClick, eller så returnerar du true från funktionen om du vill submitta.

Samt har det blivit lite problem med submitt knappen för när man får upp ett error och klickar ingen så får man upp du har redan postat varningsmeddelandet..

Joo... submitcount-värdet nollställs aldrig. Vad är meningen att den skall göra?

------------------
/ J

Inte jag som har gjort scriptet.. bara ihopsaxat.. Är inte så high på Javascript..

Den är till för att inte .dcd eller .dsx filen ska laddas upp två gånger.. hm.. kanske finns något bättre sätt att lösa detta på?