Decadal

Поставьте в самом начале функции $city = [] ;

И добавьте строку на которую ругается. Номер строки, файл, саму строку
А то оно может ругаться совсем на другой файл

Есть две разных версии php: та, которая запускается как модуль Apache, и та, которая в CLI. В CLI, судя по вашему комментарию, версия 5.6, вам нужно апгрейднуть консольный php.
Версию CLI php можно проверить, набрав в консоли php -v
Версию php вашего сервера можно проверить по phpinfo или аналогичной функции для вашего фреймворка

Вроде бы нормально, но не слишком замудренно?

Хмм
замудрённо? Разве? Пожалуй, ссылку на аватарку можно было бы вынести вообще, а сделать таблицу файлов, а юзеру присвоить внешний ключ avatar_id, который будет ссылаться на запись таблицы файлов. Техническую информацию о профиле типа токенов можно отделить от информации о человеке (типа интересов, профессии, даты рождения). И т.д.
Нет предела усложнениям. Но есть конкретная задача. Сейчас будет слишком очевидная фраза: если в рамках этой задачи и вашего виденья проекта усложнять не нужно - усложнять не нужно. Если нужно - нужно.
Принимайте решения сами, если вы один, и советуйтесь с командой, если вы в команде. Все остальные советы по развитию и проектированию вашего приложения \ базы \ апи \ ..., которые начинаются со слов "не слишком ли сложно...", виднее только вам и вашей команде. На подобных ресурсах много спецов разных уровней которые набегут и насоветуют вам по вашему приложению кучу всего, но не факт, что их варианты будут лучше.
Вот например: я считаю, что ваше решение вполне уместное, да, следует разделить модели и таблицы так как вы указали, а более короткое решение просто не требуется. Но это моё мнение, тут может оказаться много других. Я ведь не вижу, как развивается ваш проект, какие у него цели и на каком он этапе, чтобы как-то аргументировать свои слова согласно вашей ситуации в проекте