[SPIGOT-3233] Books: limitation to the number of pages Created: 13/May/17  Updated: 23/Aug/17  Resolved: 13/May/17

Status: Resolved
Project: Spigot
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Minor
Reporter: Pyves Assignee: Unassigned
Resolution: Won't Fix Votes: 1
Labels: Books, Items


 Description   

Commit 4acd0f49e07 has imposed a maximum of 50 pages per book. Even though an in-game player cannot himself create books with more than 50 pages, the client can handle books with many more pages without any issues.

This limitation has broken books functionality on many servers. For instance, speaking as the author of the Advanced Achievements plugin, players that have received over 50 achievements have been reporting that their books (listing one achievement per page) are now trimmed down to 50 pages, which is a very low limit.

Similar discussions were done in SPIGOT-3214 (and related commit [
fecf4b4137c|https://hub.spigotmc.org/stash/projects/SPIGOT/repos/craftbukkit/commits/fecf4b4137c5ce01fb18b697da244a6753b35c2b]).

 



 Comments   
Comment by Brokkonaut [ 23/Aug/17 ]

I would still recommend to limit the book size only for books that are created by the client (book update/signing and creative item creation) Books that are created by the server/plugins should only be limited to the clients limits.

Comment by Kimberly Boynton [ 22/Aug/17 ]

@md_5 This really screws over some servers such as mine that use these just for a comprehensive reading piece to our server.  I honestly think this should be allowed to be overridden at the plugin developers own risk since it can handle it.  This completely screws with a lot of the textbooks on my HP server bc now we can't even finish 3 chapters of some of them.

Comment by md_5 [ 13/May/17 ]

Each page causes lag on the server.
50 pages is the maximum the client can create, so this is the best number for us to cap it at.
Capping it at the maximum the client could read (32768) would be the same as just not capping it at all. This isn't acceptable from a security standpoint.

Generated at Tue Apr 22 03:43:28 UTC 2025 using Jira 10.3.5#10030005-sha1:190c783f2bd6c69cd5accdb70f97e48812a78d14.