@@ -298,45 +298,45 @@ const menuMap = {
298
298
function checkPrivilege ( item : MenuFirstOption | MenuChildOption , params : InterceptorMenuParams ) {
299
299
const { $table, column } = params
300
300
const { code, params : chartParams = { } } = item
301
- if ( column ) {
302
- switch ( code ) {
303
- case 'CHART_BAR_X_AXIS' :
304
- case 'CHART_BAR_Y_AXIS' :
305
- case 'CHART_LINE' : {
301
+ switch ( code ) {
302
+ case 'CHART_BAR_X_AXIS' :
303
+ case 'CHART_BAR_Y_AXIS' :
304
+ case 'CHART_LINE' :
305
+ case 'CHART_PIE' : {
306
+ item . disabled = ! column
307
+ if ( column ) {
306
308
const cellAreas = $table . getCellAreas ( )
307
- if ( cellAreas . length === 1 ) {
309
+ const validArea = cellAreas . length === 1
310
+ item . disabled = ! validArea
311
+ if ( validArea ) {
308
312
const { rows, cols } = cellAreas [ 0 ]
309
313
const { category } = chartParams
310
- if ( category ) {
311
- const serieColumns = cols . filter ( ( column ) => column . property !== category )
312
- item . disabled = ! rows . length || serieColumns . length < 1
313
- } else {
314
- item . disabled = ! rows . length || cols . length < 2
314
+ switch ( code ) {
315
+ case 'CHART_BAR_X_AXIS' :
316
+ case 'CHART_BAR_Y_AXIS' :
317
+ case 'CHART_LINE' : {
318
+ if ( category ) {
319
+ const serieColumns = cols . filter ( ( column ) => column . property !== category )
320
+ item . disabled = ! rows . length || serieColumns . length < 1
321
+ } else {
322
+ item . disabled = ! rows . length || cols . length < 2
323
+ }
324
+ break
325
+ }
326
+ case 'CHART_PIE' : {
327
+ if ( category ) {
328
+ const serieColumns = cols . filter ( ( column ) => column . property !== category )
329
+ item . disabled = ! rows . length || serieColumns . length !== 1
330
+ } else {
331
+ item . disabled = ! rows . length || cols . length !== 2
332
+ }
333
+ break
334
+ }
315
335
}
316
- } else {
317
- item . disabled = true
318
336
}
319
- break
320
- }
321
- case 'CHART_PIE' : {
322
- const cellAreas = $table . getCellAreas ( )
323
- if ( cellAreas . length === 1 ) {
324
- const { rows, cols } = cellAreas [ 0 ]
325
- const { category } = chartParams
326
- if ( category ) {
327
- const serieColumns = cols . filter ( ( column ) => column . property !== category )
328
- item . disabled = ! rows . length || serieColumns . length !== 1
329
- } else {
330
- item . disabled = ! rows . length || cols . length !== 2
331
- }
332
- } else {
333
- item . disabled = true
334
- }
335
- break
336
337
}
338
+ break
337
339
}
338
- } else {
339
- item . disabled = true
340
340
}
341
341
}
342
342
0 commit comments