Guriddo » Topic: inline edit with select and multiple:true

inline edit with select and multiple:true

Guriddo
inline select multiple
inline edit with select and multiple:true

Tagged: inline select multiple

This topic has 6 replies, 2 voices, and was last updated 10 years, 9 months ago by guriddo.support.

Creator

Topic
July 3, 2015 at 11:46 am #123997

gzorzi
Participant

Hi,

I’m using jqGrid 4.8.3.

I have a table with inline edit enables.

I’ve defined a column with edittype select and in the editoptions I have multiple:true.

With this setup I can’t add a new record (I get an Uncaught TypeError: Cannot read property ‘join’ of null).

If I edit a row (inline or with navgrid’s edit dialog) everything is ok.

If I remove multiple:true I can add a new record just fine

Below is part of the code:

Any hint about solving this will be really appreciated 🙂

thanks

g.

colModel:[
{name:’id’,index:’id’,hidden: true},
…

{name:’months’,index:’months’, editable: true, edittype:’select’, formatter:’select’, editoptions: monthslist},
…
],

edioptions are stored in:

var monthslist={
value: {
1:’Gen’,
2:’Feb’,
3:’Mar’,
4:’Apr’,
5:’Mag’,
6:’Giu’,
7:’Lug’,
8:’Ago’,
9:’Set’,
10:’Ott’,
11:’Nov’,
12:’Dic’
},
multiple:true,
size:12
}
Creator

Topic

Viewing 6 replies - 1 through 6 (of 6 total)

Author

Replies

July 6, 2015 at 10:01 am #124003

guriddo.support

Participant

Hello,

Thank you for the expalnation.

The only missing part from your description code is how you add a record – i.e the code with which you add a record.

We have successfully implemented exactly your code with inline edit navigator and everything work as expected.

Please, provide a code with which you add a record or send us a link to the problem!

Also the last version is 4.8.2. It is good to point which version of jQuery (jQuery UI) is used too.

Kind Regards,

Will

Guriddo Support Team

July 6, 2015 at 10:59 am #124009

gzorzi

Participant

Hi,

I have no specific code to add a new record, as I use the default navGrid add button. Here’s my grid code:


var lastSelO=&quot;&quot;;
$(&#039;#config-profili-orari&#039;).jqGrid({
datatype: &#039;json&#039;,
url: &#039;/admin/getConfigOrariProfiles&#039;,
editurl: &#039;/admin/setConfigOrariProfiles&#039;,
method: &#039;GET&#039;,
width: $(&#039;body&#039;).width()-60,
shrinkToFit: true,
height: &quot;auto&quot;,
sortorder: &quot;desc&quot;,
regional: &#039;it&#039;,
loadui: &#039;disable&#039;,
pager: &#039;#config-profili-orari_pager&#039;,
jsonReader : jsonReader,
colNames:[&#039;id&#039;,&#039;Nome&#039;,&#039;Descrizione&#039;,&#039;Mesi&#039;,&#039;Giorni&#039;,&#039;Ore&#039;,&#039;Domain&#039;],
colModel:[
{name:&#039;id&#039;,index:&#039;id&#039;,hidden: true},
{name:&#039;name&#039;,index:&#039;name&#039;, editable: true, edittype:&#039;text&#039;},
{name:&#039;description&#039;,index:&#039;description&#039;, editable: true, edittype:&#039;text&#039;},
{name:&#039;months&#039;,index:&#039;months&#039;, editable: true, edittype:&#039;select&#039;, formatter:&#039;select&#039;, editoptions: monthslist},
{name:&#039;weekdays&#039;,index:&#039;weekdays&#039;, editable: true, edittype:&#039;select&#039;, formatter:&#039;select&#039;, editoptions: weekdayslist},
{name:&#039;hours&#039;,index:&#039;hours&#039;, editable: true, edittype:&#039;select&#039;, formatter:&#039;select&#039;, editoptions: hourslist},
{name:&#039;domain&#039;,index:&#039;domain&#039;, editable: true, edittype:&#039;select&#039;, formatter:&#039;select&#039;, editoptions: domainslist}
],
onSelectRow: function(id){
if(id &amp;&amp; id!==lastSelO){
jQuery(&#039;#config-profili-orari&#039;).saveRow(lastSelO);
lastSelO=id;
}
jQuery(&#039;#config-profili-orari&#039;).editRow(id, true);
},
loadComplete: function() {
$(&quot;#config-profili-orari tr.jqgrow:even&quot;).addClass(&#039;evenRowClass&#039;);
$(&quot;#config-profili-orari tr.jqgrow:odd&quot;).addClass(&#039;oddRowClass&#039;);
}
})
.jqGrid(&#039;navGrid&#039;,&#039;#config-profili-orari_pager&#039;,{
add: true,
del:true,
edit: true,
search: false,
refresh: false
});

var lastSelO="";

$('#config-profili-orari').jqGrid({

datatype: 'json',

url: '/admin/getConfigOrariProfiles',

editurl: '/admin/setConfigOrariProfiles',

method: 'GET',

width: $('body').width()-60,

shrinkToFit: true,

height: "auto",

sortorder: "desc",

regional: 'it',

loadui: 'disable',

pager: '#config-profili-orari_pager',

jsonReader : jsonReader,

colNames:['id','Nome','Descrizione','Mesi','Giorni','Ore','Domain'],

colModel:[

{name:'id',index:'id',hidden: true},

{name:'name',index:'name', editable: true, edittype:'text'},

{name:'description',index:'description', editable: true, edittype:'text'},

{name:'months',index:'months', editable: true, edittype:'select', formatter:'select', editoptions: monthslist},

{name:'weekdays',index:'weekdays', editable: true, edittype:'select', formatter:'select', editoptions: weekdayslist},

{name:'hours',index:'hours', editable: true, edittype:'select', formatter:'select', editoptions: hourslist},

{name:'domain',index:'domain', editable: true, edittype:'select', formatter:'select', editoptions: domainslist}

onSelectRow: function(id){

if(id && id!==lastSelO){

jQuery('#config-profili-orari').saveRow(lastSelO);

lastSelO=id;

}

jQuery('#config-profili-orari').editRow(id, true);

loadComplete: function() {

$("#config-profili-orari tr.jqgrow:even").addClass('evenRowClass');

$("#config-profili-orari tr.jqgrow:odd").addClass('oddRowClass');

}

})

.jqGrid('navGrid','#config-profili-orari_pager',{

add: true,

del:true,

edit: true,

search: false,

refresh: false

});

This reply was modified 10 years, 9 months ago by admin. Reason: code formatting

July 6, 2015 at 11:48 am #124014

guriddo.support

Participant

Hello,

Thank you for the test case. We have reproduced the bug.

The bug is fixed and you can get the fixed version at end of this week when the new release will be out.

Our problem was in communication – you speak about inline edit, but actually the problem was in form edit module.

Kind Regards,

Will

Guriddo Support Team

July 6, 2015 at 12:34 pm #124015

gzorzi

Participant

thanks a lot 🙂

July 7, 2015 at 9:28 am #124019

gzorzi

Participant

Hi,

I’ve had to change this to have it working:

if($(elc).get(0).type === ‘select-multiple’ && tmp) {

-> if($(elc).get(0).type === ‘select-multiple’ && tmp!==null) {

giuliano

This reply was modified 10 years, 9 months ago by gzorzi.

July 7, 2015 at 10:49 am #124021

guriddo.support

Participant

Hello,

Thank you for the hint. IMHO the better way will be

if($(elc).get(0).type === ‘select-multiple’ &amp;&amp; tmp != null) {
...
}

if($(elc).get(0).type === ‘select-multiple’ && tmp != null) {

...

}

tmp != null means null and undefined while tmp !== null means only null

P.S But this will cause problems if tmp is empty string, so I think the fixed expression is fine
Kind Regards,
Will

inline edit with select and multiple:true

Search Forum

Recent Replies

Forum Statistics

UserOnline