jQuery Grid Plugin - jqGridForum

Topic RSS

allow to use strings as values of cellattr and rowattr

09/11/2014
15:02

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

I suggest to allow to use string values forÂ cellattr andÂ rowattr in the same way like one can now to use strings as the value for template (see the pull request):

$.extend($.jgrid,{
    cellattr: {
        someCellAttrValue: function (rowId, val, rawObject) {
            ...
        }
    },
    rowattr: {
        someRowAttrValue: function (rd) {
            ...
        }
    }
});

The corresponding changes in jqGrid code are very easy. To allow the usage string values forÂ cellattrÂ one need to change the line

var cm = ts.p.colModel[pos],

var cm = ts.p.colModel[pos], cellAttrFunc,

and the lines

} else if (cm.cellattr && $.isFunction(cm.cellattr))
{
    celp = cm.cellattr.call(ts, rowId, tv, rawObject, cm, rdata);

to the following

} else if ($.isFunction(cm.cellattr) || (typeof cm.cellattr === "string" && $.jgrid.cellattr != null && $.isFunction($.jgrid.cellattr[cm.cellattr]))) {
    cellAttrFunc = $.isFunction(cm.cellattr) ? cm.cellattr : $.jgrid.cellattr[cm.cellattr];
    celp = cellAttrFunc.call(ts, rowId, tv, rawObject, cm, rdata);

In the same way to allows the usage of string as the value of rowattr one need to change the lineÂ

rowAttrObj = $.isFunction(ts.p.rowattr) ? ts.p.rowattr.call(ts, rd, cur, id) :{};

to the lines

rowAttrObj = $.isFunction(ts.p.rowattr) ? ts.p.rowattr.call(ts, rd, cur, id) :
    (typeof ts.p.rowattr === "string" && $.jgrid.rowattr != null && $.isFunction($.jgrid.rowattr[ts.p.rowattr]) ?
        $.jgrid.rowattr[ts.p.rowattr].call(ts, rd, cur, id) : {});

One can modify the code more to allow to use cellattr andÂ rowattr as comma-separated string with the values $.jgrid.cellattr and $.jgrid.rowattr. In the way one could allow applying of multiple callback functionsÂ (the stack) at once. In general such extension could have sense for formatterÂ andÂ template values too.

The demoÂ demonstrates the changes working.

Best regards
Oleg

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
39 Guest(s)

Currently Browsing this Page:
1 Guest(s)