Now I am the one with fat fingers (or brain). Somehow I typed a curly bracket opening bracket in this text, but finished it with an ordinary closing bracket.
1) When saving Status: System logs: Settings, if the Log File Size field is left blank, it gets cast to (int) and ends pu as zero "0" in config.xml. That breaks the creation of clog files in /etc/rc during boot. /etc/rc expects logfilesize to be unset, blank or a valid integer >= 5120.
So, if the field is empty, then make sure that it is unset in config.xml
2) For data items that are cast to (int) whenputting into the config (nentries and logfilesize), copy the resulting value back to the $pconfig array. When the user enters 123.45 the in field, is correctly truncated to (int) 123 in config.xml. By copying that back to $pconfig, the "123" also appears correctly on the GUI page after the save.
3) Mention the units of measure of Log File Size (Bytes)
4) Mention that the user should press Save first when changing values, then they can use "Reset Log Files" to set the log file sizes to the just-saved value.
5) Display a message when the log files have been reset - it's a bit more user-friendly IMHO.
If I have an internal gateway to other private subnets behind LAN, and add a static route to those private subnets through the internal gateway, then pfSense very nicely generates NAT rules on WAN-style interfaces to NAT those internal subnets on the way out of WAN. That is very good and convenient. This outbound NAT is actually correctly in the ruleset.
However, the new Outbound NAT GUI page shows the Automatic Outbound NAT rules that are in effect (a great new feature in 2.2). But it did not show the rules for the static routes.
This fixes it. (But the code for all of this is a bit ugly, because it all depends on having a global variable $GatewaysList in view - I am not going to attempt to re-engineer that)
Thought I would tidy this up while coming across it. IMHO "Package settings" reads better than "Packages settings"... and I reworded the explanatory text a bit also.
Added check to see if $config['ipsec']["ipsec_{$lkey}"] is set before executing unnecessary unset, was causing issues when trying to enable/disable ("Do NOT check packages signature")
Add "cloeahead" variable and manually close HEAD
Add CDATA sections to scripts
Close INPUT
Add SUMMARY to tables
Update HTML Boolean operators
Close IMG and add ALT
Remove empty TFOOT tag
Move TFOOT tag between THEAD and TBODY (quirk of HTML)
Add dummy table row, but don't display it
Add missing closing DIV and closing FORM
Add "cloeahead" variable and manually close HEAD
Add CDATA sections to scripts
Add SUMMARY to tables
Update HTML Boolean operators
Close INPUT
Remove duplicate closing TD tag
Move TFOOT tag between THEAD and TBODY (quirk of HTML)
Add dummy table row, but don't display it
Remove duplicate opening and closing TR tags
Closing IMG and add ALT
Deprecate ampersand
Add "closehead" variable and manually close HEAD
Add CDATA sections to scripts
Add SUMMARY to tables
Update HTML Boolean operators
Close INPUT
Move TFOOT between THEAD and TBODY (quirk of HTML)
Move NOWRAP into class statement
Close IMG and add ALT
Deprecate ampersand
Add table row, but don't display it
Remove duplicate opening and closing TR tags
Add missing closing BODY tag and closing HTML tag
Update alert message
Add "closehead" variable and manually close HEAD
Add CDATA sections to script
Add SUMMARY to tables
Update HTML Boolean operators
Close INPUT
Add missing closing P tag
Remove empty TFOOT tag
Deprecate ampersand
