NIFI-4436: - Minor UX tweaks. - Updating imports due to refactoring of Registry models. - Fixing checkstyle issues.
Project: http://git-wip-us.apache.org/repos/asf/nifi/repo Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/c5b0931e Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/c5b0931e Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/c5b0931e Branch: refs/heads/master Commit: c5b0931e558f7043b29ae1d2edc38dbd5607d289 Parents: 416b861 Author: Matt Gilman <[email protected]> Authored: Fri Dec 15 09:49:13 2017 -0500 Committer: Bryan Bende <[email protected]> Committed: Mon Jan 8 12:44:56 2018 -0500 ---------------------------------------------------------------------- .../main/java/org/apache/nifi/web/NiFiServiceFacade.java | 6 +++--- .../java/org/apache/nifi/web/StandardNiFiServiceFacade.java | 2 +- .../WEB-INF/partials/canvas/import-flow-version-dialog.jsp | 2 +- .../WEB-INF/partials/canvas/save-flow-version-dialog.jsp | 8 ++++---- .../nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css | 2 +- .../nifi-web-ui/src/main/webapp/css/flow-status.css | 4 ++++ .../nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js | 9 --------- .../src/main/webapp/js/nf/canvas/nf-context-menu.js | 4 +++- .../main/webapp/views/nf-ng-breadcrumbs-directive-view.html | 2 +- 9 files changed, 18 insertions(+), 21 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java index 6c20eac..ab96747 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/NiFiServiceFacade.java @@ -1289,7 +1289,7 @@ public interface NiFiServiceFacade { * Returns a FlowComparisonEntity that contains all of the local modifications since the Process Group * was last synchronized with the Flow Registry * - * @param processGroupId + * @param processGroupId the ID of the Process Group * @return a FlowComparisonEntity that contains all of the local modifications since the Process Group * was last synchronized with the Flow Registry * @throws IllegalStateException if the Process Group with the given ID is not under version control @@ -1455,8 +1455,8 @@ public interface NiFiServiceFacade { * update the contents of that Process Group * @return the Process Group */ - ProcessGroupEntity updateProcessGroupContents(NiFiUser user, Revision revision, String groupId, VersionControlInformationDTO versionControlInfo, VersionedFlowSnapshot snapshot, String componentIdSeed, - boolean verifyNotModified, boolean updateSettings, boolean updateDescendantVersionedFlows); + ProcessGroupEntity updateProcessGroupContents(NiFiUser user, Revision revision, String groupId, VersionControlInformationDTO versionControlInfo, VersionedFlowSnapshot snapshot, + String componentIdSeed, boolean verifyNotModified, boolean updateSettings, boolean updateDescendantVersionedFlows); // ---------------------------------------- // Component state methods http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java index 1da63c2..1865bff 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java @@ -87,6 +87,7 @@ import org.apache.nifi.history.History; import org.apache.nifi.history.HistoryQuery; import org.apache.nifi.history.PreviousValue; import org.apache.nifi.registry.ComponentVariableRegistry; +import org.apache.nifi.registry.authorization.Permissions; import org.apache.nifi.registry.bucket.Bucket; import org.apache.nifi.registry.client.NiFiRegistryException; import org.apache.nifi.registry.flow.FlowRegistry; @@ -115,7 +116,6 @@ import org.apache.nifi.registry.flow.mapping.InstantiatedVersionedProcessGroup; import org.apache.nifi.registry.flow.mapping.InstantiatedVersionedProcessor; import org.apache.nifi.registry.flow.mapping.InstantiatedVersionedRemoteGroupPort; import org.apache.nifi.registry.flow.mapping.NiFiRegistryFlowMapper; -import org.apache.nifi.registry.model.authorization.Permissions; import org.apache.nifi.remote.RemoteGroupPort; import org.apache.nifi.remote.RootGroupPort; import org.apache.nifi.reporting.Bulletin; http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/import-flow-version-dialog.jsp ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/import-flow-version-dialog.jsp b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/import-flow-version-dialog.jsp index 5ea622a..01e03d5 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/import-flow-version-dialog.jsp +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/import-flow-version-dialog.jsp @@ -25,7 +25,7 @@ </div> </div> <div class="setting"> - <div class="setting-name">Location</div> + <div class="setting-name">Bucket</div> <div class="setting-field"> <div id="import-flow-version-bucket-combo"></div> <div id="import-flow-version-bucket" class="hidden"></div> http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/save-flow-version-dialog.jsp ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/save-flow-version-dialog.jsp b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/save-flow-version-dialog.jsp index a7f54b6..6017f37 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/save-flow-version-dialog.jsp +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/WEB-INF/partials/canvas/save-flow-version-dialog.jsp @@ -19,14 +19,13 @@ <div class="dialog-content"> <div class="setting"> <div class="setting-name">Registry</div> - <div id="save-flow-version-registry-container" class="setting-field"> + <div class="setting-field"> <div id="save-flow-version-registry-combo" class="hidden"></div> <div id="save-flow-version-registry" class="hidden"></div> - <div id="save-flow-version-label"></div> </div> </div> <div class="setting"> - <div class="setting-name">Location</div> + <div class="setting-name">Bucket</div> <div class="setting-field"> <div id="save-flow-version-bucket-combo" class="hidden"></div> <div id="save-flow-version-bucket" class="hidden"></div> @@ -34,10 +33,11 @@ </div> <div class="setting"> <div class="setting-name">Name</div> - <div class="setting-field"> + <div id="save-flow-version-registry-container" class="setting-field"> <span id="save-flow-version-process-group-id" class="hidden"></span> <input type="text" id="save-flow-version-name-field" class="setting-input hidden"/> <div id="save-flow-version-name" class="hidden"></div> + <div id="save-flow-version-label"></div> </div> </div> <div class="setting"> http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css index 710c0fb..82ffc38 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/dialog.css @@ -224,7 +224,7 @@ div.progress-label { height: 85px; } -#save-flow-version-registry-combo, #save-flow-version-registry { +#save-flow-version-name-field, #save-flow-version-name { width: 100%; margin-right: 10px; } http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/flow-status.css ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/flow-status.css b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/flow-status.css index 42f9afd..35ef016 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/flow-status.css +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/css/flow-status.css @@ -67,6 +67,10 @@ padding-left: 5px; } +#flow-status-container { + width: 90%; +} + #search-container { overflow: hidden; height: 32px; http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js index 692dd58..00dd4a8 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js @@ -609,14 +609,6 @@ }); }; - // define a function for update the flow status dimensions - var updateFlowStatusContainerSize = function () { - $('#flow-status-container').css({ - 'width': ((($('#nifi-logo').width() + $('#component-container').width())/$(window).width())*100)*2 + '%' - }); - }; - updateFlowStatusContainerSize(); - // listen for events to go to components $('body').on('GoTo:Component', function (e, item) { nfCanvasUtils.showComponent(item.parentGroupId, item.id); @@ -641,7 +633,6 @@ } updateGraphSize(); - updateFlowStatusContainerSize(); // resize shell when appropriate var shell = $('#shell-dialog'); http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-context-menu.js ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-context-menu.js b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-context-menu.js index 0c21838..5571e08 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-context-menu.js +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-context-menu.js @@ -541,7 +541,9 @@ } // check the selection for version control information - return versionControlInformation.state !== 'LOCALLY_MODIFIED' && versionControlInformation.state !== 'LOCALLY_MODIFIED_AND_STALE'; + return versionControlInformation.state !== 'LOCALLY_MODIFIED' && + versionControlInformation.state !== 'LOCALLY_MODIFIED_AND_STALE' && + versionControlInformation.state !== 'SYNC_FAILURE'; }; /** http://git-wip-us.apache.org/repos/asf/nifi/blob/c5b0931e/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/views/nf-ng-breadcrumbs-directive-view.html ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/views/nf-ng-breadcrumbs-directive-view.html b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/views/nf-ng-breadcrumbs-directive-view.html index a69de46..19018d3 100644 --- a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/views/nf-ng-breadcrumbs-directive-view.html +++ b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/views/nf-ng-breadcrumbs-directive-view.html @@ -22,7 +22,7 @@ limitations under the License. <span ng-if="separatorFunc(crumb.parentBreadcrumb)" style="margin: 0 12px;"> » </span> - <span ng-if="isTracking(crumb)" title="{{getVersionControlTooltip(crumb)}}" class="{{getVersionControlClass(crumb)}}" style="margin: 0 6px;"></span> + <span ng-if="isTracking(crumb)" title="{{getVersionControlTooltip(crumb)}}" class="{{getVersionControlClass(crumb)}}" style="margin-right: 6px;"></span> <span class="link" ng-class="(highlightCrumbId === crumb.id) ? 'link-bold' : ''" ng-click="clickFunc(crumb.id)">
