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.

Sign up for GitHub

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

Jump to bottom

A couple of placeholders at /plot info are not replaced on manual entry #3339

Closed
2 tasks done
DevJoey opened this issue Nov 18, 2021 · 2 comments
Closed
2 tasks done

A couple of placeholders at /plot info are not replaced on manual entry #3339

DevJoey opened this issue Nov 18, 2021 · 2 comments
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Good first issue Simple issue which would be a good first PR for newbies interested in contributing to the project

Comments

@DevJoey
Copy link

DevJoey commented Nov 18, 2021

Server Implementation

Paper

Server Version

1.16.5

Describe the bug

If i use /p i size, the variable is not replaced with the plot size

To Reproduce

  1. Type /p i size

Expected behaviour

It must show the size of the plot. On unmerged plot it has to be show 1 and on merges the amount of plots

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/3e3fda787b4443519c0019a21c43a57a

PlotSquared Version

6.1.4

Checklist

Anything else?

No response
@DevJoey DevJoey added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Nov 18, 2021
@NotMyFault NotMyFault changed the title <size> variable not replaced A couple of placeholders at /plot info are not replaced on manual entry Nov 18, 2021
@NotMyFault NotMyFault added Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Good first issue Simple issue which would be a good first PR for newbies interested in contributing to the project and removed Requires Testing This is a new issue which needs to be approved before labeled with "bug" labels Nov 18, 2021
@NotMyFault
Copy link
Member

The same applies to likes, creationdate and inv, though the latter just misses components.

@DevJoey
Copy link
Author

DevJoey commented Nov 18, 2021

The same applies to likes, creationdate and inv, though the latter just misses components.

Yes, on creation date this is also be the case

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Good first issue Simple issue which would be a good first PR for newbies interested in contributing to the project
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NotMyFault @DevJoey