Skip to content

Commit 8d44c71

Browse files
Minor change
1 parent df5f482 commit 8d44c71

File tree

1 file changed

+27
-25
lines changed

1 file changed

+27
-25
lines changed

src/component/json-diff.tsx

+27-25
Original file line numberDiff line numberDiff line change
@@ -28,35 +28,37 @@ export default function JsonDifference() {
2828
})
2929

3030
React.useEffect(() => {
31-
if (isComparing && first?.trim().length && second?.trim().length) {
32-
try {
33-
compare.compare(JSON.parse(first.trim()), JSON.parse(second.trim()))
34-
.then(data => setSummary(data.get() as IterableSummary))
35-
.catch((e) => {console.log(e)})
36-
.finally(() => setIsComparing(false))
37-
} catch(e: any) {
31+
if (isComparing) {
32+
if (first?.trim().length && second?.trim().length) {
33+
try {
34+
compare.compare(JSON.parse(first.trim()), JSON.parse(second.trim()))
35+
.then(data => setSummary(data.get() as IterableSummary))
36+
.catch((e) => {console.log(e)})
37+
.finally(() => setIsComparing(false))
38+
} catch(e: any) {
39+
showMessage({
40+
severity: AlertSeverity.Error,
41+
header: 'Error',
42+
message: e.message
43+
})
44+
} finally {
45+
setIsComparing(false)
46+
}
47+
} else if (!first || !first.trim().length) {
48+
showMessage({
49+
severity: AlertSeverity.Info,
50+
header: 'No JSON Input',
51+
message: 'No JSON header input was defined for first JSON input.'
52+
})
53+
setIsComparing(false)
54+
} else if (!second || !second.trim().length) {
3855
showMessage({
39-
severity: AlertSeverity.Error,
40-
header: 'Error',
41-
message: e.message
56+
severity: AlertSeverity.Info,
57+
header: 'No JSON Input',
58+
message: 'No JSON header input was defined for second JSON input.'
4259
})
43-
} finally {
4460
setIsComparing(false)
4561
}
46-
} else if (!first || !first.trim().length) {
47-
showMessage({
48-
severity: AlertSeverity.Info,
49-
header: 'No JSON Input',
50-
message: 'No JSON header input was defined for first JSON input.'
51-
})
52-
setIsComparing(false)
53-
} else if (!second || !second.trim().length) {
54-
showMessage({
55-
severity: AlertSeverity.Info,
56-
header: 'No JSON Input',
57-
message: 'No JSON header input was defined for second JSON input.'
58-
})
59-
setIsComparing(false)
6062
}
6163
}, [isComparing])
6264

0 commit comments

Comments
 (0)