Javascript error i vbulletin_global.js

Detta är inte ett fel, därav warning. I JavaScript är det tillåtet att tilldela variabler värden på det här sättet. Efter tilldelningen testas variabeln för sant eller falskt precis som vanligt.

Liknande syntax finns ju även i exempelvis C/C++.

Men varför ha kod som ger varningar? Finns ju ingen vits med det.

Josef skrev:

Men varför ha kod som ger varningar? Finns ju ingen vits med det.

I mitt tycke är det inte språket i sig som skall ändras, utan den webbläsares JS-parser som inte följer reglerna.

Peter S skrev:

I mitt tycke är det inte språket i sig som skall ändras, utan den webbläsares JS-parser som inte följer reglerna.

Må så vara.

Plats för en dålig liknelse: Du går på trottoaren. Det kommer en cykel, egentligen borde cykeln flytta på sig, men för att slippa bli omkullvräkt tar du ett steg åt sidan.

Är trött så ta det inte för allvarligt! ;)

Att skriva bara = är ett ganska vanligt misstag när man vill jämföra. Antagligen därför den varnar.

Jag tycker det är ganska dumt att använda =. Visst man kan få gjort mer med mindre kod, men det blir sämre läsbarhet. Svårare att hänga med i vad koden gör.
C# har väl tagit bort det just p.g.a. detta..?

Peter S skrev:

Detta är inte ett fel, därav warning. I JavaScript är det tillåtet att tilldela variabler värden på det här sättet. Efter tilldelningen testas variabeln för sant eller falskt precis som vanligt.

Liknande syntax finns ju även i exempelvis C/C++.

Den typen av tilldelningar har en förmåga att ge upphov till svårfunna buggar. Det är inte ofta man stöter på den typen av kod.

Att jag skrev error i rubriken var ett dumt val iofs :OO

Men att parsern varnar är helt riktigt...

Jag som inte har någon koll på javascript, är följande korrekt

//ersätt
if (r = this.id.match(/^([a-z]{1})([0-9]+)$/))

//med
r = this.id.match(/^([a-z]{1})([0-9]+)$/)
if(r)

?

Har inte provkört koden, men det ser helt rätt ut.

Har ändrat nu, är problemet borta?

Problemet borta :)

Duktig Peeer :bire

Dock märkte jag att det dök upp en varningstriangel när mitt svar blev submittat...

JavaScript Console skrev:

Warning: reference to undefined property subjectText.length
Source File: http://www.webforum.nu/clientscript/vbulletin_global.js
Line: 255

Underligt, där har jag inte varit och ändat något. Fler som får felet, eller är det kanske rent av någon som vet hur man löser det? Har som sagt ingen koll på javascript.. ingen alls!

Problemet ligger i att validatePost() (i /clientscript/vbulletin_stdedit.js) anropas med 0 som andra argument. validateMessage() (som anropas av ovan nämnda funktion) försöker sedan läsa egenskapen length av 0, vilket orsakar felet.

Problemet går nog att lösa på ett par olika sätt. Ett av dessa kan vara att göra följande ändring i den senast nämnda funktionen:

// check for completed subject
if ([b]typeof subjectText == "object" &&[/b] subjectText.length < 1)
{
	alert(vbphrase["must_enter_subject"]);
	return false;
}

Fetstilt text är tillagd.