AuthMe / AuthMeReloaded

The best authentication plugin for the Bukkit/Spigot API!
https://www.spigotmc.org/resources/authmereloaded.6269/
GNU General Public License v3.0
628 stars 516 forks source link

AuthMe 5.1 Snapshot inventory bug. #246

Closed andris155 closed 8 years ago

andris155 commented 8 years ago

Server: Spigot 1.8.8 latest Authme version: http://ci.xephi.fr/job/AuthMeReloaded/167/ Video: https://www.youtube.com/watch?v=qNpYGp-Kslo&feature=youtu.be

Platinteufel commented 8 years ago

http://dev.bukkit.org/bukkit-plugins/authme-reloaded/tickets/987-force-survival-inventory-wipes/

I'm waiting for a fix too ^^

The last working version is 5.0-Dev http://dev.bukkit.org/bukkit-plugins/authme-reloaded/files/72-auth-me-5-0-dev/

sgdc3 commented 8 years ago

Which multiinv plugin are you using?

andris155 commented 8 years ago

I don't using multiinv plugin.

Platinteufel commented 8 years ago

But you're running a plugin to handle inventories? Sth. like Multiverse-Inventories or Per-World-Inventories?

Platinteufel commented 8 years ago

But I've tested it with several Inventory-Plugins. It's definitely an authme issue @sgdc3

And as I wrote it's working fine in 5.0-Dev.

andris155 commented 8 years ago

Used inv edit plugin(creative, and survival inventories): Limitedcreative: http://dev.bukkit.org/bukkit-plugins/limited-creative/

Platinteufel commented 8 years ago

I've reported it three month ago on BukkitDev and there's still no fix or at least a statement about this bug. That's a bit disappointing. :(

NickFirenzo commented 8 years ago

This is not authme bug. Wich plugin for creative use? LimitedCreative/Creative Control?

NickFirenzo commented 8 years ago

Aham, I saw post above. Yes, Limited Creative is problem and I have this problem 3-4 months ago, but I fixed it. You need other plugin wich save inventory on players in BASE, not limited creative. You can use MultiInv http://dev.bukkit.org/bukkit-plugins/multiinv/ http://dev.bukkit.org/bukkit-plugins/tuxtwolib/ MultiInv + TuxTwoLib must put in your plugins folder to work.

  1. When you put 2 plugins above, you need going in config.yml at LimitedCreative and edit it. LimitedCreative config.yml CONFIGURE!: http://pastebin.com/9X6XHvnd
  2. MultiInv plugin to save inventory on all players on the server you must configure, pastebin Edit MultiInv: http://pastebin.com/Kj60Agf3

DON'T REMEMBER TO CONFIGURE LimitedCreative in config.YML how on my PASTEBIN! When you make this steps you can ENJOY :D. If you have any problem, comment on topic

Platinteufel commented 8 years ago

There are inventory wipes if you logout in another gamemode than it's defined by Multiverse (e.g. logout with Survival in a Creative-world). It doesn't matter which inventory-plugin you're using.

Platinteufel commented 8 years ago

Just tested it with the latest dev-build (487).

https://youtu.be/MHF6BxvNlAw

DNx5 commented 8 years ago

it fixed? @Platinteufel

Platinteufel commented 8 years ago

Nope, it's still not fixed in build 487.

games647 commented 8 years ago

Do we actually need to save and restore the gamemode? I think it's useless.

sgdc3 commented 8 years ago

@games647 I agree

games647 commented 8 years ago

@Platinteufel Can I send you new version in order to test if the issue still exists?

I already tested my changes, but it's very likely that I forgot something.

Platinteufel commented 8 years ago

Seems like everything works fine now, thank you @games647 ! Just one suggestion: Maybe you could disable gamemode restoring by default and the user can enable it in config if they need to. There should be a warning like: "gamemode restoring can cause inventory swaps".

games647 commented 8 years ago

@Platinteufel it should be already disabled by default because of this commit https://github.com/AuthMe-Team/AuthMeReloaded/commit/0beb937b8fcfef8e1e1b06f42ea65088e403ee5b. The warning hint is really good point. We might add it to the config.

I'm glad that it works now.

andris155 commented 8 years ago

Thanks!