From 6b8e8d50f0af724f1675d96904094208f6a4379a Mon Sep 17 00:00:00 2001
From: Will Abson <will.abson@gmail.com>
Date: Wed, 1 Jun 2016 13:26:45 +0100
Subject: [PATCH] Extract search box into its own component

- Upgraded to latest 0.1.5 version of search
- Search is now only shown if the user is logged in
- Heading now lives inside the demo app not inside search

Refs #69
---
 demo-shell-ng2/app/app.component.html         | 14 ++-----
 demo-shell-ng2/app/app.component.ts           | 32 ++++++---------
 .../components/search/search.component.html   |  4 ++
 .../app/components/search/search.component.ts | 41 +++++++++++++++++++
 demo-shell-ng2/package.json                   |  2 +-
 5 files changed, 62 insertions(+), 31 deletions(-)
 create mode 100644 demo-shell-ng2/app/components/search/search.component.html
 create mode 100644 demo-shell-ng2/app/components/search/search.component.ts

diff --git a/demo-shell-ng2/app/app.component.html b/demo-shell-ng2/app/app.component.html
index 29bc3d8b14..38a3a96c97 100644
--- a/demo-shell-ng2/app/app.component.html
+++ b/demo-shell-ng2/app/app.component.html
@@ -7,17 +7,9 @@
             <!-- Add spacer, to align navigation to the right -->
             <div class="mdl-layout-spacer"></div>
 
-            <form [ngFormModel]="searchForm" (submit)="onSearch(searchForm.value, $event)">
-                <div class="mdl-textfield mdl-js-textfield mdl-textfield--expandable">
-                    <label class="mdl-button mdl-js-button mdl-button--icon" for="searchTerm">
-                        <i class="material-icons">search</i>
-                    </label>
-                    <div class="mdl-textfield__expandable-holder">
-                        <input class="mdl-textfield__input" type="text" id="searchTerm" ngControl="searchTerm">
-                        <label class="mdl-textfield__label" for="searchTerm">Expandable Input</label>
-                    </div>
-                </div>
-            </form>
+            <!-- Search box -->
+            <alfresco-search-control *ngIf="isLoggedIn()" [searchTerm]="searchTerm"
+            (searchChange)="searchTermChange($event);"></alfresco-search-control>
 
             <!-- Navigation. We hide it in small screens. -->
             <nav class="mdl-navigation mdl-layout--large-screen-only">
diff --git a/demo-shell-ng2/app/app.component.ts b/demo-shell-ng2/app/app.component.ts
index 97790f2fe0..77c5b59760 100644
--- a/demo-shell-ng2/app/app.component.ts
+++ b/demo-shell-ng2/app/app.component.ts
@@ -16,7 +16,6 @@
  */
 
 import { Component } from 'angular2/core';
-import { ControlGroup, FormBuilder, Validators } from 'angular2/common';
 import { Router, RouteConfig, ROUTER_DIRECTIVES } from 'angular2/router';
 import { MDL } from 'ng2-alfresco-core/dist/ng2-alfresco-core';
 import { FilesComponent } from './components/files/files.component';
@@ -29,7 +28,8 @@ import {
 } from 'ng2-alfresco-core/dist/ng2-alfresco-core';
 import { UploadButtonComponent } from 'ng2-alfresco-upload/dist/ng2-alfresco-upload';
 import { DataTableDemoComponent } from './components/datatable/datatable-demo.component';
-import { AlfrescoSearchComponent } from 'ng2-alfresco-search/dist/ng2-alfresco-search';
+import { SearchComponent } from './components/search/search.component';
+import { ALFRESCO_SEARCH_DIRECTIVES } from 'ng2-alfresco-search/dist/ng2-alfresco-search';
 import { LoginDemoComponent } from './components/login/login-demo.component';
 import { ViewerFileComponent } from './components/viewer/viewer.component';
 
@@ -38,7 +38,7 @@ declare var document: any;
 @Component({
     selector: 'my-app',
     templateUrl: 'app/app.component.html',
-    directives: [ROUTER_DIRECTIVES, AuthRouterOutlet, MDL],
+    directives: [ALFRESCO_SEARCH_DIRECTIVES, ROUTER_DIRECTIVES, AuthRouterOutlet, MDL],
     pipes: [AlfrescoPipeTranslate]
 })
 @RouteConfig([
@@ -47,24 +47,19 @@ declare var document: any;
     {path: '/datatable', name: 'DataTable', component: DataTableDemoComponent},
     {path: '/uploader', name: 'Uploader', component: UploadButtonComponent},
     {path: '/login', name: 'Login', component: LoginDemoComponent},
-    {path: '/search', name: 'Search', component: AlfrescoSearchComponent},
+    {path: '/search', name: 'Search', component: SearchComponent},
     {path: '/viewer', name: 'Viewer', component: ViewerFileComponent}
 ])
 export class AppComponent {
     translate: AlfrescoTranslationService;
-    searchForm: ControlGroup;
+    searchTerm: string = '';
 
-    constructor(private _fb: FormBuilder,
-                public auth: AlfrescoAuthenticationService,
+    constructor(public auth: AlfrescoAuthenticationService,
                 public router: Router,
                 translate: AlfrescoTranslationService,
                 alfrescoSettingsService: AlfrescoSettingsService) {
         alfrescoSettingsService.host = 'http://192.168.99.100:8080';
 
-        this.searchForm = this._fb.group({
-            searchTerm: ['', Validators.compose([Validators.required, Validators.minLength(3)])]
-        });
-
         this.translate = translate;
         this.translate.translationInit();
     }
@@ -95,14 +90,13 @@ export class AppComponent {
     }
 
     /**
-     * Method called on submit form
-     * @param value
-     * @param event
+     * Called when a new search term is submitted
+     *
+     * @param params Parameters relating to the search
      */
-    onSearch(value: any, event) {
-        if (event) {
-            event.preventDefault();
-        }
-        this.router.navigate(['Search', value]);
+    searchTermChange(params) {
+        this.router.navigate(['Search', {
+            q: params.value
+        }]);
     }
 }
diff --git a/demo-shell-ng2/app/components/search/search.component.html b/demo-shell-ng2/app/components/search/search.component.html
new file mode 100644
index 0000000000..8adba65974
--- /dev/null
+++ b/demo-shell-ng2/app/components/search/search.component.html
@@ -0,0 +1,4 @@
+<div>
+    <h1>Search results</h1>
+    <alfresco-search></alfresco-search>
+</div>
\ No newline at end of file
diff --git a/demo-shell-ng2/app/components/search/search.component.ts b/demo-shell-ng2/app/components/search/search.component.ts
new file mode 100644
index 0000000000..f1a5913113
--- /dev/null
+++ b/demo-shell-ng2/app/components/search/search.component.ts
@@ -0,0 +1,41 @@
+/**
+ * @license
+ * Copyright 2016 Alfresco Software, Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { Component } from 'angular2/core';
+import { ALFRESCO_SEARCH_DIRECTIVES } from 'ng2-alfresco-search/dist/ng2-alfresco-search';
+
+declare let __moduleName: string;
+
+@Component({
+    moduleId: __moduleName,
+    selector: 'search-component',
+    templateUrl: './search.component.html',
+    styles: [`
+        :host div {
+            padding: 0 20px;
+        }
+        :host h1 {
+            font-size: 22px;
+        }
+    `],
+    directives: [ ALFRESCO_SEARCH_DIRECTIVES ]
+})
+export class SearchComponent {
+    constructor() {
+        console.log('SearchComponent constructor');
+    }
+}
diff --git a/demo-shell-ng2/package.json b/demo-shell-ng2/package.json
index ed16d8b342..15ddbe71fa 100644
--- a/demo-shell-ng2/package.json
+++ b/demo-shell-ng2/package.json
@@ -60,7 +60,7 @@
     "ng2-alfresco-datatable": "^0.1.0",
     "ng2-alfresco-documentlist": "^0.1.0",
     "ng2-alfresco-login": "^0.1.0",
-    "ng2-alfresco-search": "^0.1.2",
+    "ng2-alfresco-search": "^0.1.5",
     "ng2-alfresco-upload": "^0.1.0",
     "ng2-alfresco-viewer": "^0.1.6",
     "ng2-translate": "^1.11.3",