Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adding Blank Lines throw StringIndexOutOfBoundsOfException #72

Open
frozenMC opened this issue Feb 28, 2024 · 0 comments
Open

Adding Blank Lines throw StringIndexOutOfBoundsOfException #72

frozenMC opened this issue Feb 28, 2024 · 0 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@frozenMC
Copy link

While editing an existing hologram, deleting a line moved the line down so I assumed that adding an blank line will push the position of line 1 back to where it was previously.

The command used was '/hologram edit The_White_Mane-WFP3A insertafter 1 ' and it throws an string index, "begin 0, end -1, length 0".
Expected Behavior: Adding Line 2 which is blank.

Build 51

@OliverSchlueter OliverSchlueter added bug Something isn't working enhancement New feature or request labels Feb 29, 2024
devpeaqe pushed a commit to devpeaqe/FancyHolograms that referenced this issue Nov 2, 2024
devpeaqe pushed a commit to devpeaqe/FancyHolograms that referenced this issue Nov 3, 2024
cancelt the running command if the provided text is empty
implemented changes to the InsertBeforeCMD.java
devpeaqe pushed a commit to devpeaqe/FancyHolograms that referenced this issue Nov 3, 2024
devpeaqe pushed a commit to devpeaqe/FancyHolograms that referenced this issue Nov 3, 2024
removed command lines
devpeaqe pushed a commit to devpeaqe/FancyHolograms that referenced this issue Nov 3, 2024
changed from "return true" to "return false"
OliverSchlueter pushed a commit that referenced this issue Nov 3, 2024
* fixed issue #72
#72

* Issue #72
cancelt the running command if the provided text is empty
implemented changes to the InsertBeforeCMD.java

* Issue #72
removed command lines

---------

Co-authored-by: peaqe <[email protected]>
OliverSchlueter pushed a commit that referenced this issue Nov 3, 2024
* fixed issue #72
#72

* fixed issue #15
#15

* Issue #15
changed the permissions that contain a "-" with an "_"

* Issue #15
Removed issue #72 from the #15

* Issue #72
cancelt the running command if the provided text is empty
implemented changes to the InsertBeforeCMD.java

* Issue #72
removed command lines

* Issue #72
changed from "return true" to "return false"

---------

Co-authored-by: peaqe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants