Remove duplicated code from stac_fastapi.types

[StijnCaerts]

Related Issue(s):

• N/A

Description: We have duplicated code from stac_fastapi.types in the code base. This could introduce hard to find bugs when method signatures in these abstract classes change and we do not propagate this change (eg. the path parameter collection_id that was introduced recently in the collection update from the transaction extension). By removing the duplicated code, we should catch these errors faster and it should be easier to upgrade to newer stac_fastapi versions.

PR Checklist:

• [x] Code is formatted and linted (run pre-commit run --all-files)
• [x] Tests pass (run make test)
• [x] Documentation has been updated to reflect changes, if applicable
• [ ] Changes are added to the changelog

[jonhealy1]

I created this so that I could add: database = attr.ib(default=BaseDatabaseLogic) to the AsyncBaseTransactionsClient and AsyncBaseCoreClient. I am sure there is a better way to do this as this is clearly not very maintainable. This was done to create the core library to support different databases. It looks like we don't need to do this but I'm not sure why tests are failing