Github user scottyaslan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2722#discussion_r189689454
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-settings.js
---
@@ -264,12 +264,20 @@
var bType =
nfCommon.isDefinedAndNotNull(b.component[sortDetails.columnId]) ?
nfCommon.substringAfterLast(b.component[sortDetails.columnId], '.') : '';
return aType === bType ? 0 : aType > bType ? 1 : -1;
} else if (sortDetails.columnId === 'state') {
- var aState = 'Invalid';
- if (nfCommon.isEmpty(a.component.validationErrors)) {
+ var aState;
+ if (a.component.validationStatus === 'VALIDATING') {
+ aState = 'Validating';
+ } else if (a.component.validationStatus === 'INVALID')
{
+ aState = 'Invalid';
+ } else {
aState =
nfCommon.isDefinedAndNotNull(a.component[sortDetails.columnId]) ?
a.component[sortDetails.columnId] : '';
}
- var bState = 'Invalid';
- if (nfCommon.isEmpty(b.component.validationErrors)) {
+ var bState;
+ if (a.component.validationStatus === 'VALIDATING') {
+ bState = 'Validating';
--- End diff --
should this be checking b.component.validationStatus
---
