This commit is contained in:
Dennis Eichhorn 2023-04-17 22:57:01 +00:00
commit e912e6b36d
6 changed files with 26 additions and 11 deletions

View File

@ -8,9 +8,11 @@ assignees: ''
--- ---
# Bug Description # Bug Description
A clear and concise description of what the bug is. A clear and concise description of what the bug is.
# How to Reproduce # How to Reproduce
Steps to reproduce the behavior: Steps to reproduce the behavior:
1. Go to '...' 1. Go to '...'
@ -19,16 +21,20 @@ Steps to reproduce the behavior:
4. See error 4. See error
# Expected Behavior # Expected Behavior
A clear and concise description of what you expected to happen. A clear and concise description of what you expected to happen.
# Screenshots # Screenshots
If applicable, add screenshots to help explain your problem. If applicable, add screenshots to help explain your problem.
# System Information # System Information
- System: [e.g. PC or iPhone11, ...]
- OS: [e.g. iOS] - System: [e.g. PC or iPhone11, ...]
- Browser [e.g. chrome, safari] - OS: [e.g. iOS]
- KarakaVersion [e.g. 22] - Browser [e.g. chrome, safari]
- KarakaVersion [e.g. 22]
# Additional Information # Additional Information
Add any other context about the problem here. Add any other context about the problem here.

View File

@ -8,11 +8,14 @@ assignees: ''
--- ---
# What is the feature you request # What is the feature you request
* A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] * A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
* A clear and concise description of what you want to happen. * A clear and concise description of what you want to happen.
# Alternatives # Alternatives
A clear and concise description of any alternative solutions or features you've considered. A clear and concise description of any alternative solutions or features you've considered.
# Additional Information # Additional Information
Add any other context or screenshots about the feature request here. Add any other context or screenshots about the feature request here.

View File

@ -46,8 +46,8 @@ final class ApiController extends Controller
private function validateTaxCodeCreate(RequestAbstract $request) : array private function validateTaxCodeCreate(RequestAbstract $request) : array
{ {
$val = []; $val = [];
if (($val['abbr'] = empty($request->getData('abbr'))) if (($val['abbr'] = !$request->hasData('abbr'))
|| ($val['title'] = empty($request->getData('title'))) || ($val['title'] = !$request->hasData('title'))
) { ) {
return $val; return $val;
} }
@ -100,7 +100,7 @@ final class ApiController extends Controller
$code->percentageSales = $request->getDataInt('percentage_sales_tax') ?? 0; $code->percentageSales = $request->getDataInt('percentage_sales_tax') ?? 0;
$code->percentageInput = $request->getDataInt('percentage_input_tax') ?? 0; $code->percentageInput = $request->getDataInt('percentage_input_tax') ?? 0;
if (!empty($request->getData('title'))) { if ($request->hasData('title')) {
$code->l11n->title = (string) ($request->getData('title')); $code->l11n->title = (string) ($request->getData('title'));
$code->l11n->short = $request->getDataString('short') ?? ''; $code->l11n->short = $request->getDataString('short') ?? '';
$code->l11n->long = $request->getDataString('long') ?? ''; $code->l11n->long = $request->getDataString('long') ?? '';
@ -122,8 +122,8 @@ final class ApiController extends Controller
private function validateTaxCodeL11nCreate(RequestAbstract $request) : array private function validateTaxCodeL11nCreate(RequestAbstract $request) : array
{ {
$val = []; $val = [];
if (($val['title'] = empty($request->getData('title'))) if (($val['title'] = !$request->hasData('title'))
|| ($val['code'] = empty($request->getData('code'))) || ($val['code'] = !$request->hasData('code'))
) { ) {
return $val; return $val;
} }

View File

@ -23,6 +23,9 @@ use phpOMS\DataStorage\Database\Mapper\DataMapperFactory;
* @license OMS License 2.0 * @license OMS License 2.0
* @link https://jingga.app * @link https://jingga.app
* @since 1.0.0 * @since 1.0.0
*
* @template T of BaseStringL11n
* @extends DataMapperFactory<T>
*/ */
final class TaxCodeL11nMapper extends DataMapperFactory final class TaxCodeL11nMapper extends DataMapperFactory
{ {

View File

@ -23,6 +23,9 @@ use phpOMS\DataStorage\Database\Mapper\DataMapperFactory;
* @license OMS License 2.0 * @license OMS License 2.0
* @link https://jingga.app * @link https://jingga.app
* @since 1.0.0 * @since 1.0.0
*
* @template T of TaxCode
* @extends DataMapperFactory<T>
*/ */
final class TaxCodeMapper extends DataMapperFactory final class TaxCodeMapper extends DataMapperFactory
{ {

View File

@ -223,7 +223,7 @@ echo $this->getData('nav')->render();
?> ?>
<tr> <tr>
<td><?= $values['month'] . '/' . \substr((string) $values['year'], -2); ?> <td><?= $values['month'] . '/' . \substr((string) $values['year'], -2); ?>
<td><?= (new Money(((int) $values['net_sales']) / 1000))->getCurrency(); ?> <td><?= (new Money(((int) $values['net_sales']) / 10000))->getCurrency(); ?>
<td><?= ((int) $values['customers']); ?> <td><?= ((int) $values['customers']); ?>
<?php endforeach; ?> <?php endforeach; ?>
<tr> <tr>
@ -343,7 +343,7 @@ echo $this->getData('nav')->render();
?> ?>
<tr> <tr>
<td><?= (string) $values['year']; ?> <td><?= (string) $values['year']; ?>
<td><?= (new Money(((int) $values['net_sales']) / 1000))->getCurrency(); ?> <td><?= (new Money(((int) $values['net_sales']) / 10000))->getCurrency(); ?>
<td><?= ((int) $values['customers']); ?> <td><?= ((int) $values['customers']); ?>
<?php endforeach; ?> <?php endforeach; ?>
</table> </table>