I have many useful changes in this branch now, feel free to cherry pick whatever you want#99
I have many useful changes in this branch now, feel free to cherry pick whatever you want#99intangir wants to merge 18 commits intoExultant:masterfrom
Conversation
Conflicts: src/com/untamedears/citadel/listener/BlockListener.java
A more elegant and simple fix for the item removal issue that appeared with 1.4.6 Log protected block breaks on snitch block even if they still have reinforcement left Damage tools when they are used on protected blocks
|
Cool, Cool, I will have Erocs review the pull just because he has been doing maintenance then merge it and put it up for a short testing before placing it online. Thanks for your work man. |
|
Some of these changes may not be ideal as is for civcraft removing diamond tools from the damaged list might be prudent |
|
I made some quick comments, likely more as notes to myself. I like the tool damage, but it may need some adjustment for Civcraft. Thanks! |
|
Ya there is already rioting over it, might want to exclude damage to diamond |
…nd uprading of default protections also bumped up the protection stats and added a couple more securables
…k (defaults to hide)
…t protection, tuned up the extend code a bit
|
I added a new feature that you probably wont want on civcraft, but its pretty cool nonetheless a default level of protection, "by hand" protection, (uses material 'air' hehe) i also added a feature to allow hiding of /ctinfo information if your using it on blocks you don't have access to they can be left off, or turned on based on configuration options |
|
For the "by hand" protection, it looks like a free low-grade low-annoyance reinforcement, correct? Pretty cool. We'll have to ask ttk2 but I'm suspecting he won't want to enable it. We should cherry-pick ed25241, for the /ctinfo change. |
|
We already add nearly 60,000 reinforcements per day there is no reason to
|
|
might want to include it anyway and leave the option off for civcraft if the mod is publicly released it is a good idea to offer more options the reason i wanted it is because im having a hard time keeping new players, even harder when before they learn how to protect their things they are all burned down or griefed the free protection prevents burning also i made a quick youtube video explaining the basics of citadel usage since so few of my newer players ever bother to reinforce their stuff.., it didnt help though ;) i can link it later (dont have access to youtube at work) |
…eset state faction also
better fire protection
better lava protection
better for the 1.4.6 item removal issue i mentioned in another thread on cancelled messages
logging of reinforcement breaks to preciousstones snitches
tools are damaged when breaking reinforced blocks even if the block doesnt break
i made a priority change that probably has no difference for you, it was to improve compatibility with a mod you don't use
(also some housekeeping on files that i didn't want in my branch)