PiggyArchive/PiggyAuth

Register wont work

Closed this issue · 9 comments

FunnyBuddys commented

DO NOT REMOVE THIS

Please make sure your issue complies with these guidelines:

    • Be using PMMP
    • Make sure the issue hasn't already been reported
    • Make sure you are on the latest version of PMMP & PiggyAuth
    • Have a detailed title like "Players are being kicked randomly"
    • Provide any crash dumps or errors

General

Server Software:

PMMP

Version of PiggyAuth:

3.0.0.20

Data Provider:

  • MySQL
  • SQLite
  • YAML
  • JSON

Steps to Reproduce the Issue

  1. Try to Register

Extra Information

[Server thread/CRITICAL]: Could not pass event 'pocketmine\event\player\PlayerCommandPreprocessEvent' to 'PiggyAuth v3.0.0.20': Uninitialized string offset: 1 on PiggyAuth\EventListener

[Server thread/CRITICAL]: ErrorException: "Uninitialized string offset: 1" (EXCEPTION) in "plugins/PiggyAuth-master/src/PiggyAuth/EventListener" at line 233 --

nasiridrishi commented

do ./register password password none
you can type none in place of email . email dont work .idk why

SleepSpace9 commented

Because the "else" (https://github.com/DaPigGuy/PiggyAuth/blob/8404e1fda18e1d727de93d4d0cd3e728ee01884a/src/PiggyAuth/Commands/PreregisterCommand.php#L48) is not needed.
If the user enters no email then the argument is automatically set to "none", because of the "else" the rest of the code will not be executed, but should!

nasiridrishi commented

Actually I started using it on my main server and having lots of problems!!

SalmonDE commented

@SleepSpace9 Could you create a pr for that? Also, is this really why this issue occurs?

SleepSpace9 commented

@SalmonDE Sorry, I skimmed a bit too quickly through the issue description and only remembered a previously fixed issue on @AppleMinerHD's server when the player enters no value for email address. Sadly I didn't have much time so far to create a PR, I was always in a rush and didn't want to use the web editor. 😉

SalmonDE commented

darn, I was planning to fix this. ;(

nasiridrishi commented

Email thingy is still not working ..so can you add where players don't have/forced to enter email/"none" coz they are kid's they only know to type /register password password email and it doesn't work .. so they just have you type /register password password ..it would be awesome

DaPigGuy commented

We just did that -_-

nasiridrishi commented

What I tested with latest version/commit.. I will try again