feat(http): Change HttpModule to HttpClientModule

rucken · Nov 18, 2017 · 880619b · 880619b
1 parent c036cb3
commit 880619b
Show file tree

Hide file tree

Showing 20 changed files with 262 additions and 273 deletions.
diff --git a/apps/demo/src/app/app.module.ts b/apps/demo/src/app/app.module.ts
@@ -1,32 +1,23 @@
+import { HttpClientModule } from '@angular/common/http';
 import { ModuleWithProviders, NgModule } from '@angular/core';
 import { FormsModule } from '@angular/forms';
-import { Http, HttpModule, RequestOptions } from '@angular/http';
 import { BrowserModule } from '@angular/platform-browser';
 import { PreloadAllModules, RouterModule } from '@angular/router';
 import { TranslateFakeLoader, TranslateLoader, TranslateModule } from '@ngx-translate/core';
-import {
-  AccountService,
-  AppService,
-  EndpointHelper,
-  HttpHelper,
-  RepositoryHelper,
-  RuckenCoreServices,
-  ThemesService,
-} from '@rucken/core';
+import { AppService, EndpointHelper, HttpHelper, RepositoryHelper, RuckenCoreServices, ThemesService } from '@rucken/core';
+import { TokenService } from '@rucken/core';
 import {
   AlertModalModule,
-  AuthHttpFactory,
   BaseResourceSelectInputConfig,
   RuckenWebServices,
   SelectInputConfig,
   SharedModule,
   TableColumnConfig,
   TextInputConfig,
-  WebAccountService,
   WebAppService,
   WebThemesService,
+  WebTokenService,
 } from '@rucken/web';
-import { AuthHttp } from 'angular2-jwt';
 import { LaddaModule } from 'angular2-ladda';
 import {
   ComponentLoaderFactory,
@@ -50,7 +41,7 @@ import { DemoHttpHelper } from './shared/helpers/http.helper';
   imports: [
     BrowserModule,
     FormsModule,
-    HttpModule,
+    HttpClientModule,
     LaddaModule.forRoot({
       style: 'expand-left',
       spinnerColor: 'white',
@@ -79,8 +70,7 @@ import { DemoHttpHelper } from './shared/helpers/http.helper';
     TableColumnConfig,
     { provide: ThemesService, useClass: WebThemesService },
     { provide: AppService, useClass: WebAppService },
-    { provide: AccountService, useClass: WebAccountService },
-    { provide: AuthHttp, useFactory: AuthHttpFactory.create, deps: [Http, RequestOptions] },
+    { provide: TokenService, useClass: WebTokenService },
     { provide: EndpointHelper, useClass: DemoEndpointHelper },
     { provide: RepositoryHelper, useClass: RepositoryHelper },
     { provide: HttpHelper, useClass: DemoHttpHelper }

diff --git a/apps/demo/src/app/i18n/ru.i18n.ts b/apps/demo/src/app/i18n/ru.i18n.ts
@@ -1,32 +1,32 @@
 export const RuckenDemoRuI18n = {
-  '': {
-    'project-id-version': '',
-    'pot-creation-date': '',
-    'po-revision-date': '',
-    'last-translator': '',
-    'language-team': '',
-    'language': 'ru',
-    'mime-version': '1.0',
-    'content-type': 'text/plain; charset=UTF-8',
-    'content-transfer-encoding': '8bit',
-    'x-generator': 'Poedit 2.0.4',
-    'plural-forms': 'nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);'
-  },
-  'Loading...': 'Загрузка...',
-  'Russian': 'Русский',
-  'English': 'Английский',
-  'Account': 'Личный кабинет',
-  'Profile': 'Профиль',
-  'Admin': 'Администрирование',
-  'Groups': 'Группы',
-  'Users': 'Пользователи',
-  'Components': 'Компоненты',
-  'Controls': 'Контролы',
-  'Home': 'Домашняя страница',
-  'Themes': 'Темы',
-  'Language': 'Язык',
-  'Menu': 'Меню',
-  'Rucken: Demo': 'Rucken: Todo',
-  'Logout': 'Выйти',
-  'Login': 'Вход'
+    '': {
+        'project-id-version': '',
+        'pot-creation-date': '',
+        'po-revision-date': '',
+        'last-translator': '',
+        'language-team': '',
+        'language': 'ru',
+        'mime-version': '1.0',
+        'content-type': 'text/plain; charset=UTF-8',
+        'content-transfer-encoding': '8bit',
+        'x-generator': 'Poedit 2.0.4',
+        'plural-forms': 'nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);'
+    },
+    'Loading...': 'Загрузка...',
+    'Russian': 'Русский',
+    'English': 'Английский',
+    'Account': 'Личный кабинет',
+    'Profile': 'Профиль',
+    'Admin': 'Администрирование',
+    'Groups': 'Группы',
+    'Users': 'Пользователи',
+    'Components': 'Компоненты',
+    'Controls': 'Контролы',
+    'Home': 'Домашняя страница',
+    'Themes': 'Темы',
+    'Language': 'Язык',
+    'Menu': 'Меню',
+    'Rucken: Demo': 'Rucken: Todo',
+    'Logout': 'Выйти',
+    'Login': 'Вход'
 };
diff --git a/libs/core/src/i18n/ru.i18n.ts b/libs/core/src/i18n/ru.i18n.ts
@@ -1,63 +1,63 @@
 export const RuckenCoreRuI18n = {
-  '': {
-    'project-id-version': '',
-    'pot-creation-date': '',
-    'po-revision-date': '',
-    'last-translator': '',
-    'language-team': '',
-    'language': 'ru',
-    'mime-version': '1.0',
-    'content-type': 'text/plain; charset=UTF-8',
-    'content-transfer-encoding': '8bit',
-    'x-generator': 'Poedit 2.0.4',
-    'plural-forms': 'nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);'
-  },
-  'Yes': 'Да',
-  'No': 'Нет',
-  'Loading...': 'Загрузка...',
-  'Not found': 'Нет данных',
-  'Error, you not set perPage count': 'Ошибка, вы не установили текущую страницу',
-  'Error in creating': 'Ошибка при создании',
-  'Creating...': 'Создание...',
-  'Updating...': 'Изменение...',
-  'Error in updating': 'Ошибка при изменении',
-  'Removing...': 'Удаление...',
-  'OK': 'Ок',
-  'Error on deleting': 'Ошибка при удалении',
-  'Unknown error': 'Неизвестная ошибка',
-  'Id': 'Ид.',
-  'Name': 'Наименование',
-  'Title': 'Заголовок',
-  'Icon': 'Иконка',
-  'Code': 'Код',
-  'Class': 'Класс',
-  'Group': 'Группа',
-  'Permission': 'Разрешения',
-  'Permissions': 'Разрешения',
-  'Total results': 'Всего записей',
-  'Current page': 'Текущ. стр.',
-  'Total pages': 'Всего стр.',
-  'Per page': 'Кол-во стр.',
-  'Content type': 'Тип содержимого',
-  'Url': 'Адрес',
-  'User': 'Пользователь',
-  'Username': 'Имя пользователя',
-  'Password': 'Пароль',
-  'Repeat password': 'Пароль (повторно)',
-  'Username/Email': 'Имя пользователя/ Эл. почта',
-  'Roles': 'Роли',
-  'Staff': 'Персонал',
-  'Administrator': 'Администратор',
-  'Full name': 'Полное имя',
-  'First name': 'Имя',
-  'Last name': 'Фамилия',
-  'Email': 'Эл. почта',
-  'Last login': 'Дата последнего входа',
-  'Date joined': 'Дата регистрации',
-  'Groups': 'Группы',
-  'Date of birth': 'Дата рождени',
-  'Password does not match the repeat password': 'Введённые пароли отличаются',
-  'Error': 'Ошибка',
-  'Login...': 'Вход...',
-  'Logout...': 'Выход...'
+    '': {
+        'project-id-version': '',
+        'pot-creation-date': '',
+        'po-revision-date': '',
+        'last-translator': '',
+        'language-team': '',
+        'language': 'ru',
+        'mime-version': '1.0',
+        'content-type': 'text/plain; charset=UTF-8',
+        'content-transfer-encoding': '8bit',
+        'x-generator': 'Poedit 2.0.4',
+        'plural-forms': 'nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);'
+    },
+    'Yes': 'Да',
+    'No': 'Нет',
+    'Loading...': 'Загрузка...',
+    'Not found': 'Нет данных',
+    'Error, you not set perPage count': 'Ошибка, вы не установили текущую страницу',
+    'Error in creating': 'Ошибка при создании',
+    'Creating...': 'Создание...',
+    'Updating...': 'Изменение...',
+    'Error in updating': 'Ошибка при изменении',
+    'Removing...': 'Удаление...',
+    'OK': 'Ок',
+    'Error on deleting': 'Ошибка при удалении',
+    'Unknown error': 'Неизвестная ошибка',
+    'Id': 'Ид.',
+    'Name': 'Наименование',
+    'Title': 'Заголовок',
+    'Icon': 'Иконка',
+    'Code': 'Код',
+    'Class': 'Класс',
+    'Group': 'Группа',
+    'Permission': 'Разрешения',
+    'Permissions': 'Разрешения',
+    'Total results': 'Всего записей',
+    'Current page': 'Текущ. стр.',
+    'Total pages': 'Всего стр.',
+    'Per page': 'Кол-во стр.',
+    'Content type': 'Тип содержимого',
+    'Url': 'Адрес',
+    'User': 'Пользователь',
+    'Username': 'Имя пользователя',
+    'Password': 'Пароль',
+    'Repeat password': 'Пароль (повторно)',
+    'Username/Email': 'Имя пользователя/ Эл. почта',
+    'Roles': 'Роли',
+    'Staff': 'Персонал',
+    'Administrator': 'Администратор',
+    'Full name': 'Полное имя',
+    'First name': 'Имя',
+    'Last name': 'Фамилия',
+    'Email': 'Эл. почта',
+    'Last login': 'Дата последнего входа',
+    'Date joined': 'Дата регистрации',
+    'Groups': 'Группы',
+    'Date of birth': 'Дата рождени',
+    'Password does not match the repeat password': 'Введённые пароли отличаются',
+    'Error': 'Ошибка',
+    'Login...': 'Вход...',
+    'Logout...': 'Выход...'
 };
diff --git a/libs/core/src/index.ts b/libs/core/src/index.ts
@@ -52,6 +52,8 @@ import { PermissionsService } from './shared/services/permissions.service';
 export { PermissionsService } from './shared/services/permissions.service';
 import { ThemesService } from './shared/services/themes.service';
 export { ThemesService } from './shared/services/themes.service';
+import { TokenService } from './shared/services/token.service';
+export { TokenService } from './shared/services/token.service';
 import { UserGroupsService } from './shared/services/user-groups.service';
 export { UserGroupsService } from './shared/services/user-groups.service';
 import { UsersService } from './shared/services/users.service';
@@ -67,5 +69,5 @@ export { BaseRepositoryService } from './shared/base/services/base-repository.se
 export const RuckenCoreModules: any[] = [];
 export const RuckenCoreComponents: any[] = [];
 export const RuckenCoreShareds: any[] = [RuckenCoreRuI18n, translate, isJson, inValues, EndpointStatusEnum, EndpointHelper, HttpHelper, RepositoryHelper, FontawesomeItemsMock, ThemeItemsMock, ContentType, Fontawesome, GroupPermission, Group, MetaModel, Permission, Theme, UserGroup, User, BaseResourceModel];
-export const RuckenCoreServices: any[] = [AccountService, AppService, ContentTypesService, FontawesomeService, GroupPermissionsService, GroupsService, PermissionsService, ThemesService, UserGroupsService, UsersService, BaseLocalRepositoryService, BaseRemoteRepositoryService, BaseRepositoryService];
+export const RuckenCoreServices: any[] = [AccountService, AppService, ContentTypesService, FontawesomeService, GroupPermissionsService, GroupsService, PermissionsService, ThemesService, TokenService, UserGroupsService, UsersService, BaseLocalRepositoryService, BaseRemoteRepositoryService, BaseRepositoryService];
 export const RuckenCorePipes: any[] = [];
diff --git a/libs/core/src/package-lock.json b/libs/core/src/package-lock.json
diff --git a/libs/core/src/package.json b/libs/core/src/package.json
@@ -20,8 +20,7 @@
     },
     "dependencies": {
         "@types/lodash": "*",
-        "angular2-jwt": "*",
         "lodash": "*",
         "moment": "*"
     }
-}
+}
diff --git a/libs/core/src/shared/base/services/base-remote-repository.service.ts b/libs/core/src/shared/base/services/base-remote-repository.service.ts
@@ -140,13 +140,14 @@ export class BaseRemoteRepositoryService extends BaseLocalRepositoryService {
         );
       }
     } else {
-      if (error.json && error.json().detail === 'Invalid page.' && filter.curPage > 1) {
+      const errorBody = error.json && _.isFunction(error.json) ? error.json() : error;
+      if (errorBody && errorBody.detail === 'Invalid page.' && filter.curPage > 1) {
         filter.curPage = 1;
         this.ignoreCache = true;
         this.loadAll(filter.q, filter);
       } else {
         this.items$.next([]);
-        result.error(this.repositoryHelper.extractError(error));
+        result.error(this.repositoryHelper.extractError(errorBody));
         this.setStatusList(EndpointStatusEnum.NotFound,
           translate('Not found')
         );

diff --git a/libs/core/src/shared/helpers/endpoint.helper.ts b/libs/core/src/shared/helpers/endpoint.helper.ts
@@ -43,7 +43,7 @@ export class EndpointHelper {
     return this.httpHelper.post(this.actionUrl(endpointService, action, null), this.actionRequestBody(endpointService, action, data));
   }
   actionRequestBody(endpointService: any, action?: any, data?: any) {
-    if (data === undefined) {
+    if (!data) {
       data = {};
     }
     if (data.format !== undefined) {
@@ -64,11 +64,11 @@ export class EndpointHelper {
     if (message === undefined) {
       message = translate('Unknown error');
     }
-    if (!error._body || !isJson(error._body) || error.json().type === 'error') {
+    if (!error._body || !isJson(error._body) || error.type === 'error') {
       console.log(error);
       return { message: [error.statusText ? error.statusText : message] };
     } else {
-      const errorBody = error.json();
+      const errorBody = error.json && _.isFunction(error.json) ? error.json() : error;
       if (_.isString(errorBody)) {
         return { message: [errorBody] };
       }