Remove additional csrf_form_base calls, general tidy up

This commit is contained in:
FrancescoUK
2018-05-27 21:43:18 +01:00
committed by jekkos
parent c9a50cdeaf
commit fea0462407
5 changed files with 53 additions and 47 deletions

View File

@@ -226,17 +226,17 @@
//validation and submit handling
$(document).ready(function()
{
$("span").tooltip();
$('span').tooltip();
$("#currency_symbol, #thousands_separator").change(function() {
$('#currency_symbol, #thousands_separator').change(function() {
var field = $(this).attr('id');
var value = $(this).is(":checkbox") ? $(this).is(":checked") : $(this).val();
var data = { number_locale: $("#number_locale").val() };
var value = $(this).is(':checkbox') ? $(this).is(':checked') : $(this).val();
var data = { number_locale: $('#number_locale').val() };
data[field] = value;
$.post("<?php echo site_url($controller_name . '/ajax_check_number_locale')?>",
data,
function(response) {
$("#number_locale_example").text(response.number_locale_example);
$('#number_locale_example').text(response.number_locale_example);
},
'json'
);
@@ -252,17 +252,17 @@ $(document).ready(function()
remote:
{
url: "<?php echo site_url($controller_name . '/ajax_check_number_locale')?>",
type: 'post',
type: 'POST',
data: {
'number_locale': $("#number_locale").val(),
'thousands_separator': $("#thousands_separator").is(":checked")
'number_locale': $('#number_locale').val(),
'thousands_separator': $('#thousands_separator').is(':checked')
},
dataFilter: function(data) {
setup_csrf_token();
var response = JSON.parse(data);
$("#number_locale_example").text(response.number_locale_example);
$("#currency_symbol").val(response.currency_symbol);
$("#thousands_separator").prop('checked', response.thousands_separator);
$('#number_locale_example').text(response.number_locale_example);
$('#currency_symbol').val(response.currency_symbol);
$('#thousands_separator').prop('checked', response.thousands_separator);
return response.success;
}
}

View File

@@ -451,18 +451,17 @@ $(document).ready(function()
rules:
{
first_name: "required",
last_name: "required",
consent: "required",
first_name: 'required',
last_name: 'required',
consent: 'required',
email:
{
remote:
{
url: "<?php echo site_url($controller_name . '/ajax_check_email')?>",
type: "post",
data:
{
"person_id" : "<?php echo $person_info->person_id; ?>",
type: 'POST',
data: {
'person_id': '<?php echo $person_info->person_id; ?>'
// email is posted by default
}
}
@@ -472,9 +471,9 @@ $(document).ready(function()
remote:
{
url: "<?php echo site_url($controller_name . '/ajax_check_account_number')?>",
type: "post",
type: 'POST',
data: {
"person_id" : "<?php echo $person_info->person_id; ?>"
'person_id': '<?php echo $person_info->person_id; ?>'
// account_number is posted by default
}
}

View File

@@ -72,7 +72,7 @@ $(document).ready(function()
$("input[name='person_name']").val(ui.item.label);
};
var autocompleter = $("#person_name").autocomplete({
var autocompleter = $('#person_name').autocomplete({
source: '<?php echo site_url("customers/suggest"); ?>',
minChars: 0,
delay: 15,
@@ -94,7 +94,7 @@ $(document).ready(function()
{
table_support.handle_submit('<?php echo site_url($controller_name); ?>', {message: errorThrown});
},
dataType:'json'
dataType: 'json'
});
};
@@ -106,27 +106,32 @@ $(document).ready(function()
},
rules:
{
<?php if($this->config->item('giftcard_number') == "series") { ?>
<?php
if($this->config->item('giftcard_number') == "series")
{
?>
giftcard_number:
{
required: true,
number: true
},
<?php } ?>
<?php
}
?>
giftcard_amount:
{
required: true,
remote:
{
url: "<?php echo site_url($controller_name . '/ajax_check_number_giftcard')?>",
type: 'post',
type: 'POST',
data: {
'amount': $("#giftcard_amount").val()
'amount': $('#giftcard_amount').val()
},
dataFilter: function(data) {
setup_csrf_token();
var response = JSON.parse(data);
$("#giftcard_amount").text(response.value);
$('#giftcard_amount').text(response.value);
return response.success;
}
}
@@ -134,13 +139,18 @@ $(document).ready(function()
},
messages:
{
<?php if($this->config->item('giftcard_number') == "series"){ ?>
<?php
if($this->config->item('giftcard_number') == "series")
{
?>
giftcard_number:
{
required: "<?php echo $this->lang->line('giftcards_number_required'); ?>",
number: "<?php echo $this->lang->line('giftcards_number'); ?>"
},
<?php } ?>
<?php
}
?>
giftcard_amount:
{
required: "<?php echo $this->lang->line('giftcards_value_required'); ?>",

View File

@@ -376,10 +376,10 @@ $(document).ready(function()
$("#custom" + <?php echo $i; ?>).autocomplete({
source:function (request, response) {
$.ajax({
type: "POST",
type: 'POST',
url: "<?php echo site_url('items/suggest_custom');?>",
dataType: "json",
data: $.extend(request, $extend(csrf_form_base(), {field_no: <?php echo $i; ?>})),
data: $.extend(request, {field_no: <?php echo $i; ?>}),
success: function(data) {
response($.map(data, function(item) {
return {
@@ -438,15 +438,13 @@ $(document).ready(function()
remote:
{
url: "<?php echo site_url($controller_name . '/check_item_number')?>",
type: "post",
data: $.extend(csrf_form_base(),
{
type: 'POST',
data: {
"item_id": "<?php echo $item_info->item_id; ?>",
"item_number": function()
{
"item_number": function() {
return $("#item_number").val();
},
})
}
}
}
},
cost_price:

View File

@@ -96,13 +96,13 @@
$(document).ready(function()
{
<?php if(!empty($sale_info['email'])): ?>
$("#send_invoice").click(function(event) {
$('#send_invoice').click(function(event) {
if (confirm("<?php echo $this->lang->line('sales_invoice_confirm') . ' ' . $sale_info['email'] ?>")) {
$.get('<?php echo site_url() . "/sales/send_pdf/" . $sale_info['sale_id']; ?>',
function(response) {
dialog_support.hide();
table_support.handle_submit('<?php echo site_url('sales'); ?>', response);
}, "json"
}, 'json'
);
}
});
@@ -143,7 +143,7 @@ $(document).ready(function()
$("input[name='customer_name']").val(ui.item.label);
};
$("#customer_name").autocomplete(
$('#customer_name').autocomplete(
{
source: '<?php echo site_url("customers/suggest"); ?>',
minChars: 0,
@@ -190,14 +190,13 @@ $(document).ready(function()
remote:
{
url: "<?php echo site_url($controller_name . '/check_invoice_number')?>",
type: "POST",
type: 'POST',
data: {
"sale_id" : <?php echo $sale_info['sale_id']; ?>,
"invoice_number" : function()
{
return $("#invoice_number").val();
'sale_id': <?php echo $sale_info['sale_id']; ?>,
'invoice_number': function() {
return $('#invoice_number').val();
}
})
}
}
}
},