Skip to content

Return chat component with empty text instead of throwing exception.#3843

Closed
CDFN wants to merge 3 commits into
PaperMC:masterfrom
CDFN:fix/inventory-title
Closed

Return chat component with empty text instead of throwing exception.#3843
CDFN wants to merge 3 commits into
PaperMC:masterfrom
CDFN:fix/inventory-title

Conversation

@CDFN
Copy link
Copy Markdown

@CDFN CDFN commented Jul 7, 2020

Fixes #3328

@Proximyst Proximyst added status: rebase required type: bug Something doesn't work as it was intended to. labels Aug 24, 2020
@CDFN CDFN requested a review from a team as a code owner August 25, 2020 02:40
* Get the title of this inventory window.
*
- * @return The title.
+ * @return The title or empty string when title is null.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should get a // Paper tag on it. Just at the end of the line.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The javadoc is HTML, so:

Suggested change
+ * @return The title or empty string when title is null.
+ * @return The title or empty string when title is {@code null}. <!-- Paper -->

@MiniDigger
Copy link
Copy Markdown
Member

#5070

thanks

@MiniDigger MiniDigger closed this Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: rebase required type: bug Something doesn't work as it was intended to.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Console Error - InventoryClickEvent

6 participants