Это синтаксис ECMAScript 2015. Мне кажется у вас все правильно написано:
then() принимает два аргумента, первый из которых это функция которая выполнится в случае если обещание выполнено, а второй - функция, которая выполнится если произошла ошибка. Функции даны стрелочные. Эта конструкция даже работает: говорит мне что произошла ошибка, т.к.
$http не существует.
Может быть PHPStorm считает, что это плохо, когда анонимная функция не выполнена моментально и не передана какой-нибудь переменной. Посмотрите
тут, мне кажется там такого же рода ошибка.
А вообще, это ведь даже не ошибка, а warning, разве нет? Можно ведь забить на это.