BetonQuest / BetonQuest-Editor

[End of Life] Quest editor for BetonQuest plugin.
GNU General Public License v3.0
17 stars 12 forks source link

Bugs #14

Closed RiledUpCrow closed 8 years ago

RiledUpCrow commented 8 years ago

Please post all info about discovered bugs here :bug:

RiledUpCrow commented 8 years ago
RiledUpCrow commented 8 years ago
HollishKid commented 8 years ago

When trying to load my default package into the editor, I get the following error. (you know my vast knowledge of error reading, so I have no idea whether this is relevant for you) In any case, it causes the package to be loaded very partially; only 3 conversations out of the ... eh ... many!

The follwing error I get when opening another package. This one does seem to be fully loaded nevertheless.

RiledUpCrow commented 8 years ago

You probably have an error in one of the conditions, in one of the player options in one of the conversations. The condition is prefixed with the name of a package which does not exist or at least is not loaded to the editor. Try to load all packages or fix that condition, it won't work anyway if there is such no package.

I think I need to add a nice error handling to package loading.

Second error is caused by the same thing, only with point category. I added issue #25 to address that problem.

RiledUpCrow commented 8 years ago

Hey @HollishKid, could you pull the latest changes and load those packages again? It should handle these errors.

RiledUpCrow commented 8 years ago

Things I found:

rosedragon commented 8 years ago

Found a massive bug that while importing my folder works great, when I compile it on the editor it take out all the player responses. Here are sample of 'before' and 'after'. If you need to get the whole folder, poke me on spigot forum. Before (notepad++) http://pastebin.com/KyVpCDSt

After (editor) http://pastebin.com/iyJtG39Q

Also, the editor tends to make its own line-break with / , is it necessary?

regards, rosedragon

RiledUpCrow commented 8 years ago

Alright, I see the bug. Thanks ^^

No, it's not necessary, it's just how the YAML library saves its output. There is probably a way of changing that, but not now.

The options are loaded, they are simply not in the list of options.

Fixed.