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

Import Some Solarus Built-In entities. #1

Open
ekureina opened this issue May 24, 2014 · 4 comments
Open

Import Some Solarus Built-In entities. #1

ekureina opened this issue May 24, 2014 · 4 comments

Comments

@ekureina
Copy link
Owner

Each entity should be named builtin_${entity}.lua, to allow for other versions of the same entities. See solarus-games/solarus#517 for discussion.

@ekureina
Copy link
Owner Author

List of entities to be converted (in no particular order):

  • Arrow
  • Hookshot
  • Bomb
  • Boomerang
  • Fire
  • Crystal
  • Crystal Block
  • Stairs ?
  • Shop Treasure ?
  • Explosion

Any more to be added?

@Renkineko
Copy link
Collaborator

Explosion (a bomb lead to an explosion, but explosions can happens without bombs, so it has to be a different entity)

@ekureina
Copy link
Owner Author

Thank you, I did not think of that.

@ekureina ekureina added this to the Packaging milestone Jun 14, 2014
@Diarandor
Copy link
Contributor

I think it's not possible to make bombs with only one script, because at present there is no method to make the hero lift/carry a custom entity. First, you would need the custom bomb to follow the hero above him when lifted, which is easy, but you would also need to change the tunic of the hero to simulate the carrying state, and change the "custom state" to avoid using other weapons when carrying the bomb (which involves modifying the event game:on_command_pressed(command), and maybe more things). So it would be necessary to modify several important events and other things. I think that scripting the bombs is not worth it.

PS: I made scripts for items that can be carried and not destroyed when thrown (they just bounce three times), but I don't think it would be useful since it is needed to modify important functions which are usually defined in other scripts. (I will probably get rid of these scripts the day when the engine allow to carry custom entities.) Anyway, I can share that code if you want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants