jQuery Grid Plugin - jqGridForum

Topic RSS

Related Topics sp_TopicIcon

wrong priority in colModel template and colModel properties

Tags: colModel template

05/02/2011
20:28

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony!

I found small bug in my original suggestion about the usage of templates for the colModel properties. The problem is that in the current implementation the settings from cmTemplate parameter of jqGrid or the template properties has higher priority as the settings from the colModel properties. If for example the width property exist in all template, cmTemplate and in colModel, the values from template insted of colModel property will be used. So I suggest to change the line 1895 of the grid.base.js from

clm = $.extend(clm, ts.p.cmTemplate, clm.template || {});

ts.p.colModel[i] = $.extend({}, ts.p.cmTemplate, clm.template || {}, clm);

Because the colModel templates are not yet described in the documentation, the feature will be probably not used and the problem which I describe here not found by other people.

Best regards

Oleg

21/03/2011
21:58

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Sorry Tony,

but the line have to be changed one more time (How many errors one can make in one line of code?) from

ts.p.colModel[i] = $.extend({}, ts.p.cmTemplate, ts.p.colModel[i].template || {}, ts.p.colModel[i]);

to:

ts.p.colModel[i] = $.extend(true, {}, ts.p.cmTemplate, ts.p.colModel[i].template || {}, ts.p.colModel[i]);

I hope that it is the last my error in the column templates.

Best regards
Oleg

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
57 Guest(s)

Currently Browsing this Page:
1 Guest(s)