phpbb-extensions/ad-management

Delete Ad bug

iMattPro opened this issue · 0 comments

The UI for deleting ads is a little bit messed. Turn off Javascript, go to delete an AD, but choose No in the confirm box. You will be returned to an empty ads list. This is incorrect behavior, user should be returned to the ads list they were seeing before. It's because the list_ads() in the ACP controller is in an else statement, so it never gets run when you abort deleting an ad.