Filled Updated Easy Elevators Plugin for 1.12

I would love to see this plugin updated. It doesn't seem to quite work with 1.12. There seems to be an issue, and I have outlined it in the comments for the plugin. In addition to compatibility, I would also like to be able to set the floor to various blocks rather than just double smooth stone slab. Let me know how feasible you think it would be get this plugin updated. Thanks!

 

I'll update it if the developer gives me the permission to do so (in the plugin page, he/she mentioned that if anyone would like to update it, they should contact them)
no promise

 

Thanks! I assume you have already sent a PM to the developer?

 

@GuitarMaster5001
Given that the plugin is licensed under GNU GPLv3, you don't need permission from the original author to update it. I decided to take a look at updating this, and after fixing a minor error with the sound effect on arrival at a floor, everything seems to work perfectly. I can't seem to reproduce the error about 'next' you're getting. Could you please post some more thorough instructions on how to reproduce this error?

 

Thanks for the reply, Alvin! I've here is a video briefly outlining my current issue with the plugin. Let me know if you need any more information or context that the video/commentary may not provide.

 

@GuitarMaster5001
Huh. I still can't replicate it, are you sure you're using the latest version (1.21)?

 

How did you fix the sound effect on arrival? That may be where my problem lies.

 

@GuitarMaster5001
Well, I opened up the source code and changed it (the field name had changed in later versions). But that error was a NoSuchFieldError, not a NumberFormatException (I also can't find any code setting "next" on a sign anywhere).

Anyway, here's the jar I updated, try it:
http://bringholm.com/downloads/EasyElevator.jar

And as per the license, I'll provide you with the source code:
https://gist.github.com/Alvin-LB/f03e39ddb1d29f41b634e34ff765a98b

 

Well, that updated jar seems to have fixed it for now. Thanks for your help, Alvin!

 

@GuitarMaster5001
Well, glad that it worked (you must've been using an old version somehow, since I cannot for the life of me find how that 'next' error would've occurred)!

Something to note: This code is probably soon running on its last legs. It's using API methods deprecated more than 3 years ago. All I did was the bare minimum to make it work; this urgently needs a complete rewrite, but that is something I do not currently have time for (and if I were to put in the work, I'd rather have support from the original author, so it could be posted on the devbukkit page).