Repository: nifi
Updated Branches:
  refs/heads/master af2717932 -> a25b651da


NIFI-2837: - Immediately hiding the template dialog after clicking the Upload 
button to prevent an accidental re-submission.

This closes #1086.

Signed-off-by: Bryan Bende <bbe...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/nifi/repo
Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/a25b651d
Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/a25b651d
Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/a25b651d

Branch: refs/heads/master
Commit: a25b651da9da57a20431b9eedcd8a8860aaa4242
Parents: af27179
Author: Matt Gilman <matt.c.gil...@gmail.com>
Authored: Thu Sep 29 14:57:07 2016 -0400
Committer: Bryan Bende <bbe...@apache.org>
Committed: Wed Oct 12 09:18:58 2016 -0400

----------------------------------------------------------------------
 .../nf-ng-canvas-operate-controller.js          | 28 +++++++++++++-------
 1 file changed, 18 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/nifi/blob/a25b651d/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/controllers/nf-ng-canvas-operate-controller.js
----------------------------------------------------------------------
diff --git 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/controllers/nf-ng-canvas-operate-controller.js
 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/controllers/nf-ng-canvas-operate-controller.js
index 5832332..c74ca48 100644
--- 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/controllers/nf-ng-canvas-operate-controller.js
+++ 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/controllers/nf-ng-canvas-operate-controller.js
@@ -121,34 +121,39 @@ nf.ng.Canvas.OperateCtrl = function () {
                             options.url += 
(encodeURIComponent(nf.Canvas.getGroupId()) + '/templates/upload');
                         },
                         success: function (response, statusText, xhr, form) {
-                            // see if the import was successful
+                            // see if the import was successful and inform the 
user
                             if (response.documentElement.tagName === 
'templateEntity') {
-                                // close the dialog
-                                $('#upload-template-dialog').modal('hide');
-
-                                // close the settings dialog
                                 nf.Dialog.showOkDialog({
                                     headerText: 'Success',
                                     dialogContent: 'Template successfully 
imported.'
                                 });
                             } else {
                                 // import failed
-                                var status = 'Unable to import template. 
Please check the log for errors.';
+                                var statusText = 'Unable to import template. 
Please check the log for errors.';
                                 if (response.documentElement.tagName === 
'errorResponse') {
                                     // if a more specific error was given, use 
it
                                     var errorMessage = 
response.documentElement.getAttribute('statusText');
                                     if (!nf.Common.isBlank(errorMessage)) {
-                                        status = errorMessage;
+                                        statusText = errorMessage;
                                     }
                                 }
-                                $('#upload-template-status').text(status);
+
+                                // show reason
+                                nf.Dialog.showOkDialog({
+                                    headerText: 'Unable to Upload',
+                                    dialogContent: 
nf.Common.escapeHtml(statusText)
+                                });
                             }
                         },
                         error: function (xhr, statusText, error) {
-                            
$('#upload-template-status').text(xhr.responseText);
+                            // request failed
+                            nf.Dialog.showOkDialog({
+                                headerText: 'Unable to Upload',
+                                dialogContent: 
nf.Common.escapeHtml(xhr.responseText)
+                            });
                         }
                     });
-                    
+
                     // configure the upload template dialog
                     this.getElement().modal({
                         headerText: 'Upload Template',
@@ -168,6 +173,9 @@ nf.ng.Canvas.OperateCtrl = function () {
                                         $('#upload-template-status').text('No 
template selected. Please browse to select a template.');
                                     } else {
                                         templateForm.submit();
+
+                                        // hide the dialog
+                                        
$('#upload-template-dialog').modal('hide');
                                     }
                                 }
                             }

Reply via email to