Several more issues with this:

1. If a variable is renamed in upgrades.sql then it have no effect on clean In-Portal installation. To make clean installation work it needs to be renamed in INSERT INTO sql in the install_data.sql file as well.
2. You have renamed variable only, but not it's usages. I've counted 2 usages at least in the Core module.

The DisplayOrder column value (the 10.04 in above SQL) should be unique across same configuration group. Right now it exactly matches value of NoPermissionTemplate system setting.

You need to:

1. shift down all configuration variables in same sub-section
   
2. use 10.05 as DisplayOrder column value for that new system setting
3. update install_data.sql to reflect new value in DisplayOrder column

I usually use SQL similar to this to shift up/down system settings in particular sub-group:

UPDATE SystemSettings
SET DisplayOrder = ROUND(DisplayOrder + 0.01, 2)
WHERE Section = 'x' and Heading = 'y' AND DisplayOrder > z

Just noticed that used label doesn't match system setting name and should be la_config_ FatalErrorTemplate. You can see on screenshot above that two system settings have identical translation.

This code will be used when it's Front-End and Debug Mode is Turned Off. In all other cases please use original error reporting code.

I think we can benefit from using error_fatal.tpl in Admin Console as well, when debug mode is turned off. To make this happen please replace

• this: $this->Application->isDebugMode() || $this->Application->isAdmin
• with this: $this->Application->isDebugMode(false)

Please replace text in <strong> tags with la_text_FatalError (also add this phrase to the /core/install/english.lang file).

1. Only show error message text when we're in Debug Mode.
2. Remove error message from session in any case.

When we're in admin then use error_fatal as error template and don't read it from system setting to allow Front-End having different template without affecting Admin Console where template name is always the same.

The ternary operator must be bracketed because otherwise the 'margin: ' . $this->Application->isAdmin will become it's condition.

This looks awful compared to other error pages in theme (e.g. Error Not Found or No Permission).

How this looks:

At least how it should look:

Key missing elements:

1. navigation bar
2. heading
3. paddings
4. phrases

Maybe something else is missing as well.

The no_permissions title preset is used resulting in No Permissions page title instead of correct one:

You shouldn't do any massive unrelated changes unless arc lint complains about them. So please revert.

Actually don't change that line, because we're only getting to _displayFatalError method in debug mode, when debug report is invisible and this happens during ajax request only in which case I want to see failed ajax request instead of redirect.

Please also do this (for debug mode only):

1. add the system-log_id parameter to redirect params, that would contain ID of just created log record
2. on the error template use <inp2:system-log_PrintBacktrace/> tag to display expandable stack trace of the error

Please don't do it, because we have separate discussion about this at http://community.in-portal.org/x/ZAD3.