fix: set sourceHeader=false not work, will auto find sourceHeader(#16…

[lefex]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

if sourceHeader=false

const sourceHeader = retrieve2(newMetaRawOption.sourceHeader, upMetaRawOption.sourceHeader) || null;

the sourceHeader will set null. If sourceHeader is null, echarts will find sourceHeader auto.

if (sourceHeader === 'auto' || sourceHeader == null) {
    arrayRowsTravelFirst(function (val) {
        // '-' is regarded as null/undefined.
        if (val != null && val !== '-') {
            if (isString(val)) {
                startIndex == null && (startIndex = 1);
            }
            else {
                startIndex = 0;
            }
        }
    // 10 is an experience number, avoid long loop.
    }, seriesLayoutBy, dataArrayRows, 10);
}
else {
    startIndex = isNumber(sourceHeader) ? sourceHeader : sourceHeader ? 1 : 0;
}

Fixed issues

版本从5.1.2升到5.2.2之后柱状图第一项数据被吞掉 Close #16343
Close #16084

Details

Before: What was the problem?

After: How is it fixed in this PR?

Misc

• The API has been changed (apache/echarts-doc#xxx).
• This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

• Please squash the commits into a single one when merging.

Other information

[echarts-bot]

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

[pissang]

Will it change the behavior when sourceHeader is set to false because it will be changed to 'auto'

[echarts-bot]

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

[susiwen8]

Does this PR fix #16084 ?