Project

General

Profile

Actions
Copy link

Regression #11747

closed
VG JP

Firewall rule schedule cannot be changed

Regression #11747: Firewall rule schedule cannot be changed

Added by Viktor Gurov over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Jim Pingle
Category:
Rules / NAT
Target version:
2.5.1
Start date:
03/28/2021
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Default
Affected Version:
2.5.0
Affected Architecture:

Description

Running on latest Version 2.5.0
When I modify the schedule, it will not save any longer

I get the following error:

The following input errors were detected:

Invalid start time - ''

When I added an additional time it worked
When I deleted a time from a few days ago and try to save it, it shows above error

https://forum.netgate.com/topic/161476/schedule-can-not-saved
When I deleted a time from a few days ago and try to save it, it shows above error

VG Updated by Viktor Gurov over 5 years ago Actions
Copy link
 #1

it also fixes error:

[28-Mar-2021 15:16:30 Europe/Moscow] PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /usr/local/www/firewall_schedule_edit.php on line 461
[28-Mar-2021 15:16:30 Europe/Moscow] PHP Warning:  Invalid argument supplied for foreach() in /usr/local/www/firewall_schedule_edit.php on line 463

when deleting the last timerange + saving

https://gitlab.netgate.com/pfSense/pfSense/-/merge_requests/210

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #2

  • Tracker changed from Bug to Regression
  • Subject changed from Schedule cannot be changed to Firewall rule schedule cannot be changed
  • Target version set to 2.5.1

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #3

  • Status changed from New to Pull Request Review

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #4

  • Status changed from Pull Request Review to Feedback

PR merged to master, after it's tested there we can consider cherry picking to 21.02.2/2.5.1 since it's a significant regression.

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #5

  • % Done changed from 0 to 100

Patch works for me when I test it, picked back so it doesn't get missed.

RB Updated by Renato Botelho over 5 years ago Actions
Copy link
 #6

  • Assignee set to Jim Pingle

AZ Updated by Alhusein Zawi over 5 years ago Actions
Copy link
 #7

I was able to modify Schedules when it is applied to FW rule (added/deleted)

2.5.1-RC (amd64)
built on Tue Apr 06 13:08:56 EDT 2021
FreeBSD 12.2-STABLE

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #8

  • Status changed from Feedback to Resolved
Actions
Copy link

Also available in: Atom