This revision aims to simplify the platforms export method by removing the first "if" statement for the "publish_default_platform" boolean property, modifying the "class PublishAddPlatform(bpy.types.Operator)" and setting the "default platform" specifications out of the box, thus only using one "def WriteRuntime()" instead of two via a for loop.
Details
Diff Detail
Event Timeline
One of the goals of the "default' platform is that it does not require any setup, and it is something that is always available. If it is treated like other platforms, it could be accidentally removed. Also, this looks like it will add a "Default" platform every time another platform is added. That being said, I am fine with having some better defaults for newly added platforms that they are actually publishable from the get go.
| game_engine_publishing.py | ||
|---|---|---|
| 234 | This can be removed. | |
| game_engine_publishing.py | ||
|---|---|---|
| 509 | From a recent discussion on #gameblender, Moguri and I have decided that it will be necessary to leave this to not break old blend files. | |
I didn't really get what this patch would bring to the end user.
Apart from that, what did you mean in your last reply, are you abandoning this patch? or only that part you specifically commented on?