Project

General

Profile

Actions

Bug #5021

closed

can't not select queue/ackqueue on firewall_rules_edit.php where queues exist

Added by Chris Buechler over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
High
Target version:
Start date:
08/26/2015
Due date:
% Done:

100%

Estimated time:
Spent time:

Description

When you have traffic shaper queues configured, and edit a firewall rule, it sets the queue to the first in the list and doesn't allow you to not select a queue.

Actions #1

Updated by Chris Buechler over 8 years ago

  • Priority changed from Normal to High

this one really problematic on anything with queues defined. example: https://bootstrap-test1.pfmechanics.com

Actions #2

Updated by Anonymous over 8 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Chris Buechler

Would you mind setting this one up again please? This: https://bootstrap-test1.pfmechanics.com no longer reachable.

Actions #3

Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Confirmed
  • Assignee changed from Chris Buechler to Anonymous

that system's stable again and still has an example case.

Actions #4

Updated by Anonymous over 8 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Chris Buechler

Added 'None' option and changed initial value to accommodate new rule

Actions #5

Updated by Anonymous over 8 years ago

  • % Done changed from 0 to 100
Actions #6

Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Confirmed
  • Assignee changed from Chris Buechler to Anonymous

none option exists now. But it doesn't save as such when you select that option, and it doesn't default to that for new rules.

Actions #7

Updated by Anonymous over 8 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Chris Buechler

This control is unusual in that it saves two double-quotes to indicate the default value, rather than an empty string as most other pages do.

I have made an appropriate edit, but have not tested it since https://bootstrap-test1.pfmechanics.com is currently broken.

If this fix is not effective, would you please fix the test instance so I can investigate?

Actions #8

Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Confirmed
  • Assignee changed from Chris Buechler to Anonymous
  • Target version set to 2.3
  • Affected Version set to 2.3

no change.

Access issue because of DNS rebinding not being excluded for domain on Steve's end, discussed elsewhere and he's in now.

Actions #9

Updated by Anonymous over 8 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Anonymous to Chris Buechler

Thanks for all the testing help.

Selector values were rived and queues now save correctly.

Actions #10

Updated by Anonymous over 8 years ago

Actions #11

Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Resolved

fixed

Actions

Also available in: Atom PDF