jQuery Grid Plugin - jqGrid - Topic: allow to use strings as values of cellattr and rowattr http://www.trirand.com/blog/?page_id=393/feature-request/allow-to-use-strings-as-values-of-cellattr-and-rowattr Simple:Press Version 5.7.5.3 OlegK on allow to use strings as values of cellattr and rowattr http://www.trirand.com/blog/?page_id=393/feature-request/allow-to-use-strings-as-values-of-cellattr-and-rowattr#p31359 Feature Request http://www.trirand.com/blog/?page_id=393/feature-request/allow-to-use-strings-as-values-of-cellattr-and-rowattr#p31359 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],

to

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

]]> Sun, 09 Nov 2014 15:02:42 +0200