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

MapXML creator cannot be launched from the game #843

Closed
DanVanAtta opened this issue Jun 12, 2016 · 19 comments
Closed

MapXML creator cannot be launched from the game #843

DanVanAtta opened this issue Jun 12, 2016 · 19 comments

Comments

@DanVanAtta
Copy link
Member

If I'm not missing something, it does not look like the map creator that frostion imported can be launched from within the game.
Task:

  • add button to launch it. Main class is at: tools.map.xml.creator.MapXmlCreator
@RoiEXLab
Copy link
Member

RoiEXLab commented Jun 13, 2016

@DanVanAtta Preferences Window?
I can launch the mapCreator without Problems...

@DanVanAtta
Copy link
Member Author

That launches the old map making tool, not the new one that was ported over recently. Or have I been working with 1.9 for that long that I do not realize we did not have that link in 1.8.0.9

@RoiEXLab
Copy link
Member

There is a new one?

@DanVanAtta
Copy link
Member Author

@RoiEXLab
Copy link
Member

Ah I see...
Is this new MapCreator an addition or an replacement for the old one?
If it's an replacement, then we could simply change the class name of the old Creator with the class name of the new one...

@DanVanAtta
Copy link
Member Author

Additional AFAIK. @frigoref , could you weigh in?

@DanVanAtta DanVanAtta added Discussion team communication thread, meant for coordination and decision making High Priority and removed Discussion team communication thread, meant for coordination and decision making labels Jun 15, 2016
@frigoref
Copy link
Member

The new MapCreator is a start for a replacement of the C#-based version. (sry, didn't notice the thread earlier)

@RoiEXLab
Copy link
Member

@frigoref Ah Thanks... If it's finished, is the old MapCreator obsolete?

@frigoref
Copy link
Member

The old MapCreator should be kept as reference for the moment I guess, but its usage is deprecated.

@RoiEXLab
Copy link
Member

@frigoref What would you suggest to do?
Switch out the class name to be the new one or create a new button?

@frigoref
Copy link
Member

Sorry, What class name are you referring to? I am confused.

@RoiEXLab
Copy link
Member

@frigoref The Map Creator is currently being called by calling javaw.exe (on Windows) with the class name of the old MapCreator e.g. tools.map.making.MapCreator we could replace this with the new one if it's ready to be replaced...

@frigoref
Copy link
Member

What is the new one? Sry, currently my thoughts are around work and EM ^^ and I still do not understand what should be replaced.
The MapCreator package I developed could replace the old map creator application as the number of affected users is quite small, the old map creator is not up-to-date anyway and the new MapCreator needs testing.

@RoiEXLab
Copy link
Member

@frigoref
Old one: tools.map.making.MapCreator
New one: tools.map.xml.creator.MapXmlCreator

@DanVanAtta
Copy link
Member Author

@frigoref @RoiEXLab - this is urgent. People need an in-game way to launch the map making tool. At the very least we need some good documentation on how to run, but the in-game way would fit in with the existing tools.
This is just about the last urgent issue before we release..

@RoiEXLab
Copy link
Member

@DanVanAtta I can do this... The question is should I replace the ild one or just add a new button next to the old one?

@DanVanAtta
Copy link
Member Author

A new button please - then we can take our time deciding whether to drop the old one or not.

@RoiEXLab
Copy link
Member

#877 Adds this button next to the old one...

DanVanAtta added a commit that referenced this issue Jun 25, 2016
Added button for MapCreator launch | Some workarounds(#843)
@RoiEXLab
Copy link
Member

@DanVanAtta this can be closed

@RoiEXLab RoiEXLab removed their assignment Nov 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants