Skip to content
This repository was archived by the owner on Feb 17, 2024. It is now read-only.

Conversation

@MrJeremyFisher
Copy link

Maps can be copy protected by running /copyprotect while holding a filled map. Copy protected maps may still be locked and have their scale changed, but may not be copied in either a cartography table or a crafting table.

event.getWhoClicked().sendMessage(
Component.text()
.color(TextColor.color(0xFF5555))
.content("You can not copy copy protected maps")

Choose a reason for hiding this comment

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

Perhaps "You can not duplicate copy protected maps" to avoid the consecutive "copy"?

Choose a reason for hiding this comment

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

You could also hyphenate: "You can not copy copy-protected maps"

Copy link
Author

Choose a reason for hiding this comment

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

How does clone sound? It's what the recipe is actually called.

@Protonull
Copy link

Protonull commented Dec 27, 2023

Hmmm, would it not be better to piggyback off of the map-locking system? As in, anyone is free to copy the source map, but locked maps cannot be copied? If not, then I do think that copyrighting should have a material cost, and should be done through an interface rather than a command.

@MrJeremyFisher
Copy link
Author

MrJeremyFisher commented Dec 28, 2023

I did it via command at no material cost to align with brewery's unlabel command, which fulfills a similar purpose. Adding it on to locking isn't a good option IMO because it removes the ability to distribute locked maps that you actually want to be copyable and would be confusing for anyone not expecting it.

…yprotection

# Conflicts:
#	paper/src/main/java/com/programmerdan/minecraft/simpleadminhacks/hacks/basic/MapCopyProtection.java
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants