The page stucked after clicking on the "Go" Button in the Actions sec…

…tion. Fixes #1660
Azure · Aug 8, 2017 · d9a2a4b · d9a2a4b
1 parent ecbb7b5
commit d9a2a4b
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 11 deletions.
diff --git a/AzureFunctions.AngularClient/src/app/function-new/function-new.component.html b/AzureFunctions.AngularClient/src/app/function-new/function-new.component.html
@@ -68,7 +68,7 @@ <h3>{{ 'functionNew_nameYourFunction' | translate }}</h3>
                  (remove)="onRemoveBinding($event)"
                  (update)="onUpdateBinding($event)"></binding>
 
-        <div *ngIf="model.trigger.type === 'eventGridTrigger'" class="alert alert-warning alert-dismissible" role="alert">
+        <div *ngIf="model.trigger && model.trigger.type === 'eventGridTrigger'" class="alert alert-warning alert-dismissible" role="alert">
             {{ 'eventGrid_createMessage' | translate }}
         </div>
 

diff --git a/AzureFunctions.AngularClient/src/app/template-picker/template-picker.component.ts b/AzureFunctions.AngularClient/src/app/template-picker/template-picker.component.ts
@@ -261,17 +261,19 @@ export class TemplatePickerComponent {
             this.selectedTemplate = template;
 
             // Some bindings (and templates that use them) require an AAD app; if so, show express button
-            let binding = this.bindings.find((b) => {
-                return b.type.toString() === this.selectedTemplate;
-            });
-            if (binding) {
-                this.showAADExpressRegistration = !!binding.AADPermissions;
-            } else {
-                // Could be improved by determining which bindings a template uses automatically
-                let templateObject = this.templates.find((t) => {
-                    return t.value === template;
+            if (this.bindings) {
+                let binding = this.bindings.find((b) => {
+                    return b.type.toString() === this.selectedTemplate;
                 });
-                this.showAADExpressRegistration = templateObject && !!templateObject.AADPermissions;
+                if (binding) {
+                    this.showAADExpressRegistration = !!binding.AADPermissions;
+                } else {
+                    // Could be improved by determining which bindings a template uses automatically
+                    let templateObject = this.templates.find((t) => {
+                        return t.value === template;
+                    });
+                    this.showAADExpressRegistration = templateObject && !!templateObject.AADPermissions;
+                }
             }
 
             if (!this.showFooter) {