tonytomov/jqGrid — comment on commit d40ab3737a1415a5a9d5659ca2d157a60a647c0b
File:  line 0
By OlegKi:

Hello Tony,

In the implementation the `xmlReader.root`, `xmlReader.row` can't be functions. In my suggestion I get first `drows = getXmlData(data, dReader.root, true);` which returns an object which will be used as input to `drows = getXmlData(drows, dReader.row, true);` and the `drows` will be set to the needed value.

The `xmlReader.id` seems be not changed. Do you don't want to make changes here or you want to do the changes later?

The `xmlReader.userdata` have to return the data in very specific way with `name` attributes. In my suggestion the function `xmlReader.userdata` can directly return `p.userData` and the `name` attribute will be used only in case of non-function datatype of `xmlReader.userdata`.

Best regards
Oleg