PlotSquared icon indicating copy to clipboard operation
PlotSquared copied to clipboard

Database error when setting too many use flags.

Open ch4ika opened this issue 1 year ago • 3 comments

Server Implementation

Paper

Server Version

1.20.1

Describe the bug

When setting many use materials in flags, the database error occurs.

To Reproduce

Just add as many materials in the Use flag.

Expected behaviour

MySQL value limited to varchar 512

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://paste.gg/p/anonymous/3f819349c91048e5813e3b1a53c95f88

PlotSquared Version

PlotSquared-7.0.0 RC3

Checklist

  • [X] I have included a Plot debugpaste.
  • [X] I am using the newest build from https://www.spigotmc.org/resources/77506/ and the issue still persists.

Anything else?

No response

ch4ika avatar Jul 31 '23 01:07 ch4ika

https://github.com/IntellectualSites/PlotSquared/issues/2686

RedstoneFuture avatar Oct 05 '23 23:10 RedstoneFuture

https://github.com/IntellectualSites/PlotSquared/issues/2686

Seems like it was not fixed in v5 lol (or just implemented again). The create statement still assigns a VARCHAR(512) for the flag value

PierreSchwang avatar Oct 06 '23 01:10 PierreSchwang

Well since we used 512 not 511 anyway, we've got almost an entire byte of space we're not using

dordsor21 avatar Dec 03 '23 15:12 dordsor21