From aa2c67c92dd4a9a2a37dc3ee3f18d72d9a29ef2a Mon Sep 17 00:00:00 2001 From: Paul Eggleton Date: Fri, 13 Sep 2013 11:24:21 +0100 Subject: [PATCH] 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 --- templates/layerindex/classicrecipedetail.html | 12 ++++++------ templates/layerindex/editlayer.html | 12 ++++++------ 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/templates/layerindex/classicrecipedetail.html b/templates/layerindex/classicrecipedetail.html index f37db37..9df9a84 100644 --- a/templates/layerindex/classicrecipedetail.html +++ b/templates/layerindex/classicrecipedetail.html @@ -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'); } diff --git a/templates/layerindex/editlayer.html b/templates/layerindex/editlayer.html index e2b4fcb..b7ff5dc 100644 --- a/templates/layerindex/editlayer.html +++ b/templates/layerindex/editlayer.html @@ -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); } };