Есть ли какие-либо проблемы с производительностью или безопасностью при этом?
Нет, но это плохая практика.Хранение данных в глобальном масштабе может быть признаком плохого дизайна.Чрезмерные глобальные переменные - это «запах кода», но я не понимаю этого от этого конкретного использования.
Как уже упоминалось в комментариях, они, похоже, злоупотребляют глобальными глобальными переменными вместо реализации шаблона реестра , который предназначен для учета именно такого типа поведения без загрязнения глобальной области действия .
Многие другие языки на самом деле не имеют глобальной области видимости для переменных, что означает, что у них будет для использования Registry (или чего-то подобного) вместо этого.PHP просто помогает легко поступать неправильно.
Использование реестра, а не глобальных переменных также может упростить автоматическое тестирование кода.