GDevelop icon indicating copy to clipboard operation
GDevelop copied to clipboard

Fixed Ordering of Parameter for event: "Test the elapsed time of a scene timer"

Open gautamv95 opened this issue 5 years ago • 7 comments

Fixes #843

Tasks:

  • [x] Update the parameters of the action declared in the extension
  • [x] Update the game engine functions
  • [x] Add a UpdateInstructionsFromGD4098 that exchange the parameters
  • [x] Bump the version of GD Core to 4.0.99
  • [x] Compile libGD.js and check that when you open a file it's properly updated

@4ian I cannot find game engine functions related to Time extensions, can you guide me?

gautamv95 avatar Apr 21 '20 18:04 gautamv95

@4ian I cannot find game engine functions related to Time extensions, can you guide me?

The link between a GDevelop action/condition and the implementation in the engine is made using SetFunctionName (in C++ files)/setFunctionName (in JS). In the case of builtin extensions, they are implemented into the platform (GDJS) folder, in the folder "Extensions/Builtin": https://github.com/4ian/GDevelop/blob/master/GDJS/GDJS/Extensions/Builtin/TimeExtension.cpp

This should give you the name of the function you're searching for :)

4ian avatar Apr 22 '20 17:04 4ian

@4ian can you guide me on how do I increase the GDCore version? I figured it is stored in VersionPriv.h, should I make commits to that file?

EDIT - resolved

gautamv95 avatar Apr 23 '20 17:04 gautamv95

@4ian, please review

gautamv95 avatar Apr 24 '20 17:04 gautamv95

Sorry for the delay! I think this is working fine 👍 The only downside is that this is a breaking change, so I want to wait a bit before merging this - because games opened with the version with this won't be compatible with older version. This can be bad if we release a version that is buggy and ask people to go back to the previous version if we can't release another one quickly.

4ian avatar Apr 30 '20 07:04 4ian

@4ian can you point to the code which is responsible for auto-filling the parameters, when clicked on an instruction?

gautamv95 avatar Apr 30 '20 16:04 gautamv95

Sorry for the late reply @gautamv95 Here where are filled the parameters in the instruction editor.

https://github.com/4ian/GDevelop/blob/master/newIDE/app/src/EventsSheet/InstructionEditor/InstructionParametersEditor.js#L287-L342

Bouh avatar Sep 12 '20 09:09 Bouh

I propose we make another PR to add compatibility code to core for when this will be added, to have 2-3 version prior supporting the new project file and having a fixed date from which on we can merge this.

arthuro555 avatar Oct 06 '20 10:10 arthuro555