fix #9182 Graph label.rotate is invalid

[cuijian-dexter]

when series[i].type is graph, series[i].label.rotate is invalid

[cuijian-dexter]

Duplicate of #9182

[Ovilia]

Please use == undefined to check undefined so that null will also match the comparing.
Here, you'd better use var labelRotate = seriesModel.get('label.rotate') || 0 for simplification.

[cuijian-dexter]

@Ovilia ok, the code has been optimized.

[Ovilia]

data.label should also be supported. Test example:

var option = {
    title: {
        text: 'Graph 简单示例'
    },
    tooltip: {},
    animationDurationUpdate: 1500,
    animationEasingUpdate: 'quinticInOut',
    series : [
        {
            type: 'graph',
            layout: 'none',
            symbolSize: 50,
            roam: true,
            label: {
                   show: true,
                    rotate: 30,
                    fontWeight:5,
                    fontSize: 26,
                    color: "#000",
                    distance: 15,
                    position: 'inside',
                    verticalAlign: 'middle'
            },
            edgeSymbol: ['circle', 'arrow'],
            edgeSymbolSize: [4, 10],
            edgeLabel: {
                normal: {
                    textStyle: {
                        fontSize: 20
                    }
                }
            },
            data: [{
                name: 'rotate 30',
                x: 300,
                y: 300
            }, {
                name: 'rotate -30',
                x: 500,
                y: 600,
                label: {
                    rotate: -30
                }
            }],
            emphasis: {
                label: {
                    rotate: 45
                }
            }
        }
    ]
}

[cuijian-dexter]

@Ovilia I have a question.If you exist at the same time, do you take data.label priority?

[cuijian-dexter]

@Ovilia ,It's already commit. You can try it.

[Ovilia]

Very close now!

[Ovilia]

If itemModel.get('label.rotate') is set to be 0, it will use labelRotate. But in fact, we wish it to be 0.
It should be something like

var itemRotate = itemModel.get('label.rotate');
if (itemRotate != undefined) {
    labelRotate = itemRotate;
}

[cuijian-dexter]

@Ovilia Already commit again.😄

[Ovilia]

@100pah please review this PR.

[100pah]

By convention, some != null is recommended. (not a big deal, just probably because that null can not be redefined)

[cuijian-dexter]

@100pah you can check it again.thanks

[100pah]

Checked, thanks @cuijian-dexter !

[cuijian-dexter]

@100pah Thanks