OPNSense/core d43b14esrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php

Firewall: Rules [new]: Exclude loopback from interface selectpicker (#9723)

DeltaFile
+7-5src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+7-51 files

OPNSense/core 06e3733src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php

Firewall: Rules [new]: Exclude loopback from selectpicker, clean up use statements
DeltaFile
+7-5src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+7-51 files

OPNSense/core 2528b45src/opnsense/mvc/app/controllers/OPNsense/Firewall/forms dialogFilterRule.xml, src/opnsense/mvc/app/library/OPNsense/Firewall FilterRule.php

Firewall: Rules [new]: Add tcpflags_any for parity with legacy rules
DeltaFile
+10-0src/opnsense/mvc/app/controllers/OPNsense/Firewall/forms/dialogFilterRule.xml
+8-1src/opnsense/mvc/app/models/OPNsense/Firewall/Filter.php
+1-1src/opnsense/mvc/app/library/OPNsense/Firewall/FilterRule.php
+1-0src/opnsense/mvc/app/models/OPNsense/Firewall/Filter.xml
+20-24 files

OPNSense/core 3e438a8src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Firewall - Rules [new]: Add all rules option to interface selectpicker and make it default selection (#9713)

* Firewall - Rules [new]: Add all rules option to interface selectpicker and select it by default
* Interface select default only on null or no match, not on empty string
* To fix URL hash weirdness, it's best to special case __floating and __any in the frontend
* Lower diff in controller by folding null case into is_if
* firewall: tweak the interface selector

Show group name and description.  Could be inconvenient when
the description is long, but better for the auto-groups.
That also removes the hint at the end.  See note below.

Use fixed width logos for the interface groupings.

Remove muted from the any selection.

TODO/Remarks:

The groupings being shown as muted is a bit inconvenient also

    [16 lines not shown]
DeltaFile
+40-19src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+13-6src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+53-252 files

OPNSense/core 6dce1desrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Firewall - Rules [new]: Add all rules option to interface selectpicker and make it default selection (#9713)

* Firewall - Rules [new]: Add all rules option to interface selectpicker and select it by default
* Interface select default only on null or no match, not on empty string
* To fix URL hash weirdness, it's best to special case __floating and __any in the frontend
* Lower diff in controller by folding null case into is_if
* firewall: tweak the interface selector

Show group name and description.  Could be inconvenient when
the description is long, but better for the auto-groups.
That also removes the hint at the end.  See note below.

Use fixed width logos for the interface groupings.

Remove muted from the any selection.

TODO/Remarks:

The groupings being shown as muted is a bit inconvenient also

    [14 lines not shown]
DeltaFile
+40-19src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+13-6src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+53-252 files

OPNSense/core 44328cbsrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

firewall: tweak the interface selector

Show group name and description.  Could be inconvenient when
the description is long, but better for the auto-groups.
That also removes the hint at the end.  See note below.

Use fixed width logos for the interface groupings.

Remove muted from the any selection.

TODO/Remarks:

The groupings being shown as muted is a bit inconvenient also
as it muddies the perspective of what this does and how important
it is.  Would be nicer to not have it in this particular selector.

When an interface is selected it would be perfect if the grouping
icon would show so people see better what they selected.  Coloring
is nice so that would take it to the next level.

    [5 lines not shown]
DeltaFile
+14-11src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+1-2src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+15-132 files

OPNSense/core f2b9f50src/opnsense/www/js opnsense_ui.js opnsense.js

ui: move refresh of selectpicker types into setFormData() and improve type detection.

PR: https://github.com/opnsense/core/pull/9717

(cherry picked from commit cf0341f13928015813019ca5fd3509d8b710e245)
(cherry picked from commit 7a8f6bee113ec9994ca0c2b03ff43698ad0656a5)
DeltaFile
+10-9src/opnsense/www/js/opnsense_ui.js
+4-0src/opnsense/www/js/opnsense.js
+0-1src/opnsense/www/js/opnsense_bootgrid.js
+14-103 files

OPNSense/core e4b7d65src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

firewall: tweak the interface selector

Show group name and description.  Could be inconvenient when
the description is long, but better for the auto-groups.
That also removes the hint at the end.  See note below.

Use fixed width logos for the interface groupings.

Remove muted from the any selection.

TODO/Remarks:

The groupings being shown as muted is a bit inconvenient also
as it muddies the perspective of what this does and how important
it is.  Would be nicer to not have it in this particular selector.

When an interface is selected it would be perfect if the grouping
icon would show so people see better what they selected.  Coloring
is nice so that would take it to the next level.

    [5 lines not shown]
DeltaFile
+14-11src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+1-2src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+15-132 files

OPNSense/core 7a8f6besrc/opnsense/www/js opnsense_ui.js opnsense.js

mvc - ui: fix jquery glitch when using "options" instead of val(), closes https://github.com/opnsense/core/pull/9717
DeltaFile
+10-9src/opnsense/www/js/opnsense_ui.js
+0-1src/opnsense/www/js/opnsense.js
+10-102 files

OPNSense/core cf0341fsrc/opnsense/www/js opnsense.js opnsense_bootgrid.js

mvc - ui: move refresh of selectpicker types into setFormData() and improve type detection.

When we are using a selectpicker, the original one will be moved inside a div containing the "bootstrap-select" class.

If this fixes https://github.com/opnsense/core/pull/9717, we need to remove the console output as that is merely for debugging.
DeltaFile
+5-0src/opnsense/www/js/opnsense.js
+0-1src/opnsense/www/js/opnsense_bootgrid.js
+5-12 files

OPNSense/core 93ab29dsrc/opnsense/mvc/app/models/OPNsense/Base BaseModel.php

mvc: BaseModel - improve legacy mapper support when parent item doesn't exist.

Fixes:

ErrorException: Undefined array key 0 in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:755
Stack trace:

If we can't find the specified root node, we should create one, which is similar to non legacy mapper nodes.

(cherry picked from commit 40cb82128dc8ba5f8262654a693a9b273d716eb6)
DeltaFile
+9-1src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php
+9-11 files

OPNSense/core 76f5eddsrc/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api LeasesController.php

dnsmasq: Compare lower case strings only in leases to fix edge cases in is_reserved detection (#9714)

(cherry picked from commit 5276f51dc08f5bd145a3ef92ddb2b2849c260513)
DeltaFile
+6-8src/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api/LeasesController.php
+6-81 files

OPNSense/core 551c4bcsrc/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Firewall: Rules [new]: Theory for race condition during selectpicker population
DeltaFile
+22-0src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+22-01 files

OPNSense/core 40cb821src/opnsense/mvc/app/models/OPNsense/Base BaseModel.php

mvc: BaseModel - improve legacy mapper support when parent item doesn't exist.

Fixes:

ErrorException: Undefined array key 0 in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:755
Stack trace:

If we can't find the specified root node, we should create one, which is similar to non legacy mapper nodes.
DeltaFile
+9-1src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php
+9-11 files

OPNSense/core 2a750e1src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Firewall: Rules [new]: Theory for race condition during selectpicker population
DeltaFile
+15-6src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+15-61 files

OPNSense/core 5276f51src/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api LeasesController.php

dnsmasq: Compare lower case strings only in leases to fix edge cases in is_reserved detection (#9714)

DeltaFile
+6-8src/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api/LeasesController.php
+6-81 files

OPNSense/core 0329175src/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api LeasesController.php

dnsmasq: Compare lower case strings only in leases to fix edge cases in is_reserved detection
DeltaFile
+6-8src/opnsense/mvc/app/controllers/OPNsense/Dnsmasq/Api/LeasesController.php
+6-81 files

OPNSense/core 7ae42d9src/opnsense/scripts/firmware upgrade.sh

firmware: do not fail upgrade if new kernel is already booted

PR: https://forum.opnsense.org/index.php?topic=50654.0
DeltaFile
+2-0src/opnsense/scripts/firmware/upgrade.sh
+2-01 files

OPNSense/core 94d8a11src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php

Shuffle is_if condition around for clarity
DeltaFile
+2-2src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+2-21 files

OPNSense/core d048419src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Turn __* into __any
DeltaFile
+3-3src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+1-1src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+4-42 files

OPNSense/core 0935abbsrc/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Remove some unneeded newlines
DeltaFile
+0-3src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+0-31 files

OPNSense/core 0ba302dsrc/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

That comment went missing
DeltaFile
+1-1src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+1-11 files

OPNSense/core 1b42262src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Some more style
DeltaFile
+9-9src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+0-1src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+9-102 files

OPNSense/core defe87fsrc/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Style
DeltaFile
+0-1src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+0-11 files

OPNSense/core c3bb8b5src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php

Lower diff in controller by folding null case into is_if
DeltaFile
+5-9src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+5-91 files

OPNSense/core b8bf4afsrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php

Change label from any to floating (for floating)
DeltaFile
+1-1src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+1-11 files

OPNSense/core 15ef4besrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

To fix URL hash weirdness, it's best to special case __floating and __* in the frontend
DeltaFile
+17-16src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+2-2src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+19-182 files

OPNSense/core 3dd547csrc/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Interface select default only on null or no match, not on empty string
DeltaFile
+1-1src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+1-11 files

OPNSense/core e6e59fasrc/opnsense/mvc/app/controllers/OPNsense/Firewall/Api FilterController.php, src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Firewall - Rules [new]: Add all rules option to interface selectpicker and select it by default
DeltaFile
+33-10src/opnsense/mvc/app/controllers/OPNsense/Firewall/Api/FilterController.php
+14-3src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+47-132 files

OPNSense/core 9b67006src/opnsense/mvc/app/views/OPNsense/Firewall filter_rule.volt

Select all rules as default
DeltaFile
+4-0src/opnsense/mvc/app/views/OPNsense/Firewall/filter_rule.volt
+4-01 files