Painting Selector Plugin

jadenPete

I was bored today, so I decided to make a plugin based on an idea from @JeterPan: A painting selector plugin. You can find it on GitHub - jadenPete/OpenPaintings: Painting selector plugin for Spigot., it includes the Eclipse project, the usage, and instructions on how to build a jar. I'd like to see this plugin get into MeepCraft, as I think it's very useful.

MasterTurtle

I think this is a really cool plugin, it would be cool to see it on Meepcraft in the future.

Sent from my iPad using Tapatalk

TheTastyNacho

I don't code and can't read yours. Could you explain what the plugin does?

jadenPete

Sorry if i didn't include enough on the Github description, but it basically works as follows:
1. You do /painting list to see the names of the various paintings
2. You do /painting select to select the type of the next painting you place.
3. You place a painting and it attempts to place the painting you selected. If it's too big for the surface, it spawns a random one.
4. Once you place the painting your selection automatically resets.
5. Or you can reset it manually using /painting cancel.

JeterPan

+1, this would be amazing!!

SuperDyl

We should include this. It'd be a nice tool for those who want to, even if paintings aren't as unruly as they used to be.

LordInateur

This would be a cool plugin for sure, I think... maybe as a donor rank or something. However, I am of the opinion that techs on the server should look into fixing current server issues before adding mini-features :) As far as your project goes, I definitely think it's a good start, but there are a couple of places in the code that can be optimized and/or cleaned up a bit. Definitely let me know if you would like me to help out with the project.

Muunkee

I bet this would be too laggy for the servers really

LordInateur

I bet this would be too laggy for the servers really
~ @Muunkee

Not as much as you would think, actually. A painting selector would really be a one-time-use per painting plugin, and there shouldn't be any continuous event listeners (those are really what lag a server). The most lag from this plugin would come from it being another plugin in general--the solution to which, of course, is consolidate small donor plugins into a single plugin.

jadenPete

Not as much as you would think, actually. A painting selector would really be a one-time-use per painting plugin, and there shouldn't be any continuous event listeners (those are really what lag a server). The most lag from this plugin would come from it being another plugin in general--the solution to which, of course, is consolidate small donor plugins into a single plugin.
~ @LordInateur

Bingo. The only listener this uses is HangingPlaceEvent, which is very specific in that it runs only when an item frame or painting is placed. Even if you place a lot of item frames, a single if statement terminates the function if it isn't a painting. This is a very simple plugin, it just replaces the paintings that you place.

ENIbeast

When people know how to code and you're just sitting there trying to understand what they are saying even though there is no chance of understanding it.

jadenPete

When people know how to code and you're just sitting there trying to understand what they are saying even though there is no chance of understanding it.
~ @ENIbeast

Lmao, see post #4.

CluelessKlutz

Denied after further consideration