kenmirrors/firefly-iii
1
0
Fork 0
mirror of https://github.com/firefly-iii/firefly-iii.git synced 2025-10-12 15:35:15 +00:00
Code Issues Projects Releases Wiki Activity

More audit logs.

Browse Source
This commit is contained in:
James Cole
2024-01-04 08:34:57 +01:00
parent 84ae6a633e
commit 09c18d6d44
2 changed files with 31 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
app/Rules/IsValidAmount.php
View File

@@ -22,24 +22,27 @@ class IsValidAmount implements ValidationRule
// must not be empty: // must not be empty:
if($this->emptyString($value)) { if($this->emptyString($value)) {
$fail('validation.filled')->translate(); $fail('validation.filled')->translate();
Log::info(sprintf('IsValidAmount: "%s" cannot be empty.', $value)); $message = sprintf('IsValidAmount: "%s" cannot be empty.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must be a number: // must be a number:
if(!$this->isValidNumber($value)) { if(!$this->isValidNumber($value)) {
$fail('validation.numeric')->translate(); $fail('validation.numeric')->translate();
Log::info(sprintf('IsValidAmount: "%s" is not a number.', $value)); $message = sprintf('IsValidAmount: "%s" is not a number.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must not be scientific notation: // must not be scientific notation:
if($this->scientificNumber($value)) { if($this->scientificNumber($value)) {
$fail('validation.scientific_notation')->translate(); $fail('validation.scientific_notation')->translate();
Log::info(sprintf('IsValidAmount: "%s" cannot be in the scientific notation.', $value)); $message = sprintf('IsValidAmount: "%s" cannot be in the scientific notation.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
@@ -47,15 +50,18 @@ class IsValidAmount implements ValidationRule
if($this->lessThanLots($value)) { if($this->lessThanLots($value)) {
$amount = bcmul('-1', self::BIG_AMOUNT); $amount = bcmul('-1', self::BIG_AMOUNT);
$fail('validation.gte.numeric')->translate(['value' => $amount]); $fail('validation.gte.numeric')->translate(['value' => $amount]);
Log::info(sprintf('IsValidAmount: "%s" must be more than %s.', $value, $amount)); $message = sprintf('IsValidAmount: "%s" must be more than %s.', $value, $amount);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must be less than 100 million and 1709: // must be less than 100 million and 1709:
if($this->moreThanLots($value)) { if($this->moreThanLots($value)) {
Log::info(sprintf('IsValidPositiveAmount: "%s" must be more than %s.', $value, self::BIG_AMOUNT));
$fail('validation.lte.numeric')->translate(['value' => self::BIG_AMOUNT]); $fail('validation.lte.numeric')->translate(['value' => self::BIG_AMOUNT]);
$message = sprintf('IsValidAmount: "%s" must be more than %s.', $value, self::BIG_AMOUNT);
Log::info($message);
Log::channel('audit')->info($message);
} }
Log::info(sprintf('IsValidAmount: "%s" is a valid positive amount.', $value)); Log::info(sprintf('IsValidAmount: "%s" is a valid positive amount.', $value));
} }

26
app/Rules/IsValidPositiveAmount.php
View File

@@ -21,37 +21,43 @@ class IsValidPositiveAmount implements ValidationRule
// must not be empty: // must not be empty:
if($this->emptyString($value)) { if($this->emptyString($value)) {
$fail('validation.filled')->translate(); $fail('validation.filled')->translate();
Log::info(sprintf('IsValidPositiveAmount: "%s" cannot be empty.', $value)); $message = sprintf('IsValidPositiveAmount: "%s" cannot be empty.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must be a number: // must be a number:
if(!$this->isValidNumber($value)) { if(!$this->isValidNumber($value)) {
$fail('validation.numeric')->translate(); $fail('validation.numeric')->translate();
Log::info(sprintf('IsValidPositiveAmount: "%s" is not a number.', $value)); $message = sprintf('IsValidPositiveAmount: "%s" is not a number.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must not be scientific notation: // must not be scientific notation:
if($this->scientificNumber($value)) { if($this->scientificNumber($value)) {
$fail('validation.scientific_notation')->translate(); $fail('validation.scientific_notation')->translate();
Log::info(sprintf('IsValidPositiveAmount: "%s" cannot be in the scientific notation.', $value)); $message = sprintf('IsValidPositiveAmount: "%s" cannot be in the scientific notation.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must be more than zero: // must be more than zero:
if($this->lessOrEqualToZero($value)) { if($this->lessOrEqualToZero($value)) {
$fail('validation.more_than_zero')->translate(); $fail('validation.more_than_zero')->translate();
Log::info(sprintf('IsValidPositiveAmount: "%s" must be more than zero.', $value)); $message = sprintf('IsValidPositiveAmount: "%s" must be more than zero.', $value);
Log::info($message);
Log::channel('audit')->info($message);
return; return;
} }
// must be less than 100 million and 1709: // must be less than 100 million and 1709:
if($this->moreThanLots($value)) { if($this->moreThanLots($value)) {
Log::info(sprintf('IsValidPositiveAmount: "%s" must be less than %s.', $value, self::BIG_AMOUNT));
$fail('validation.lte.numeric')->translate(['value' => self::BIG_AMOUNT]); $fail('validation.lte.numeric')->translate(['value' => self::BIG_AMOUNT]);
$message = sprintf('IsValidPositiveAmount: "%s" must be less than %s.', $value, self::BIG_AMOUNT);
Log::info($message);
Log::channel('audit')->info($message);
} }
Log::info(sprintf('IsValidPositiveAmount: "%s" is a valid positive amount.', $value)); Log::debug(sprintf('IsValidPositiveAmount: "%s" is a valid positive amount.', $value));
} }
} }