Project

General

Profile

Bug #9334

bogus dialogue on Limiter deletion

Added by David Burns about 1 year ago. Updated 13 days ago.

Status:
Resolved
Priority:
Low
Category:
Limiters
Target version:
Start date:
02/17/2019
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.4.4_2
Affected Architecture:

Description

When deleting the last row of the Limiter config - an error "The last row may not be deleted." appears.

Clicking OK then presents "Are you sure you wish to delete limiter?".

On clicking OK the limiter is deleted.

Steps to reproduce:
1. Firewall / Traffic Shaper / Limiters - click New Limiter
2. Save limiter(s)
3. Apply changes
4. Diagnostics / Limiter Info - verify limiter config
5. Firewall / Traffic Shaper / Limiters - select Limiter
6. Click Delete Limiter
7. Observe dialog "The last row may not be deleted."
8. Click OK
9. Observe dialogue "Are you sure you wish to delete limiter?"
10. Click OK
11. Note that Limiter has been deleted

Associated revisions

Revision 86c560d9 (diff)
Added by Viktor Gurov 24 days ago

Remove bogus warning on limiter/shaper deletion. Issue #9334

History

#1 Updated by Johnny Good 29 days ago

David Burns wrote:

When deleting the last row of the Limiter config - an error "The last row may not be deleted." appears.

Clicking OK then presents "Are you sure you wish to delete limiter?".

On clicking OK the limiter is deleted.

Steps to reproduce:
1. Firewall / Traffic Shaper / Limiters - click New Limiter
2. Save limiter(s)
3. Apply changes
4. Diagnostics / Limiter Info - verify limiter config
5. Firewall / Traffic Shaper / Limiters - select Limiter
6. Click Delete Limiter
7. Observe dialog "The last row may not be deleted."
8. Click OK
9. Observe dialogue "Are you sure you wish to delete limiter?"
10. Click OK
11. Note that Limiter has been deleted

Same issue here. I was going to create a new report but after searching came across this old report. I'm surprised nothing was done about it?

#3 Updated by Jim Pingle 21 days ago

  • Status changed from New to Pull Request Review
  • Target version set to 2.5.0

#4 Updated by Renato Botelho 17 days ago

  • Status changed from Pull Request Review to Feedback
  • Assignee set to Renato Botelho
  • % Done changed from 0 to 100

Pull request has been merged. Thanks!

#5 Updated by Viktor Gurov 13 days ago

  • Status changed from Feedback to Resolved

works as expected on 2.5.0.a.20200211.1811

Also available in: Atom PDF