Repository navigation
Commit cf5a453
authored
[ZEPPELIN-6593] Preserve false interpreter property values
Fixes [ZEPPELIN-6593](https://issues.apache.org/jira/browse/ZEPPELIN-6593).
The New UI's view-mode property cell used a truthy fallback, so a boolean `false` value rendered as blank. Route the cell through a nullish formatter instead, preserving `false` and `true` while keeping nullish values empty. Password masking and checkbox editing remain unchanged.
Verification:
- `npm run test:shell -- src/app/pages/workspace/interpreter/item/item.component.spec.ts` — 4 tests passed. Vite emitted a missing Monaco source-map warning.
- Angular workspace lint completed with 0 errors (existing warnings remain).
- The combined `npm run lint` did not complete: the separate React remote lint could not resolve its uninstalled `eslint-plugin-react` dependency. No React files are changed.
Closes #5592 from dajiaohuang/ZEPPELIN-6593-preserve-false-interpreter-values.
Signed-off-by: YONGJAE LEE <dev.yongjaelee@gmail.com>1 parent 5fdc82c commit cf5a453
3 files changed
Lines changed: 47 additions & 1 deletion
File tree
- zeppelin-web-angular/src/app/pages/workspace/interpreter/item
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
361 | 361 | | |
362 | 362 | | |
363 | 363 | | |
364 | | - | |
| 364 | + | |
365 | 365 | | |
366 | 366 | | |
367 | 367 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
100 | 104 | | |
101 | 105 | | |
102 | 106 | | |
| |||
0 commit comments