#636 You can light fire on top of the fire (Contribution)

Closed
opened 3 months ago by wuniversales · 2 comments

With the lighter, you can light fire on top of the fire.

Updated

With the charge of fire (The fireball) also passes.

With the lighter, you can light fire on top of the fire. **Updated** With the charge of fire (The fireball) also passes.
Wuzzy added the
bug
label 3 months ago
Wuzzy added the
items
label 3 months ago
wuniversales commented 2 months ago
Poster

I update, I provide the code for the solution of this problem.

I've replaced this

mcl_fire.set_fire(pointed_thing, user)

with this

-- Do not light a fire on top of a fire
if minetest.get_node(pointed_thing.under).name ~= "mcl_fire:fire" then
mcl_fire.set_fire(pointed_thing, user)
end

In the files "/mods/ITEMS/mcl_fire/fire_charge.lua” and "/mods/ITEMS/mcl_fire/flint_and_steel.lua”

I do not know exactly how this forks work, I hope I have done well.

Fork (File 1): b63b70db2b Fork (File 2): 287021f3ea

I update, I provide the code for the solution of this problem. I've replaced this ``` mcl_fire.set_fire(pointed_thing, user) ``` with this ``` -- Do not light a fire on top of a fire if minetest.get_node(pointed_thing.under).name ~= "mcl_fire:fire" then mcl_fire.set_fire(pointed_thing, user) end ``` In the files **"/mods/ITEMS/mcl_fire/fire_charge.lua"** and **"/mods/ITEMS/mcl_fire/flint_and_steel.lua"** **I do not know exactly how this forks work, I hope I have done well.** Fork (File 1): https://git.minetest.land/wuniversales/MineClone2/commit/b63b70db2bfd8d0d8a644ac2f29ee96855141331 Fork (File 2): https://git.minetest.land/wuniversales/MineClone2/commit/287021f3ea078367707bd03104ad38708f8d8b9e
wuniversales changed title from You can light fire on top of the fire to You can light fire on top of the fire (Contribution) 2 months ago
Wuzzy added the
contribution inside
label 2 months ago
Wuzzy commented 2 months ago
Owner

Fixed with my own code.

Fixed with my own code.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.