search

basxsoftwareassociation / bread

Engine to create database applications based on Django and the IBM Carbon Design System
BSD 3-Clause "New" or "Revised" License
19 stars 2 forks source link

add: view for bulk restore, did: refactoring #62

Closed saemideluxe closed 3 years ago

saemideluxe commented 3 years ago

Refactoring: bulk functions no longer determine what happens after the function finished, the BrowseView is responsible for further processing, e.g. construct the correct redirect. The reason for this is that the bulk action function are not supposed to know the "calling convention", they only get the queryset. Therefore, in order to construct the correct redirect, the redirect needs to be create in the according BrowseView. I think this is sufficient for now, if we need more we can make the behaviour parameterizable later.

saemideluxe commented 3 years ago

Ah, and a small improvment for the bulk-delete function: The success message does no longer count already deleted entries.