[0.8.2] Bug in gridpanel (invalid columnname)

[plykkegaard]

Hi

I came across a minor bug today in the gridpanel (dll ver 0.8.2.2648)

I have a table I am displaying as a crosstab query with little control over fieldnames
Some of the values are three letters 'var' and it will become a columnname in the gridpanel

Eg a layout like this
date | var | omk

Obvious 'var' is a reserved word in javascript and I get a runtimeerror on the clientside
In coolite version 8.1 (dll version 0.8.1.987) the above is working without problems

I have done an intermediate fix by replacing var with a longer more descriptive name I am using for label/column header

rgds /Peter

[Daniil]

Hi,

Could you provide a sample to reproduce?

[plykkegaard]

Could you provide a sample to reproduce?

Hmm I see your point
I was unable to reproduce using one of your examples
Maybe the difference lies in markup versus codebehind?

I'll dig into it this weekend

[plykkegaard]

I'll dig into it this weekend

I am sorry, it has been impossible to reproduce the bug in a small example

Will a dump from the html page send to the client (IE) be of any help?
I can probably provide an example from both dll versions

[Daniil]

Hi,

Yes, I will try to see the difference.

P.S. By the way why 0.8.x?

I would suggest you to use Ext.Net 1.0. Many-many things are easy and possible with Ext.Net 1.0.

It's not so difficult to migrate but be sure, the benefits are great, see
http://forums.ext.net/showthread.php...to-Ext.Net-1.0
http://forums.ext.net/showthread.php...0-step-by-step

Most developers has already migrated to Ext.Net 1.0 and all of them is very glad.

[plykkegaard]

P.S. By the way why 0.8.x?

It's a huge application (+200 files) and I need to close development/testing and send latest changes to production before I can start the migration and also due to lack of time and new requirements/wishes and ...

Most developers has already migrated to Ext.Net 1.0 and all of them is very glad.

I've already done a small two pages app with 1.0, works like a charm :-)

[plykkegaard]

Yes, I will try to see the difference.

Offending code in ver 8.2:

gridPanel.insertRecord(
    rowIndex, { 
        Date: template.data['Date'], 
        omk: template.data['omk'], 
        var: template.data['var'] }
    );

After renaming var to varekoeb

gridPanel.insertRecord(
    rowIndex, { 
        Date: template.data['Date'], 
        omkostning: template.data['omkostning'], 
        varekoeb: template.data['varekoeb'] });

The above code does not exist in ver 8.1
And firefox does not produce an error only IE (various versions)

Error message (in danish)

Detaljer om fejl på websiden
Brugeragent: Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; GTB6.6; 
.NET CLR 2.0.50727; .NET CLR 1.1.4322; OfficeLiveConnector.1.3; OfficeLivePatch.0.0; 
.NET CLR 3.0.4506.2152; .NET CLR 3.5.30729; .NET4.0C; .NET4.0E)
Tidsstempling: Thu, 2 Dec 2010 16:09:22 UTC
Meddelelse: Expected identifier, string or number
Linje: 324
Tegn: 899
Kode: 0

Character/column 899 is the comma after "['omk']" but before "var" in the above code

hth :-)