Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decode MDT string to JSON/PHP array and vice versa #96

Closed
Wotuu opened this issue Jan 4, 2019 · 0 comments
Closed

Decode MDT string to JSON/PHP array and vice versa #96

Wotuu opened this issue Jan 4, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@Wotuu
Copy link
Contributor

Wotuu commented Jan 4, 2019

From a string encoded by MDT, we need to decode it so we can properly parse it and know what to import. For how MDT does it, to quote /u/BlackFayah: In case they don't want to collaborate, this is where the magic happens: https://github.com/nnogga/MethodDungeonTools/blob/master/Transmission.lua. They serialize > compress > encode.

@Wotuu Wotuu added the enhancement New feature or request label Jan 4, 2019
@Wotuu Wotuu added this to the Implement MDT importer/exporter milestone Jan 4, 2019
Wotuu added a commit that referenced this issue Jan 6, 2019
…#96

I initially tried to convert the LUA implementation to PHP but found out that that wasn't going to work. The Huffman encoding library didn't seem to produce the output I was expecting (or perhaps the decoding), then even if it did I'd ran into the issue of the deserializing. In the end I probably could have done it, but going the LUA route also made sure the code wasn't going to be maintained by me, which always helps.

I added some hints and pointers to docs/lua.md for when I ever decide to reinstall the current server where Keystone.guru is running on (bless me when that happens) so I can get LUA working there.

Kept the PHP implementation of what I have for the time being, perhaps it'll come in useful later on.
@Wotuu Wotuu closed this as completed Jan 6, 2019
@Wotuu Wotuu self-assigned this Feb 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant