FIX_Minor Issues

This commit is contained in:
sanjeev.p 2026-03-13 17:31:48 +05:30
parent 0498873ec5
commit b70811dff7
11 changed files with 107 additions and 34 deletions

View File

@ -247,7 +247,7 @@ class AppContentManagementController extends AdminController
'errors' => [ 'errors' => [
'required' => 'Type is required', 'required' => 'Type is required',
'max_length' => 'Type cannot exceed 255 characters', 'max_length' => 'Type cannot exceed 255 characters',
'regex_match' => 'Type contains invalid characters' 'regex_match' => 'Type can contain only letters, numbers, spaces, _ and -'
] ]
], ],
'content_section' => [ 'content_section' => [
@ -255,7 +255,7 @@ class AppContentManagementController extends AdminController
'errors' => [ 'errors' => [
'required' => 'Content Section is required', 'required' => 'Content Section is required',
'max_length' => 'Content Section cannot exceed 255 characters', 'max_length' => 'Content Section cannot exceed 255 characters',
'regex_match' => 'Content Section contains invalid characters' 'regex_match' => 'Content Section can contain only letters, numbers, spaces, _ and -'
] ]
], ],
'heading' => [ 'heading' => [
@ -263,7 +263,7 @@ class AppContentManagementController extends AdminController
'errors' => [ 'errors' => [
'required' => 'Heading is required', 'required' => 'Heading is required',
'max_length' => 'Heading cannot exceed 255 characters', 'max_length' => 'Heading cannot exceed 255 characters',
'regex_match' => 'Heading contains invalid characters. Only letters, numbers, spaces and basic punctuation are allowed' 'regex_match' => 'Heading can contain only letters, numbers, spaces, and these characters: . , ; : ! ? ( ) & / -'
] ]
], ],
'content' => [ 'content' => [
@ -465,18 +465,22 @@ class AppContentManagementController extends AdminController
if ($method === 'post') { if ($method === 'post') {
$rules = [ $rules = [
'category' => [ 'category' => [
'rules' => 'required|max_length[100]|alpha_numeric_space', // Allow letters, numbers, spaces and / - . , " '
'rules' => 'required|max_length[100]|regex_match[/^[a-zA-Z0-9 \\/\\-\\.\,\"\\\']+$/]',
'errors' => [ 'errors' => [
'required' => 'Category is required', 'required' => 'Category is required',
'max_length' => 'Category cannot exceed 100 characters', 'max_length' => 'Category cannot exceed 100 characters',
'alpha_numeric_space' => 'Category contains invalid characters' 'regex_match' => 'Category can contain only letters, numbers, spaces, and these characters: / - . , " \'',
] ]
], ],
'question' => [ 'question' => [
'rules' => 'required|max_length[1000]', // Allow only letters, numbers, spaces and basic punctuation . , ; : ! ? ( ) & / -
'rules' => 'required|max_length[1000]|regex_match[/^[a-zA-Z0-9 _\\-.,;:!?()&\\/]+$/]',
'errors' => [ 'errors' => [
'required' => 'Question is required', 'required' => 'Question is required',
'max_length' => 'Question cannot exceed 1000 characters', 'max_length' => 'Question cannot exceed 1000 characters',
'regex_match' => 'Question can contain only letters, numbers, spaces, and these characters: . , ; : ! ? ( ) & / -',
] ]
], ],
'answer' => [ 'answer' => [
@ -787,9 +791,10 @@ class AppContentManagementController extends AdminController
'/<\s*object/i', // <object> '/<\s*object/i', // <object>
'/<\s*embed/i', // <embed> '/<\s*embed/i', // <embed>
'/<\s*applet/i', // <applet> '/<\s*applet/i', // <applet>
'/on\w+\s*=/i', // on...= (e.g., onclick=) '/on\w+\s*=/i', // on...= (e.g., onclick=, onmouseover=, onerror=)
'/data\s*:\s*text\/html/i', // data:text/html '/data\s*:\s*text\/html/i', // data:text/html
'/expression\s*\(/i', // CSS expression() '/expression\s*\(/i', // CSS expression()
'/\balert\s*\(/i', // alert(...)
]; ];
// Loop through the patterns and check if any of them exist in the decoded string. // Loop through the patterns and check if any of them exist in the decoded string.

View File

@ -129,7 +129,7 @@
if(res.status == true){ if(res.status == true){
$('#cd_ac_no_for_cd_master_errorr').text(res.message); $('#cd_ac_no_for_cd_master_errorr').text(res.message);
// toastr.warning(res.message, 'warning'); // toastr.warning(res.message, 'Warning');
// $('#cd_ac_no').val(''); // $('#cd_ac_no').val('');
$('#cd_master_btn_Submit').prop('disabled',true); $('#cd_master_btn_Submit').prop('disabled',true);
return; return;

View File

@ -697,7 +697,7 @@ $('body').on('click', '.btnBranchEdit', function() {
setTimeout(function() { setTimeout(function() {
$('.loader').fadeOut(); $('.loader').fadeOut();
$('.loader-mask').delay(350).fadeOut('slow'); $('.loader-mask').delay(350).fadeOut('slow');
toastr.warning('Something Wrong!', 'warning'); toastr.warning('Something Wrong!', 'Warning');
}, 1000); }, 1000);
} }
}); });
@ -916,10 +916,10 @@ function removeClientBranch(element) {
// console.log(res.status == true); // console.log(res.status == true);
if (res) { if (res) {
if (res.status == true) { if (res.status == true) {
toastr.success(res.message, 'success'); toastr.success(res.message, 'Success');
location.reload(); location.reload();
} else { } else {
toastr.warning(res.message, 'warning'); toastr.warning(res.message, 'Warning');
} }
} }
}, },

View File

@ -377,7 +377,7 @@
setTimeout(function() { setTimeout(function() {
$('.loader').fadeOut(); $('.loader').fadeOut();
$('.loader-mask').delay(350).fadeOut('slow'); $('.loader-mask').delay(350).fadeOut('slow');
toastr.warning('Something Wrong!', 'warning'); toastr.warning('Something Wrong!', 'Warning');
}, 1000); }, 1000);
} }
}); });

View File

@ -376,6 +376,24 @@
} }
// Simple client-side XSS pattern check to give instant feedback
function hasUnsafeHtml(html) {
if (!html) return false;
const decoded = $('<textarea/>').html(html).text().toLowerCase();
const patterns = [
/<\s*script/i,
/on\w+\s*=/i,
/javascript\s*:/i,
/vbscript\s*:/i,
/data\s*:\s*text\/html/i,
/expression\s*\(/i
];
return patterns.some(p => p.test(decoded));
}
$('#FAQForm').on('submit', function(e) { $('#FAQForm').on('submit', function(e) {
e.preventDefault(); e.preventDefault();
const form = this; const form = this;
@ -395,6 +413,12 @@
toastr.warning("Answer is required", 'Warning'); toastr.warning("Answer is required", 'Warning');
return; // Stop the function here return; // Stop the function here
} }
// Block unsafe HTML/JS patterns
if (hasUnsafeHtml(editor.value)) {
toastr.error("Answer contains restricted tags or attributes. Script tags, iframes and event handlers (like onclick, onerror, onmouseover) are not allowed.", "Validation Error");
return;
}
} }
// If valid, submit via AJAX // If valid, submit via AJAX

View File

@ -532,6 +532,24 @@
openModal(); openModal();
} }
// Simple client-side XSS pattern check to give instant feedback
function hasUnsafeHtml(html) {
if (!html) return false;
const decoded = $('<textarea/>').html(html).text().toLowerCase();
const patterns = [
/<\s*script/i,
/on\w+\s*=/i,
/javascript\s*:/i,
/vbscript\s*:/i,
/data\s*:\s*text\/html/i,
/expression\s*\(/i
];
return patterns.some(p => p.test(decoded));
}
$('#frontEndContentForm').on('submit', function(e) { $('#frontEndContentForm').on('submit', function(e) {
e.preventDefault(); e.preventDefault();
const form = this; const form = this;
@ -548,6 +566,11 @@
toastr.warning("Content is required", 'Warning'); toastr.warning("Content is required", 'Warning');
return; return;
} }
if (hasUnsafeHtml(content_editor.value)) {
toastr.error("Content contains restricted tags or attributes. Script tags, iframes and event handlers (like onclick, onerror, onmouseover) are not allowed.", "Validation Error");
return;
}
} }
// Notes validation // Notes validation
@ -556,6 +579,11 @@
toastr.warning("Notes is required", 'Warning'); toastr.warning("Notes is required", 'Warning');
return; return;
} }
if (hasUnsafeHtml(notes_editor.value)) {
toastr.error("Notes contains restricted tags or attributes. Script tags, iframes and event handlers (like onclick, onerror, onmouseover) are not allowed.", "Validation Error");
return;
}
} }
// If valid, submit via AJAX // If valid, submit via AJAX

View File

@ -488,7 +488,7 @@
setTimeout(function() { setTimeout(function() {
$('.loader').fadeOut(); $('.loader').fadeOut();
$('.loader-mask').delay(350).fadeOut('slow'); $('.loader-mask').delay(350).fadeOut('slow');
toastr.warning('Something Wrong!', 'warning'); toastr.warning('Something Wrong!', 'Warning');
}, 1000); }, 1000);
}, },
complete: function() { complete: function() {

View File

@ -102,8 +102,12 @@
border-width: 1px !important; border-width: 1px !important;
} }
</style> </style>
<?php
$isLeadEditEb = isset($lead_edit_data) && ! empty($lead_edit_data);
$ebSubtitle = $isLeadEditEb ? 'Edit Opportunities - (EB)' : 'Add Opportunities - (EB)';
?>
<script> <script>
var pageSubTitle = 'Add Opportunity'; var pageSubTitle = '<?= $ebSubtitle ?>';
var pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>'; var pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>';
</script> </script>
@ -773,14 +777,11 @@
} }
} }
hide_list_show_add(); hide_list_show_add();
var page_title = 'Edit Opportunity'; pageSubTitle = 'Edit Opportunities - (EB)';
var pageSubTitle = 'Edit Opportunity'; pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>';
var pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>';
$('#page_title').text(page_title);
$('#leads_primarykey').val(res.data.id); $('#leads_primarykey').val(res.data.id);
$('#actual_lead_id').val(res.data.actual_lead_id || 0); $('#actual_lead_id').val(res.data.actual_lead_id || 0);
$('#policy_start_date').val(res.data.policy_end_date); $('#policy_start_date').val(res.data.policy_end_date);
@ -1617,8 +1618,19 @@
console.log('--- End of calculatePolicyMetrics ---\n'); console.log('--- End of calculatePolicyMetrics ---\n');
} }
$(document).on("input", "#incept_no_of_lives", function () {
if (this.value.length > 10) {
this.value = this.value.slice(0, 10); // max 10 digits
}
this.value = this.value.replace(/\D/g, '');
});
$(document).on("input", "#incept_emp_count, #incept_dept_count, #renewal_emp_count, #renewal_dept_count, #exp_emp_count, #exp_dept_count", function() { $(document).on("input", "#incept_emp_count, #incept_dept_count, #renewal_emp_count, #renewal_dept_count, #exp_emp_count, #exp_dept_count", function() {
if (this.value.length > 10) {
this.value = this.value.slice(0, 10); // max 10 digits
}
this.value = this.value.replace(/\D/g, '');
calculateTotalLives(this); calculateTotalLives(this);
}); });

View File

@ -47,8 +47,12 @@
border-width: 1px !important; border-width: 1px !important;
} }
</style> </style>
<?php
$isLeadEditNonEb = isset($lead_edit_data) && ! empty($lead_edit_data);
$nonEbSubtitle = $isLeadEditNonEb ? 'Edit Opportunities - (Non EB)' : 'Add Opportunities - (Non EB)';
?>
<script> <script>
var pageSubTitle = 'Add Opportunity'; var pageSubTitle = '<?= $nonEbSubtitle ?>';
var pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>'; var pageBackButton = '<a href="<?= base_url('leads/list') ?>" data-toggle="tooltip" data-placement="top" title="Back"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>';
</script> </script>

View File

@ -724,8 +724,8 @@ $(document).ready(function(){
function hide_list_show_add() function hide_list_show_add()
{ {
// $('#page_title').text('Add Policy') // $('#page_title').text('Add Policy')
pageSubTitle = 'Add Policy'; var pageSubTitle = 'Add Policy';
pageBackButton = '<a href="<?= base_url("policy_tranction/inception/list"); ?>" aria-label="Back to list"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>'; var pageBackButton = '<a href="<?= base_url("policy_tranction/inception/list"); ?>" aria-label="Back to list"><i class="mdi mdi-arrow-left" style="font-size: 17px;"></i></a>';
$('#inception_form_id')[0].reset(); $('#inception_form_id')[0].reset();
$('#client_id').val('').change().prop('disabled', false); $('#client_id').val('').change().prop('disabled', false);
$('#tpa').val('').change().prop('disabled', false); $('#tpa').val('').change().prop('disabled', false);

View File

@ -129,7 +129,7 @@
setTimeout(function() { setTimeout(function() {
$('.loader').fadeOut(); $('.loader').fadeOut();
$('.loader-mask').delay(350).fadeOut('slow'); $('.loader-mask').delay(350).fadeOut('slow');
toastr.warning('Something Went Wrong!', 'warning'); toastr.warning('Something Went Wrong!', 'Warning');
}, 1000); }, 1000);
} }
}) })