WebsiteRepository delete method should not delete DB and DB user when doing a soft delete

tenancy / multi-tenant

Run multiple websites using the same Laravel installation while keeping tenant specific data separated for fully independent multi-domain setups, previously github.com/hyn/multi-tenant

https://tenancy.dev

MIT License

2.56k stars 394 forks source link

WebsiteRepository delete method should not delete DB and DB user when doing a soft delete #945

Open Bibendus83 opened 4 years ago

Bibendus83 commented 4 years ago

How to reproduce

Set TENANCY_DATABASE_AUTO_DELETE and/or TENANCY_DATABASE_AUTO_DELETE_USER to true in .env settings
Delete a website with soft delete using app(WebsiteRepository::class)->delete($website, false);

Result

The website is soft deleted from the websites table, however the database and user are hard deleted from the DBMS too.

Expected behavior

Database and user should not be removed when soft deleting a website

bkintanar commented 4 years ago

@Bibendus83 can you try my branch if it works for you?