Fix nil pointer panic in error handling for employee report

ccremer commented 2 years ago

Summary

Adds a custom error implementation for "contract found"
Fixes the following panic:
```
..."method":"POST","uri":"/report"
```

panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x84903e]

goroutine 199 [running]: github.com/vshn/odootools/pkg/web/employeereport.(*EmployeeReport).ignoreNoContractFound(0xc0003e7cc0?, {0x84cc19?, 0xc00029a480?}, {0x0, 0x0}) /home/runner/work/odootools/odootools/pkg/web/employeereport/employeereport_controller.go:191 +0x1e



## Checklist

- [x] Categorize the PR by setting a good title and adding one of the labels:
      `bug`, `enhancement`, `documentation`, `change`, `breaking`, `dependency`
      as they show up in the changelog
- [x] Update the documentation.
- [x] Update tests.
- [x] Link this PR to related issues.

<!--
Remove items that do not apply. For completed items, change [ ] to [x].

NOTE: these things are not required to open a PR and can be done afterwards,
while the PR is open.
-->

github-actions[bot] commented 2 years ago

🚀 Preview deployment active


App URL	https://odootools-pr-112-vshn-odoo-test.apps.cloudscale-lpg-2.appuio.cloud
Revision	26840f786c0f5b4d872261679f5698b8082e168b
Helm release	vshn-odoo-test/odootools-pr-112
Cluster	https://api.cloudscale-lpg-2.appuio.cloud:6443

To uninstall this deployment, close or merge this PR.

github-actions[bot] commented 2 years ago

✔️ Preview deployment uninstalled


Helm release	vshn-odoo-test/odootools-pr-112
Cluster	https://api.cloudscale-lpg-2.appuio.cloud:6443

vshn / odootools