recursivetree / seat-transport

MIT License
2 stars 1 forks source link

BUG: Parsing error #12

Open martinskalicky opened 1 year ago

martinskalicky commented 1 year ago

recursivetree commented 1 year ago

yeah that's a new formatting I haven't encountered before. From where did you get that?

recursivetree commented 1 year ago

Just to provide some context, CCP uses different copy paste formats everywhere. Jut look at the code responsible for parsing from evepraisal. https://github.com/evepraisal/go-evepraisal/tree/master/parsers

recursivetree commented 1 year ago

You can get it to work by removing the group name

martinskalicky commented 1 year ago

This is from the personal assets so if people wanna do contracts remotely they wont be able to do it. This will be frequent. Can we make it working somehow?

recursivetree commented 1 year ago

Bildschirmfoto 2023-02-18 um 22 17 35 well, it looks different to me

recursivetree commented 1 year ago

it's the personal assets window

martinskalicky commented 1 year ago

Screenshot 2023-02-19 at 13 07 10

martinskalicky commented 1 year ago

You are able to hide some of the stuff. I might test if caregory is gone if the parsing will work. However this should be default if u did not do any changes.

recursivetree commented 1 year ago

I need to rework the parser anyways. I planned to do it since a long time, but new, small feature requests and bugs keep coming in almost every day :) With the current way parsing is handled, it is almost impossible to add more features.

On another note, If the group is hidden, it isn't possible to reliably detect containers when they're renamed

recursivetree commented 1 year ago

for now, just make sure the columns like in my screenshot are visible

martinskalicky commented 1 year ago

I've created a guide to workaround this in case anyone will go into trouble.

recursivetree commented 1 year ago

That doesn't mean it won't happen once the new parsers are done! reopening

recursivetree commented 1 year ago

I'll be able to fix this soon, now that the new parser infrastructure is ready

martinskalicky commented 1 year ago

"Isogen 719951 Megacyte 1 Mexallon 47760 Nocxium 214598 Pyerite 121676 Tritanium 6060433 Zydrine 4"

Throwing: Error You need to enter at least one item!

using DEV branch

recursivetree commented 1 year ago

Is this still an issue? if so, please send a parser report from the tool found under https:your-seat.com/treelib/debug/parsers