Skip to content

Conversation

@gautamv95
Copy link
Contributor

@gautamv95 gautamv95 commented Apr 21, 2020

Fixes #843

Tasks:

  • Update the parameters of the action declared in the extension
  • Update the game engine functions
  • Add a UpdateInstructionsFromGD4098 that exchange the parameters
  • Bump the version of GD Core to 4.0.99
  • 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?

@4ian
Copy link
Owner

4ian commented Apr 22, 2020

@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 :)

@gautamv95
Copy link
Contributor Author

gautamv95 commented Apr 23, 2020

@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
Copy link
Contributor Author

@4ian, please review

@gautamv95 gautamv95 changed the title [WIP] Fixed Ordering of Parameter for event: "Test the elapsed time of a scene timer" Fixed Ordering of Parameter for event: "Test the elapsed time of a scene timer" Apr 25, 2020
@4ian
Copy link
Owner

4ian commented Apr 30, 2020

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 4ian added the 🧐Breaking change This contains a breaking change, be careful about releasing this label Apr 30, 2020
@gautamv95
Copy link
Contributor Author

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

@Bouh
Copy link
Collaborator

Bouh commented Sep 12, 2020

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

@arthuro555
Copy link
Contributor

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.

@4ian 4ian self-requested a review as a code owner September 21, 2023 16:59
@4ian 4ian force-pushed the master branch 3 times, most recently from 8a32fc5 to 846afd9 Compare August 30, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧐Breaking change This contains a breaking change, be careful about releasing this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Object & scene timers have different ordering of parameters

4 participants