Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(label): fix endLabel fails with null data #18841

Merged
merged 3 commits into from
Jul 5, 2023
Merged

fix(label): fix endLabel fails with null data #18841

merged 3 commits into from
Jul 5, 2023

Conversation

Ovilia
Copy link
Contributor

@Ovilia Ovilia commented Jul 4, 2023

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Check setLabelText exists before calling.

Fixed issues

#18839

Details

Before: What was the problem?

endLabel fail when data contain null or '-'.

After: How does it behave after the fixing?

endLabel not fail when data contain null or '-'.

Document Info

One of the following should be checked.

  • This PR doesn't relate to document changes
  • The document should be updated later
  • The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

  • 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
Copy link

echarts-bot bot commented Jul 4, 2023

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.

The pull request is marked to be PR: author is committer because you are a committer of this project.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 4, 2023

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-18841@1d10ba8

@Ovilia
Copy link
Contributor Author

Ovilia commented Jul 4, 2023

View fixed case

@Ovilia Ovilia merged commit a92b0df into master Jul 5, 2023
@echarts-bot
Copy link

echarts-bot bot commented Jul 5, 2023

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

@Ovilia Ovilia deleted the fix-endLabel branch July 5, 2023 02:47
plainheart added a commit that referenced this pull request Jul 5, 2023
@plainheart plainheart added this to the 5.4.3 milestone Jul 16, 2023
@plainheart plainheart changed the title fix(endLabel): endLabel fail with null data #18839 fix(label): fix endLabel fails with null data Jul 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Script error in line charts when endLabels are shown for a data series containing nulls
2 participants