Slow discovery selection

src/src/app/app.component.ts

@@ -1,6 +1,5 @@
 import {Config, Events, Nav, Platform} from 'ionic-angular';
 import {Component, ViewChild} from '@angular/core';
-import { ReconfigurePage } from '../pages/welcome/reconfigure';
 import { ProfilePage } from '../pages/profile/profile';
 import { ConfigurationScreen } from '../pages/configScreen/configScreen';
 import { ScreenOrientation } from '@ionic-native/screen-orientation/ngx';
@@ -43,6 +42,7 @@ export class GeteduroamApp {
   lastTimeBackPress = 0;
 
   timePeriodToExit = 2000;
+
   /**
    * @constructor
    *
@@ -79,22 +79,7 @@ export class GeteduroamApp {
       this.enableWifi();
     }
     if (!this.checkExtFile) {
-      const isAssociated = await this.isAssociatedNetwork();
-
-      if (!!isAssociated.success) {
-        // this.rootPage = !!isAssociated.success ? ConfigurationScreen : ReconfigurePage;
-        this.rootPage = ConfigurationScreen;
-      } else{
-        if (!isAssociated.message.includes('alreadyAssociated')) {
-          this.rootPage = ConfigurationScreen;
-        } else {
-          this.rootPage = ReconfigurePage;
-          this.getAssociation(isAssociated);
-          this.global.setOverrideProfile(true);
-
-          !isAssociated.success && !isAssociated.overridable ? this.removeAssociatedManually() : '';
-        }
-      }
+      this.rootPage = ConfigurationScreen;
     }
   }
   async checkExternalOpen() {
@@ -149,7 +134,7 @@ export class GeteduroamApp {
       this.platform.registerBackButtonAction(() => {
         // get current active page
         let view = this.navCtrl.getActive();
-        if (view.component.name == "ConfigurationScreen" || view.component.name == "ReconfigurePage") {
+        if (view.component.name == "ConfigurationScreen") {
           //Double check to exit app
           if (new Date().getTime() - this.lastTimeBackPress < this.timePeriodToExit) {
             this.platform.exitApp(); //Exit from app
@@ -199,12 +184,11 @@ export class GeteduroamApp {
    */
   async notConnectionNetwork() {
     if (!this.checkExtFile) {
-      this.rootPage = ReconfigurePage;
+      this.rootPage = ConfigurationScreen;
 
       const isAssociated = await this.isAssociatedNetwork();
       this.getAssociation(isAssociated);
 
-
       if (!isAssociated.success && !isAssociated.overridable) {
         this.removeAssociatedManually();
 
@@ -232,8 +216,10 @@ export class GeteduroamApp {
     if (!this.checkExtFile) {
       this.connectionEvent(connectionStatus);
 
-      if (!connectionStatus.connected){
+      if (!connectionStatus.connected) {
         this.notConnectionNetwork();
+      } else {
+        this.global.setDiscovery(await this.getEduroamServices.discovery());
       }
     }
   }

src/src/app/app.module.ts

@@ -3,7 +3,6 @@ import { BrowserModule } from '@angular/platform-browser';
 import { IonicApp, IonicModule } from 'ionic-angular';
 import { GeteduroamApp } from './app.component';
 import { ScreenOrientation } from '@ionic-native/screen-orientation/ngx';
-
 import { StatusBar } from '@ionic-native/status-bar/ngx';
 import { SplashScreen } from '@ionic-native/splash-screen/ngx';
 import { AndroidPermissions } from '@ionic-native/android-permissions/ngx';
@@ -19,15 +18,18 @@ import { GlobalProvider } from '../providers/global/global';
 import { CryptoUtil } from '../providers/util/crypto-util';
 import { ErrorServiceProvider } from "../providers/error-service/error-service";
 import { OauthConfProvider } from '../providers/oauth-conf/oauth-conf';
+import { InstitutionSearch } from '../pages/institutionSearch/institutionSearch';
+import { ReactiveFormsModule } from '@angular/forms';
 
 @NgModule({
   declarations: [
     GeteduroamApp
   ],
   imports: [
     BrowserModule,
+    ReactiveFormsModule,
     PagesModule,
-    IonicModule.forRoot(GeteduroamApp)
+    IonicModule.forRoot(GeteduroamApp, { preload: InstitutionSearch }),
   ],
   bootstrap: [IonicApp],
   entryComponents: [
@@ -50,7 +52,7 @@ import { OauthConfProvider } from '../providers/oauth-conf/oauth-conf';
     OauthConfProvider,
     ErrorServiceProvider
   ],
-  exports:[]
+  exports:[],
 })
 export class AppModule {
 

src/src/pages/configScreen/configScreen.html

@@ -14,9 +14,9 @@ <h1>{{getString('header', 'config')}}</h1>
     <div class="content-form" margin-top padding-top text-center>
         <ion-label class="label-conf" no-margin padding-top text-left>{{getString('label', 'institution')}}</ion-label>
 
-        <div class="row" padding-top>
+        <div *ngIf="this.instances" class="row" padding-top>
 
-            <ion-searchbar (click)="showModal($event)" [(ngModel)]="instanceName" cancelButtonText
+            <ion-searchbar (click)="showModal($event)" [(ngModel)]="instanceName" cancelButtonText debounce="500"
                            class="searchbar-input" no-padding placeholder="{{getString('placeholder', 'institution')}}">
 
             </ion-searchbar>
@@ -44,4 +44,3 @@ <h1>{{getString('header', 'config')}}</h1>
         </div>
     </div>
 </div>
-

src/src/pages/configScreen/configScreen.ts

@@ -1,5 +1,5 @@
-import { Component, NgZone } from '@angular/core';
-import {Events, ModalController, NavController} from 'ionic-angular';
+import { Component, NgModule, NgZone } from '@angular/core';
+import { Events, Modal, ModalController, NavController, NavParams, ViewController } from 'ionic-angular';
 import {GeteduroamServices} from "../../providers/geteduroam-services/geteduroam-services";
 import { ProfilePage } from '../profile/profile';
 import { OauthFlow } from '../oauthFlow/oauthFlow';
@@ -20,7 +20,6 @@ declare var window;
   selector: 'page-config-screen',
   templateUrl: 'configScreen.html',
 })
-
 export class ConfigurationScreen extends BasePage{
 
   showAll: boolean = false;
@@ -80,7 +79,6 @@ export class ConfigurationScreen extends BasePage{
    */
   showButton: boolean = true;
 
-
   /**
    * Constructor
    * */
@@ -89,34 +87,45 @@ export class ConfigurationScreen extends BasePage{
               protected dictionary: DictionaryServiceProvider, protected global: GlobalProvider,
               private errorHandler: ErrorHandlerProvider) {
     super(loading, dictionary, event, global);
+    Keyboard.addListener('keyboardWillShow', async () => {
+     await Keyboard.hide();
+    });
+
+    this.event.subscribe('connection', async (res: any) => {
+      if (!!res.connected && !this.global.discovery) {
+        this.chargeDiscovery()
+      }
+    });
   }
 
+
   /**
    * Method executes when the search bar is tapped.
    * */
   async showModal(e: any) {
     e.preventDefault();
-    await Keyboard.hide();
     if (!!this.instances) {
       let searchModal = this.modalCtrl.create(InstitutionSearch, {
-        instances: this.instances,
-        instanceName: this.instanceName}
-      );
+        instanceName: this.instanceName
+      });
 
       searchModal.onDidDismiss((data) => {
 
+        data = this.instances.filter((res: any) => {
+           if (res.name === data) return res
+        });
+
         if (data !== undefined) {
-          this.instance = data;
-          this.instanceName = data.name;
+          this.instance = data[0];
+          this.instanceName = data[0].name;
 
           this.initializeProfiles(this.instance);
-
         }
       });
 
       return await searchModal.present();
     } else {
-      await this.getDiscovery();
+      await this.chargeDiscovery();
       this.showModal(e);
     }
   }
@@ -226,12 +235,18 @@ export class ConfigurationScreen extends BasePage{
     this.resetValues();
   }
 
+  async ngOnInit() {
+    this.instances = this.global.discovery;
+  }
+
+  ionViewWillEnter() {
+    this.loading.create();
+  }
   /**
    *  Lifecycle when entering a page, before it becomes the active one
    *  Load the discovery data and show the spinner
    */
   async ionViewDidEnter() {
-    await this.getDiscovery();
     this.removeSpinner();
     this.showAll = true;
   }
@@ -251,7 +266,6 @@ export class ConfigurationScreen extends BasePage{
       this.instanceName = '';
 
     });
-
   }
 
   async checkEap(profile: ProfileModel) {
@@ -276,6 +290,11 @@ export class ConfigurationScreen extends BasePage{
     }
   }
 
+  async chargeDiscovery() {
+    await this.getDiscovery();
+    this.global.setDiscovery(this.instances)
+  }
+
   /**
    *
    */

src/src/pages/institutionSearch/institutionSearch.html

@@ -11,7 +11,14 @@
     </ion-toolbar>
 </ion-header>
 <ion-content fullscreen>
-    <ion-list *ngFor="let institution of filteredInstances" no-margin>
-        <ion-item (click)="selectInstitution(institution)">{{institution.name}}</ion-item>
+    <ion-list no-margin class="list list-ios list-institution">
+        <div *ngFor="let institution of filteredInstances;" class="item item-block">
+            <div class="item-inner" (click)="selectInstitution(institution)">
+                <div class="input-wrapper">
+                    <div class="label label-ios label-institution">{{ institution }} </div>
+                </div>
+            </div>
+            <div class="button-effect"></div>
+        </div>
     </ion-list>
 </ion-content>

src/src/pages/institutionSearch/institutionSearch.scss

@@ -2,11 +2,11 @@ page-institution-search {
   //---------------------------
   // ion-item custom style
   //---------------------------
-  ion-item {
-    background-color: unset;
-    div.item-inner {
-      padding: 0 !important;
-      border-bottom: 0 !important;
-    }
+  .list-institution {
+    background-color: white;
+  }
+  .label-institution {
+    font-size: 20px;
+    padding-left: 16px
   }
 }

src/src/pages/institutionSearch/institutionSearch.ts

@@ -58,6 +58,7 @@ export class InstitutionSearch extends BasePage{
               protected dictionary: DictionaryServiceProvider,
               protected event: Events, protected global: GlobalProvider) {
     super(loading, dictionary, event, global);
+    Keyboard.removeAllListeners();
   }
 
   /**
@@ -91,9 +92,8 @@ export class InstitutionSearch extends BasePage{
    */
   filterInstances(stringAux: string){
     if (stringAux && stringAux.trim() != '') {
-
       this.filteredInstances = this.instances.filter((item:any) => {
-        return (item.name.toLowerCase().indexOf(stringAux.toLowerCase()) > -1);
+        return (item.toLowerCase().indexOf(stringAux.toLowerCase()) > -1);
       })
     } else {
       this.clearInstance();
@@ -131,16 +131,20 @@ export class InstitutionSearch extends BasePage{
     this.selectedProfileId = '';
   }
 
+  ngOnInit() {
+    this.instances = Object.values(this.global.getInstitutionNames());
+    this.filteredInstances = Object.values(this.instances);
+  }
   /**
    * Lifecycle when entering a page, after it becomes the active page.
    *  this sets focus on search bar
    */
-  ionViewDidEnter() {
+  ionViewWillEnter() {
     this.ios = !!this.platform.is('ios');
-    this.instances = this.navParams.get('instances');
     this.instanceName = this.navParams.get('instanceName');
-    this.filterInstances(this.instanceName);
 
+  }
+  ionViewDidEnter() {
     setTimeout(() => {
       this.searchBar.setFocus()
     }, 10);

src/src/pages/wifiConfirmation/wifiConfirmation.html

@@ -1,11 +1,4 @@
-<div *ngIf="!global.isAndroid() && viewCtrl.enableBack() && showAll" class="arrow-back">
-    <button ion-button class="arrow-btn" (click)="backToConfig()" id="btn-back">
-        <span class="button-inner"><span class="caret">⌃</span>
-            Back
-        </span>
-    </button>
-</div>
-<div *ngIf="showAll" class="container-content" id="dismissable-back">
+<div *ngIf="showAll" class="container-content">
     <div class="header-content" margin padding-top>
         <img #imgLogo [ngClass]="!!logo ? 'imgLogo' : ''" [src]="converted_image" alt="setting" margin />
         <h1>{{getString('header', 'confirmation')}}</h1>
@@ -27,3 +20,4 @@ <h1>{{getString('header', 'confirmation')}}</h1>
         </div>
     </div>
 </div>
+