Error in the middle of Multiverse-Inventories import #229


  • Defect
  • Started
Open
Assigned to tux2
  • _ForgeUser8636634 created this issue Jan 10, 2015

    What steps will reproduce the problem?
    1. Run the command /multiinv mvimport
    2. After about an hour, I receive the error, and the conversion process is interrupted

    What version of the product are you using (if you say "latest" you will NOT be helped)?
    3.3.2-dev

    Do you have an error log of what happened?
    http://pastebin.com/M92KVh5R

    What are your world folder names (with proper capitalization)?
    I started from a clean install of MultiInv. Worlds are: Build, Creative, DPvP, End, Enmity, Events, Games, Nether, PvP, Spawn, Staff, Towny, and Vanilla.

    What is the contents of your config.yml and groups.yml?
    config.yml & groups.yml

    What is the entire list of plugins on your server (doing /plugins on the console and pasting the output is a quick method to do this)?
    Plugins (61): TuxTwoLib, WorldBorder, ExecuteEverywhere, FrameProtect, VoxelSniper, PluginManager, Slack, ServerConnect, Vault, Multiverse-Core, BanHammer, PhatLoots, SurvivalGames, SQLibrary, ChestCommands, PotionFix, OpenInv, ProtocolLib, HolographicDisplays, PwnFilter, CleanroomGenerator, LagMeter, Citizens, MultiInv, WorldEdit, Towny, Multiverse-Inventories, Skript, DmToys, SimpleAnnouncer, Multiverse-NetherPortals, VanishNoPacket, PermissionsEx, Questioner, GlobalMarket, Lottery, iConomy, ReportRTS, CoreProtect, Craft_Inc._Gates, MyWarp, Herochat, BungeeChatClient, LibsDisguises, CommandBook, skRayFall, ClickEdit, Sentry, WildSkript, CompatNoCheatPlus, SkQuery, NoCheatPlus, PlayerHeads, LWC, WorldGuard, PreciousStones, RandomSK, HomeSpawnPlus, CraftBook, ChestShop, EchoPet

    Please provide any additional information below.

    This error looks a lot like the error which comes from inventories with banners that have strange banner patterns.

  • _ForgeUser8636634 added the tags New Defect Jan 10, 2015
  • _ForgeUser8636634 posted a comment Jan 10, 2015

    To be specific, I'm using build #41 from Jenkins (http://build.yu8.me:8080/job/MultiInv3/41)

    I was trying to convert 13,833 player inventories for one group (legit), but the process was cut short at 3,114 players. This happened after 1 hour and 28 minutes to be exact.

    The only solution I have found is to delete all inventories with banners in them found by
    'grep -r -l "BANNER" /plugins/Multiverse-Inventories/groups/legit/'
    but this leads to some people losing their inventories as you'd expect.

    This was done on our beta (clone) server, but we do not want to have to delete many player's inventories on the main server in order to change plugins :(


    Edited Jan 10, 2015
  • Tux2 posted a comment Jan 10, 2015

    @Grant1204: Go

    Build #42 on the Jenkins server fixes the import error by skipping broken inventories. http://build.yu8.me:8080/job/MultiInv3/42/ (It also spits out the names of the players it was unable to import properly) Unfortunately since banners are broken on the Multiverse-Inventories side I can't code around this easily. I'll see if I can somehow override the class though and see about getting those inventories to pull in properly.

  • Tux2 posted a comment Jan 10, 2015

    @Grant1204: Go

    Could I get one of those inventory files from Multiverse-Inventories with a custom banner? I'd love to be able to look at it in order to see if I can figure out another way to import those easily.

  • Tux2 removed a tag New Jan 12, 2015
  • Tux2 added a tag Started Jan 12, 2015
  • _ForgeUser8636634 posted a comment Jan 15, 2015

    @Tux2: Go

    I ran only the inventories I had with banners in them through the MV Import (took me some time to figure out how to run only them and not 16k+ others). I have created a folder on dropbox with each player's inventory, vanilla player file, and multiverse-inventories player file (read the txt files):
    https://www.dropbox.com/sh/rpobjc8i73syqzf/AACgrLR-FJulzcJwM2Jjw1a0a?dl=0

    Relevant part of the server log:

    >multiinv mvimport
    [21:35:01 INFO]: [MultiInv] No groups.yml found. Creating example file...
    [21:35:01 INFO]: Error importing survival inventory for player rudmer18
    [21:35:01 INFO]: Error importing adventure inventory for player rudmer18
    [21:35:01 INFO]: Error importing survival inventory for player TheArtist14
    [21:35:01 INFO]: Error importing adventure inventory for player TheArtist14
    [21:35:01 INFO]: Error importing survival inventory for player BLT007
    [21:35:01 INFO]: Error importing adventure inventory for player BLT007
    [21:35:01 INFO]: Error importing survival inventory for player josuelacalidad
    [21:35:01 INFO]: Error importing adventure inventory for player josuelacalidad
    [21:35:01 INFO]: Error importing survival inventory for player KentuckyPerson
    [21:35:01 INFO]: Error importing adventure inventory for player KentuckyPerson
    [21:35:03 INFO]: [Multiverse-Inventories] Disabling Multiverse-Inventories v2.5-b383
    [21:35:03 INFO]: Multiverse-Inventories inventories imported successfuly!
    [21:35:03 INFO]: Please disable/delete Multiverse-Inventories now.
    

    I also included the files that also had banners, but worked fine just as a reference :)
    Sorry this took so long, I just saw your comment today. If you need anything more, please tell me. I'd love to see this fixed.

    P.S. Even if there's a group.yml already present, it says there isn't when importing (though I don't know if it actually remakes the yml every time).


    Edited Jan 15, 2015
  • _ForgeUser8636634 posted a comment Jan 15, 2015

    @Grant1204: Go

    I looked at the JSON files and it seems the problem is with parts like...

    This is from one of the files that has the error:
    "Patterns":[{"==":"BannerPattern","Pattern":"cre","Color":0}]

    This is from one of the files that works:
    "patterns":[{"==":"Pattern","color":"PINK","pattern":"gra"}]

  • Tux2 posted a comment Jan 23, 2015

    @Grant1204: Go

    Thanks for the additional information! Extracting and reading the files manually will probably take some time to accomplish but I'm hoping to have that build out by the end of next week. Sorry about the delay on posting this but I didn't see that you had replied either.


To post a comment, please login or register a new account.