I am moving the check for duplicated values before the if-statement such that the if-statement is entirely skipped if no useful data is found. The place where you put it had the risk of delivering an empty features_temp object to the next block of code.
Checklist before merging:
[ ] Existing tests still work (check by running the test suite, e.g. from RStudio).
[ ] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[ ] Updated or expanded the documentation.
[ ] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[ ] Added your name to the contributors lists in the DESCRIPTION file, if you think you made a significant contribution.
This PR is an edit to PR #1033:
I am moving the check for duplicated values before the if-statement such that the if-statement is entirely skipped if no useful data is found. The place where you put it had the risk of delivering an empty features_temp object to the next block of code.
Checklist before merging:
inst/NEWS.Rd
with a user-readable summary. Please, include references to relevant issues or PR discussions.DESCRIPTION
file, if you think you made a significant contribution.