mirror of
https://github.com/opensourcepos/opensourcepos.git
synced 2026-09-13 22:08:16 -04:00
6e273a2fb1
* refactor: Replace var with let/const in JavaScript files - Replace var with let for variables that are reassigned - Replace var with const for variables that are never reassigned - Modernize manage_tables.js and nominatim.autocomplete.js - Skip third-party libraries (imgpreview.full.jquery.js, clipboard.min.js) Closes #4491 * refactor: Replace var with let/const in inline JavaScript - Fixed CodeRabbit review: changed enable_actions and load_success from const to let in manage_tables.js (they are reassigned in init) - Replaced all var declarations in inline JavaScript in Views with let (for reassigned) or const (for never reassigned) - Modernized 48 additional files with inline JavaScript * refactor: Replace var with let/const in remaining JS files - Modernized gulpfile.js: 3 var declarations replaced - Modernized app/Views/errors/html/debug.js: all var declarations replaced - Used const for never-reassigned, let for reassigned variables * fix: Replace remaining var declarations in Views - Changed var to const in sales/register.php - Changed var to const in configs/receipt_config.php These were missed in the initial pass. * fix: Replace remaining var declarations in gulpfile.js - Converted 12 remaining var declarations to const - All variables are function-scoped and never reassigned - Complete coverage for this file now * fix: Address CodeRabbit review comments - items/manage.php: Remove duplicate let declaration for start_date (partial/daterangepicker already declares it) - header_js.php: Escape CSRF hash in JavaScript context - tax_jurisdictions.php: Fix mismatched selector (remove_tax_jurisdictions -> remove_tax_jurisdiction) * style(views): Replace var with let/const and fix comment casing - Convert var to let in items/manage.php for JS modernization - Capitalize \"Submit\" in validation comments across tax view files Signed-off-by: Travis Garrison <travis@chiraqbookstore.com> --------- Signed-off-by: Travis Garrison <travis@chiraqbookstore.com> Co-authored-by: Ollama <ollama@steganos.dev> Co-authored-by: objecttothis <17935339+objecttothis@users.noreply.github.com> Co-authored-by: Travis Garrison <travis@chiraqbookstore.com>
85 lines
2.5 KiB
PHP
85 lines
2.5 KiB
PHP
<?php
|
|
/**
|
|
* @var array $labels_1
|
|
* @var array $series_data_1
|
|
* @var bool $show_currency
|
|
* @var array $config
|
|
*/
|
|
?>
|
|
|
|
<script type="text/javascript">
|
|
// Labels and data series
|
|
const data = {
|
|
labels: <?= json_encode(esc($labels_1, 'js')) ?>,
|
|
series: <?= json_encode(esc($series_data_1, 'js')) ?>
|
|
};
|
|
|
|
// We are setting a few options for our chart and override the defaults
|
|
const options = {
|
|
|
|
// Specify a fixed width for the chart as a string (i.e. '100px' or '50%')
|
|
width: '100%',
|
|
|
|
// Specify a fixed height for the chart as a string (i.e. '100px' or '50%')
|
|
height: '100%',
|
|
|
|
// Padding of the chart drawing area to the container element and labels as a number or padding object {top: 5, right: 5, bottom: 5, left: 5}
|
|
chartPadding: 20,
|
|
|
|
// This option can be set to 'inside', 'outside' or 'center'.
|
|
// Show the labels on the border with the pie chart
|
|
labelPosition: 'outside',
|
|
labelDirection: 'explode',
|
|
|
|
<?php
|
|
$currency_symbol = esc($config['currency_symbol'], 'js');
|
|
$currency_prefix = '';
|
|
$currency_suffix = '';
|
|
|
|
if ($show_currency) {
|
|
if (is_right_side_currency_symbol()) {
|
|
$currency_suffix = $currency_symbol;
|
|
} else {
|
|
$currency_prefix = $currency_symbol;
|
|
}
|
|
}
|
|
?>
|
|
|
|
plugins: [
|
|
Chartist.plugins.tooltip({
|
|
transformTooltipTextFnc: function(value) {
|
|
return '<?= $currency_prefix ?>' + value + '<?= $currency_suffix ?>';
|
|
}
|
|
})
|
|
] };
|
|
|
|
const responsiveOptions = [
|
|
['screen and (min-width: 640px)', {
|
|
height: '80%',
|
|
chartPadding: 20
|
|
|
|
}]
|
|
/* ,
|
|
* ['screen and (min-width: 1024px)', {
|
|
* labelOffset: 80,
|
|
* chartPadding: 20
|
|
* }]
|
|
*/
|
|
];
|
|
|
|
chart = new Chartist.Pie('#chart1', data, options, responsiveOptions);
|
|
|
|
// Generate random colours for the pie sliced because Chartist is currently limited to 15 colours
|
|
chart.on('draw', function(data) {
|
|
if (data.type === 'slice') {
|
|
const r = Math.floor(Math.random() * 256);
|
|
const g = Math.floor(Math.random() * 256);
|
|
const b = Math.floor(Math.random() * 256);
|
|
|
|
data.element.attr({
|
|
style: 'fill: #' + ((1 << 24) + (r << 16) + (g << 8) + b).toString(16).slice(1)
|
|
});
|
|
}
|
|
});
|
|
</script>
|