alexitocr / epgp

Automatically exported from code.google.com/p/epgp
0 stars 0 forks source link

EPGP import feature reports "The imported data is invalid" on fresh export and from epgpweb #711

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
EPGP import fails with "EPGP: The imported data is invalid" when copy+pasting 
an export freshly made.  Testing was done with EPGP 5.6.4 with two different 
guilds.  The error is reproducible with all other addons disabled.

What steps will reproduce the problem?
1. /epgp -> Log -> Export
2. Copy the contents 
3. Import
4. Paste the contents

or
3. Import to EPGPWeb
4. Export from EPGPWeb
5. Paste the contents

Tested both as an officer in a large guild and the GM of a personal bank guild 
(to rule out any guild permissions issues)

What is the expected output? What do you see instead?
I expected EPGP to import the data.  Instead, I receive "EPGP: The imported 
data is invalid"

What version of the product are you using? What locale is your client (en,
ru, kr, ch, es, tw)? What other addons do you have installed and what
versions? How many members do you have in your guild? (/guildinfo).

EPGP 5.6.4 from Curse, which Google Code reports is the latest version.
en locale, US region
Problem still occurs with all other addons disabled (except Addon Control 
Panel, which was used to disable all other addons temporarily for testing) 

Bank guild consists of 3 players, 2 accounts.  Real guild has ~600 players, but 
only about 60 have EP/GP data in the export.  

Paste the text surrounded by -EPGP- in Guild Information here:
Not set in bank guild.

Real guild: 
-EPGP-
@BASE_GP:100
@DECAY_P:10
@MIN_EP:500
@EXTRAS_P:0
-EPGP-

If this is about http://www.epgpweb.com integration, what is the realm
and region of your guild?
Earthen Ring-US.  (Bank guild only, real guild is not on epgpweb)

Please provide any additional information below.
Exported data is:

{"guild":"Unaffiliated","region":"us","min_ep":0,"base_gp":1,"roster":[["Isaerya
",10000,5537]],"decay_p":0,"loot":[[1314085020,"Isaerya",11225,10000]],"timestam
p":1314211200,"extras_p":100,"realm":"Earthen Ring"}

Original issue reported on code.google.com by thisgene...@gmail.com on 24 Aug 2011 at 3:52

GoogleCodeExporter commented 8 years ago
This same thing happened to me as well today, hoping to get a response!

Original comment by jana.flo...@gmail.com on 24 Aug 2011 at 8:49

GoogleCodeExporter commented 8 years ago
Same here, looks like the whitespaces in the guild/realmname are screwed (tho 
not sure)

Original comment by a.denijs...@gmail.com on 6 Nov 2011 at 8:46

GoogleCodeExporter commented 8 years ago
found it: 

log.lua, replace (starting line 264):
  for k,t in pairs(types) do
    if type(d[k]) ~= t then
      EPGP:Print(L["The imported data is invalid"])
      return
    end
  end

with
  for k,t in pairs(types) do
    if(d[k] ~= nil) then
        if type(d[k]) ~= t then
          EPGP:Print(L["The imported data is invalid"])
          return
        end
    end 
  end

this will only fix the error, already debugging further why it won't import now

Original comment by a.denijs...@gmail.com on 6 Nov 2011 at 8:55

GoogleCodeExporter commented 8 years ago
Theeere we go:

epgp.lua, replace (starting line: 708) 
function EPGP:SetGlobalConfiguration(decay_p, extras_p, base_gp, min_ep, 
outsiders)

with
function EPGP:SetGlobalConfiguration(decay_p, extras_p, base_gp, min_ep, 
outsiders)
  local DEFAULT_DECAY_P = 0
  local DEFAULT_EXTRAS_P = 0
  local DEFAULT_MIN_EP = 0
  local DEFAULT_BASE_GP = 1
  local DEFAULT_OUTSIDERS = 0

that imported for me perfectly

Original comment by a.denijs...@gmail.com on 6 Nov 2011 at 9:01

GoogleCodeExporter commented 8 years ago
trying to commit changes :P just never worked with code.google.com in terms of 
repository

Original comment by a.denijs...@gmail.com on 6 Nov 2011 at 9:08

GoogleCodeExporter commented 8 years ago
This bug is still occuring in EPGP 5.7.1, and the posted fixes in comment #3/#4 
still work.  

Original comment by thisgene...@gmail.com on 25 Jun 2012 at 6:38