search

jlkatz / OCTGN.OCTGNDeckConverter

A comprehensive OCTGN deck converter for OCTGN 3 which can convert several formats, websites, and plain text into the OCTGN 3 format.
http://octgngames.com/deckconverter/
7 stars 9 forks source link

Converter stopped working...again #23

Closed bykr closed 6 years ago

bykr commented 6 years ago

Version 3.6 fixed the issue, for a while. Stopped working last night. Generates the following error:

Details: System.NullReferenceException: Object reference not set to an instance of an object. at OCTGNDeckConverter.Model.ConverterDeck.PopulateConverterMappings(Dictionary2 converterSets) at OCTGNDeckConverter.Model.ConvertEngine.TextConverter.ConvertText(Dictionary2 sectionsText, Dictionary2 converterSets, IEnumerable1 deckSectionNames) at OCTGNDeckConverter.Model.ConvertEngine.Game.GameConverter.ConvertText(Dictionary`2 sectionsText, ConverterGame converterGame) at OCTGNDeckConverter.Model.Converter.Convert()

Bjorgh commented 6 years ago

An error occurred while trying to convert the deck. Please try again.

Details: System.NullReferenceException: Object reference not set to an instance of an object. at OCTGNDeckConverter.Model.ConverterDeck.PopulateConverterMappings(Dictionary2 converterSets) at OCTGNDeckConverter.Model.ConvertEngine.TextConverter.ConvertText(Dictionary2 sectionsText, Dictionary2 converterSets, IEnumerable1 deckSectionNames) at OCTGNDeckConverter.Model.ConvertEngine.Game.GameConverter.ConvertText(Dictionary`2 sectionsText, ConverterGame converterGame) at OCTGNDeckConverter.Model.Converter.Convert()

jlkatz commented 6 years ago

Thanks for alerting me to this, I was able to replicate the problem. Brine changed some of OCTGN's data structure on July 12 2017, which caused the bug with v3.5.0. I fixed to match, releasing v3.6.0. Now it looks like Brine reverted his change on August 13 2017, so I will revert my code as well. I will release v3.7.0 shortly.

jlkatz commented 6 years ago

v3.7.0 is released now, which should resolve the problem.