refactor: read daia service url from app config

This commit is contained in:
Rainer Killinger
2022-02-08 13:02:03 +01:00
parent 2cd0a70fd4
commit 988236e646
4 changed files with 85 additions and 51 deletions

View File

@@ -29,6 +29,7 @@ import {DaiaAvailabilityComponent} from './daia-availability.component';
import {Observable, of} from 'rxjs'; import {Observable, of} from 'rxjs';
import {StorageProvider} from '../../storage/storage.provider'; import {StorageProvider} from '../../storage/storage.provider';
import {DaiaDataProvider} from '../daia-data.provider'; import {DaiaDataProvider} from '../daia-data.provider';
import {LoggerConfig, LoggerModule, NGXLogger} from 'ngx-logger';
const translations: any = {data: {detail: {TITLE: 'Foo'}}}; const translations: any = {data: {detail: {TITLE: 'Foo'}}};
@@ -79,6 +80,7 @@ describe('DaiaAvailabilityComponent', () => {
TranslateModule.forRoot({ TranslateModule.forRoot({
loader: {provide: TranslateLoader, useClass: TranslateFakeLoader}, loader: {provide: TranslateLoader, useClass: TranslateFakeLoader},
}), }),
LoggerModule,
], ],
providers: [ providers: [
{ {
@@ -89,6 +91,8 @@ describe('DaiaAvailabilityComponent', () => {
provide: StorageProvider, provide: StorageProvider,
useValue: storageProviderSpy, useValue: storageProviderSpy,
}, },
NGXLogger,
LoggerConfig,
], ],
schemas: [CUSTOM_ELEMENTS_SCHEMA], schemas: [CUSTOM_ELEMENTS_SCHEMA],
}).compileComponents(); }).compileComponents();
@@ -97,6 +101,8 @@ describe('DaiaAvailabilityComponent', () => {
beforeEach(async () => { beforeEach(async () => {
dataProvider = TestBed.get(DaiaDataProvider); dataProvider = TestBed.get(DaiaDataProvider);
const workingDAIAurl = 'https://daia.hebis.de/DAIA2/UB_Frankfurt';
dataProvider.backendUrl = workingDAIAurl;
translateService = TestBed.get(TranslateService); translateService = TestBed.get(TranslateService);
refresher = jasmine.createSpyObj('refresher', ['complete']); refresher = jasmine.createSpyObj('refresher', ['complete']);
spyOn(dataProvider, 'get' as any).and.returnValue( spyOn(dataProvider, 'get' as any).and.returnValue(

View File

@@ -19,7 +19,10 @@ import {StorageProvider} from '../storage/storage.provider';
import {StAppsWebHttpClient} from '../data/stapps-web-http-client.provider'; import {StAppsWebHttpClient} from '../data/stapps-web-http-client.provider';
import {HttpClient} from '@angular/common/http'; import {HttpClient} from '@angular/common/http';
import {DataProvider} from '../data/data.provider'; import {DataProvider} from '../data/data.provider';
import {ConfigProvider} from '../config/config.provider';
import {SCFeatureConfiguration} from '@openstapps/core';
import {environment} from '../../../environments/environment'; import {environment} from '../../../environments/environment';
import {NGXLogger} from 'ngx-logger';
/** /**
* Generated class for the DataProvider provider. * Generated class for the DataProvider provider.
@@ -38,7 +41,9 @@ export class DaiaDataProvider extends DataProvider {
httpClient: HttpClient; httpClient: HttpClient;
backendUrl = environment.daia_url; configProvider: ConfigProvider;
backendUrl = environment.backend_url;
/** /**
* TODO * TODO
@@ -46,15 +51,19 @@ export class DaiaDataProvider extends DataProvider {
* @param stAppsWebHttpClient TODO * @param stAppsWebHttpClient TODO
* @param storageProvider TODO * @param storageProvider TODO
* @param httpClient TODO * @param httpClient TODO
* @param ConfigProvider TODO
*/ */
constructor( constructor(
stAppsWebHttpClient: StAppsWebHttpClient, stAppsWebHttpClient: StAppsWebHttpClient,
storageProvider: StorageProvider, storageProvider: StorageProvider,
httpClient: HttpClient, httpClient: HttpClient,
configProvider: ConfigProvider,
private readonly logger: NGXLogger,
) { ) {
super(stAppsWebHttpClient, storageProvider); super(stAppsWebHttpClient, storageProvider);
this.storageProvider = storageProvider; this.storageProvider = storageProvider;
this.httpClient = httpClient; this.httpClient = httpClient;
this.configProvider = configProvider;
this.client = new Client( this.client = new Client(
stAppsWebHttpClient, stAppsWebHttpClient,
this.backendUrl, this.backendUrl,
@@ -63,59 +72,80 @@ export class DaiaDataProvider extends DataProvider {
} }
async getAvailability(id: string): Promise<SCDaiaHoldings[]> { async getAvailability(id: string): Promise<SCDaiaHoldings[]> {
if (this.backendUrl === environment.backend_url) {
try {
const features = (await this.configProvider.getValue(
'features',
)) as SCFeatureConfiguration;
if (features.extern?.daia?.url) {
this.backendUrl = features.extern?.daia?.url;
} else {
this.logger.error('Daia service url undefined');
return [];
}
} catch (error) {
this.logger.error(error);
return [];
}
}
return new Promise(resolve => return new Promise(resolve =>
this.httpClient this.httpClient
.get<SCDaiaAvailabilityResponse>(this.backendUrl, {params: {id}}) .get<SCDaiaAvailabilityResponse>(this.backendUrl, {params: {id}})
.subscribe((response: SCDaiaAvailabilityResponse) => { .subscribe(
console.error(response); (response: SCDaiaAvailabilityResponse) => {
const holdings: SCDaiaHoldings[] = []; const holdings: SCDaiaHoldings[] = [];
if (response && Array.isArray(response.document)) { if (response && Array.isArray(response.document)) {
response.document.map(document => { response.document.map(document => {
Array.isArray(document.item) && Array.isArray(document.item) &&
document.item.map(element => { document.item.map(element => {
try { try {
const { const {
department: { department: {
id: departmentId, id: departmentId,
content: departmentLabel, content: departmentLabel,
href: departmentLink, href: departmentLink,
} = {id: 'noDep', content: '', href: ''}, } = {id: 'noDep', content: '', href: ''},
label, label,
about,
available,
storage,
} = element;
const holdingIndex = holdings.findIndex(
holding => holding.id === departmentId,
);
if (holdingIndex === -1) {
holdings.push({
id: departmentId,
label: departmentLabel,
href: departmentLink,
signature: label,
available:
(Array.isArray(available) &&
(available.find(
item => item.service === 'openaccess',
) ||
available.find(
item => item.service === 'loan',
))) ||
undefined,
storage,
about, about,
}); available,
storage,
} = element;
const holdingIndex = holdings.findIndex(
holding => holding.id === departmentId,
);
if (holdingIndex === -1) {
holdings.push({
id: departmentId,
label: departmentLabel,
href: departmentLink,
signature: label,
available:
(Array.isArray(available) &&
(available.find(
item => item.service === 'openaccess',
) ||
available.find(
item => item.service === 'loan',
))) ||
undefined,
storage,
about,
});
}
} catch {
// No element available
} }
} catch { });
// No element available });
} }
}); resolve(holdings);
}); },
} error => {
resolve(holdings); this.logger.error(error);
}), resolve([]);
},
),
); );
} }
} }

View File

@@ -19,7 +19,6 @@
export const environment = { export const environment = {
backend_url: 'https://mobile.server.uni-frankfurt.de', backend_url: 'https://mobile.server.uni-frankfurt.de',
daia_url: 'https://daia.hebis.de/DAIA2/UB_Frankfurt',
app_host: 'mobile.app.uni-frankfurt.de', app_host: 'mobile.app.uni-frankfurt.de',
custom_url_scheme: 'de.anyschool.app', custom_url_scheme: 'de.anyschool.app',
backend_version: '2.0.0', backend_version: '2.0.0',

View File

@@ -19,7 +19,6 @@
export const environment = { export const environment = {
backend_url: 'https://mobile.server.uni-frankfurt.de', backend_url: 'https://mobile.server.uni-frankfurt.de',
daia_url: 'https://daia.hebis.de/DAIA2/UB_Frankfurt',
app_host: 'mobile.app.uni-frankfurt.de', app_host: 'mobile.app.uni-frankfurt.de',
custom_url_scheme: 'de.anyschool.app', custom_url_scheme: 'de.anyschool.app',
backend_version: '2.0.0', backend_version: '2.0.0',