pfSense

Display additional ooptions if value not empty

Merge pull request #2447 from phil-davis/patch-7

Merge pull request #2446 from phil-davis/patch-6

Fixed #5772

Internationalization of diag_defaults

A couple of typical fixes here:
1) Actually wrap the "Are you sure...?" question in gettext()
2) Use a single gettext() call combined with sprintf() and %s etc to construct a sentence in a single go.
3) Use %s rather than just appending a variable at the end of a sentence of text. That way there is no assumption that the variable text must go at the end in every language....

diag_confbak make a single gettext

Sentences that are constructed in pieces like this are a potential problem for translation, if the target language needs to use a different grammatical word order. e.g. in Nepali we would say datetime-dekhi datetime-samma (datetime-from datetime-to)....

Fixes some hide/show issues. An lucky bug in 2.2 was not duplicated in 2.3 so the page operated as written rather that as desired.

Merge pull request #2445 from phil-davis/patch-5

Merge pull request #2444 from phil-davis/patch-3

diag_confbak move tab_array code above diff display

At the moment the tabs "Config History" and "Backup/Restore" jump down the page below the config diff listing when a diff is displayed.
It seems better to me that the tab text stays where it is and the diff listing comes directly below it.