Fix fields not saving their values when disabled

We grey out some fields because we don't want to allow the user to
change them. However, we were setting the "disabled" jQuery property to
do this, and this results in the read value for the form field being
blank and thus the field in the database being blank. Set the "readonly"
property instead to get the correct behaviour.

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
This commit is contained in:
Paul Eggleton 2013-09-13 11:24:21 +01:00
parent 97cf2f8afb
commit aa2c67c92d
2 changed files with 12 additions and 12 deletions

View File

@ -142,16 +142,16 @@
enable_value_field = function() {
cover_status = $('#id_cover_status').val()
if( cover_status == 'U' || cover_status == 'N' ) {
$('#id_cover_pn').prop('disabled', true);
$('#id_cover_layerbranch').prop('disabled', true);
$('#id_cover_verified').prop('disabled', true);
$('#id_cover_pn').prop('readonly', true);
$('#id_cover_layerbranch').prop('readonly', true);
$('#id_cover_verified').prop('readonly', true);
$('#id_label_cover_verified').addClass('muted');
$('#id_span_cover_opts').addClass('muted');
}
else {
$('#id_cover_pn').prop('disabled', false);
$('#id_cover_layerbranch').prop('disabled', false);
$('#id_cover_verified').prop('disabled', false);
$('#id_cover_pn').prop('readonly', false);
$('#id_cover_layerbranch').prop('readonly', false);
$('#id_cover_verified').prop('readonly', false);
$('#id_label_cover_verified').removeClass('muted');
$('#id_span_cover_opts').removeClass('muted');
}

View File

@ -221,14 +221,14 @@
$('#id_vcs_web_tree_base_url').val(awf.vcs_web_tree_base_url)
$('#id_vcs_web_file_base_url').val(awf.vcs_web_file_base_url)
}
$('#id_vcs_web_url').prop('disabled', true);
$('#id_vcs_web_tree_base_url').prop('disabled', true);
$('#id_vcs_web_file_base_url').prop('disabled', true);
$('#id_vcs_web_url').prop('readonly', true);
$('#id_vcs_web_tree_base_url').prop('readonly', true);
$('#id_vcs_web_file_base_url').prop('readonly', true);
}
else {
$('#id_vcs_web_url').prop('disabled', false);
$('#id_vcs_web_tree_base_url').prop('disabled', false);
$('#id_vcs_web_file_base_url').prop('disabled', false);
$('#id_vcs_web_url').prop('readonly', false);
$('#id_vcs_web_tree_base_url').prop('readonly', false);
$('#id_vcs_web_file_base_url').prop('readonly', false);
}
};