报错是因为getMenus function 没有返回值，js默认返回undefined，然后又在undefined 上调then，所以报错
解决方法：

在getMenus 第一行初始化

var defer = $q.defer()

最后把defer.promise作为返回值返回
最终应该是这样

function getMenus() {
    var defer = $q.defer()
    if (...) {...}
    return defer.promise
}

你在 success和error回调里再返回promise当然会有问题

function getMenus() {

    var defer = $q.defer();
    
    if (!localStorage.getItem('menus')) {
        
        $http.get('api/menu').success(function (res) {
            // scope.menuList = res;
            localStorage.setItem('menus', res)
            defer.resolve();
        }).error(function (err) {
            defer.reject()
        });

    }
    
    return defer.promise;
}