-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Asset report fatal exception #2499
Comments
That is indeed a bug. I renamed the method but must have missed this occurance in reportcontroller... |
Missed this instance when renaming the method. Sorry!
Great, thanks! If I run git pull will I get this fix? |
Not yet, it will need to be merged by snipe first. You can just edit Line 170 of reportscontroller to say Helper::formatCurrencyOutput instead of Helper::parsePurchaseCost temporarily, and then overwrite it when pulling later. |
Fantastic, That works a treat. Thanks for the great support! |
Expected Behavior (or desired behavior if a feature request)
When I attempt to download a CSV asset report I should get an asset report downloaded, as in the demo,
Actual Behavior
The system throws the following error:
FatalErrorException in ReportsController.php line 170:
Call to undefined method App\Helpers\Helper::parsePurchasedCost()
in ReportsController.php line 170
Initally when trying to reproduce in the demo I wondered if it was becuase purchase cost was blank, so I have gone in an changed the assets to £0 cost, but it throws the same error.
The local is set to EN-GB and the currency is set to £ (with this sign)
Please confirm you have done the following before posting your bug report:
Please provide answers to these questions before posting your bug report:
app/storage/logs
and your webserver's logs.The text was updated successfully, but these errors were encountered: