This code would be rather more readable with a local var to hold CConfigHelper::getInstance(), and using if rather than ?: