Did what I should have done to begin with: changed _mcl_copper_waxed_variant to _mcl_waxed_variant so that it intuitively applies to more than just copper.

Cleaned up commented out code.

.gitignore

@@ -4,5 +4,4 @@
 *.blend2
 *.blend3
 /.idea/
-*.xcf
+*.xcf
-.Rproj.user

CONTRIBUTING.md

@@ -76,7 +76,7 @@ in singleplayer, post a screenshot of the message that Minetest showed
 when the crash happened (or copy the message into your issue). If you
 are a server admin, you can find error messages in the log file of the
 server.
-* Tell us which MineClone2 and Minetest versions you are using (from Minetest 5.7 type /ver, for previous versions, check the game.conf or README.md file).
+* Tell us which MineClone2 and Minetest versions you are using.
 * Tell us how to reproduce the problem: What you were doing to trigger
 the bug, e.g. before the crash happened or what causes the faulty
 behavior.

CREDITS.md

@@ -36,8 +36,7 @@
 * MrRar
 * talamh
 * Faerraven / Michieal
-* FossFanatic 
+* FossFanatic
-* SmokeyDope
 
 ## Contributors
 * Laurent Rocher
@@ -82,12 +81,15 @@
 * aldum
 * Dieter44
 * Pepebotella
+* MrRar
 * Lazerbeak12345
 * mrminer
 * Thunder1035
 * opfromthestart
 * snowyu
 * FaceDeer
+* Faerraven / Michieal
+* FossFanatic
 * Herbert West
 * GuyLiner
 * 3raven
@@ -99,8 +101,8 @@
 * b3nderman
 * CyberMango
 * gldrk
+* SmokeyDope
 * atomdmac
-* emptyshore
 
 ## MineClone5
 * kay27
@@ -174,7 +176,6 @@
 * cora
 * Faerraven / Michieal
 * Nicu
-* Exhale
 
 ## Translations
 * Wuzzy
@@ -190,7 +191,6 @@
 * 3raven
 * SakuraRiu
 * anarquimico
-* syl
 
 ## Funders
 * 40W
@@ -203,7 +203,4 @@
 * wsor for working tirelessly in the shadows for the good of all of us, particularly helping with solving contentDB and copyright issues.
 * The workaholics who spent way too much time writing for the Minecraft Wiki. It's an invaluable resource for creating this game
 * Notch and Jeb for being the major forces behind Minecraft
-* Dark Reaven Music (https://soundcloud.com/dark-reaven-music) for the main menu theme (Calmed Cube) and Traitor (horizonchris96), which is licensed under https://creativecommons.org/licenses/by-sa/3.0/
+* Dark Reaven Music (https://soundcloud.com/dark-reaven-music) for the main menu theme (Calmed Cube), which is licensed under https://creativecommons.org/licenses/by-sa/3.0/
-* Jester for helping to finely tune MineClone2 (https://www.youtube.com/@Jester-8-bit). Songs: Hailing Forest, Gift, 0dd BL0ck, Flock of One (License CC BY-SA 4.0)
-* Exhale & Tim Unwin for some wonderful MineClone2 tracks (https://www.youtube.com/channel/UClFo_JDWoG4NGrPQY0JPD_g). Songs: Valley of Ghosts, Lonely Blossom, Farmer (License CC BY-SA 4.0)
-* Diminixed for 3 fantastic tracks and remastering and leveling volumes. Songs: Afternoon Lullaby (pianowtune02), Spooled (ambientwip02), Never Grow Up (License CC BY-SA 4.0)

MODELS.md

@@ -1,70 +0,0 @@
-#Models in Minetest/Mineclone2
-
-Models are an important part of all entities & unique nodes in Mineclone2. They provide a 3 dimensional map of an object for which textures are then applied to. This document is for modders, it quickly highlights some important information for the software needed to open models in Mineclone2.
-
-## Minetest Wiki
-For more detailed information on actually using blender to create and modify models for Minetest/Mineclone2, please visit the Minetest wiki's page on using Blender [Here](https://wiki.minetest.net/Using_Blender)
-
-##Recommended software
-
-###Blender
-
-Blender is a very popular and free modeling software supported on Windows, MacOS, and most Linux distributions. It is recommended to use Blender to create and modify 3D models within the minetest engine. 
-
-Download blender [Here](https://www.blender.org/download/)
-
-### .b3d addon for blender
-
-Blitz 3D (.b3d) Is one of the main animated model formats used for entities in the minetest engine. It cannot be imported to blender without a plugin called "Import-Export:Bitz 3D format (.b3d)".
-
-The most up to date version of this Blender plugin can be downloaded [Here](https://github.com/GreenXenith/io_scene_b3d/releases/tag/f189786)
-
-##Types of model formats
-
-###Animated, skinned models
-* Blitz 3D files (.b3d)
-
-* Microsoft DirectX (.x) (binary & text, compression is not supported)
-
-###Static meshes
-* Wavefront OBJ (.obj)
-
-Note: The sometimes accompanying .mtl files are not supported and can safely be deleted.
-
-Note: Do not use .b3d and .x files for static meshes at the moment, Minetest currently spawns animated mesh scene nodes for these, which may result in reduced performance.
-
-### Supported texture formats
-
-* .png
-
-* .jpg
-
-* .bmp (depreciated, please use .png or .jpg)
-
-* .tga (depreciated, please use .png or .jpg)
-
-Note: Any formats not mentioned above but known to work in the past were removed in 5.5.0 and aren't supported anymore.
-
-##Pros & Cons of .b3d vs .x
-
-###B3D
-* [+] Binary format means a small size
-
-* [-] Difficult to postprocess after exporting
-
-* [-] Difficult to debug problems
-
-###X (text version)
-* [+] Can be parsed easily with lua scripts
-
-* [+] Can be easily generated by scripts
-
-* [+] Easy to debug issues (you can just read it)
-
-* [+] Can be optimized by quantizing some data
-
-* [-] Blender exporter is kinda buggy and inefficient
-
-* [-] Probably still bigger than an equivalent .b3d
-
-Note: Avoid using the binary X format! It's actually just a tokenized version of the ASCII representation, and may actually be less efficient than a sufficiently optimized text .x file!

README.md

@@ -78,7 +78,7 @@ The MineClone2 repository is hosted at Mesehub. To contribute or report issues,
 
 * Mesehub: <https://git.minetest.land/MineClone2/MineClone2>
 * Discord: <https://discord.gg/xE4z8EEpDC>
-* YouTube: <https://www.youtube.com/channel/UClI_YcsXMF3KNeJtoBfnk9A>
+* YouTube <https://www.youtube.com/channel/UClI_YcsXMF3KNeJtoBfnk9A>
 * IRC: <https://web.libera.chat/#mineclone2>
 * Matrix: <https://app.element.io/#/room/#mc2:matrix.org>
 * Reddit: <https://www.reddit.com/r/MineClone2/>

README_locale/README.fr.md

@@ -1,45 +1,48 @@
 # MineClone2
-Un jeu non-officiel similaire à Minecraft pour Minetest. Forké depuis Mineclone par davedevils. Développé par de nombreuses personnes. Pas développé ni supporté par Mojang AB.
+Un jeu non-officiel similaire à Minecraft pour Minetest. Forké depuis Mineclone par davedevils. Développé par de nombreuses personnes. Ni développé ou supporté par Mojang AB.
+
+ Version: 0.79 (en dévelopment)
 
 ### Gameplay 
+
 Vous atterissez dans un monde fait entièrement de cubes et généré aléatoirement. Vous pouvez explorer le monde, miner et construire presque n'importe quel bloc pour créer de nouvelles structures. Vous pouvez choisir de jouer en "mode survie" dans lequel vous devez combattre des monstres et la faim et progresser lentement dans différents aspects du jeu, comme l'extraction de minerai, l'agriculture, la construction de machines et ainsi de suite. Ou alors vous pouvez jouer en "mode créatif" où vous pouvez construire à peu près n'importe quoi instantanément.
 
 ### Résumé du Gameplay
 
 * Jeu de type bac-à-sable, sans objetifs
-* Survie : combattez des monstres hostiles et la faim
+* Survie : combattre des monstres hostiles et la faim
-* Creusez pour du minerai et d'autres trésors
+* Creuser pour du minerai et d'autres trésors
-* Magie : gagnez de l'expérience et enchantez les outils
+* Magie : gagner de l'expérience et enchanter les outils
-* Utilisez les blocs ramassés pour construire de magnifiques bâtiments, votre imagination est la seule limite
+* Utiliser les blocs ramassés pour construire de magnifiques bâtiments, votre imagination est la limite
-* Ramassez des fleurs (et d'autres sources de teinture) et colorez votre monde
+* Ramasser des fleurs (et d'autres sources de teinture) et colorez votre monde
 * Trouvez des graines et commencez à cultiver
 * Trouvez ou fabriquez des centaines d'objets
-* Construisez un réseau ferroviaire complexe et amusez-vous avec les wagonnets
+* Construisez un réseau ferroviaire complexe et amusez vous avec les wagonnets
 * En mode créatif vous pouvez construire presque n'importe quoi gratuitement et sans limite
 
 ## Comment jouer (démarrer rapidement)
 ### Commencer
 * **Frappez un arbre** jusqu'à ce qu'il casse et donne du bois
 * Placez le **bois dans la grille 2x2** (la "grille de fabrication" de votre menu d'inventaire) et fabriquez 4 planches de bois
-* Placer les 4 planches de bois dans la grille 2x2 et **fabriquez un établi**
+* Placer les 4 planches de bois dans la grille 2x2 et **fabriquez une table d'artisanat**
-* **Faites un clic droit sur l'établi** (icone livre) pour apprendre toutes les recettes possibles
+* **Cliquez droit la table d'artisanat** (icone livre) pour apprendre toutes les recettes possibles
 * **Fabriquez une pioche de bois** pour miner la pierre
-* Différents outils minent différentes sortes de blocs. Essayez-les !
+* Différents outils minent différentes sortes de blocs. Essayez les !
-* Continuez à jouer comme vous voulez. Amusez-vous !
+* Continuez à jouer comme vous voulez. Amusez vous !
 
 ### Agriculture
 * Trouvez des graines
 * Fabriquez une houe
-* Faites un clic droit sur la terre ou des blocs similaires avec la houe pour créer des terres agricoles
+* Cliquez droit la terre ou des blocs similaires avec la houe pour créer des terres agricoles
 * Placer des graines sur des terres agricoles et regardez les pousser
 * Récoltez les plantes une fois matûres
 * Les terres agricoles proche de l'eau deviennent humides et accélèrent la croissance
 
 ### Four
-* Fabriquez un four
+* Fabriquer un Four
 * Le four permet d'obtenir plus d'objets
-* L'emplacement du haut doit contenir un objet fondable (par ex : minerai de fer)
+* L'emplacement du haut doit contienir un objet fondable (par ex : minerai de fer)
-* L'emplacement du bas doit contenir un objet combustible (par ex : charbon)
+* L'emplacement du bas doit contienir un objet combustible (par ex : charbon)
 * Voir le guide d'artisanat pour en apprendre plus sur les objets fondables et combustibles
 
 ### Aide supplémentaire
@@ -59,43 +62,43 @@ Il n'y a pas de support de MineClone2 dans les versions développement de Minete
 Pour installer MineClone2 (si ce n'est pas déjà fait), déplacez ce dossier dans le dossier “games” de Minetest. Consultez l'aide de Minetest pour en apprendre plus.
 
 ## Liens utiles 
-Le dépôt de MineClone2 est hébergé sur Mesehub. Pour contribuer ou signaler des problèmes, allez là-bas.
+Le dépôt de MineClone2 est hébergé sur Mesehub. Pour contribuer ou rapporter des problèmes, aller là-bas.
 
-* Mesehub : <https://git.minetest.land/MineClone2/MineClone2>
+* Mesehub: <https://git.minetest.land/MineClone2/MineClone2>
-* Discord : <https://discord.gg/xE4z8EEpDC>
+* Discord: <https://discord.gg/xE4z8EEpDC>
-* YouTube : <https://www.youtube.com/channel/UClI_YcsXMF3KNeJtoBfnk9A>
+* YouTube <https://www.youtube.com/channel/UClI_YcsXMF3KNeJtoBfnk9A>
-* IRC : <https://web.libera.chat/#mineclone2>
+* IRC: <https://web.libera.chat/#mineclone2>
-* Matrix : <https://app.element.io/#/room/#mc2:matrix.org>
+* Matrix: <https://app.element.io/#/room/#mc2:matrix.org>
-* Reddit : <https://www.reddit.com/r/MineClone2/>
+* Reddit: <https://www.reddit.com/r/MineClone2/>
-* Forums Minetest : <https://forum.minetest.net/viewtopic.php?f=50&t=16407>
+* Minetest forums: <https://forum.minetest.net/viewtopic.php?f=50&t=16407>
-* ContentDB : <https://content.minetest.net/packages/wuzzy/mineclone2/>
+* ContentDB: <https://content.minetest.net/packages/wuzzy/mineclone2/>
-* OpenCollective : <https://opencollective.com/mineclone2>
+* OpenCollective: <https://opencollective.com/mineclone2>
 
 ## Objectif
-* Essentiellement, créer un clone de Minecraft stable, moddable, libre et gratuit basé sur le moteur de jeu Minetest avec des fonctionnalités abouties, utilisable à la fois en mode solo et multijoueur. Actuellement, beaucoup des fonctionnalités de **Minecraft Java Edition** sont déjà implémentées et leur amélioration est prioritaire sur les nouvelles demandes.
+* Créer un clone stable, moddable, libre et gratuit basé sur le moteur de jeu Minetest avec des fonctionalités abouties, utilisable à la fois en mode solo et multijoueur. Actuellement, beaucoup des fonctionalités de **Minecraft Java Edition** sont déjà implémentées et leur amélioration est prioritaire sur les nouvelles demandes.
-* Avec une priorité moindre, implémenter les fonctionnalités des versions **Minecraft + OptiFine** (OptiFine autant que supporté par le moteur Minetest). Cela signifie que les fonctionnalités présentes dans les versions listées sont priorisées.
+* Avec une priorité moindre, implémenter les fonctionalités des versions **Minecraft + OptiFine** (OtiFine autant que supporté par le moteur Minetest). Cela signifie que les fonctionalités présentes dans les versions listées sont priorisées.
-* Dans l'idéal, créer une expérience performante qui tourne bien sur des ordinateurs à basse performance. Malheureusement, en raison des mécanismes de Minecraft et des limitations du moteur Minetest ainsi que de la petite taille de la communauté de joueurs sur des ordinateurs à basses performances, les optimisations sont difficiles à explorer.
+* Dans l'idéal, créer une expérience performante qui tourne bien sur des ordinateurs à basse performance. Malheureusement, en raison des mécanismes de Minecraft et des limitations du moteur Minetest ainsi que de la petite taille de la communauté de joueurs sur des ordinateurs à basse performances, les optimisations sont difficiles à explorer.
 
 ## Statut de complétion
 Ce jeu est actuellement au stade **beta**.
-Il est jouable mais incomplet en fonctionnalités.
+Il est jouable mais incomplet en fonctionalités.
 La rétro-compatibilité n'est pas entièrement garantie, mettre votre monde à jour peut causer de petits bugs.
 Si vous voulez utiliser la version de développement de MineClone2 en production, la branche master est habituellement relativement stable. Les branches de test fusionnent souvent des pull requests expérimentales et doivent être considérées comme moins stable.
 
-Les principales fonctionnalités suivantes sont disponibles :
+Les principales fonctionalités suivantes sont disponibles :
 
 * Outils, armes
 * Armure
-* Système de fabrication : grille 2x2, établi (grille 3x3), four, incluant un guide de fabrication
+* Système de fabrication : grille 2x2, table d'artisanat (grille 3x3), four, incluant un guide de fabrication
-* Coffres, grands coffres, coffre ender, boites de Shulker
+* Coffres, grands coffres, coffre ender, boite de shulker
 * Fours, entonnoirs
 * Faim
 * La plupart des monstres et animaux
-* Tous les minerais de Minecraft
+* Tout les minerais de Minecraft
 * La plupart des blocs de l'overworld
 * Eau et lave
 * Météo
-* 28 biomes + 5 biomes du Nether
+* 28 biomes + 5 biomes du nether
 * Le Nether, monde souterrain brûlant dans une autre dimension
 * Circuits Redstone (partiel)
 * Effets de Statut (partiel)
@@ -104,10 +107,10 @@ Les principales fonctionnalités suivantes sont disponibles :
 * Brassage, potions, flèches trempées (partiel)
 * Bâteaux
 * Feu
-* Blocs de construction : escaliers, dalles, portes, trappes, barrières, portillons, murets
+* Blocs de construction : escaliers, dalles, portes, trappes, barrière, portillon, muret
 * Horloge
 * Boussole
-* Éponge
+* Eponge
 * Bloc de slime 
 * Petites plantes et pousses
 * Teintures
@@ -115,30 +118,26 @@ Les principales fonctionnalités suivantes sont disponibles :
 * Blocs de décoration : verre, verre teinté, vitres, barres de fer, terre cuites (et couleurs), têtes et plus
 * Cadres d'objets 
 * Juke-boxes
-* Lits
-* Menu d'inventaire
-* Inventaire créatif
-* Agriculture
 * Livres pour écrire
 * Commandes
 * Villages
 * L'End 
 * et plus !
 
-Les fonctionnalités suivantes sont incomplètes :
+Les fonctionalités suivantes sont incomplètes :
 
-* Certains monstres et animaux
+* certains monstres et animaux
-* Certains composants de Redstone
+* certains composants de Redstone
 * Wagonnets spéciaux
-* Quelques blocs et objets non-triviaux
+* quelques blocs et objets non-triviaux
 
-Fonctionnalités bonus (absentes de Minecraft) :
+Fonctionalités bonus (absentes de Minecraft) :
 
 * Guide d'artisanat intégré au jeu qui montre les recettes d'artisanat et de cuisson
 * Système d'aide intégré au jeu contenant des informations à propos des techniques de base, blocs, objets et plus
-* Recettes d'artisanat temporaires. Elles existent uniquement pour rendre des objets accessibles qui ne le seraient pas autrement sauf en mode créatif. Elles seront retirées au cours de l'avancement du développement et de l'ajout de nouvelles fonctionnalités.
+* Recettes d'artisanat temporaires. Elles existent uniquement pour rendre des objets accessibles qui ne le seraient pas autrement sauf en mode créatif. Elles seront retirées au cours de l'avancement du développement et de l'ajout de nouvelles fonctionalités.
 * Pousses dans les coffres en mapgen v6
-* Entièrement moddable (grâce la puissante API Lua de Minetest)
+* Entièrement moddable (grâce la puissante API lua de Minetest)
 * Nouveaux blocs et objets :
     * Outil de recherche, montre l'aide de ce qu'il touche
     * Plus de dalles et d'escaliers
@@ -150,24 +149,22 @@ Fonctionnalités bonus (absentes de Minecraft) :
     * Avant-poste du Nether (Forteresse)
 
 Différences techniques avec Minecraft :
-
 * Limite en hauteur de 31000 blocs (bien plus grand que Minecraft)
 * Taille horizontale du monde 62000×62000 blocs (bien plus petit que Minecraft mais toujours très grand)
 * Toujours assez incomplet et buggé
-* Des blocs, objets, ennemis et fonctionnalités manquent
+* Des blocs, objets, ennemis et fonctionalités manquent
 * Quelques objets ont des noms légèrement différents pour être plus faciles à distinguer
 * Des musiques différentes pour le juke-boxe
 * Des textures différentes (Pixel Perfection)
 * Des sons différents (sources diverses)
 * Un moteur de jeu différent (Minetest)
 * Des bonus cachés différents
-
 ...et enfin MineClone2 est un logiciel libre !
 
 ## Autres fichiers readme
 
-* `LICENSE.txt` : Le texte de la licence GPLv3
+* `LICENSE.txt`: Le texte de la license GPLv3
-* `CONTRIBUTING.md` : Information pour ceux qui veulent contribuer
+* `CONTRIBUTING.md`: Information pour ceux qui veulent contribuer
-* `API.md` : Pour les modders Minetest qui veulent modder ce jeu
+* `API.md`: Pour les modders Minetest qui veulent modder ce jeu
-* `LEGAL.md` : Information légale
+* `LEGAL.md`: Information légale
-* `CREDITS.md` : Liste de toutes les personnes qui ont contribué
+* `CREDITS.md`: Liste des contributeurs

RELEASE.md

@@ -13,15 +13,15 @@ git add game.conf
 
 #git add RELEASE.md
 
-git commit -m "Pre-release update credits and set version 0.83.0"
+git commit -m "Pre-release update credits and set version 0.82.0"
 
-git tag 0.83.0
+git tag 0.82.0
 
-git push origin 0.83.0
+git push origin 0.82.0
 
-#Update version in game.conf to the next version with -SNAPSHOT suffix
+#Update version in game.conf to -SNAPSHOT
 
-git commit -m "Post-release set version 0.84.0-SNAPSHOT"
+git commit -m "Post-release set version 0.82.0-SNAPSHOT"
 
 ### Hotfix Release
 

TEXTURES.md

@@ -1,57 +0,0 @@
-# Making Textures In Mineclone2
-
-Textures are a crucial asset for all items, nodes, and models in mineclone2. This document is for artist who would like to make and modify textures for mineclone2. While no means comprehensive, this document contains the basic important information for beginners to get started with texture curation and optimization.
-
-## Minetest Wiki
-For more detailed information on creating and modifing texture packs for Minetest/Mineclone2, please visit the Minetest wiki's page on creating a texture pack. Click [here](https://wiki.minetest.net/Creating_texture_packs) to view the wiki page on creating texture packs.
-
-## GIMP Tutorials Pixel Art Guide
-GIMP Tutorials has an excellent guide to making pixel art in GIMP. If you would like further clarification as well as screenshots for what we are about to cover, it is an excellent resource to turn to. Click [here](https://thegimptutorials.com/how-to-make-pixel-art/) to view the guide
-
-## Recommended Software
-
-### GIMP
-
-GIMP (Gnu Image Manipulation Program) is a very popular and free image editing software supported on Windows, MacOS, and most Linux distributions. It is recommended to use GIMP to create and modify textures within the minetest engine. 
-
-Download GIMP [here](http://gimp.org/)
-
-# Getting Started
-## Creating a new file
-the first thing to do is open GIMP and create a new file to work in by opening the File menu and choosing New.
-
-Choose width of 16 and height of 16 for the image size. While higher resolution textures are possible, The default size is 16x16. It is recommended you use this size as well, as it is universally supported on all systems.
-
-## Zoom In
-Next, you'll want to zoom in as the canvas is very small at the default zoom level. To do this either use CTRL + mousewheel, +/-, or navigate to the View menu > zoom > zoom in
-
-## Configure Grid
-Now, we'll want to turn on the grid. Open the edit menu and enable the 'show grid' option.
-
-The default grid size is 10 pixels, we want to change it to a 1 pixel grid. Go to the Image menu and choose 'configure grid.
-
-In the Spacing section, change both the Horizontal and Vertical pixel settings to 1.00 then click ok and the grid will update.
-
-## Pencil Tool & Color Picking
-The most useful brush type for pixel art is the Pencil tool. Its nested under the paintbrush tool in the toolbox, or you can use the keyboard shortcut 'N'.
-
-Once the pencil tool is selected, navigate to the sliders on the left side of the canvas and change brush size to 1 pixel.
-
-Now choose a color! You can do this by clicking on the two colored squares under the toolbox. The Color Picker tool is also a good option if you already have a reference image for color palette.
-
-## How to export optimally
-
-Once you have finished up a texture and are ready to export it, navigate to the file menu > export as... and make sure the file name extention is .png
-
-After clicking 'Export', a menu will appear with a bunch of options checked. Make sure to uncheck all of these options!!! This will drastically reduce the file size from multiple kilobytes to a couple of hundred bytes. Finally click 'Export' one more time.
-
-### Further optimization with OptiPNG
-For those running a GNU/linux distribution, you most likely have the 'optipng' command available to you. If it does not come with your system by default, the software homepage can be found [here](https://optipng.sourceforge.net/) where you can download and install from source.
-
-First, Open up the terminal in the directory where your exported texture is located (or navigate to the directory with the 'cd your/directory/path/to/textures'), then run this command
-```
-optipng -o7 -zm1-9 -nc -clobber -strip all *.png
-```
-This will further optimize all the textures in the directory.
-
-NOTE: If you would like to further edit a texture that has been optipng'd in GIMP, you must manually set the color palette back to RBG after opening. Navigate to Image menu > Mode > select RGB

game.conf

@@ -1,4 +1,4 @@
 title = MineClone 2
 description = A survival sandbox game. Survive, gather, hunt, build, explore, and do much more.
 disallowed_mapgens = v6
-version=0.84.0-SNAPSHOT
+version=0.82.0-SNAPSHOT

mods/CORE/_mcl_autogroup/init.lua

@@ -215,10 +215,6 @@ function mcl_autogroup.can_harvest(nodename, toolname, player)
 		return true
 	end
 
-	if minetest.get_item_group(nodename, "dig_immediate_piston") >= 1 then
-		return true
-	end
-
 	-- Check if it can be dug by tool
 	local tdef = minetest.registered_tools[toolname]
 	if tdef and tdef._mcl_diggroups then
@@ -304,10 +300,6 @@ end
 -- loading order.
 function mcl_autogroup.get_wear(toolname, diggroup)
 	local tdef = minetest.registered_tools[toolname]
-	if not tdef then
-		minetest.log("warning", "Adding wear for tool: " .. tostring(toolname) .. " failed with diggroup: " .. tostring(diggroup))
-		return nil
-	end
 	local uses = tdef._mcl_diggroups[diggroup].uses
 	return math.ceil(65535 / uses)
 end
@@ -370,6 +362,12 @@ local function overwrite()
 			minetest.override_item(tname, {
 				tool_capabilities = toolcaps
 			})
+		else
+			-- This is needed to deal damage when punching mobs
+			-- with random items in hand in survival mode
+			minetest.override_item(tname, {
+				tool_capabilities = mcl_meshhand.survival_hand_tool_caps
+			})
 		end
 	end
 end

mods/CORE/mcl_explosions/init.lua

@@ -13,7 +13,6 @@ under the LGPLv2.1 license.
 mcl_explosions = {}
 
 local mod_fire = minetest.get_modpath("mcl_fire")
-local explosions_griefing = minetest.settings:get_bool("mcl_explosions_griefing", true)
 --local CONTENT_FIRE = minetest.get_content_id("mcl_fire:fire")
 
 local math = math
@@ -192,7 +191,7 @@ local function trace_explode(pos, strength, raydirs, radius, info, direct, sourc
 	local grief_protected = info.grief_protected
 
 	-- Trace rays for environment destruction
-	if info.griefing and explosions_griefing then
+	if info.griefing then
 		for i = 1, #raydirs do
 			local rpos_x = pos.x
 			local rpos_y = pos.y
@@ -352,23 +351,6 @@ local function trace_explode(pos, strength, raydirs, radius, info, direct, sourc
 				end
 			end
 		end
-
-		-- Punch End Crystals to make them explode
-		if ent and ent.name == "mcl_end:crystal" then
-			if direct then
-				local puncher = direct:get_luaentity()
-				if puncher and puncher.name == "mcl_end:crystal" then
-					ent.object:punch(direct, 1.0, { -- End Crystal nearby, trigger it.
-						full_punch_interval = 1.0,
-							damage_groups = {fleshy = 1},
-						}, nil, nil)
-				else
-					ent.object:remove() -- Direct Exists, but it is not an end crystal, remove crystal.
-				end
-			else
-				ent.object:remove() -- Node exploded the end crystal, remove it.
-			end
-		end
 	end
 
 	local airs, fires = {}, {}

mods/CORE/mcl_explosions/locale/mcl_explosions.fr.tr

@@ -1,2 +0,0 @@
-# textdomain:mcl_explosions
-@1 was caught in an explosion.=@1 est mort dans une explosion

mods/CORE/mcl_explosions/locale/template.txt

@@ -1,2 +0,0 @@
-# textdomain:mcl_explosions
-@1 was caught in an explosion.=

mods/CORE/mcl_oxidation/README.md

@@ -1,14 +0,0 @@
-# Oxidization API for MineClone 2
-This mods adds the oxidization api, so that modders can easily use the same features that copper uses.
-
-## API
-To take advantage of the actual oxidization, put `oxidizable = 1` into the list of groups for the oxidizable node.
-You would also need to put `_mcl_oxidized_variant = itemstring of node this node will oxidize into` into the node definition.
-For example, a copper block oxidizes into exposed copper, so the defintion would be `_mcl_oxidized_variant = "mcl_copper:block_exposed"`.
-
-To utilize the ability to wax the block for protection from oxidization, put `mcl_waxed_variant = item string of waxed variant of node` into the node definition table.
-For example, Copper Blocks have the definition arguement of `_mcl_waxed_variant = "mcl_copper:waxed_block"`.
-
-For waxed nodes, scraping is easy. Start by putting `waxed = 1` into the list of groups of the waxed node.
-Next put `_mcl_stripped_variant = item string of the unwaxed variant of the node` into the defintion table.
-Wxaed Copper Blocks can be scrapped into normal Copper Blocks because of the definition `_mcl_stripped_variant = "mcl_copper:block"`.

mods/CORE/mcl_oxidation/init.lua

@@ -1,12 +0,0 @@
-minetest.register_abm({
-	label = "Oxidatize Nodes",
-	nodenames = { "group:oxidizable" },
-	interval = 500,
-	chance = 3,
-	action = function(pos, node)
-		local def = minetest.registered_nodes[node.name]
-		if def and def._mcl_oxidized_variant then
-			minetest.set_node(pos, { name = def._mcl_oxidized_variant, param2 = node.param2 })
-		end
-	end,
-})

mods/CORE/mcl_oxidation/mod.conf

@@ -1,4 +0,0 @@
-name = mcl_oxidation
-title = Oxidation API for MineClone 2
-author = PrairieWind, N011, Michael
-description = API to allow oxidizing different nodes.

mods/CORE/mcl_oxidization/API.md

@@ -0,0 +1,44 @@
+# Mineclone Oxidization API
+This document explains the API of this mod.
+
+### `register_oxidation_abm(node_name)`
+Registers the ABM for the oxidization of nodes. It expects that the variable
+`_mcl_oxidized_variant` be set with the node name of the oxidized version. 
+
+#### Parameters:
+`node_name`: the name of the node to check, and to oxidize. 
+
+#### Usage:
+To use this API, add `_mcl_oxidized_variant = my_oxidized_node_name,` to the node 
+definition of the desired node, and then call 
+`register_oxidation_abm(my_oxidizable_node_abm, my_oxidizable_node)` in your code.
+This abm will swap out the nodes with the more oxidized version of the node, one 
+stage at a time.
+
+#### Example of Usage:
+From mcl_copper:
+```lua 
+local block_oxidation = {
+    { "", "_exposed" },
+    { "_cut", "_exposed_cut" },
+    { "_exposed", "_weathered" },
+    { "_exposed_cut", "_weathered_cut" },
+    { "_weathered", "_oxidized" },
+    { "_weathered_cut", "_oxidized_cut" }
+}
+
+for _, b in pairs(block_oxidation) do
+    register_oxidation_abm("mcl_copper:block" .. b[1], "mcl_copper:block" .. b[2])
+end
+```
+
+### Oxidization Removal
+Make sure that the Oxidized Node has this in its definition:
+
+`_mcl_stripped_variant = my_less_oxidized_node,`
+
+And axes in mineclone will scrape the oxidization level down, usually by one stage.
+An example of usage: `_mcl_stripped_variant = "mcl_copper:block",`
+
+Implementation of other tools for scraping does not yet exist, but may in the future.
+

mods/CORE/mcl_oxidization/init.lua

@@ -0,0 +1,20 @@
+---
+--- Generated by EmmyLua(https://github.com/EmmyLua)
+--- Created by michieal.
+--- DateTime: 2/15/23 1:11 AM
+---
+
+function register_oxidation_abm(node_name)
+	minetest.register_abm({
+		label = node_name .. "_oxidization_abm",
+		nodenames = { node_name },
+		interval = 500,
+		chance = 3,
+		action = function(pos, node)
+			local def = minetest.registered_nodes[node_name]
+			if def and def._mcl_oxidized_variant then
+				minetest.swap_node(pos, { name = oxidized_variant, param2 = node.param2 })
+			end
+		end,
+	})
+end

mods/CORE/mcl_oxidization/mod.conf

@@ -0,0 +1,4 @@
+title = Oxidization API for Mineclone 2
+name = mcl_oxidization
+author = Michieal, NO11
+description = Turns NO11's oxidization function into an API.

mods/CORE/mcl_util/init.lua

@@ -34,64 +34,6 @@ function mcl_util.mcl_log(message, module, bypass_default_logger)
 	end
 end
 
-local player_timers = {}
-
--- This is a dtime timer than can be used in on_step functions so it works every x seconds
--- self - Object you want to store timer data on. E.g. mob or a minecart, or player_name
--- dtime - The time since last run of on_step, should be passed in to function
--- timer_name - This is the name of the timer and also the key to store the data. No spaces + lowercase.
--- threshold - The time before it returns successful. 0.2 if you want to run it 5 times a second.
-function mcl_util.check_dtime_timer(self, dtime, timer_name, threshold)
-	if not self or not threshold or not dtime then return end
-	if not timer_name or timer_name == "" then return end
-
-	if type(self) == "string" then
-		local player_name = self
-		if not player_timers[player_name] then
-			player_timers[player_name] = {}
-		end
-		self = player_timers[player_name]
-	end
-
-	if not self._timers then
-		self._timers = {}
-	end
-
-	if not self._timers[timer_name] then
-		self._timers[timer_name] = 0
-	else
-		self._timers[timer_name] = self._timers[timer_name] + dtime
-		--minetest.log("dtime: " .. tostring(self._timers[timer_name]))
-	end
-
-	if self._timers[timer_name] > threshold then
-		--minetest.log("Over threshold")
-		self._timers[timer_name] = 0
-		return true
-		--else
-		--minetest.log("Not over threshold")
-	end
-	return false
-end
-
--- Minetest 5.3.0 or less can only measure the light level. This came in at 5.4
--- This function has been known to fail in multiple places so the error handling is added increase safety and improve
--- debugging. See:
--- https://git.minetest.land/MineClone2/MineClone2/issues/1392
-function mcl_util.get_natural_light (pos, time)
-	local status, retVal = pcall(minetest.get_natural_light, pos, time)
-	if status then
-		return retVal
-	else
-		minetest.log("warning", "Failed to get natural light at pos: " .. dump(pos) .. ", time: " .. dump(time))
-		if (pos) then
-			local node = minetest.get_node(pos)
-			minetest.log("warning", "Node at pos: " .. dump(node.name))
-		end
-	end
-	return 0
-end
-
 function mcl_util.file_exists(name)
 	if type(name) ~= "string" then return end
 	local f = io.open(name)
@@ -1075,29 +1017,18 @@ function mcl_util.check_position_protection(position, player)
 	return false
 end
 
-local palette_indexes = {grass_palette_index = 0, foliage_palette_index = 0, water_palette_index = 0}
+local palette_indexes = {grass_palette_index = 0, foliage_palette_index = 0}
 function mcl_util.get_palette_indexes_from_pos(pos)
 	local biome_data = minetest.get_biome_data(pos)
 	local biome = biome_data.biome
 	local biome_name = minetest.get_biome_name(biome)
 	local reg_biome = minetest.registered_biomes[biome_name]
-	if reg_biome and reg_biome._mcl_grass_palette_index and reg_biome._mcl_foliage_palette_index and reg_biome._mcl_water_palette_index then
+	if reg_biome and reg_biome._mcl_grass_palette_index and reg_biome._mcl_foliage_palette_index then
 		local gpi = reg_biome._mcl_grass_palette_index
 		local fpi = reg_biome._mcl_foliage_palette_index
-		local wpi = reg_biome._mcl_water_palette_index
+		local palette_indexes = {grass_palette_index = gpi, foliage_palette_index = fpi}
-		local palette_indexes = {grass_palette_index = gpi, foliage_palette_index = fpi, water_palette_index = wpi}
 		return palette_indexes
 	else
 		return palette_indexes
 	end
 end
-
-function mcl_util.get_colorwallmounted_rotation(pos)
-	local colorwallmounted_node = minetest.get_node(pos)
-	for i = 0, 32, 1 do
-		local colorwallmounted_rotation = colorwallmounted_node.param2 - (i * 8)
-		if colorwallmounted_rotation < 6 then
-			return colorwallmounted_rotation
-		end
-	end
-end

mods/ENTITIES/mcl_boats/init.lua

@@ -437,9 +437,9 @@ cboat.selectionbox = {-0.7, -0.15, -0.7, 0.7, 0.75, 0.7}
 minetest.register_entity("mcl_boats:chest_boat", cboat)
 mcl_entity_invs.register_inv("mcl_boats:chest_boat","Boat",27)
 
-local boat_ids = { "boat", "boat_spruce", "boat_birch", "boat_jungle", "boat_acacia", "boat_dark_oak", "boat_obsidian", "boat_mangrove", "boat_cherry", "chest_boat", "chest_boat_spruce", "chest_boat_birch", "chest_boat_jungle", "chest_boat_acacia", "chest_boat_dark_oak", "chest_boat_mangrove", "chest_boat_cherry" }
+local boat_ids = { "boat", "boat_spruce", "boat_birch", "boat_jungle", "boat_acacia", "boat_dark_oak", "boat_obsidian", "boat_mangrove", "chest_boat", "chest_boat_spruce", "chest_boat_birch", "chest_boat_jungle", "chest_boat_acacia", "chest_boat_dark_oak", "chest_boat_mangrove" }
-local names = { S("Oak Boat"), S("Spruce Boat"), S("Birch Boat"), S("Jungle Boat"), S("Acacia Boat"), S("Dark Oak Boat"), S("Obsidian Boat"), S("Mangrove Boat"), S("Cherry Boat"), S("Oak Chest Boat"), S("Spruce Chest Boat"), S("Birch Chest Boat"), S("Jungle Chest Boat"), S("Acacia Chest Boat"), S("Dark Oak Chest Boat"), S("Mangrove Chest Boat"), S("Cherry Chest Boat") }
+local names = { S("Oak Boat"), S("Spruce Boat"), S("Birch Boat"), S("Jungle Boat"), S("Acacia Boat"), S("Dark Oak Boat"), S("Obsidian Boat"), S("Mangrove Boat"), S("Oak Chest Boat"), S("Spruce Chest Boat"), S("Birch Chest Boat"), S("Jungle Chest Boat"), S("Acacia Chest Boat"), S("Dark Oak Chest Boat"), S("Mangrove Chest Boat") }
-local craftstuffs = { "mcl_core:wood", "mcl_core:sprucewood", "mcl_core:birchwood", "mcl_core:junglewood", "mcl_core:acaciawood", "mcl_core:darkwood", "mcl_core:obsidian", "mcl_mangrove:mangrove_wood", "mcl_cherry_blossom:cherrywood" }
+local craftstuffs = { "mcl_core:wood", "mcl_core:sprucewood", "mcl_core:birchwood", "mcl_core:junglewood", "mcl_core:acaciawood", "mcl_core:darkwood", "mcl_core:obsidian", "mcl_mangrove:mangrove_wood" }
 
 for b=1, #boat_ids do
 	local itemstring = "mcl_boats:"..boat_ids[b]

mods/ENTITIES/mcl_boats/locale/mcl_boats.dk.tr

@@ -1,13 +0,0 @@
-# textdomain: mcl_boats
-Acacia Boat=Akaciebåd
-Birch Boat=Birkebåd
-Boat=Båd
-Boats are used to travel on the surface of water.=Både blier brugt til at rejse på vandoverflader.
-Dark Oak Boat=Mørk egetræsbåd
-Jungle Boat=Junglebåd
-Oak Boat=Egetræsbåd
-Rightclick on a water source to place the boat. Rightclick the boat to enter it. Use [Left] and [Right] to steer, [Forwards] to speed up and [Backwards] to slow down or move backwards. Use [Sneak] to leave the boat, punch the boat to make it drop as an item.=Højre-klik på en vand for at placere båden. Højre-klik på båden for at gå ombord. Brug [Left] og [Right] til at styre. [Forwards] for at øge hastigheden, og [Backwards] for at sænke farten eller sejle bagud. Brug [Sneak] for at forlade båden, slå båden for at lave den om til en genstand.
-Spruce Boat=Granbåd
-Water vehicle=Vandfartøj
-Sneak to dismount=Snig for at stige ud
-Obsidian Boat=Obsidianbåd

mods/ENTITIES/mcl_boats/locale/mcl_boats.es.tr

@@ -8,6 +8,3 @@ Jungle Boat=Barca de la selva
 Oak Boat=Barca de roble
 Rightclick on a water source to place the boat. Rightclick the boat to enter it. Use [Left] and [Right] to steer, [Forwards] to speed up and [Backwards] to slow down or move backwards. Rightclick the boat again to leave it, punch the boat to make it drop as an item.=Haga clic derecho en una fuente de agua para colocar el barco. Haga clic derecho en el barco para entrar. Utilice [Izquierda] y [Derecha] para dirigir, [Adelante] para acelerar y [Atrás] para reducir la velocidad o retroceder. Haga clic derecho en el barco nuevamente para dejarlo, golpee el barco para que se caiga como un artículo.
 Spruce Boat=Barca de abeto
-Water vehicle=Vehículo acuático
-Sneak to dismount=Agáchate para bajar
-Obsidian Boat=Barca de obsidiana

mods/ENTITIES/mcl_boats/locale/mcl_boats.fr.tr

@@ -1,21 +1,13 @@
 # textdomain: mcl_boats
-Acacia Boat=Bateau en acacia
+Acacia Boat=Bateau en Acacia
-Birch Boat=Bateau en bouleau
+Birch Boat=Bateau en Bouleau
 Boat=Bateau
 Boats are used to travel on the surface of water.=Les bateaux sont utilisés pour voyager à la surface de l'eau.
-Dark Oak Boat=Bateau en chêne noir
+Dark Oak Boat=Bateau en Chêne Noir
-Jungle Boat=Bateau en acajou
+Jungle Boat=Bateau en Acajou
-Oak Boat=Bateau en chêne
+Oak Boat=Bateau en Chêne
 Rightclick on a water source to place the boat. Rightclick the boat to enter it. Use [Left] and [Right] to steer, [Forwards] to speed up and [Backwards] to slow down or move backwards. Use [Sneak] to leave the boat, punch the boat to make it drop as an item.=Faites un clic droit sur une source d'eau pour placer le bateau. Faites un clic droit sur le bateau pour y entrer. Utilisez [Gauche] et [Droite] pour diriger, [Avant] pour accélérer et [Arrière] pour ralentir ou reculer. Utilisez [Sneak] pour le quitter, frappez le bateau pour le faire tomber en tant qu'objet.
-Spruce Boat=Bateau en sapin
+Spruce Boat=Bateau en Sapin
 Water vehicle=Véhicule aquatique
 Sneak to dismount=Se baisser pour descendre
-Obsidian Boat=Bateau en obsidienne
+Obsidian Boat=Bateau en Obsidienne
-Mangrove Boat=Bateau en palétuvier
-Oak Chest Boat=Bateau en chêne avec coffre
-Spruce Chest Boat=Bateau en sapin avec coffre
-Birch Chest Boat=Bateau en bouleau avec coffre
-Jungle Chest Boat=Bateau en acajou avec coffre
-Acacia Chest Boat=Bateau en acacia avec coffre
-Dark Oak Chest Boat=Bateau en chêne noir avec coffre
-Mangrove Chest Boat=Bateau en palétuvier avec coffre

mods/ENTITIES/mcl_boats/locale/template.txt

@@ -10,14 +10,4 @@ Rightclick on a water source to place the boat. Rightclick the boat to enter it.
 Spruce Boat=
 Water vehicle=
 Sneak to dismount=
-Obsidian Boat=
+Obsidian Boat=
-Mangrove Boat=
-Cherry Boat=
-Oak Chest Boat=
-Spruce Chest Boat=
-Birch Chest Boat=
-Jungle Chest Boat=
-Acacia Chest Boat=
-Dark Oak Chest Boat=
-Mangrove Chest Boat=
-Cherry Chest Boat=

mods/ENTITIES/mcl_burning/api.lua

@@ -5,12 +5,7 @@ function mcl_burning.get_storage(obj)
 end
 
 function mcl_burning.is_burning(obj)
-	local storage = mcl_burning.get_storage(obj)
+	return mcl_burning.get_storage(obj).burn_time
-	if storage then
-		return mcl_burning.get_storage(obj).burn_time
-	else
-		return false
-	end
 end
 
 function mcl_burning.is_affected_by_rain(obj)
@@ -158,11 +153,6 @@ function mcl_burning.extinguish(obj)
 end
 
 function mcl_burning.tick(obj, dtime, storage)
-	if not storage then
-		minetest.log("warning", "No storage for burning tick. Should not happen: " .. dump(obj))
-		return
-	end
-
 	if storage.burn_time then
 		storage.burn_time = storage.burn_time - dtime
 

mods/ENTITIES/mcl_falling_nodes/locale/mcl_falling_nodes.dk.tr

@@ -1,3 +0,0 @@
-# textdomain: mcl_falling_nodes
-@1 was smashed by a falling anvil.=@1 blev smadret af en nedfaldende ambolt.
-@1 was smashed by a falling block.=@1 blev smadret af en nedfaldende blok.

mods/ENTITIES/mcl_falling_nodes/locale/mcl_falling_nodes.es.tr

@@ -1,3 +0,0 @@
-# textdomain: mcl_falling_nodes
-@1 was smashed by a falling anvil.=@1 fue aplastado por un yunque.
-@1 was smashed by a falling block.=@1 fue aplastado por un bloque.

mods/ENTITIES/mcl_falling_nodes/locale/mcl_falling_nodes.fr.tr

@@ -1,3 +0,0 @@
-# textdomain: mcl_falling_nodes
-@1 was smashed by a falling anvil.=@1 a été écrasé par une enclume
-@1 was smashed by a falling block.=@1 a été écrasé par un bloc

mods/ENTITIES/mcl_falling_nodes/locale/template.txt

@@ -1,3 +0,0 @@
-# textdomain: mcl_falling_nodes
-@1 was smashed by a falling anvil.=
-@1 was smashed by a falling block.=

mods/ENTITIES/mcl_item_entity/init.lua

@@ -7,7 +7,12 @@ local pool = {}
 local tick = false
 
 
-
+local LOGGING_ON = minetest.settings:get_bool("mcl_logging_item_entities", false)
+local function mcl_log(message)
+	if LOGGING_ON then
+		mcl_util.mcl_log(message, "[Item Entities]", true)
+	end
+end
 
 minetest.register_on_joinplayer(function(player)
 	pool[player:get_player_name()] = 0
@@ -403,176 +408,114 @@ local function cxcz(o, cw, one, zero)
 	return o
 end
 
-local function nodes_destroy_items (self, moveresult, def, nn)
+local function hopper_take_item(self, pos)
-	local lg = minetest.get_item_group(nn, "lava")
+	--mcl_log("self.itemstring: ".. self.itemstring)
-	local fg = minetest.get_item_group(nn, "fire")
+	--mcl_log("self.itemstring: ".. minetest.pos_to_string(pos))
-	local dg = minetest.get_item_group(nn, "destroys_items")
 
-	if (def and (lg ~= 0 or fg ~= 0 or dg == 1)) then
+	local objs = minetest.get_objects_inside_radius(pos, 2)
-		local item_string = self.itemstring
-		local item_name = ItemStack(item_string):get_name()
 
-		--Wait 2 seconds to allow mob drops to be cooked, & picked up instead of instantly destroyed.
+	if objs and self.itemstring then
-		if self.age > 2 and minetest.get_item_group(item_name, "fire_immune") == 0 then
+		--mcl_log("there is an itemstring. Number of objs: ".. #objs)
-			if dg ~= 2 then
-				minetest.sound_play("builtin_item_lava", { pos = self.object:get_pos(), gain = 0.5 })
-			end
-			self._removed = true
-			self.object:remove()
-			return true
-		end
-	end
 
-	-- Destroy item when it collides with a cactus
+		for k, v in pairs(objs) do
-	if moveresult and moveresult.collides then
+			local ent = v:get_luaentity()
-		for _, collision in pairs(moveresult.collisions) do
+
-			local pos = collision.node_pos
+			-- Don't forget actual hoppers
-			if collision.type == "node" and minetest.get_node(pos).name == "mcl_core:cactus" then
+			if ent and ent.name == "mcl_minecarts:hopper_minecart" then
-				-- TODO We need to play a sound when it gets destroyed
+				local taken_items = false
-				self._removed = true
+
-				self.object:remove()
+				mcl_log("ent.name: " .. tostring(ent.name))
-				return true
+				mcl_log("ent pos: " .. tostring(ent.object:get_pos()))
+
+				local inv = mcl_entity_invs.load_inv(ent, 5)
+
+				if not inv then
+					mcl_log("No inv")
+					return false
+				end
+
+				local current_itemstack = ItemStack(self.itemstring)
+
+				mcl_log("inv. size: " .. ent._inv_size)
+				if inv:room_for_item("main", current_itemstack) then
+					mcl_log("Room")
+					inv:add_item("main", current_itemstack)
+					self.object:get_luaentity().itemstring = ""
+					self.object:remove()
+					taken_items = true
+				else
+					mcl_log("no Room")
+				end
+
+				if not taken_items then
+					local items_remaining = current_itemstack:get_count()
+
+					-- This will take part of a floating item stack if no slot can hold the full amount
+					for i = 1, ent._inv_size, 1 do
+						local stack = inv:get_stack("main", i)
+
+						mcl_log("i: " .. tostring(i))
+						mcl_log("Items remaining: " .. items_remaining)
+						mcl_log("Name: " .. tostring(stack:get_name()))
+
+						if current_itemstack:get_name() == stack:get_name() then
+							mcl_log("We have a match. Name: " .. tostring(stack:get_name()))
+
+							local room_for = stack:get_stack_max() - stack:get_count()
+							mcl_log("Room for: " .. tostring(room_for))
+
+							if room_for == 0 then
+								-- Do nothing
+								mcl_log("No room")
+							elseif room_for < items_remaining then
+								mcl_log("We have more items remaining than space")
+
+								items_remaining = items_remaining - room_for
+								stack:set_count(stack:get_stack_max())
+								inv:set_stack("main", i, stack)
+								taken_items = true
+							else
+								local new_stack_size = stack:get_count() + items_remaining
+								stack:set_count(new_stack_size)
+								mcl_log("We have more than enough space. Now holds: " .. new_stack_size)
+
+								inv:set_stack("main", i, stack)
+								items_remaining = 0
+
+								self.object:get_luaentity().itemstring = ""
+								self.object:remove()
+
+								taken_items = true
+								break
+							end
+
+							mcl_log("Count: " .. tostring(stack:get_count()))
+							mcl_log("stack max: " .. tostring(stack:get_stack_max()))
+							--mcl_log("Is it empty: " .. stack:to_string())
+						end
+
+						if i == ent._inv_size and taken_items then
+							mcl_log("We are on last item and still have items left. Set final stack size: " .. items_remaining)
+							current_itemstack:set_count(items_remaining)
+							--mcl_log("Itemstack2: " .. current_itemstack:to_string())
+							self.itemstring = current_itemstack:to_string()
+						end
+					end
+				end
+
+				--Add in, and delete
+				if taken_items then
+					mcl_log("Saving")
+					mcl_entity_invs.save_inv(ent)
+					return taken_items
+				else
+					mcl_log("No need to save")
+				end
 			end
 		end
 	end
-end
 
-local function push_out_item_stuck_in_solid(self, dtime, p, def, is_in_water)
+	return false
-	if not is_in_water and def and def.walkable and def.groups and def.groups.opaque == 1 then
-		local shootdir
-		local cx = (p.x % 1) - 0.5
-		local cz = (p.z % 1) - 0.5
-		local order = {}
-
-		-- First prepare the order in which the 4 sides are to be checked.
-		-- 1st: closest
-		-- 2nd: other direction
-		-- 3rd and 4th: other axis
-		if math.abs(cx) < math.abs(cz) then
-			order = cxcz(order, cx, "x", "z")
-			order = cxcz(order, cz, "z", "x")
-		else
-			order = cxcz(order, cz, "z", "x")
-			order = cxcz(order, cx, "x", "z")
-		end
-
-		-- Check which one of the 4 sides is free
-		for o = 1, #order do
-			local nn = minetest.get_node(vector.add(p, order[o])).name
-			local def = minetest.registered_nodes[nn]
-			if def and def.walkable == false and nn ~= "ignore" then
-				shootdir = order[o]
-				break
-			end
-		end
-		-- If none of the 4 sides is free, shoot upwards
-		if shootdir == nil then
-			shootdir = vector.new(0, 1, 0)
-			local nn = minetest.get_node(vector.add(p, shootdir)).name
-			if nn == "ignore" then
-				-- Do not push into ignore
-				return true
-			end
-		end
-
-		-- Set new item moving speed accordingly
-		local newv = vector.multiply(shootdir, 3)
-		self.object:set_acceleration(vector.zero())
-		self.object:set_velocity(newv)
-		disable_physics(self.object, self, false, false)
-
-
-		if shootdir.y == 0 then
-			self._force = newv
-			p.x = math.floor(p.x)
-			p.y = math.floor(p.y)
-			p.z = math.floor(p.z)
-			self._forcestart = p
-			self._forcetimer = 1
-		end
-		return true
-	end
-
-	-- This code is run after the entity got a push from above “push away” code.
-	-- It is responsible for making sure the entity is entirely outside the solid node
-	-- (with its full collision box), not just its center.
-	if self._forcetimer > 0 then
-		local cbox = self.object:get_properties().collisionbox
-		local ok = false
-		if self._force.x > 0 and (p.x > (self._forcestart.x + 0.5 + (cbox[4] - cbox[1]) / 2)) then ok = true
-		elseif self._force.x < 0 and (p.x < (self._forcestart.x + 0.5 - (cbox[4] - cbox[1]) / 2)) then ok = true
-		elseif self._force.z > 0 and (p.z > (self._forcestart.z + 0.5 + (cbox[6] - cbox[3]) / 2)) then ok = true
-		elseif self._force.z < 0 and (p.z < (self._forcestart.z + 0.5 - (cbox[6] - cbox[3]) / 2)) then ok = true end
-		-- Item was successfully forced out. No more pushing
-		if ok then
-			self._forcetimer = -1
-			self._force = nil
-			enable_physics(self.object, self)
-		else
-			self._forcetimer = self._forcetimer - dtime
-		end
-		return true
-	elseif self._force then
-		self._force = nil
-		enable_physics(self.object, self)
-		return true
-	end
-end
-
-local function move_items_in_water (self, p, def, node, is_floating, is_in_water)
-	-- Move item around on flowing liquids; add 'source' check to allow items to continue flowing a bit in the source block of flowing water.
-	if def and not is_floating and (def.liquidtype == "flowing" or def.liquidtype == "source") then
-		self._flowing = true
-
-		--[[ Get flowing direction (function call from flowlib), if there's a liquid.
-        NOTE: According to Qwertymine, flowlib.quickflow is only reliable for liquids with a flowing distance of 7.
-        Luckily, this is exactly what we need if we only care about water, which has this flowing distance. ]]
-		local vec = flowlib.quick_flow(p, node)
-		-- Just to make sure we don't manipulate the speed for no reason
-		if vec.x ~= 0 or vec.y ~= 0 or vec.z ~= 0 then
-			-- Minecraft Wiki: Flowing speed is "about 1.39 meters per second"
-			local f = 1.2
-			-- Set new item moving speed into the direciton of the liquid
-			local newv = vector.multiply(vec, f)
-			-- Swap to acceleration instead of a static speed to better mimic MC mechanics.
-			self.object:set_acceleration(vector.new(newv.x, -0.22, newv.z))
-
-			self.physical_state = true
-			self._flowing = true
-			self.object:set_properties({
-				physical = true
-			})
-			return true
-		end
-		if is_in_water and def.liquidtype == "source" then
-			local cur_vec = self.object:get_velocity()
-			-- apply some acceleration in the opposite direction so it doesn't slide forever
-			local vec = {
-				x = 0 - cur_vec.x * 0.9,
-				y = 3 - cur_vec.y * 0.9,
-				z = 0 - cur_vec.z * 0.9
-			}
-			self.object:set_acceleration(vec)
-			-- slow down the item in water
-			local vel = self.object:get_velocity()
-			if vel.y < 0 then
-				vel.y = vel.y * 0.9
-			end
-			self.object:set_velocity(vel)
-			if self.physical_state ~= false or self._flowing ~= true then
-				self.physical_state = true
-				self._flowing = true
-				self.object:set_properties({
-					physical = true
-				})
-			end
-		end
-	elseif self._flowing == true and not is_in_water and not is_floating then
-		-- Disable flowing physics if not on/in flowing liquid
-		self._flowing = false
-		enable_physics(self.object, self, true)
-		return true
-	end
 end
 
 minetest.register_entity(":__builtin:item", {
@@ -621,17 +564,13 @@ minetest.register_entity(":__builtin:item", {
 		if speed ~= nil then self.random_velocity = speed end
 
 		local vel = self.object:get_velocity()
-
-		-- There is perhaps a cleverer way of making this physical so it bounces off the wall like swords.
-		local max_vel = 6.5 -- Faster than this and it throws it into the wall / floor and turns black because of clipping.
-
 		if vel and vel.x == 0 and vel.z == 0 and self.random_velocity > 0 then
 			local v = self.random_velocity
-			local x = math.random(5, max_vel) / 10 * v
+			local x = math.random(5, 10) / 10 * v
 			if math.random(0, 10) < 5 then x = -x end
-			local z = math.random(5, max_vel) / 10 * v
+			local z = math.random(5, 10) / 10 * v
 			if math.random(0, 10) < 5 then z = -z end
-			local y = math.random(1, 2)
+			local y = math.random(2, 4)
 			self.object:set_velocity(vector.new(x, y, z))
 		end
 		self.random_velocity = 0
@@ -808,19 +747,11 @@ minetest.register_entity(":__builtin:item", {
 		if total_count > max_count then
 			return false
 		end
-
 		-- Merge the remote stack into this one
-		local self_pos = self.object:get_pos()
-		local pos = object:get_pos()
 
-		--local y = pos.y + ((total_count - count) / max_count) * 0.15
+		-- local pos = object:get_pos()
-		local x_diff = (self_pos.x - pos.x) / 2
+		-- pos.y = pos.y + ((total_count - count) / max_count) * 0.15
-		local z_diff = (self_pos.z - pos.z) / 2
+		-- self.object:move_to(pos)
-
-		local new_pos = vector.offset(pos, x_diff, 0, z_diff)
-		new_pos.y = math.max(self_pos.y, pos.y) + 0.1
-
-		self.object:move_to(new_pos)
 
 		self.age = 0 -- Handle as new entity
 		own_stack:set_count(total_count)
@@ -841,7 +772,6 @@ minetest.register_entity(":__builtin:item", {
 			self.object:set_acceleration(vector.zero())
 			return
 		end
-
 		self.age = self.age + dtime
 		if self._collector_timer then
 			self._collector_timer = self._collector_timer + dtime
@@ -855,13 +785,19 @@ minetest.register_entity(":__builtin:item", {
 		-- otherwise there might have some data corruption.
 		if self.itemstring == "" then
 			minetest.log("warning",
-				"Item entity with empty itemstring found and being deleted at: " .. minetest.pos_to_string(self.object:get_pos()))
+				"Item entity with empty itemstring found at " .. minetest.pos_to_string(self.object:get_pos()) ..
+				"! Deleting it now.")
 			self._removed = true
 			self.object:remove()
 			return
 		end
 
 		local p = self.object:get_pos()
+		-- If hopper has taken item, it has gone, and no operations should be conducted on this item
+		if hopper_take_item(self, p) then
+			return
+		end
+
 		local node = minetest.get_node(p)
 		local in_unloaded = node.name == "ignore"
 
@@ -871,9 +807,6 @@ minetest.register_entity(":__builtin:item", {
 			return
 		end
 
-
-
-
 		if self.is_clock then
 			self.object:set_properties({
 				textures = { "mcl_clock:clock_" .. (mcl_worlds.clock_works(p) and mcl_clock.old_time or mcl_clock.random_frame) }
@@ -909,12 +842,167 @@ minetest.register_entity(":__builtin:item", {
 		-- Destroy item in lava, fire or special nodes
 
 		local def = minetest.registered_nodes[nn]
+		local lg = minetest.get_item_group(nn, "lava")
+		local fg = minetest.get_item_group(nn, "fire")
+		local dg = minetest.get_item_group(nn, "destroys_items")
+		if (def and (lg ~= 0 or fg ~= 0 or dg == 1)) then
+			--Wait 2 seconds to allow mob drops to be cooked, & picked up instead of instantly destroyed.
+			if self.age > 2 and minetest.get_item_group(self.itemstring, "fire_immune") == 0 then
+				if dg ~= 2 then
+					minetest.sound_play("builtin_item_lava", { pos = self.object:get_pos(), gain = 0.5 })
+				end
+				self._removed = true
+				self.object:remove()
+				return
+			end
+		end
 
-		if nodes_destroy_items(self, moveresult, def, nn) then return end
+		-- Destroy item when it collides with a cactus
+		if moveresult and moveresult.collides then
+			for _, collision in pairs(moveresult.collisions) do
+				local pos = collision.node_pos
+				if collision.type == "node" and minetest.get_node(pos).name == "mcl_core:cactus" then
+					self._removed = true
+					self.object:remove()
+					return
+				end
+			end
+		end
 
-		if push_out_item_stuck_in_solid(self, dtime, p, def, is_in_water) then return end
+		-- Push item out when stuck inside solid opaque node
+		if not is_in_water and def and def.walkable and def.groups and def.groups.opaque == 1 then
+			local shootdir
+			local cx = (p.x % 1) - 0.5
+			local cz = (p.z % 1) - 0.5
+			local order = {}
 
-		if move_items_in_water (self, p, def, node, is_floating, is_in_water) then return end
+			-- First prepare the order in which the 4 sides are to be checked.
+			-- 1st: closest
+			-- 2nd: other direction
+			-- 3rd and 4th: other axis
+			if math.abs(cx) < math.abs(cz) then
+				order = cxcz(order, cx, "x", "z")
+				order = cxcz(order, cz, "z", "x")
+			else
+				order = cxcz(order, cz, "z", "x")
+				order = cxcz(order, cx, "x", "z")
+			end
+
+			-- Check which one of the 4 sides is free
+			for o = 1, #order do
+				local nn = minetest.get_node(vector.add(p, order[o])).name
+				local def = minetest.registered_nodes[nn]
+				if def and def.walkable == false and nn ~= "ignore" then
+					shootdir = order[o]
+					break
+				end
+			end
+			-- If none of the 4 sides is free, shoot upwards
+			if shootdir == nil then
+				shootdir = vector.new(0, 1, 0)
+				local nn = minetest.get_node(vector.add(p, shootdir)).name
+				if nn == "ignore" then
+					-- Do not push into ignore
+					return
+				end
+			end
+
+			-- Set new item moving speed accordingly
+			local newv = vector.multiply(shootdir, 3)
+			self.object:set_acceleration(vector.zero())
+			self.object:set_velocity(newv)
+			disable_physics(self.object, self, false, false)
+
+
+			if shootdir.y == 0 then
+				self._force = newv
+				p.x = math.floor(p.x)
+				p.y = math.floor(p.y)
+				p.z = math.floor(p.z)
+				self._forcestart = p
+				self._forcetimer = 1
+			end
+			return
+		end
+
+		-- This code is run after the entity got a push from above “push away” code.
+		-- It is responsible for making sure the entity is entirely outside the solid node
+		-- (with its full collision box), not just its center.
+		if self._forcetimer > 0 then
+			local cbox = self.object:get_properties().collisionbox
+			local ok = false
+			if self._force.x > 0 and (p.x > (self._forcestart.x + 0.5 + (cbox[4] - cbox[1]) / 2)) then ok = true
+			elseif self._force.x < 0 and (p.x < (self._forcestart.x + 0.5 - (cbox[4] - cbox[1]) / 2)) then ok = true
+			elseif self._force.z > 0 and (p.z > (self._forcestart.z + 0.5 + (cbox[6] - cbox[3]) / 2)) then ok = true
+			elseif self._force.z < 0 and (p.z < (self._forcestart.z + 0.5 - (cbox[6] - cbox[3]) / 2)) then ok = true end
+			-- Item was successfully forced out. No more pushing
+			if ok then
+				self._forcetimer = -1
+				self._force = nil
+				enable_physics(self.object, self)
+			else
+				self._forcetimer = self._forcetimer - dtime
+			end
+			return
+		elseif self._force then
+			self._force = nil
+			enable_physics(self.object, self)
+			return
+		end
+
+		-- Move item around on flowing liquids; add 'source' check to allow items to continue flowing a bit in the source block of flowing water.
+		if def and not is_floating and (def.liquidtype == "flowing" or def.liquidtype == "source") then
+			self._flowing = true
+
+			--[[ Get flowing direction (function call from flowlib), if there's a liquid.
+			NOTE: According to Qwertymine, flowlib.quickflow is only reliable for liquids with a flowing distance of 7.
+			Luckily, this is exactly what we need if we only care about water, which has this flowing distance. ]]
+			local vec = flowlib.quick_flow(p, node)
+			-- Just to make sure we don't manipulate the speed for no reason
+			if vec.x ~= 0 or vec.y ~= 0 or vec.z ~= 0 then
+				-- Minecraft Wiki: Flowing speed is "about 1.39 meters per second"
+				local f = 1.2
+				-- Set new item moving speed into the direciton of the liquid
+				local newv = vector.multiply(vec, f)
+				-- Swap to acceleration instead of a static speed to better mimic MC mechanics.
+				self.object:set_acceleration(vector.new(newv.x, -0.22, newv.z))
+
+				self.physical_state = true
+				self._flowing = true
+				self.object:set_properties({
+					physical = true
+				})
+				return
+			end
+			if is_in_water and def.liquidtype == "source" then
+				local cur_vec = self.object:get_velocity()
+				-- apply some acceleration in the opposite direction so it doesn't slide forever
+				local vec = {
+					x = 0 - cur_vec.x * 0.9,
+					y = 3 - cur_vec.y * 0.9,
+					z = 0 - cur_vec.z * 0.9
+				}
+				self.object:set_acceleration(vec)
+				-- slow down the item in water
+				local vel = self.object:get_velocity()
+				if vel.y < 0 then
+					vel.y = vel.y * 0.9
+				end
+				self.object:set_velocity(vel)
+				if self.physical_state ~= false or self._flowing ~= true then
+					self.physical_state = true
+					self._flowing = true
+					self.object:set_properties({
+						physical = true
+					})
+				end
+			end
+		elseif self._flowing == true and not is_in_water and not is_floating then
+			-- Disable flowing physics if not on/in flowing liquid
+			self._flowing = false
+			enable_physics(self.object, self, true)
+			return
+		end
 
 		-- If node is not registered or node is walkably solid and resting on nodebox
 		local nn = minetest.get_node(vector.offset(p, 0, -0.5, 0)).name
@@ -923,13 +1011,14 @@ minetest.register_entity(":__builtin:item", {
 		local is_on_floor = def and (def.walkable
 			and not def.groups.slippery and v.y == 0)
 
-		if not minetest.registered_nodes[nn] or is_floating or is_on_floor then
+		if not minetest.registered_nodes[nn]
-
+			or is_floating or is_on_floor then
 			local own_stack = ItemStack(self.object:get_luaentity().itemstring)
 			-- Merge with close entities of the same item
 			for _, object in pairs(minetest.get_objects_inside_radius(p, 0.8)) do
 				local obj = object:get_luaentity()
-				if obj and obj.name == "__builtin:item" and obj.physical_state == false then
+				if obj and obj.name == "__builtin:item"
+					and obj.physical_state == false then
 					if self:try_merge_with(own_stack, object, obj) then
 						return
 					end

mods/ENTITIES/mcl_minecarts/init.lua

@@ -11,14 +11,6 @@ mcl_minecarts.check_float_time = 15
 dofile(mcl_minecarts.modpath.."/functions.lua")
 dofile(mcl_minecarts.modpath.."/rails.lua")
 
-local LOGGING_ON = minetest.settings:get_bool("mcl_logging_minecarts", false)
-local function mcl_log(message)
-	if LOGGING_ON then
-		mcl_util.mcl_log(message, "[Minecarts]", true)
-	end
-end
-
-
 local function detach_driver(self)
 	if not self._driver then
 		return
@@ -59,134 +51,6 @@ end
 
 local activate_normal_minecart = detach_driver
 
-local function hopper_take_item(self, dtime)
-	local pos = self.object:get_pos()
-	if not pos then return end
-
-	if not self or self.name ~= "mcl_minecarts:hopper_minecart" then return end
-
-	if mcl_util.check_dtime_timer(self, dtime, "hoppermc_take", 0.15) then
-		--minetest.log("The check timer was triggered: " .. dump(pos) .. ", name:" .. self.name)
-	else
-		--minetest.log("The check timer was not triggered")
-		return
-	end
-
-	--mcl_log("self.itemstring: ".. self.itemstring)
-
-	local above_pos = vector.offset(pos, 0, 0.9, 0)
-	--mcl_log("self.itemstring: ".. minetest.pos_to_string(above_pos))
-	local objs = minetest.get_objects_inside_radius(above_pos, 1.25)
-
-	if objs then
-
-		mcl_log("there is an itemstring. Number of objs: ".. #objs)
-
-		for k, v in pairs(objs) do
-			local ent = v:get_luaentity()
-
-			if ent._removed or not ent.itemstring or ent.itemstring == "" then
-				--minetest.log("Ignore this item")
-				break
-			end
-
-			-- Don't forget actual hoppers
-
-			local taken_items = false
-
-			mcl_log("ent.name: " .. tostring(ent.name))
-			mcl_log("ent pos: " .. tostring(ent.object:get_pos()))
-
-			local inv = mcl_entity_invs.load_inv(self, 5)
-
-			if not inv then
-				mcl_log("No inv")
-				return false
-			end
-
-			local current_itemstack = ItemStack(ent.itemstring)
-
-			mcl_log("inv. size: " .. self._inv_size)
-			if inv:room_for_item("main", current_itemstack) then
-				mcl_log("Room")
-				inv:add_item("main", current_itemstack)
-				ent.object:get_luaentity().itemstring = ""
-				ent.object:remove()
-				taken_items = true
-			else
-				mcl_log("no Room")
-			end
-
-			if not taken_items then
-				local items_remaining = current_itemstack:get_count()
-
-				-- This will take part of a floating item stack if no slot can hold the full amount
-				for i = 1, self._inv_size, 1 do
-					local stack = inv:get_stack("main", i)
-
-					mcl_log("i: " .. tostring(i))
-					mcl_log("Items remaining: " .. items_remaining)
-					mcl_log("Name: " .. tostring(stack:get_name()))
-
-					if current_itemstack:get_name() == stack:get_name() then
-						mcl_log("We have a match. Name: " .. tostring(stack:get_name()))
-
-						local room_for = stack:get_stack_max() - stack:get_count()
-						mcl_log("Room for: " .. tostring(room_for))
-
-						if room_for == 0 then
-							-- Do nothing
-							mcl_log("No room")
-						elseif room_for < items_remaining then
-							mcl_log("We have more items remaining than space")
-
-							items_remaining = items_remaining - room_for
-							stack:set_count(stack:get_stack_max())
-							inv:set_stack("main", i, stack)
-							taken_items = true
-						else
-							local new_stack_size = stack:get_count() + items_remaining
-							stack:set_count(new_stack_size)
-							mcl_log("We have more than enough space. Now holds: " .. new_stack_size)
-
-							inv:set_stack("main", i, stack)
-							items_remaining = 0
-
-							ent.object:get_luaentity().itemstring = ""
-							ent.object:remove()
-
-							taken_items = true
-							break
-						end
-
-						mcl_log("Count: " .. tostring(stack:get_count()))
-						mcl_log("stack max: " .. tostring(stack:get_stack_max()))
-						--mcl_log("Is it empty: " .. stack:to_string())
-					end
-
-					if i == self._inv_size and taken_items then
-						mcl_log("We are on last item and still have items left. Set final stack size: " .. items_remaining)
-						current_itemstack:set_count(items_remaining)
-						--mcl_log("Itemstack2: " .. current_itemstack:to_string())
-						ent.itemstring = current_itemstack:to_string()
-					end
-				end
-			end
-
-			--Add in, and delete
-			if taken_items then
-				mcl_log("Saving")
-				mcl_entity_invs.save_inv(ent)
-				return taken_items
-			else
-				mcl_log("No need to save")
-			end
-		end
-	end
-
-	return false
-end
-
 -- Table for item-to-entity mapping. Keys: itemstring, Values: Corresponding entity ID
 local entity_mapping = {}
 
@@ -202,7 +66,6 @@ local function register_entity(entity_id, mesh, textures, drop, on_rightclick, o
 		on_rightclick = on_rightclick,
 
 		_driver = nil, -- player who sits in and controls the minecart (only for minecart!)
-		_passenger = nil, -- for mobs
 		_punched = false, -- used to re-send _velocity and position
 		_velocity = {x=0, y=0, z=0}, -- only used on punch
 		_start_pos = nil, -- Used to calculate distance for “On A Rail” achievement
@@ -223,7 +86,6 @@ local function register_entity(entity_id, mesh, textures, drop, on_rightclick, o
 		local data = minetest.deserialize(staticdata)
 		if type(data) == "table" then
 			self._railtype = data._railtype
-			self._passenger = data._passenger
 		end
 		self.object:set_armor_groups({immortal=1})
 
@@ -315,11 +177,7 @@ local function register_entity(entity_id, mesh, textures, drop, on_rightclick, o
 
 	cart.on_activate_by_rail = on_activate_by_rail
 
-	local passenger_attach_position = vector.new(0, -1.75, 0)
-
 	function cart:on_step(dtime)
-		hopper_take_item(self, dtime)
-
 		local ctrl, player = nil, nil
 		if self._driver then
 			player = minetest.get_player_by_name(self._driver)
@@ -354,29 +212,6 @@ local function register_entity(entity_id, mesh, textures, drop, on_rightclick, o
 			end
 		end
 
-		-- Grab mob
-		if math.random(1,20) > 15 and not self._passenger then
-			if self.name == "mcl_minecarts:minecart" then
-				local mobsnear = minetest.get_objects_inside_radius(self.object:get_pos(), 1.3)
-				for n=1, #mobsnear do
-					local mob = mobsnear[n]
-					if mob then
-						local entity = mob:get_luaentity()
-						if entity and entity.is_mob then
-							self._passenger = entity
-							mob:set_attach(self.object, "", passenger_attach_position, vector.zero())
-							break
-						end
-					end
-				end
-			end
-		elseif self._passenger then
-			local passenger_pos = self._passenger.object:get_pos()
-			if not passenger_pos then
-				self._passenger = nil
-			end
-		end
-
 		-- Drop minecart if it isn't on a rail anymore
 		if self._last_float_check >= mcl_minecarts.check_float_time then
 			pos = self.object:get_pos()
@@ -401,8 +236,19 @@ local function register_entity(entity_id, mesh, textures, drop, on_rightclick, o
 					return
 				end
 
-				-- Do not drop minecart. It goes off the rails too frequently, and anyone using them for farms won't
+				-- Drop items and remove cart entity
-				-- notice and lose their iron and not bother. Not cool until fixed.
+				local pname = ""
+				if player then
+					pname = player:get_player_name()
+				end
+				if not minetest.is_creative_enabled(pname) then
+					for d=1, #drop do
+						minetest.add_item(self.object:get_pos(), drop[d])
+					end
+				end
+
+				self.object:remove()
+				return
 			end
 			self._last_float_check = 0
 		end

mods/ENTITIES/mcl_minecarts/locale/mcl_minecarts.dk.tr

@@ -1,36 +0,0 @@
-# textdomain: mcl_minecarts
-Minecart=Minevogn
-Minecarts can be used for a quick transportion on rails.=Minevogne kan bruges til hurtig transport på spor.
-Minecarts only ride on rails and always follow the tracks. At a T-junction with no straight way ahead, they turn left. The speed is affected by the rail type.=Minevogne kan kun køre på spor, og følger dem altid. Ved et T-kryds uden en vej ligeud drejer de altid til venstre. Farten påvirkes af sportypen.
-You can place the minecart on rails. Right-click it to enter it. Punch it to get it moving.=Du kan placere minevogne på spor. Højre-klik for at stige ombord.
-To obtain the minecart, punch it while holding down the sneak key.=For at at få minevognen i din oppakning. 
-A minecart with TNT is an explosive vehicle that travels on rail.=En minevogn med TNT as et eksplosivt fartøj som kører på spor.
-Place it on rails. Punch it to move it. The TNT is ignited with a flint and steel or when the minecart is on an powered activator rail.=Placér den på spor. Slå den for at flytte den. TNTet bliver antændt med flint og stål eller når minevognen er på et aktiveringsspor.
-To obtain the minecart and TNT, punch them while holding down the sneak key. You can't do this if the TNT was ignited.=For at få minevognen med TNT i din oppakning skal du slå den mens du holder snigeknappen nede. Du kan ikke gøre dette hvis TNTen er antændt.
-A minecart with furnace is a vehicle that travels on rails. It can propel itself with fuel.=En minevogn med ovn er et fartøj som kører på spor. Den kan køre af sig selv med brændstof.
-Place it on rails. If you give it some coal, the furnace will start burning for a long time and the minecart will be able to move itself. Punch it to get it moving.=Placér den på spor. Hvis du putter kul i den vil ovnen brænde i lang tid, og minevognen vil køre af sig selv. Slå den for at sætte den i bevægelse.
-To obtain the minecart and furnace, punch them while holding down the sneak key.=For at få minevognen med ovn i din oppakning skal du slå den mens du holder snigeknappen nede.
-Minecart with Chest=Minevogn med kiste
-Minecart with Furnace=Minevogn med ovn
-Minecart with Command Block=Minevogn med kommandoblok
-Minecart with Hopper=Minevogn med tragt
-Minecart with TNT=Minevogn med TNT
-Place them on the ground to build your railway, the rails will automatically connect to each other and will turn into curves, T-junctions, crossings and slopes as needed.=Placér dem på jorden for at bygge din jerbane. Sporene kobler sig automatisk sammen med hinanden og laver sving, T-kryds, kryds og skråninger efter behov.
-Rail=Spor
-Rails can be used to build transport tracks for minecarts. Normal rails slightly slow down minecarts due to friction.=Spor kan bruges til at bygge jernbaner til minevogne. Normale spor sænker minevognene en smule på grund af friktionsmodstand.
-Powered Rail=Strømspor
-Rails can be used to build transport tracks for minecarts. Powered rails are able to accelerate and brake minecarts.=Spor kan bruges til at bygge jernbaner til minevogne. Strømspor kan accelerere eller bremse minevogne. 
-Without redstone power, the rail will brake minecarts. To make this rail accelerate minecarts, power it with redstone power.=Uden redstonekraft vil sporet bremse minevognen. For at accelerere minevognen skal den bruge redstoneskraft.
-Activator Rail=Aktiveringsspor
-Rails can be used to build transport tracks for minecarts. Activator rails are used to activate special minecarts.=Spor kan bruges til at bygge jernbaner til minevogne. Aktiveringsspor bruges til at aktivere specielle minevogne.
-To make this rail activate minecarts, power it with redstone power and send a minecart over this piece of rail.=For at få dette spor til at aktiere minevogne skal du give det redstonekraft og sende en minevogn over dette sporstykke.
-Detector Rail=Detektorspor
-Rails can be used to build transport tracks for minecarts. A detector rail is able to detect a minecart above it and powers redstone mechanisms.=Spor kan bruges til at bygge jernbaner til minevogne. Et detektorspor kan opdage en minevogn som kører over det og give kraft til redstonemekanismer.
-To detect a minecart and provide redstone power, connect it to redstone trails or redstone mechanisms and send any minecart over the rail.=For at opdage en minevogn og give redstonekraft skal den forbindes til redstonestøv eller redstonemekanismer og send en hvilkensomhelst minevogn hen over sporet.
-Track for minecarts=Spor til minevogne.
-Speed up when powered, slow down when not powered=Accelerérer når der er strøm, sænk hastigheden når der ikke er strøm.
-Activates minecarts when powered=Aktieverer minevogne når der er strøm.
-Emits redstone power when a minecart is detected=Udsender redstonekraft når en minevogn bliver opdaget. 
-Vehicle for fast travel on rails=Fartøj til hurtig kørsel på spor.
-Can be ignited by tools or powered activator rail=Kan antændes med værktøj eller et aktivatorspor med strøm.
-Sneak to dismount=Snig for at stige af.

mods/ENTITIES/mcl_minecarts/locale/mcl_minecarts.fr.tr

@@ -10,15 +10,15 @@ To obtain the minecart and TNT, punch them while holding down the sneak key. You
 A minecart with furnace is a vehicle that travels on rails. It can propel itself with fuel.=Une wagonnet avec un four est un véhicule qui se déplace sur rails. Il peut se propulser avec du carburant.
 Place it on rails. If you give it some coal, the furnace will start burning for a long time and the minecart will be able to move itself. Punch it to get it moving.=Placez-le sur des rails. Si vous lui donnez du charbon, le four commencera à brûler pendant longtemps et le wagonnet pourra se déplacer. Frappez-le pour le faire bouger.
 To obtain the minecart and furnace, punch them while holding down the sneak key.=Pour obtenir le wagonnet et le four, frappez-les tout en maintenant la touche furtive enfoncée.
-Minecart with Chest=Wagonnet avec coffre
+Minecart with Chest=Wagonnet avec Coffre
-Minecart with Furnace=Wagonnet avec four
+Minecart with Furnace=Wagonnet avec Four
-Minecart with Command Block=Wagonnet avec bloc de commande
+Minecart with Command Block=Wagonnet avec Bloc de Commande
-Minecart with Hopper=Wagonnet avec entonnoir
+Minecart with Hopper=Wagonnet avec Entonoir
 Minecart with TNT=Wagonnet avec TNT
 Place them on the ground to build your railway, the rails will automatically connect to each other and will turn into curves, T-junctions, crossings and slopes as needed.=Placez-les sur le sol pour construire votre chemin de fer, les rails se connecteront automatiquement les uns aux autres et se transformeront en courbes, en jonctions en T, en traversées et en pentes au besoin.
 Rail=Rail
 Rails can be used to build transport tracks for minecarts. Normal rails slightly slow down minecarts due to friction.=Les rails peuvent être utilisés pour construire des voies de transport pour les wagonnets. Les rails ralentissent légèrement les wagonnets en raison de la friction.
-Powered Rail=Rail alimenté
+Powered Rail=Rail allimenté
 Rails can be used to build transport tracks for minecarts. Powered rails are able to accelerate and brake minecarts.=Les rails peuvent être utilisés pour construire des voies de transport pour les wagonnets. Les rails motorisés sont capables d'accélérer et de freiner les wagonnets.
 Without redstone power, the rail will brake minecarts. To make this rail accelerate minecarts, power it with redstone power.=Sans énergie de redstone, le rail freinera les wagonnets. Pour que ce rail accélère les minecarts, alimentez-le avec une source d'énergie redstone.
 Activator Rail=Rail d'activation

mods/ENTITIES/mcl_mobs/api.lua

@@ -5,18 +5,23 @@ local math, vector, minetest, mcl_mobs = math, vector, minetest, mcl_mobs
 
 local PATHFINDING = "gowp"
 local CRASH_WARN_FREQUENCY = 60
-local LIFETIMER_DISTANCE = 47
 
 -- Localize
 local S = minetest.get_translator("mcl_mobs")
 
-local DEVELOPMENT = minetest.settings:get_bool("mcl_development",false)
+local LOGGING_ON = minetest.settings:get_bool("mcl_logging_mobs_villager",false)
+local function mcl_log (message)
+	if LOGGING_ON then
+		mcl_util.mcl_log (message, "[Mobs]", true)
+	end
+end
 
 -- Invisibility mod check
 mcl_mobs.invis = {}
 
 local remove_far = true
-
+local mobs_griefing = minetest.settings:get_bool("mobs_griefing") ~= false
+local spawn_protected = minetest.settings:get_bool("mobs_spawn_protected") ~= false
 local mobs_debug = minetest.settings:get_bool("mobs_debug", false) -- Shows helpful debug info above each mob
 local spawn_logging = minetest.settings:get_bool("mcl_logging_mobs_spawn",true)
 
@@ -33,6 +38,15 @@ if minetest.settings:get_bool("only_peaceful_mobs", false) then
 	end)
 end
 
+local node_ok = function(pos, fallback)
+	fallback = fallback or mcl_mobs.fallback_node
+	local node = minetest.get_node_or_nil(pos)
+	if node and minetest.registered_nodes[node.name] then
+		return node
+	end
+	return minetest.registered_nodes[fallback]
+end
+
 function mob_class:update_tag() --update nametag and/or the debug box
 	local tag
 	if mobs_debug then
@@ -61,27 +75,16 @@ function mob_class:update_tag() --update nametag and/or the debug box
 	})
 end
 
-function mob_class:jock_to(mob, reletive_pos, rot)
-	local pos = self.object:get_pos()
-	if not pos then return end
-
-	self.jockey = mob
-	local jock = minetest.add_entity(pos, mob)
-	if not jock then return end
-	jock:get_luaentity().docile_by_day = false
-	jock:get_luaentity().riden_by_jock = true
-	self.object:set_attach(jock, "", reletive_pos, rot)
-end
-
 function mob_class:get_staticdata()
 
 	for _,p in pairs(minetest.get_connected_players()) do
 		self:remove_particlespawners(p:get_player_name())
 	end
-
 	-- remove mob when out of range unless tamed
 	if remove_far
-	and self:despawn_allowed()
+	and self.can_despawn
+	and self.remove_ok
+	and ((not self.nametag) or (self.nametag == ""))
 	and self.lifetimer <= 20 then
 		if spawn_logging then
 			minetest.log("action", "[mcl_mobs] Mob "..tostring(self.name).." despawns at "..minetest.pos_to_string(vector.round(self.object:get_pos())) .. " - out of range")
@@ -90,6 +93,7 @@ function mob_class:get_staticdata()
 		return "remove"-- nil
 	end
 
+	self.remove_ok = true
 	self.attack = nil
 	self.following = nil
 	self.state = "stand"
@@ -111,21 +115,6 @@ function mob_class:get_staticdata()
 	return minetest.serialize(tmp)
 end
 
-local function valid_texture(self, def_textures)
-	if not self.base_texture then
-		return false
-	end
-
-	if self.texture_selected then
-		if #def_textures < self.texture_selected then
-			self.texture_selected = nil
-		else
-			return true
-		end
-	end
-	return false
-end
-
 function mob_class:mob_activate(staticdata, def, dtime)
 	if not self.object:get_pos() or staticdata == "remove" then
 		mcl_burning.extinguish(self.object)
@@ -148,20 +137,16 @@ function mob_class:mob_activate(staticdata, def, dtime)
 	end
 
 	--If textures in definition change, reload textures
-	if not valid_texture(self, def.textures) then
+	if not self.base_texture or (def.textures and table.indexof(def.textures, self.base_texture) == -1) then
-
 		-- compatiblity with old simple mobs textures
 		if type(def.textures[1]) == "string" then
 			def.textures = {def.textures}
 		end
 
-		if not self.texture_selected then
+		local c = 1
-			local c = 1
+		if #def.textures > c then c = #def.textures end
-			if #def.textures > c then c = #def.textures end
-			self.texture_selected = math.random(c)
-		end
 
-		self.base_texture = def.textures[self.texture_selected]
+		self.base_texture = def.textures[math.random(c)]
 		self.base_mesh = def.mesh
 		self.base_size = self.visual_size
 		self.base_colbox = self.collisionbox
@@ -283,13 +268,6 @@ function mob_class:mob_activate(staticdata, def, dtime)
 	self._current_animation = nil
 	self:set_animation( "stand")
 
-
-	if self.riden_by_jock then --- Keep this function before self.on_spawn() is run.
-		self.object:remove()
-		return
-	end
-
-
 	if self.on_spawn and not self.on_spawn_run then
 		if self.on_spawn(self) then
 			self.on_spawn_run = true
@@ -306,9 +284,6 @@ function mob_class:mob_activate(staticdata, def, dtime)
 		self._run_armor_init = true
 	end
 
-
-
-
 	if def.after_activate then
 		def.after_activate(self, staticdata, def, dtime)
 	end
@@ -316,33 +291,46 @@ end
 
 -- execute current state (stand, walk, run, attacks)
 -- returns true if mob has died
-function mob_class:do_states(dtime, player_in_active_range)
+function mob_class:do_states(dtime)
 	--if self.can_open_doors then check_doors(self) end
 
-	-- knockback timer. set in on_punch
+	if self.state == "stand" then
-	if self.pause_timer > 0 then
+		self:do_states_stand()
-		self.pause_timer = self.pause_timer - dtime
+	elseif self.state == PATHFINDING then
-		return
-	end
-
-	self:env_danger_movement_checks(player_in_active_range)
-
-	if self.state == PATHFINDING then
 		self:check_gowp(dtime)
+	elseif self.state == "walk" then
+		self:do_states_walk()
+	elseif self.state == "runaway" then
+		-- runaway when punched
+		self:do_states_runaway()
 	elseif self.state == "attack" then
+		-- attack routines (explode, dogfight, shoot, dogshoot)
 		if self:do_states_attack(dtime) then
 			return true
 		end
-	else
+	end
-		if mcl_util.check_dtime_timer(self, dtime, "onstep_dostates", 1) then
+end
-			if self.state == "stand" then
+
-				self:do_states_stand(player_in_active_range)
+local function update_timers (self, dtime)
-			elseif self.state == "walk" then
+	-- knockback timer. set in on_punch
-				self:do_states_walk()
+	if self.pause_timer > 0 then
-			elseif self.state == "runaway" then
+		self.pause_timer = self.pause_timer - dtime
-				self:do_states_runaway()
+		return true
-			end
+	end
+
+	-- attack timer. Not anymore, it seems. Used for also occassionally processing mob step too!
+	self.timer = self.timer + dtime
+
+	if self.state ~= "attack" and self.state ~= PATHFINDING then
+		if self.timer < 1 then
+			return true
 		end
+		self.timer = 0
+	end
+
+	-- never go over 100
+	if self.timer > 100 then
+		self.timer = 1
 	end
 end
 
@@ -370,8 +358,6 @@ function mob_class:outside_limits()
 	end
 end
 
-
-
 local function on_step_work (self, dtime)
 	local pos = self.object:get_pos()
 	if not pos then return end
@@ -387,62 +373,71 @@ local function on_step_work (self, dtime)
 	end
 
 	if self:falling(pos) then return end
-	if self:step_damage (dtime, pos) then return end
+	self:check_suspend()
+
+	if not self.fire_resistant then
+		mcl_burning.tick(self.object, dtime, self)
+		if not self.object:get_pos() then return end -- mcl_burning.tick may remove object immediately
+
+		if self:check_for_death("fire", {type = "fire"}) then
+			return true
+		end
+	end
+
+	if self:env_damage (dtime, pos) then return end
 
 	if self.state == "die" then return end
 	-- End: Death/damage processing
 
-	local player_in_active_range = self:player_in_active_range()
-	self:check_suspend(player_in_active_range)
-
 	self:check_water_flow()
+	self:env_danger_movement_checks (dtime)
 
-	if not self._jumping_cliff then
+	self:follow_flop() -- Mob following code.
-		self._can_jump_cliff = self:can_jump_cliff()
-	else
-		self._can_jump_cliff = false
-	end
 
-	self:flop()
+	self:set_animation_speed() -- set animation speed relative to velocity
 
 	self:check_smooth_rotation(dtime)
+	self:check_head_swivel(dtime)
 
-	if player_in_active_range then
+	if self.jump_sound_cooloff > 0 then self.jump_sound_cooloff = self.jump_sound_cooloff - dtime end
-		self:set_animation_speed() -- set animation speed relative to velocity
+	self:do_jump()
-
-		self:check_head_swivel(dtime)
-
-		if mcl_util.check_dtime_timer(self, dtime, "onstep_engage", 0.2) then
-			self:check_follow()
-			self:check_runaway_from()
-			self:monster_attack()
-			self:npc_attack()
-		end
-
-		self:check_herd(dtime)
-
-		if self.jump_sound_cooloff > 0 then self.jump_sound_cooloff = self.jump_sound_cooloff - dtime end
-		self:do_jump()
-	end
-
-	if mcl_util.check_dtime_timer(self, dtime, "onstep_occassional", 1) then
-
-		if player_in_active_range then
-			self:check_item_pickup()
-			self:set_armor_texture()
-			self:step_opinion_sound(dtime)
-		end
-
-		self:check_breeding()
-	end
 
+	self:check_runaway_from()
+	self:monster_attack()
+	self:npc_attack()
 	self:check_aggro(dtime)
 
-	self:check_particlespawners(dtime)
-
 	if self.do_custom and self.do_custom(self, dtime) == false then return end
 
-	if self:do_states(dtime, player_in_active_range) then return end
+
+	-- In certain circumstances, we abandon processing of certain functionality
+	local skip_processing = false
+	if update_timers(self, dtime) then
+		skip_processing = true
+	end
+
+
+
+	if not skip_processing then
+		self:check_breeding()
+
+		self:check_item_pickup()
+		self:set_armor_texture()
+
+		self:check_particlespawners(dtime)
+
+		if self.opinion_sound_cooloff > 0 then
+			self.opinion_sound_cooloff = self.opinion_sound_cooloff - dtime
+		end
+		-- mob plays random sound at times. Should be 120. Zombie and mob farms are ridiculous
+		if math.random(1, 70) == 1 then
+			self:mob_sound("random", true)
+		end
+
+		if self:do_states(dtime) then return end
+	end
+
+
 
 	if mobs_debug then self:update_tag() end
 
@@ -453,17 +448,9 @@ end
 
 local last_crash_warn_time = 0
 
-local function log_error (stack_trace, info, info2)
+local on_step_error_handler = function ()
-	minetest.log("action", "--- Bug report start (please provide a few lines before this also for context) ---")
+	local info = debug.getinfo(1, "SnlufL")
-	minetest.log("action", "Error: " .. stack_trace)
-	minetest.log("action", "Bug info: " .. info)
-	if info2 then
-		minetest.log("action", "Bug info additional: " .. info2)
-	end
-	minetest.log("action", "--- Bug report end ---")
-end
 
-local function warn_user_error ()
 	local current_time = os.time()
 	local time_since_warning = current_time - last_crash_warn_time
 
@@ -475,48 +462,27 @@ local function warn_user_error ()
 		last_crash_warn_time = current_time
 		minetest.log("A game crashing bug was prevented. Please provide debug.log information to MineClone2 dev team for investigation. (Search for: --- Bug report start)")
 	end
+	minetest.log("action", "--- Bug report start (please provide a few lines before this also for context) ---")
+	minetest.log("action", "Stack trace: ".. tostring(debug.traceback()))
+	minetest.log("action", "Bug info: ".. dump(info))
+	minetest.log("action", "--- Bug report end ---")
 end
 
-local on_step_error_handler = function ()
-	warn_user_error ()
-	local info = debug.getinfo(1, "SnlufL")
-	log_error(tostring(debug.traceback()), dump(info))
-end
-
-
-
 -- main mob function
 function mob_class:on_step(dtime)
-	if not DEVELOPMENT then
+	local status, retVal = xpcall(on_step_work, on_step_error_handler, self, dtime)
-		-- Removed as bundled Lua (5.1 doesn't support xpcall)
+	if status then
-		--local status, retVal = xpcall(on_step_work, on_step_error_handler, self, dtime)
+		return retVal
-		local status, retVal = pcall(on_step_work, self, dtime)
-		if status then
-			return retVal
-		else
-			warn_user_error ()
-			local pos = self.object:get_pos()
-			if pos then
-				local node = minetest.get_node(pos)
-				if node and node.name == "ignore" then
-					minetest.log("warning", "Pos is ignored: " .. dump(pos))
-				end
-			end
-			log_error (dump(retVal), dump(pos), dump(self))
-		end
-	else
-		return on_step_work (self, dtime)
 	end
 end
 
 local timer = 0
-
+minetest.register_globalstep(function(dtime)
-local function update_lifetimer(dtime)
 	timer = timer + dtime
 	if timer < 1 then return end
 	for _, player in pairs(minetest.get_connected_players()) do
 		local pos = player:get_pos()
-		for _, obj in pairs(minetest.get_objects_inside_radius(pos, LIFETIMER_DISTANCE)) do
+		for _, obj in pairs(minetest.get_objects_inside_radius(pos, 47)) do
 			local lua = obj:get_luaentity()
 			if lua and lua.is_mob then
 				lua.lifetimer = math.max(20, lua.lifetimer)
@@ -525,125 +491,22 @@ local function update_lifetimer(dtime)
 		end
 	end
 	timer = 0
-end
-
-minetest.register_globalstep(function(dtime)
-	update_lifetimer(dtime)
 end)
 
-
+minetest.register_chatcommand("clearmobs",{
-minetest.register_chatcommand("clearmobs", {
+	privs={maphack=true},
-	privs = { maphack = true },
+	params = "<all>|<nametagged>|<range>",
-	params = "[all|monster|passive|<mob name> [<range>|nametagged|tamed]]",
+	description=S("Removes all spawned mobs except nametagged and tamed ones. all removes all mobs, nametagged only nametagged ones and with the range paramter all mobs in a distance of the current player are removed."),
-	description = S("Removes specified mobs except nametagged and tamed ones. For the second parameter, use nametagged/tamed to select only nametagged/tamed mobs, or a range to specify a maximum distance from the player."),
+	func=function(n,param)
-	func = function(player, param)
+		local p = minetest.get_player_by_name(n)
-		local default = false
+		local num=tonumber(param)
-		if not param or param == "" then
-			default = true
-			minetest.chat_send_player(player,
-					S("Default usage. Clearing hostile mobs. For more options please type: /help clearmobs"))
-		end
-		local mob, unsafe = param:match("^([%w]+)[ ]?([%w%d]*)$")
-
-		local all = false
-		local nametagged = false
-		local tamed = false
-
-		local mob_name, mob_type, range
-
-		-- Param 1 resolve
-		if mob and mob ~= "" then
-			if mob == "all" then
-				all = true
-			elseif mob == "passive" or mob == "monster" then
-				mob_type = mob
-			elseif mob then
-				mob_name = mob
-			end
-			--minetest.log ("mob: [" .. mob .. "]")
-		else
-			--minetest.log("No valid first param")
-			if default then
-				--minetest.log("Use default")
-				mob_type = "monster"
-			end
-			--return
-		end
-
-		-- Param 2 resolve
-		if unsafe and unsafe ~= "" then
-			--minetest.log ("unsafe: [" .. unsafe .. "]")
-			if unsafe == "nametagged" then
-				nametagged = true
-			elseif unsafe == "tamed" then
-				tamed = true
-			end
-
-			local num = tonumber(unsafe)
-			if num then range = num end
-		end
-
-		local p = minetest.get_player_by_name(player)
-
 		for _,o in pairs(minetest.luaentities) do
-			if o and o.is_mob then
+			if o.is_mob then
-				local mob_match = false
+				if  param == "all" or
-
+				( param == "nametagged" and o.nametag ) or
-				if all then
+				( param == "" and ( not o.nametag or o.nametag == "" ) and not o.tamed ) or
-					--minetest.log("Match - All mobs specified")
+				( num and num > 0 and vector.distance(p:get_pos(),o.object:get_pos()) <= num ) then
-					mob_match = true
+					o.object:remove()
-				elseif mob_type then
-
-					--minetest.log("Match - o.type: ".. tostring(o.type))
-					--minetest.log("mob_type: ".. tostring(mob_type))
-					if mob_type == "monster" and o.type == mob_type then
-						--minetest.log("Match - monster")
-						mob_match = true
-					elseif mob_type == "passive" and o.type ~= "monster" and o.type ~= "npc" then
-						--minetest.log("Match - passive")
-						mob_match = true
-					else
-						--minetest.log("No match for type.")
-					end
-
-				elseif mob_name and (o.name == mob_name or string.find(o.name, mob_name)) then
-					--minetest.log("Match - mob_name = ".. tostring(o.name))
-					mob_match = true
-				else
-					--minetest.log("No match - o.type = ".. tostring(o.type))
-					--minetest.log("No match - mob_name = ".. tostring(o.name))
-					--minetest.log("No match - mob_type = ".. tostring(mob_name))
-				end
-
-				if mob_match then
-					local in_range = true
-					if (not range or range <= 0 ) then
-						in_range = true
-					else
-						if ( vector.distance(p:get_pos(),o.object:get_pos()) <= range ) then
-							in_range = true
-						else
-							--minetest.log("Out of range")
-							in_range = false
-						end
-					end
-
-					--minetest.log("o.nametag: ".. tostring(o.nametag))
-
-					if nametagged then
-						if o.nametag then
-							--minetest.log("Namedtagged and it has a name tag. Kill it")
-							o.object:remove()
-						end
-					elseif tamed then
-						if o.tamed then
-							--minetest.log("Tamed. Kill it")
-							o.object:remove()
-						end
-					elseif in_range and (not o.nametag or o.nametag == "") and not o.tamed then
-						--minetest.log("No nametag or tamed. Kill it")
-						o.object:remove()
-					end
 				end
 			end
 		end

mods/ENTITIES/mcl_mobs/api.txt

@@ -262,7 +262,6 @@ functions needed for the mob to work properly which contains the following:
 	'custom_visual_size' will not reset visual_size from the base class on reload
 	'noyaw'				If true this mob will not automatically change yaw
 	'particlespawners'	Table of particlespawners attached to the mob. This is implemented in a coord safe manner i.e. spawners are only sent to players within the player_transfer_distance (and automatically removed). This enables infinitely lived particlespawners.
-    'attack_frequency'  Attack frequency in seconds. If unset, this defaults to 1. Implemented for melee only atm.
 
 	mobs:gopath(self,target,callback_arrived)		pathfind a way to target and run callback on arrival
 

mods/ENTITIES/mcl_mobs/breeding.lua

@@ -74,7 +74,6 @@ function mob_class:feed_tame(clicker, feed_count, breed, tame, notake)
 			if self.food >= feed_count then
 				self.food = 0
 				self.horny = true
-				self.persistent = true
 			end
 		end
 

mods/ENTITIES/mcl_mobs/combat.lua

@@ -10,8 +10,6 @@ local stuck_path_timeout = 10 -- how long will mob follow path before giving up
 
 local enable_pathfinding = true
 
-local TIME_TO_FORGET_TARGET = 15
-
 local atann = math.atan
 local function atan(x)
 	if not x or x ~= x then
@@ -331,7 +329,10 @@ end
 
 -- find someone to attack
 function mob_class:monster_attack()
-	if not damage_enabled or self.passive ~= false or self.state == "attack" or self:day_docile() then
+	if not damage_enabled
+	or self.passive ~= false
+	or self.state == "attack"
+	or self:day_docile() then
 		return
 	end
 
@@ -340,10 +341,8 @@ function mob_class:monster_attack()
 	local player, obj, min_player
 	local type, name = "", ""
 	local min_dist = self.view_range + 1
-
-	local blacklist_attack = {}
-
 	local objs = minetest.get_objects_inside_radius(s, self.view_range)
+	local blacklist_attack = {}
 
 	for n = 1, #objs do
 		if not objs[n]:is_player() then
@@ -360,29 +359,32 @@ function mob_class:monster_attack()
 	end
 
 	for n = 1, #objs do
+
+
 		if objs[n]:is_player() then
 			if mcl_mobs.invis[ objs[n]:get_player_name() ] or (not self:object_in_range(objs[n])) then
 				type = ""
 			elseif (self.type == "monster" or self._aggro) then
-				-- self.aggro made player be attacked by npc again if out of range then back in again
-				-- Does it serve a purpose other than that?
 				player = objs[n]
 				type = "player"
 				name = "player"
 			end
 		else
 			obj = objs[n]:get_luaentity()
+
 			if obj then
 				player = obj.object
 				type = obj.type
 				name = obj.name or ""
 			end
+
 		end
 
 		-- find specific mob to attack, failing that attack player/npc/animal
 		if specific_attack(self.specific_attack, name)
-				and (type == "player" or ( type == "npc" and self.attack_npcs )
+		and (type == "player" or ( type == "npc" and self.attack_npcs )
-				or (type == "animal" and self.attack_animals == true)) then
+			or (type == "animal" and self.attack_animals == true)) then
+
 			p = player:get_pos()
 			sp = s
 
@@ -398,10 +400,10 @@ function mob_class:monster_attack()
 					attacked_p = true
 				end
 			end
-
 			-- choose closest player to attack
-			local line_of_sight = self:line_of_sight( sp, p, 2) == true
+			if dist < min_dist
-			if dist < min_dist and not attacked_p and line_of_sight then
+			and not attacked_p
+			and self:line_of_sight( sp, p, 2) == true then
 				min_dist = dist
 				min_player = player
 			end
@@ -432,9 +434,11 @@ function mob_class:npc_attack()
 	local objs = minetest.get_objects_inside_radius(s, self.view_range)
 
 	for n = 1, #objs do
+
 		obj = objs[n]:get_luaentity()
 
 		if obj and obj.type == "monster" then
+
 			p = obj.object:get_pos()
 			sp = s
 
@@ -444,7 +448,8 @@ function mob_class:npc_attack()
 			p.y = p.y + 1
 			sp.y = sp.y + 1
 
-			if dist < min_dist and self:line_of_sight( sp, p, 2) == true then
+			if dist < min_dist
+			and self:line_of_sight( sp, p, 2) == true then
 				min_dist = dist
 				min_player = obj.object
 			end
@@ -791,57 +796,23 @@ end
 
 function mob_class:check_aggro(dtime)
 	if not self._aggro or not self.attack then return end
-
+	if not self._check_aggro_timer or self._check_aggro_timer > 5 then
-	if not self._check_aggro_timer then
 		self._check_aggro_timer = 0
-	end
+		if not self.attack:get_pos() or vector.distance(self.attack:get_pos(),self.object:get_pos()) > 128 then
-
+			self._aggro = nil
-	if self._check_aggro_timer > 5 then
+			self.attack = nil
-		self._check_aggro_timer = 0
+			self.state = "stand"
-
-		if self.attack then
-			-- TODO consider removing this in favour of what is done in do_states_attack
-			-- Attack is dropped in do_states_attack if out of range, so won't even trigger here
-			-- I do not think this code does anything. Are mobs still loaded in at 128?
-			if not self.attack:get_pos() or vector.distance(self.attack:get_pos(),self.object:get_pos()) > 128 then
-				self._aggro = nil
-				self.attack = nil
-				self.state = "stand"
-			end
 		end
 	end
 	self._check_aggro_timer = self._check_aggro_timer + dtime
 end
 
-
-
-local function clear_aggro(self)
-	self.state = "stand"
-	self:set_velocity( 0)
-	self:set_animation( "stand")
-
-	self.attack = nil
-	self._aggro = nil
-
-	self.v_start = false
-	self.timer = 0
-	self.blinktimer = 0
-	self.path.way = nil
-end
-
 function mob_class:do_states_attack (dtime)
-	self.timer = self.timer + dtime
+	local yaw = self.object:get_yaw() or 0
-	if self.timer > 100 then
-		self.timer = 1
-	end
 
 	local s = self.object:get_pos()
-	if not s then return end
-
 	local p = self.attack:get_pos() or s
 
-	local yaw = self.object:get_yaw() or 0
-
 	-- stop attacking if player invisible or out of range
 	if not self.attack
 			or not self.attack:get_pos()
@@ -849,24 +820,16 @@ function mob_class:do_states_attack (dtime)
 			or self.attack:get_hp() <= 0
 			or (self.attack:is_player() and mcl_mobs.invis[ self.attack:get_player_name() ]) then
 
-		clear_aggro(self)
+		self.state = "stand"
-		return
+		self:set_velocity( 0)
-	end
+		self:set_animation( "stand")
+		self.attack = nil
+		self.v_start = false
+		self.timer = 0
+		self.blinktimer = 0
+		self.path.way = nil
 
-	local target_line_of_sight = self:line_of_sight(s, p, 2)
+		return
-	if not target_line_of_sight then
-		if self.target_time_lost then
-			local time_since_seen = os.time() - self.target_time_lost
-			if time_since_seen > TIME_TO_FORGET_TARGET then
-				self.target_time_lost = nil
-				clear_aggro(self)
-				return
-			end
-		else
-			self.target_time_lost = os.time()
-		end
-	else
-		self.target_time_lost = nil
 	end
 
 	-- calculate distance from mob and enemy
@@ -874,27 +837,36 @@ function mob_class:do_states_attack (dtime)
 
 	if self.attack_type == "explode" then
 
-		if target_line_of_sight then
+		local vec = {
-			local vec = { x = p.x - s.x, z = p.z - s.z }
+			x = p.x - s.x,
-			yaw = (atan(vec.z / vec.x) +math.pi/ 2) - self.rotate
+			z = p.z - s.z
-			if p.x > s.x then yaw = yaw +math.pi end
+		}
-			yaw = self:set_yaw( yaw, 0, dtime)
+
-		end
+		yaw = (atan(vec.z / vec.x) +math.pi/ 2) - self.rotate
+
+		if p.x > s.x then yaw = yaw +math.pi end
+
+		yaw = self:set_yaw( yaw, 0, dtime)
 
 		local node_break_radius = self.explosion_radius or 1
 		local entity_damage_radius = self.explosion_damage_radius
 				or (node_break_radius * 2)
 
 		-- start timer when in reach and line of sight
-		if not self.v_start and dist <= self.reach and target_line_of_sight then
+		if not self.v_start
+				and dist <= self.reach
+				and self:line_of_sight( s, p, 2) then
+
 			self.v_start = true
 			self.timer = 0
 			self.blinktimer = 0
 			self:mob_sound("fuse", nil, false)
 
 			-- stop timer if out of reach or direct line of sight
-		elseif self.allow_fuse_reset and self.v_start
+		elseif self.allow_fuse_reset
-				and (dist >= self.explosiontimer_reset_radius or not target_line_of_sight) then
+				and self.v_start
+				and (dist >= self.explosiontimer_reset_radius
+				or not self:line_of_sight( s, p, 2)) then
 			self.v_start = false
 			self.timer = 0
 			self.blinktimer = 0
@@ -903,10 +875,10 @@ function mob_class:do_states_attack (dtime)
 		end
 
 		-- walk right up to player unless the timer is active
-		if self.v_start and (self.stop_to_explode or dist < self.reach) or not target_line_of_sight then
+		if self.v_start and (self.stop_to_explode or dist < self.reach) then
-			self:set_velocity(0)
+			self:set_velocity( 0)
 		else
-			self:set_velocity(self.run_velocity)
+			self:set_velocity( self.run_velocity)
 		end
 
 		if self.animation and self.animation.run_start then
@@ -916,20 +888,25 @@ function mob_class:do_states_attack (dtime)
 		end
 
 		if self.v_start then
+
 			self.timer = self.timer + dtime
 			self.blinktimer = (self.blinktimer or 0) + dtime
 
 			if self.blinktimer > 0.2 then
+
 				self.blinktimer = 0
+
 				if self.blinkstatus then
 					self:remove_texture_mod("^[brighten")
 				else
 					self:add_texture_mod("^[brighten")
 				end
+
 				self.blinkstatus = not self.blinkstatus
 			end
 
 			if self.timer > self.explosion_timer then
+
 				local pos = self.object:get_pos()
 
 				if mobs_griefing and not minetest.is_protected(pos, "") then
@@ -1043,45 +1020,54 @@ function mob_class:do_states_attack (dtime)
 
 		-- move towards enemy if beyond mob reach
 		if dist > self.reach then
+
 			-- path finding by rnd
-			if enable_pathfinding and self.pathfinding then
+			if self.pathfinding -- only if mob has pathfinding enabled
+					and enable_pathfinding then
+
 				self:smart_mobs(s, p, dist, dtime)
 			end
 
 			if self:is_at_cliff_or_danger() then
+
 				self:set_velocity( 0)
 				self:set_animation( "stand")
 				local yaw = self.object:get_yaw() or 0
 				yaw = self:set_yaw( yaw + 0.78, 8)
 			else
+
 				if self.path.stuck then
-					self:set_velocity(self.walk_velocity)
+					self:set_velocity( self.walk_velocity)
 				else
-					self:set_velocity(self.run_velocity)
+					self:set_velocity( self.run_velocity)
 				end
+
 				if self.animation and self.animation.run_start then
-					self:set_animation("run")
+					self:set_animation( "run")
 				else
-					self:set_animation("walk")
+					self:set_animation( "walk")
 				end
 			end
+
 		else -- rnd: if inside reach range
+
 			self.path.stuck = false
 			self.path.stuck_timer = 0
 			self.path.following = false -- not stuck anymore
 
 			self:set_velocity( 0)
 
-			local attack_frequency = self.attack_frequency or 1
+			if not self.custom_attack then
 
-			if self.timer > attack_frequency then
+				if self.timer > 1 then
-				self.timer = 0
 
-				if not self.custom_attack then
+					self.timer = 0
-					if self.double_melee_attack and math.random(1, 2) == 1 then
+
-						self:set_animation("punch2")
+					if self.double_melee_attack
+							and math.random(1, 2) == 1 then
+						self:set_animation( "punch2")
 					else
-						self:set_animation("punch")
+						self:set_animation( "punch")
 					end
 
 					local p2 = p
@@ -1091,6 +1077,8 @@ function mob_class:do_states_attack (dtime)
 					s2.y = s2.y + .5
 
 					if self:line_of_sight( p2, s2) == true then
+
+						-- play attack sound
 						self:mob_sound("attack")
 
 						-- punch player (or what player is attached to)
@@ -1103,7 +1091,13 @@ function mob_class:do_states_attack (dtime)
 							damage_groups = {fleshy = self.damage}
 						}, nil)
 					end
-				else
+				end
+			else	-- call custom attack every second
+				if self.custom_attack
+						and self.timer > 1 then
+
+					self.timer = 0
+
 					self.custom_attack(self, p)
 				end
 			end
@@ -1129,7 +1123,7 @@ function mob_class:do_states_attack (dtime)
 
 		yaw = self:set_yaw( yaw, 0, dtime)
 
-		local stay_away_from_player = vector.zero()
+		local stay_away_from_player = vector.new(0,0,0)
 
 		--strafe back and fourth
 
@@ -1146,13 +1140,7 @@ function mob_class:do_states_attack (dtime)
 			if math.random(40) == 1 then
 				self.strafe_direction = self.strafe_direction*-1
 			end
-
+			self.acc = vector.add(vector.multiply(vector.rotate_around_axis(vector.direction(s, p), vector.new(0,1,0), self.strafe_direction), 0.3*self.walk_velocity), stay_away_from_player)
-			local dir = vector.rotate_around_axis(vector.direction(s, p), vector.new(0,1,0), self.strafe_direction)
-			local dir2 = vector.multiply(dir, 0.3 * self.walk_velocity)
-
-			if dir2 and stay_away_from_player then
-				self.acc = vector.add(dir2, stay_away_from_player)
-			end
 		else
 			self:set_velocity( 0)
 		end

mods/ENTITIES/mcl_mobs/effects.lua

@@ -4,8 +4,6 @@ local active_particlespawners = {}
 local disable_blood = minetest.settings:get_bool("mobs_disable_blood")
 local DEFAULT_FALL_SPEED = -9.81*1.5
 
-local PATHFINDING = "gowp"
-
 local player_transfer_distance = tonumber(minetest.settings:get("player_transfer_distance")) or 128
 if player_transfer_distance == 0 then player_transfer_distance = math.huge end
 
@@ -138,19 +136,6 @@ function mob_class:mob_sound(soundname, is_opinion, fixed_pitch)
 	end
 end
 
-function mob_class:step_opinion_sound(dtime)
-	if self.state ~= "attack" and self.state ~= PATHFINDING then
-
-		if self.opinion_sound_cooloff > 0 then
-			self.opinion_sound_cooloff = self.opinion_sound_cooloff - dtime
-		end
-		-- mob plays random sound at times. Should be 120. Zombie and mob farms are ridiculous
-		if math.random(1, 70) == 1 then
-			self:mob_sound("random", true)
-		end
-	end
-end
-
 function mob_class:add_texture_mod(mod)
 	local full_mod = ""
 	local already_added = false
@@ -256,20 +241,16 @@ function mob_class:set_animation(anim, fixed_frame)
 	if not self.animation or not anim then
 		return
 	end
-
-	if self.jockey and self.object:get_attach() then
-		anim = "jockey"
-	elseif not self.object:get_attach() then
-		self.jockey = nil
-	end
-	
 	if self.state == "die" and anim ~= "die" and anim ~= "stand" then
 		return
 	end
 
+	if self.jockey then
+		anim = "jockey"
+	end
 
 
-	if self.fly and self:flight_check() and anim == "walk" then anim = "fly" end
+	if self:flight_check() and self.fly and anim == "walk" then anim = "fly" end
 
 	self._current_animation = self._current_animation or ""
 
@@ -308,7 +289,7 @@ local function dir_to_pitch(dir)
 	return -math.atan2(-dir.y, xz)
 end
 
-local function who_are_you_looking_at (self, dtime)
+local function who_are_you_looking_at (self)
 	local pos = self.object:get_pos()
 
 	local stop_look_at_player_chance = math.random(833/self.curiosity)
@@ -316,26 +297,20 @@ local function who_are_you_looking_at (self, dtime)
 
 	local stop_look_at_player = stop_look_at_player_chance == 1
 
-	if self.attack then
+	if self.attack or self.following then
-		if not self.target_time_lost then
+		self._locked_object = self.attack or self.following
-			self._locked_object = self.attack
-		else
-			self._locked_object = nil
-		end
-	elseif self.following then
-		self._locked_object = self.following
 	elseif self._locked_object then
 		if stop_look_at_player then
 			--minetest.log("Stop look: ".. self.name)
 			self._locked_object = nil
 		end
 	elseif not self._locked_object then
-		if mcl_util.check_dtime_timer(self, dtime, "step_look_for_someone", 0.2) then
+		if math.random(1, 30) then
 			--minetest.log("Change look check: ".. self.name)
 
 			-- For the wither this was 20/60=0.33, so probably need to rebalance and divide rates.
 			-- but frequency of check isn't good as it is costly. Making others too infrequent requires testing
-			local chance = 150/self.curiosity
+			local chance = 20/self.curiosity
 
 			if chance < 1 then chance = 1 end
 			local look_at_player_chance = math.random(chance)
@@ -366,10 +341,9 @@ end
 function mob_class:check_head_swivel(dtime)
 	if not self.head_swivel or type(self.head_swivel) ~= "string" then return end
 
+	who_are_you_looking_at (self)
 
-	who_are_you_looking_at (self, dtime)
+	local final_rotation = vector.new(0,0,0)
-
-	local final_rotation = vector.zero()
 	local oldp,oldr = self.object:get_bone_position(self.head_swivel)
 
 	if self._locked_object and (self._locked_object:is_player() or self._locked_object:get_luaentity()) and self._locked_object:get_hp() > 0 then
@@ -381,34 +355,33 @@ function mob_class:check_head_swivel(dtime)
 			_locked_object_eye_height = self._locked_object:get_properties().eye_height
 		end
 		if _locked_object_eye_height then
-
 			local self_rot = self.object:get_rotation()
-			-- If a mob is attached, should we really be messing with what they are looking at?
+			if self.object:get_attach() then
-			-- Should this be excluded?
-			if self.object:get_attach() and self.object:get_attach():get_rotation() then
 				self_rot = self.object:get_attach():get_rotation()
 			end
 
-			local player_pos = self._locked_object:get_pos()
+			if self.rot then
-			local direction_player = vector.direction(vector.add(self.object:get_pos(), vector.new(0, self.head_eye_height*.7, 0)), vector.add(player_pos, vector.new(0, _locked_object_eye_height, 0)))
+				local player_pos = self._locked_object:get_pos()
-			local mob_yaw = math.deg(-(-(self_rot.y)-(-minetest.dir_to_yaw(direction_player))))+self.head_yaw_offset
+				local direction_player = vector.direction(vector.add(self.object:get_pos(), vector.new(0, self.head_eye_height*.7, 0)), vector.add(player_pos, vector.new(0, _locked_object_eye_height, 0)))
-			local mob_pitch = math.deg(-dir_to_pitch(direction_player))*self.head_pitch_multiplier
+				local mob_yaw = math.deg(-(-(self_rot.y)-(-minetest.dir_to_yaw(direction_player))))+self.head_yaw_offset
+				local mob_pitch = math.deg(-dir_to_pitch(direction_player))*self.head_pitch_multiplier
 
-			if (mob_yaw < -60 or mob_yaw > 60) and not (self.attack and self.state == "attack" and not self.runaway) then
+				if (mob_yaw < -60 or mob_yaw > 60) and not (self.attack and self.state == "attack" and not self.runaway) then
-				final_rotation = vector.multiply(oldr, 0.9)
+					final_rotation = vector.multiply(oldr, 0.9)
-			elseif self.attack and self.state == "attack" and not self.runaway then
+				elseif self.attack and self.state == "attack" and not self.runaway then
-				if self.head_yaw == "y" then
+					if self.head_yaw == "y" then
-					final_rotation = vector.new(mob_pitch, mob_yaw, 0)
+						final_rotation = vector.new(mob_pitch, mob_yaw, 0)
-				elseif self.head_yaw == "z" then
+					elseif self.head_yaw == "z" then
-					final_rotation = vector.new(mob_pitch, 0, -mob_yaw)
+						final_rotation = vector.new(mob_pitch, 0, -mob_yaw)
-				end
+					end
 
-			else
+				else
 
-				if self.head_yaw == "y" then
+					if self.head_yaw == "y" then
-					final_rotation = vector.new(((mob_pitch-oldr.x)*.3)+oldr.x, ((mob_yaw-oldr.y)*.3)+oldr.y, 0)
+						final_rotation = vector.new(((mob_pitch-oldr.x)*.3)+oldr.x, ((mob_yaw-oldr.y)*.3)+oldr.y, 0)
-				elseif self.head_yaw == "z" then
+					elseif self.head_yaw == "z" then
-					final_rotation = vector.new(((mob_pitch-oldr.x)*.3)+oldr.x, 0, -(((mob_yaw-oldr.y)*.3)+oldr.y)*3)
+						final_rotation = vector.new(((mob_pitch-oldr.x)*.3)+oldr.x, 0, -(((mob_yaw-oldr.y)*.3)+oldr.y)*3)
+					end
 				end
 			end
 		end

mods/ENTITIES/mcl_mobs/init.lua

@@ -154,7 +154,6 @@ function mcl_mobs.register_mob(name, def)
 		description = def.description,
 		type = def.type,
 		attack_type = def.attack_type,
-		attack_frequency = def.attack_frequency,
 		fly = def.fly or false,
 		fly_in = def.fly_in or {"air", "__airlike"},
 		owner = def.owner or "",
@@ -217,7 +216,6 @@ function mcl_mobs.register_mob(name, def)
 		replace_with = def.replace_with,
 		replace_offset = def.replace_offset or 0,
 		on_replace = def.on_replace,
-		replace_delay = def.replace_delay or 0,
 		timer = 0,
 		env_damage_timer = 0,
 		tamed = false,
@@ -499,11 +497,12 @@ function mcl_mobs.register_egg(mob, desc, background_color, overlay_color, addeg
 				return def.on_rightclick(pointed_thing.under, under, placer, itemstack)
 			end
 
-			if pos and within_limits(pos, 0)  and not minetest.is_protected(pos, placer:get_player_name()) then
+			if pos
+			and within_limits(pos, 0)
+			and not minetest.is_protected(pos, placer:get_player_name()) then
+
 				local name = placer:get_player_name()
 				local privs = minetest.get_player_privs(name)
-
-
 				if under.name == "mcl_mobspawners:spawner" then
 					if minetest.is_protected(pointed_thing.under, name) then
 						minetest.record_protection_violation(pointed_thing.under, name)
@@ -513,14 +512,7 @@ function mcl_mobs.register_egg(mob, desc, background_color, overlay_color, addeg
 						minetest.chat_send_player(name, S("You need the “maphack” privilege to change the mob spawner."))
 						return itemstack
 					end
-
+					mcl_mobspawners.setup_spawner(pointed_thing.under, itemstack:get_name())
-					local dim = mcl_worlds.pos_to_dimension(placer:get_pos())
-					local mob_light_lvl = {mcl_mobs:mob_light_lvl(itemstack:get_name(),dim)}
-
-					--minetest.log("min light: " .. mob_light_lvl[1])
-					--minetest.log("max light: " .. mob_light_lvl[2])
-
-					mcl_mobspawners.setup_spawner(pointed_thing.under, itemstack:get_name(), mob_light_lvl[1], mob_light_lvl[2])
 					if not minetest.is_creative_enabled(name) then
 						itemstack:take_item()
 					end
@@ -558,7 +550,7 @@ function mcl_mobs.register_egg(mob, desc, background_color, overlay_color, addeg
 						nametag = string.sub(nametag, 1, MAX_MOB_NAME_LENGTH)
 					end
 					ent.nametag = nametag
-					ent:update_tag()
+					update_tag(ent)
 				end
 
 				-- if not in creative then take item

mods/ENTITIES/mcl_mobs/locale/mcl_mobs.dk.tr

@@ -1,11 +0,0 @@
-# textdomain: mcl_mobs
-Peaceful mode active! No monsters will spawn.=Fredelig tilstand aktiveret! Ingen monstre vil spawne.
-This allows you to place a single mob.=Dette gør dig i stand til at placere et enkelt monster.
-Just place it where you want the mob to appear. Animals will spawn tamed, unless you hold down the sneak key while placing. If you place this on a mob spawner, you change the mob it spawns.=Placér det blot der hvor du ønsker, at monsteret skal komme. Dyr vil spawne tamme, medmindre du holder snige-knappen nede mens du placerer dem. Hvis du placerer denne på et monsterspawn, ændrer du hvilket monster det spawner.
-You need the “maphack” privilege to change the mob spawner.=Du skal have "maphack" privilegier for at ændre monsterspawneren.
-Name Tag=Navneskilt
-A name tag is an item to name a mob.=Et navneskilt bruges til at navngive et monster.
-Before you use the name tag, you need to set a name at an anvil. Then you can use the name tag to name a mob. This uses up the name tag.=Før du bruger navneskiltet, skal du vælge navnet ved en ambolt. Derefter kan du bruge navneskiltet til at navngive et monster. Dette opbruger navneskiltet.
-Only peaceful mobs allowed!=Kun fredelige monstre er tilladt!
-Give names to mobs=Giv navne til monstre
-Set name at anvil=Vælg navn ved en ambolt.

mods/ENTITIES/mcl_mobs/locale/mcl_mobs.fr.tr

@@ -9,5 +9,3 @@ Before you use the name tag, you need to set a name at an anvil. Then you can us
 Only peaceful mobs allowed!=Seuls les mobs pacifiques sont autorisées!
 Give names to mobs=Donne des noms aux mobs
 Set name at anvil=Définir le nom sur l'enclume
-Removes specified mobs except nametagged and tamed ones. For the second parameter, use nametagged/tamed to select only nametagged/tamed mobs, or a range to specify a maximum distance from the player.=Enlève les mobs spécifiés sauf ceux qui sont nommés et apprivoisés. Pour le deuxième paramètre, utiliser nametagged/tamed pour ne sélectionner que les mobs nommés/apprivoisés, ou une distance pour spécifier la distance maximale par rapport au joueur.
-Default usage. Clearing hostile mobs. For more options please type: /help clearmobs=Usage par défaut. Enlève les mobs hostiles. Pour plus d'options saisir : /help clearmobs

mods/ENTITIES/mcl_mobs/locale/mcl_mobs.ru.tr

@@ -1,11 +1,11 @@
 # textdomain: mcl_mobs
-Peaceful mode active! No monsters will spawn.=Мирный режим включён! Чудовища не будут появляться.
+Peaceful mode active! No monsters will spawn.=Мирный режим включён! Монстры не будут появляться.
-This allows you to place a single mob.=Позволяет вам породить одно существо.
+This allows you to place a single mob.=Позволяет вам разместить одного моба.
-Just place it where you want the mob to appear. Animals will spawn tamed, unless you hold down the sneak key while placing. If you place this on a mob spawner, you change the mob it spawns.=Просто нажмите на блок, где хотите, чтобы появилось существо. Животные будут появляться уже прирученные, если это не нужно, удерживайте клавишу [Красться] при размещении. Если использовать на порождателе, тогда существо будет изменено.
+Just place it where you want the mob to appear. Animals will spawn tamed, unless you hold down the sneak key while placing. If you place this on a mob spawner, you change the mob it spawns.=Просто поместите это туда, где хотите, чтобы появился моб. Животные будут появляться уже прирученные, если это не нужно, удерживайте клавишу [Красться] при размещении. Если поместить это на спаунер, появляющийся из него моб будет изменён.
-You need the “maphack” privilege to change the mob spawner.=Вам нужно обладать привилегией «maphack», чтобы изменить порождатель существ.
+You need the “maphack” privilege to change the mob spawner.=Вам нужно обладать привилегией “maphack”, чтобы изменить спаунер моба.
 Name Tag=Именная бирка
-A name tag is an item to name a mob.=Именная бирка — это предмет, чтобы дать существу имя.
+A name tag is an item to name a mob.=Именная бирка это предмет, чтобы дать мобу имя.
 Before you use the name tag, you need to set a name at an anvil. Then you can use the name tag to name a mob. This uses up the name tag.=Прежде чем использовать именную бирку, нужно задать имя на наковальне. Тогда вы сможете использовать бирку, чтобы дать имя мобу.
-Only peaceful mobs allowed!=Разрешены только мирные существа!
+Only peaceful mobs allowed!=Разрешены только мирные мобы!
-Give names to mobs=Даёт имена существам
+Give names to mobs=Даёт имена мобам
 Set name at anvil=Задайте имя при помощи наковальни

mods/ENTITIES/mcl_mobs/locale/template.txt

@@ -9,5 +9,3 @@ Before you use the name tag, you need to set a name at an anvil. Then you can us
 Only peaceful mobs allowed!=
 Give names to mobs=
 Set name at anvil=
-Removes specified mobs except nametagged and tamed ones. For the second parameter, use nametagged/tamed to select only nametagged/tamed mobs, or a range to specify a maximum distance from the player.=
-Default usage. Clearing hostile mobs. For more options please type: /help clearmobs=

mods/ENTITIES/mcl_mobs/mod.conf

@@ -2,4 +2,4 @@ name = mcl_mobs
 author = PilzAdam
 description = Adds a mob API for mods to add animals or monsters, etc.
 depends = mcl_particles
-optional_depends = mcl_weather, mcl_explosions, mcl_hunger, mcl_worlds, invisibility, lucky_block, cmi, doc_identifier, mcl_armor, mcl_portals, mcl_experience, mcl_sculk
+optional_depends = mcl_weather, mcl_explosions, mcl_hunger, mcl_worlds, invisibility, lucky_block, cmi, doc_identifier, mcl_armor, mcl_portals, mcl_experience

mods/ENTITIES/mcl_mobs/movement.lua

@@ -3,13 +3,13 @@ local mob_class = mcl_mobs.mob_class
 local DEFAULT_FALL_SPEED = -9.81*1.5
 local FLOP_HEIGHT = 6
 local FLOP_HOR_SPEED = 1.5
-
-local CHECK_HERD_FREQUENCY = 4
-
 local PATHFINDING = "gowp"
 
+local node_ice = "mcl_core:ice"
+local node_snowblock = "mcl_core:snowblock"
 local node_snow = "mcl_core:snow"
 
+
 local mobs_griefing = minetest.settings:get_bool("mobs_griefing") ~= false
 
 local atann = math.atan
@@ -21,19 +21,14 @@ local function atan(x)
 	end
 end
 
-local registered_fallback_node = minetest.registered_nodes[mcl_mobs.fallback_node]
-
 -- get node but use fallback for nil or unknown
 local node_ok = function(pos, fallback)
+	fallback = fallback or mcl_mobs.fallback_node
 	local node = minetest.get_node_or_nil(pos)
 	if node and minetest.registered_nodes[node.name] then
 		return node
 	end
-	if fallback then
+	return minetest.registered_nodes[fallback]
-		return minetest.registered_nodes[fallback]
-	else
-		return registered_fallback_node
-	end
 end
 
 -- Returns true is node can deal damage to self
@@ -206,21 +201,19 @@ end
 
 -- is mob facing a cliff or danger
 function mob_class:is_at_cliff_or_danger()
-	if self.fear_height == 0 or self._jumping_cliff or self._can_jump_cliff or not self.object:get_luaentity() then -- 0 for no falling protection!
+	if self.fear_height == 0 or self:can_jump_cliff() or self._jumping_cliff or not self.object:get_luaentity() then -- 0 for no falling protection!
 		return false
 	end
 
 	local yaw = self.object:get_yaw()
 	local dir_x = -math.sin(yaw) * (self.collisionbox[4] + 0.5)
 	local dir_z = math.cos(yaw) * (self.collisionbox[4] + 0.5)
-
 	local pos = self.object:get_pos()
 	local ypos = pos.y + self.collisionbox[2] -- just above floor
 
 	local free_fall, blocker = minetest.line_of_sight(
-			vector.new(pos.x + dir_x, ypos, pos.z + dir_z),
+		{x = pos.x + dir_x, y = ypos, z = pos.z + dir_z},
-			vector.new(pos.x + dir_x, ypos - self.fear_height, pos.z + dir_z))
+		{x = pos.x + dir_x, y = ypos - self.fear_height, z = pos.z + dir_z})
-
 	if free_fall then
 		return true
 	else
@@ -242,15 +235,7 @@ end
 
 -- copy the 'mob facing cliff_or_danger check' from above, and rework to avoid water
 function mob_class:is_at_water_danger()
-	if self.water_damage == 0 and self.breath_max == -1 then
+	if not self.object:get_luaentity() or self:can_jump_cliff() or self._jumping_cliff then
-		--minetest.log("Do not need a water check for: " .. self.name)
-		return
-	end
-
-	local in_water_danger = self:is_node_waterhazard(self.standing_in) or self:is_node_waterhazard(self.standing_on)
-	if in_water_danger then return false end -- If you're in trouble, do not stop
-
-	if not self.object:get_luaentity() or self._jumping_cliff or self._can_jump_cliff then
 		return false
 	end
 	local yaw = self.object:get_yaw()
@@ -265,57 +250,54 @@ function mob_class:is_at_water_danger()
 
 	local ypos = pos.y + self.collisionbox[2] -- just above floor
 
-	local los, blocker = minetest.line_of_sight(
+	local free_fall, blocker = minetest.line_of_sight(
-		vector.new(pos.x + dir_x, ypos, pos.z + dir_z),
+		{x = pos.x + dir_x, y = ypos, z = pos.z + dir_z},
-		vector.new(pos.x + dir_x, ypos - 3, pos.z + dir_z))
+		{x = pos.x + dir_x, y = ypos - 3, z = pos.z + dir_z})
-
+	if free_fall then
-	if not los then
+		return true
+	else
 		local bnode = minetest.get_node(blocker)
 		local waterdanger = self:is_node_waterhazard(bnode.name)
-
+		if
-		if waterdanger and not in_water_danger then
+			waterdanger and (self:is_node_waterhazard(self.standing_in) or self:is_node_waterhazard( self.standing_on)) then
+			return false
+		elseif waterdanger and (self:is_node_waterhazard(self.standing_in) or self:is_node_waterhazard(self.standing_on)) == false then
 			return true
+		else
+			local def = minetest.registered_nodes[bnode.name]
+			if def and def.walkable then
+				return false
+			end
 		end
 	end
 
 	return false
 end
 
-function mob_class:env_danger_movement_checks(player_in_active_range)
+function mob_class:env_danger_movement_checks(dtime)
 	local yaw = 0
-
+	if self:is_at_water_danger() and self.state ~= "attack" then
-	if not player_in_active_range then return end
+		if math.random(1, 10) <= 6 then
-
-	if self.state == PATHFINDING
-			or self.state == "attack"
-			or self.state == "stand"
-			or self.state == "runaway" then
-		return
-	end
-
-	if self:is_at_water_danger() then
-		--minetest.log("At water danger for mob, stop?: " .. self.name)
-		if math.random(1, 10) <= 7 then
-			if self.state ~= "stand" then
-				self:set_velocity(0)
-				self.state = "stand"
-				self:set_animation( "stand")
-			end
-			yaw = yaw + math.random(-0.5, 0.5)
-			yaw = self:set_yaw( yaw, 8)
-			return
-		end
-	end
-
-	--[[if self:is_at_cliff_or_danger(can_jump_cliff) then
-		if self.state ~= "stand" then
 			self:set_velocity(0)
 			self.state = "stand"
 			self:set_animation( "stand")
+			yaw = yaw + math.random(-0.5, 0.5)
+			yaw = self:set_yaw( yaw, 8)
 		end
+	else
+		-- This code should probably be moved to movement code
+		if self.move_in_group ~= false then
+			self:check_herd(dtime)
+		end
+	end
+
+	if self:is_at_cliff_or_danger() then
+		self:set_velocity(0)
+		self.state = "stand"
+		self:set_animation( "stand")
 		local yaw = self.object:get_yaw() or 0
 		yaw = self:set_yaw( yaw + 0.78, 8)
-	end--]]
+	end
 end
 
 -- jump if facing a solid node (not fences or gates)
@@ -355,11 +337,9 @@ function mob_class:do_jump()
 		jump_c_multiplier = v2/self.walk_velocity/2
 	end
 
-	local yaw_dir = minetest.yaw_to_dir(self.object:get_yaw())
-
 	-- where is front
-	local dir_x = -math.sin(yaw) * (self.collisionbox[4] + 0.5)*jump_c_multiplier+yaw_dir.x
+	local dir_x = -math.sin(yaw) * (self.collisionbox[4] + 0.5)*jump_c_multiplier+0.6
-	local dir_z = math.cos(yaw) * (self.collisionbox[4] + 0.5)*jump_c_multiplier+yaw_dir.z
+	local dir_z = math.cos(yaw) * (self.collisionbox[4] + 0.5)*jump_c_multiplier+0.6
 
 	-- what is in front of mob?
 	nod = node_ok({
@@ -388,7 +368,7 @@ function mob_class:do_jump()
 	end
 
 	local ndef = minetest.registered_nodes[nod.name]
-	if self.walk_chance == 0 or ndef and ndef.walkable or self._can_jump_cliff then
+	if self.walk_chance == 0 or ndef and ndef.walkable or self:can_jump_cliff() then
 
 		if minetest.get_item_group(nod.name, "fence") == 0
 		and minetest.get_item_group(nod.name, "fence_gate") == 0
@@ -398,7 +378,7 @@ function mob_class:do_jump()
 
 			v.y = self.jump_height + 0.1 * 3
 
-			if self._can_jump_cliff then
+			if self:can_jump_cliff() then
 				v=vector.multiply(v, vector.new(2.8,1,2.8))
 			end
 
@@ -482,7 +462,6 @@ end
 -- find and replace what mob is looking for (grass, wheat etc.)
 function mob_class:replace_node(pos)
 
-
 	if not self.replace_rate
 	or not self.replace_what
 	or self.child == true
@@ -513,21 +492,18 @@ function mob_class:replace_node(pos)
 
 		local oldnode = {name = what, param2 = node.param2}
 		local newnode = {name = with, param2 = node.param2}
-		local on_replace_return = false
+		local on_replace_return
+
 		if self.on_replace then
 			on_replace_return = self.on_replace(self, pos, oldnode, newnode)
 		end
 
-
 		if on_replace_return ~= false then
 
 			if mobs_griefing then
-				minetest.after(self.replace_delay, function()
+				minetest.set_node(pos, newnode)
-					if self and self.object and self.object:get_velocity() and self.health > 0 then
-						minetest.set_node(pos, newnode)
-					end
-				end)
 			end
+
 		end
 	end
 end
@@ -637,52 +613,76 @@ function mob_class:check_runaway_from()
 end
 
 
--- follow player if owner or holding item
+-- follow player if owner or holding item, if fish outta water then flop
-function mob_class:check_follow()
+function mob_class:follow_flop()
+
 	-- find player to follow
-	if (self.follow ~= "" or self.order == "follow") and not self.following
+	if (self.follow ~= ""
+	or self.order == "follow")
+	and not self.following
 	and self.state ~= "attack"
 	and self.order ~= "sit"
 	and self.state ~= "runaway" then
+
 		local s = self.object:get_pos()
 		local players = minetest.get_connected_players()
+
 		for n = 1, #players do
-			if (self:object_in_range(players[n])) and not mcl_mobs.invis[ players[n]:get_player_name() ] then
+
+			if (self:object_in_range(players[n]))
+			and not mcl_mobs.invis[ players[n]:get_player_name() ] then
+
 				self.following = players[n]
+
 				break
 			end
 		end
 	end
 
-	if self.type == "npc" and self.order == "follow"
+	if self.type == "npc"
-			and self.state ~= "attack" and self.order ~= "sit" and self.owner ~= "" then
+	and self.order == "follow"
+	and self.state ~= "attack"
+	and self.order ~= "sit"
+	and self.owner ~= "" then
 
-		if self.following and self.owner and self.owner ~= self.following:get_player_name() then
+		-- npc stop following player if not owner
+		if self.following
+		and self.owner
+		and self.owner ~= self.following:get_player_name() then
 			self.following = nil
 		end
 	else
 		-- stop following player if not holding specific item,
 		-- mob is horny, fleeing or attacking
-		if self.following and self.following:is_player()
+		if self.following
-				and (self:follow_holding(self.following) == false or self.horny or self.state == "runaway") then
+		and self.following:is_player()
+		and (self:follow_holding(self.following) == false or
+		self.horny or self.state == "runaway") then
 			self.following = nil
 		end
+
 	end
 
 	-- follow that thing
 	if self.following then
-		local s = self.object:get_pos()
 
+		local s = self.object:get_pos()
 		local p
+
 		if self.following:is_player() then
+
 			p = self.following:get_pos()
+
 		elseif self.following.object then
+
 			p = self.following.object:get_pos()
 		end
 
 		if p then
+
 			local dist = vector.distance(p, s)
 
+			-- dont follow if out of range
 			if (not self:object_in_range(self.following)) then
 				self.following = nil
 			else
@@ -692,12 +692,17 @@ function mob_class:check_follow()
 				}
 
 				local yaw = (atan(vec.z / vec.x) +math.pi/ 2) - self.rotate
+
 				if p.x > s.x then yaw = yaw +math.pi end
+
 				self:set_yaw( yaw, 2.35)
 
 				-- anyone but standing npc's can move along
-				if dist > 3 and self.order ~= "stand" then
+				if dist > 3
+				and self.order ~= "stand" then
+
  					self:set_velocity(self.follow_velocity)
+
 					if self.walk_chance ~= 0 then
 						self:set_animation( "run")
 					end
@@ -705,18 +710,17 @@ function mob_class:check_follow()
 					self:set_velocity(0)
 					self:set_animation( "stand")
 				end
+
 				return
 			end
 		end
 	end
-end
 
-function mob_class:flop()
 	-- swimmers flop when out of their element, and swim again when back in
 	if self.fly then
 		local s = self.object:get_pos()
+		if self:flight_check( s) == false then
 
-		if self:flight_check(s) == false then
 			self.state = "flop"
 			self.object:set_acceleration({x = 0, y = DEFAULT_FALL_SPEED, z = 0})
 
@@ -735,10 +739,11 @@ function mob_class:flop()
 			end
 
 			self:set_animation( "stand", true)
+
 			return
 		elseif self.state == "flop" then
 			self.state = "stand"
-			self.object:set_acceleration(vector.zero())
+			self.object:set_acceleration({x = 0, y = 0, z = 0})
 			self:set_velocity(0)
 		end
 	end
@@ -765,12 +770,9 @@ end
 local check_herd_timer = 0
 function mob_class:check_herd(dtime)
 	local pos = self.object:get_pos()
-	if not pos or self.state == "attack" then return end
+	if not pos then return end
-	-- Does any mob not move in group. Weird check for something not set?
-	if self.move_in_group == false then return end
-
 	check_herd_timer = check_herd_timer + dtime
-	if check_herd_timer < CHECK_HERD_FREQUENCY then return end
+	if check_herd_timer < 4 then return end
 	check_herd_timer = 0
 	for _,o in pairs(minetest.get_objects_inside_radius(pos,self.view_range)) do
 		local l = o:get_luaentity()
@@ -901,7 +903,7 @@ function mob_class:do_states_walk()
 	end
 end
 
-function mob_class:do_states_stand(player_in_active_range)
+function mob_class:do_states_stand()
 	local yaw = self.object:get_yaw() or 0
 
 	if math.random(1, 4) == 1 then
@@ -945,16 +947,14 @@ function mob_class:do_states_stand(player_in_active_range)
 	if self.order == "stand" or self.order == "sleep" or self.order == "work" then
 
 	else
-		if player_in_active_range then
+		if self.walk_chance ~= 0
-			if self.walk_chance ~= 0
+				and self.facing_fence ~= true
-					and self.facing_fence ~= true
+				and math.random(1, 100) <= self.walk_chance
-					and math.random(1, 100) <= self.walk_chance
+				and self:is_at_cliff_or_danger() == false then
-					and self:is_at_cliff_or_danger() == false then
 
-				self:set_velocity(self.walk_velocity)
+			self:set_velocity(self.walk_velocity)
-				self.state = "walk"
+			self.state = "walk"
-				self:set_animation( "walk")
+			self:set_animation( "walk")
-			end
 		end
 	end
 end

mods/ENTITIES/mcl_mobs/physics.lua

@@ -184,7 +184,7 @@ function mob_class:collision()
 end
 
 function mob_class:check_death_and_slow_mob()
-	local d = 0.7
+	local d = 0.85
 	local dying = self:check_dying()
 	if dying then d = 0.92 end
 
@@ -198,8 +198,6 @@ end
 
 -- move mob in facing direction
 function mob_class:set_velocity(v)
-	if not v then return end
-
 	local c_x, c_y = 0, 0
 
 	-- can mob be pushed, if so calculate direction
@@ -209,15 +207,18 @@ function mob_class:set_velocity(v)
 
 	-- halt mob if it has been ordered to stay
 	if self.order == "stand" or self.order == "sit" then
-		self.acc = vector.zero()
+	  self.acc=vector.new(0,0,0)
-		return
+	  return
 	end
 
 	local yaw = (self.object:get_yaw() or 0) + self.rotate
 	local vv = self.object:get_velocity()
-
+	if vv then
-	if vv and yaw then
+		self.acc={
-		self.acc = vector.new(((math.sin(yaw) * -v) + c_x) * .4, 0, ((math.cos(yaw) * v) + c_y) * .4)
+		  x = ((math.sin(yaw) * -v) + c_x)*.27,
+		  y = 0,
+		  z = ((math.cos(yaw) * v) + c_y)*.27,
+		}
 	end
 end
 
@@ -327,9 +328,9 @@ function mob_class:set_yaw(yaw, delay, dtime)
 	end
 
 	if math.deg(yaw) > 360 then
-		yaw=math.rad(math.deg(yaw)%360)
+		yaw=yaw%360
 	elseif math.deg(yaw) < 0 then
-		yaw=math.rad(((360*5)-math.deg(yaw))%360)
+		yaw=((360*5)-yaw)%360
 	end
 
 	--calculate the shortest way to turn to find our target
@@ -353,7 +354,7 @@ function mob_class:set_yaw(yaw, delay, dtime)
 		ddtime = dtime
 	end
 
-	if math.abs(target_shortest_path_nums) > 10 then
+	if math.abs(target_shortest_path_nums) > 5 then
 		self.object:set_yaw(self.object:get_yaw()+(target_shortest_path*(3.6*ddtime)))
 		if self.acc then
 			self.acc=vector.rotate_around_axis(self.acc,vector.new(0,1,0), target_shortest_path*(3.6*ddtime))
@@ -487,21 +488,9 @@ function mob_class:check_for_death(cause, cmi_cause)
 			self:item_drop(cooked, looting)
 
 			if ((not self.child) or self.type ~= "animal") and (minetest.get_us_time() - self.xp_timestamp <= math.huge) then
-				local pos = self.object:get_pos()
+				mcl_experience.throw_xp(self.object:get_pos(), math.random(self.xp_min, self.xp_max))
-				local xp_amount = math.random(self.xp_min, self.xp_max)
-
-				if not mcl_sculk.handle_death(pos, xp_amount) then
-					--minetest.log("Xp not thrown")
-					if minetest.is_creative_enabled("") ~= true then
-						mcl_experience.throw_xp(pos, xp_amount)
-					end
-				else
-					--minetest.log("xp thrown")
-				end
 			end
 		end
-
-
 	end
 
 	-- execute custom death function
@@ -521,12 +510,6 @@ function mob_class:check_for_death(cause, cmi_cause)
 		end
 	end
 
-	if self.jockey or self.riden_by_jock then
-		self.riden_by_jock = nil
-		self.jockey = nil
-	end
-
-
 	local collisionbox
 	if self.collisionbox then
 		collisionbox = table.copy(self.collisionbox)
@@ -633,34 +616,24 @@ function mob_class:do_env_damage()
 		return true
 	end
 
-	local node = minetest.get_node(pos)
+	local sunlight = minetest.get_natural_light(pos, self.time_of_day)
-	if node then
+
-		if node.name ~= "ignore" then
+	-- bright light harms mob
-			-- put below code in this block if we can prove that unloaded maps are causing crash.
+	if self.light_damage ~= 0 and (sunlight or 0) > 12 then
-			-- it should warn then error
+		if self:deal_light_damage(pos, self.light_damage) then
-		else
+			return true
-			--minetest.log("warning", "Pos is ignored: " .. dump(pos))
 		end
-
+	end
-		local sunlight = mcl_util.get_natural_light(pos, self.time_of_day)
+	local _, dim = mcl_worlds.y_to_layer(pos.y)
-
+	if (self.sunlight_damage ~= 0 or self.ignited_by_sunlight) and (sunlight or 0) >= minetest.LIGHT_MAX and dim == "overworld" then
-		if self.light_damage ~= 0 and (sunlight or 0) > 12 then
+		if self.armor_list and not self.armor_list.helmet or not self.armor_list or self.armor_list and self.armor_list.helmet and self.armor_list.helmet == "" then
-			if self:deal_light_damage(pos, self.light_damage) then
+			if self.ignited_by_sunlight then
+				mcl_burning.set_on_fire(self.object, 10)
+			else
+				self:deal_light_damage(pos, self.sunlight_damage)
 				return true
 			end
 		end
-		local _, dim = mcl_worlds.y_to_layer(pos.y)
-		if (self.sunlight_damage ~= 0 or self.ignited_by_sunlight) and (sunlight or 0) >= minetest.LIGHT_MAX and dim == "overworld" then
-			if self.armor_list and not self.armor_list.helmet or not self.armor_list or self.armor_list and self.armor_list.helmet and self.armor_list.helmet == "" then
-				if self.ignited_by_sunlight then
-					mcl_burning.set_on_fire(self.object, 10)
-				else
-					self:deal_light_damage(pos, self.sunlight_damage)
-					return true
-				end
-			end
-		end
-
 	end
 
 	local y_level = self.collisionbox[2]
@@ -820,19 +793,11 @@ function mob_class:do_env_damage()
 	return self:check_for_death("unknown", {type = "unknown"})
 end
 
-function mob_class:step_damage (dtime, pos)
+function mob_class:env_damage (dtime, pos)
-	if not self.fire_resistant then
-		mcl_burning.tick(self.object, dtime, self)
-		if not self.object:get_pos() then return true end -- mcl_burning.tick may remove object immediately
-
-		if self:check_for_death("fire", {type = "fire"}) then
-			return true
-		end
-	end
-
 	-- environmental damage timer (every 1 second)
 	self.env_damage_timer = self.env_damage_timer + dtime
 
+
 	if self.env_damage_timer > 1 then
 		self.env_damage_timer = 0
 
@@ -911,36 +876,47 @@ function mob_class:falling(pos)
 	-- floating in water (or falling)
 	local v = self.object:get_velocity()
 	if v then
-		local new_acceleration
-
 		if v.y > 0 then
 			-- apply gravity when moving up
-			new_acceleration = vector.new(0, DEFAULT_FALL_SPEED, 0)
+			self.object:set_acceleration({
+				x = 0,
+				y = DEFAULT_FALL_SPEED,
+				z = 0
+			})
+
 		elseif v.y <= 0 and v.y > self.fall_speed then
 			-- fall downwards at set speed
-			new_acceleration = vector.new(0, self.fall_speed, 0)
+			self.object:set_acceleration({
+				x = 0,
+				y = self.fall_speed,
+				z = 0
+			})
 		else
 			-- stop accelerating once max fall speed hit
-			new_acceleration =vector.zero()
+			self.object:set_acceleration({x = 0, y = 0, z = 0})
 		end
-
-		self.object:set_acceleration(new_acceleration)
 	end
 
 	local acc = self.object:get_acceleration()
 
-	local registered_node = minetest.registered_nodes[node_ok(pos).name]
+	if minetest.registered_nodes[node_ok(pos).name].groups.lava then
-
-	if registered_node.groups.lava then
 		if acc and self.floats_on_lava == 1 then
-			self.object:set_acceleration(vector.new(0, -self.fall_speed / (math.max(1, v.y) ^ 2), 0))
+			self.object:set_acceleration({
+				x = 0,
+				y = -self.fall_speed / (math.max(1, v.y) ^ 2),
+				z = 0
+			})
 		end
 	end
 
 	-- in water then float up
-	if registered_node.groups.water then
+	if minetest.registered_nodes[node_ok(pos).name].groups.water then
 		if acc and self.floats == 1 then
-			self.object:set_acceleration(vector.new(0, -self.fall_speed / (math.max(1, v.y) ^ 2), 0))
+			self.object:set_acceleration({
+				x = 0,
+				y = -self.fall_speed / (math.max(1, v.y) ^ 2),
+				z = 0
+			})
 		end
 	else
 		-- fall damage onto solid ground
@@ -1013,10 +989,10 @@ function mob_class:check_dying()
 	end
 end
 
-function mob_class:check_suspend(player_in_active_range)
+function mob_class:check_suspend()
 	local pos = self.object:get_pos()
 
-	if pos and not player_in_active_range then
+	if pos and not self:player_in_active_range() then
 		local node_under = node_ok(vector.offset(pos,0,-1,0)).name
 
 		self:set_animation( "stand", true)
@@ -1024,8 +1000,11 @@ function mob_class:check_suspend(player_in_active_range)
 		local acc = self.object:get_acceleration()
 		if acc then
 			if acc.y > 0 or node_under ~= "air" then
-				self.object:set_acceleration(vector.zero())
+				self.object:set_acceleration(vector.new(0,0,0))
-				self.object:set_velocity(vector.zero())
+				self.object:set_velocity(vector.new(0,0,0))
+			end
+			if acc.y == 0 and node_under == "air" then
+				self:falling(pos)
 			end
 		end
 		return true

mods/ENTITIES/mcl_mobs/spawning.lua

@@ -27,11 +27,8 @@ local table_remove   = table.remove
 local pairs = pairs
 
 local LOGGING_ON = minetest.settings:get_bool("mcl_logging_mobs_spawning", false)
-local function mcl_log (message, property)
+local function mcl_log (message)
 	if LOGGING_ON then
-		if property then
-			message = message .. ": " .. dump(property)
-		end
 		mcl_util.mcl_log (message, "[Mobs spawn]", true)
 	end
 end
@@ -55,25 +52,21 @@ local MOB_CAP_INNER_RADIUS = 32
 local aoc_range = 136
 
 local MISSING_CAP_DEFAULT = 15
-local MOBS_CAP_CLOSE = 10
+local MOBS_CAP_CLOSE = 5
 
 local SPAWN_MAPGEN_LIMIT  = mcl_vars.mapgen_limit - 150
 
 local mob_cap = {
 	hostile = tonumber(minetest.settings:get("mcl_mob_cap_monster")) or 70,
-	passive = tonumber(minetest.settings:get("mcl_mob_cap_animal")) or 10,
+	passive = tonumber(minetest.settings:get("mcl_mob_cap_animal")) or 13,
 	ambient = tonumber(minetest.settings:get("mcl_mob_cap_ambient")) or 15,
 	water = tonumber(minetest.settings:get("mcl_mob_cap_water")) or 8,
-	water_ambient = tonumber(minetest.settings:get("mcl_mob_cap_water_ambient")) or 20,
+	water_ambient = tonumber(minetest.settings:get("mcl_mob_cap_water_ambient")) or 20, --currently unused
-	water_underground = tonumber(minetest.settings:get("mcl_mob_cap_water_underground")) or 5,
-	axolotl = tonumber(minetest.settings:get("mcl_mob_cap_axolotl")) or 2, -- TODO should be 5 when lush caves added
 	player = tonumber(minetest.settings:get("mcl_mob_cap_player")) or 75,
-	global_hostile = tonumber(minetest.settings:get("mcl_mob_cap_hostile")) or 300,
-	global_non_hostile = tonumber(minetest.settings:get("mcl_mob_cap_non_hostile")) or 300,
 	total = tonumber(minetest.settings:get("mcl_mob_cap_total")) or 500,
 }
 
-local peaceful_percentage_spawned = tonumber(minetest.settings:get("mcl_mob_peaceful_percentage_spawned")) or 30
+local peaceful_percentage_spawned = tonumber(minetest.settings:get("mcl_mob_peaceful_percentage_spawned")) or 35
 local peaceful_group_percentage_spawned = tonumber(minetest.settings:get("mcl_mob_peaceful_group_percentage_spawned")) or 15
 local hostile_group_percentage_spawned = tonumber(minetest.settings:get("mcl_mob_hostile_group_percentage_spawned")) or 20
 
@@ -347,32 +340,15 @@ local function count_mobs_all(categorise_by, pos)
 end
 
 local function count_mobs_total_cap(mob_type)
-	local total = 0
 	local num = 0
-	local hostile = 0
-	local non_hostile = 0
 	for _,l in pairs(minetest.luaentities) do
 		if l.is_mob then
-			total = total + 1
+			if ( mob_type == nil or l.type == mob_type ) and l.can_despawn and not l.nametag then
-			local nametagged = l.nametag and l.nametag ~= ""
-			if ( mob_type == nil or l.type == mob_type ) and not nametagged then
-				if l.spawn_class == "hostile" then
-					hostile = hostile + 1
-				else
-					non_hostile = non_hostile + 1
-				end
 				num = num + 1
-			else
-				mcl_log("l.name", l.name)
-				mcl_log("l.nametag", l.nametag)
-
 			end
 		end
 	end
-	mcl_log("Total mobs", total)
+	return num
-	mcl_log("hostile", hostile)
-	mcl_log("non_hostile", non_hostile)
-	return num, non_hostile, hostile
 end
 
 local function output_mob_stats(mob_counts, total_mobs, chat_display)
@@ -437,8 +413,6 @@ WARNING: BIOME INTEGRATION NEEDED -> How to get biome through lua??
 
 --this is where all of the spawning information is kept
 local spawn_dictionary = {}
---this is where all of the spawning information  is kept for mobs that don't naturally spawn
-local non_spawn_dictionary = {}
 local summary_chance = 0
 
 function mcl_mobs:spawn_setup(def)
@@ -504,69 +478,7 @@ function mcl_mobs:spawn_setup(def)
 	summary_chance = summary_chance + chance
 end
 
-function mcl_mobs:mob_light_lvl(mob_name, dimension)
+function mcl_mobs:spawn_specific(name, dimension, type_of_spawning, biomes, min_light, max_light, interval, chance, aoc, min_height, max_height, day_toggle, on_spawn)
-	local spawn_dictionary_consolidated = {}
-
-	if non_spawn_dictionary[mob_name] then
-		local mob_dimension = non_spawn_dictionary[mob_name][dimension]
-		if mob_dimension then
-			--minetest.log("Found in non spawn dictionary for dimension")
-			return mob_dimension.min_light, mob_dimension.max_light
-		else
-			--minetest.log("Found in non spawn dictionary but not for dimension")
-			local overworld_non_spawn_def = non_spawn_dictionary[mob_name]["overworld"]
-			if overworld_non_spawn_def then
-				return overworld_non_spawn_def.min_light, overworld_non_spawn_def.max_light
-			end
-		end
-	else
-		--minetest.log("must be in spawning dictionary")
-		for i,v in pairs(spawn_dictionary) do
-			local current_mob_name = spawn_dictionary[i].name
-			local current_mob_dim = spawn_dictionary[i].dimension
-			if mob_name == current_mob_name then
-				if not spawn_dictionary_consolidated[current_mob_name] then
-					spawn_dictionary_consolidated[current_mob_name] = {}
-				end
-				spawn_dictionary_consolidated[current_mob_name][current_mob_dim] = {
-					["min_light"] = spawn_dictionary[i].min_light,
-					["max_light"] = spawn_dictionary[i].max_light
-				}
-			end
-		end
-
-		if spawn_dictionary_consolidated[mob_name] then
-			--minetest.log("is in consolidated")
-			local mob_dimension = spawn_dictionary_consolidated[mob_name][dimension]
-			if mob_dimension then
-				--minetest.log("found for dimension")
-				return mob_dimension.min_light, mob_dimension.max_light
-			else
-				--minetest.log("not found for dimension, use overworld def")
-				local mob_dimension_default = spawn_dictionary_consolidated[mob_name]["overworld"]
-				if mob_dimension_default then
-					return mob_dimension_default.min_light, mob_dimension_default.max_light
-				end
-			end
-		else
-			--minetest.log("not in consolidated")
-		end
-	end
-
-	minetest.log("action", "There are no light levels for mob (" .. tostring(mob_name) .. ") in dimension (" .. tostring(dimension) .. "). Return defaults")
-	return 0, minetest.LIGHT_MAX+1
-end
-
-function mcl_mobs:non_spawn_specific(mob_name,dimension,min_light,max_light)
-	table.insert(non_spawn_dictionary, mob_name)
-	non_spawn_dictionary[mob_name] = {
-		[dimension] = {
-			min_light = min_light , max_light = max_light
-		}
-	}
-end
-
-function mcl_mobs:spawn_specific(name, dimension, type_of_spawning, biomes, min_light, max_light, interval, chance, aoc, min_height, max_height, day_toggle, on_spawn, check_position)
 
 	-- Do mobs spawn at all?
 	if not mobs_spawn then
@@ -603,7 +515,6 @@ function mcl_mobs:spawn_specific(name, dimension, type_of_spawning, biomes, min_
 	spawn_dictionary[key]["min_height"] = min_height
 	spawn_dictionary[key]["max_height"] = max_height
 	spawn_dictionary[key]["day_toggle"] = day_toggle
-	spawn_dictionary[key]["check_position"] = check_position
 
 	summary_chance = summary_chance + chance
 end
@@ -678,17 +589,14 @@ end
 
 
 local function spawn_check(pos, spawn_def)
-	if not spawn_def or not pos then return end
+	if not spawn_def then return end
-
 	dbg_spawn_attempts = dbg_spawn_attempts + 1
 	local dimension = mcl_worlds.pos_to_dimension(pos)
 	local mob_def = minetest.registered_entities[spawn_def.name]
 	local mob_type = mob_def.type
 	local gotten_node = get_node(pos).name
 	local gotten_biome = minetest.get_biome_data(pos)
-
 	if not gotten_node or not gotten_biome then return end
-
 	gotten_biome = get_biome_name(gotten_biome.biome) --makes it easier to work with
 
 	local is_ground = minetest.get_item_group(gotten_node,"solid") ~= 0
@@ -704,37 +612,24 @@ local function spawn_check(pos, spawn_def)
 	local is_bedrock  = gotten_node == "mcl_core:bedrock"
 	local is_grass = minetest.get_item_group(gotten_node,"grass_block") ~= 0
 
-	if pos.y >= spawn_def.min_height
+	if pos and spawn_def
-			and pos.y <= spawn_def.max_height
+	and pos.y >= spawn_def.min_height
-			and spawn_def.dimension == dimension
+	and pos.y <= spawn_def.max_height
-			and biome_check(spawn_def.biomes, gotten_biome) then
+	and spawn_def.dimension == dimension
-
+	and biome_check(spawn_def.biomes, gotten_biome)
-		--mcl_log("Level 1 spawn check passed")
+	and (is_ground or spawn_def.type_of_spawning ~= "ground")
-		--minetest.log("Mob: " .. mob_def.name)
+	and (spawn_def.type_of_spawning ~= "ground" or not is_leaf)
-
+	and has_room(mob_def,pos)
-		if  (is_ground or spawn_def.type_of_spawning ~= "ground")
+	and (spawn_def.check_position and spawn_def.check_position(pos) or true)
-				and (spawn_def.type_of_spawning ~= "ground" or not is_leaf)
+	and (not is_farm_animal(spawn_def.name) or is_grass)
-				and (not is_farm_animal(spawn_def.name) or is_grass)
+	and (spawn_def.type_of_spawning ~= "water" or is_water)
-				and (spawn_def.type_of_spawning ~= "water" or is_water)
+	and ( not spawn_protected or not minetest.is_protected(pos, "") )
-				and not is_bedrock
+	and not is_bedrock then
-				and has_room(mob_def,pos)
+		--only need to poll for node light if everything else worked
-				and (spawn_def.check_position and spawn_def.check_position(pos) or spawn_def.check_position == nil)
+		local gotten_light = get_node_light(pos)
-				and ( not spawn_protected or not minetest.is_protected(pos, "") ) then
+		if gotten_light >= spawn_def.min_light and gotten_light <= spawn_def.max_light then
-
+			return true
-			--mcl_log("Level 2 spawn check passed")
-
-			local gotten_light = get_node_light(pos)
-			if gotten_light >= spawn_def.min_light and gotten_light <= spawn_def.max_light then
-				--mcl_log("Level 3 spawn check passed")
-				return true
-			else
-				--mcl_log("Spawn check level 3 failed")
-			end
-		else
-			--mcl_log("Spawn check level 2 failed")
 		end
-	else
-		--mcl_log("Spawn check level 1 failed")
 	end
 	return false
 end
@@ -855,7 +750,7 @@ if mobs_spawn then
 	-- Get pos to spawn, x and z are randomised, y is range
 
 
-	local function mob_cap_space (pos, mob_type, mob_counts_close, mob_counts_wide, cap_space_hostile, cap_space_non_hostile)
+	local function mob_cap_space (pos, mob_type, mob_counts_close, mob_counts_wide)
 
 		-- Some mob examples
 		--type = "monster", spawn_class = "hostile",
@@ -871,18 +766,9 @@ if mobs_spawn then
 			mob_total_wide = 0
 		end
 
-		local cap_space_wide = math.max(type_cap - mob_total_wide, 0)
+		local cap_space_wide = type_cap - mob_total_wide
-
+		if cap_space_wide < 1 then
-		mcl_log("mob_type", mob_type)
+			cap_space_wide = 0
-		mcl_log("cap_space_wide", cap_space_wide)
-
-		local cap_space_available = 0
-		if mob_type == "hostile" then
-			mcl_log("cap_space_global", cap_space_hostile)
-			cap_space_available = math.min(cap_space_hostile, cap_space_wide)
-		else
-			mcl_log("cap_space_global", cap_space_non_hostile)
-			cap_space_available = math.min(cap_space_non_hostile, cap_space_wide)
 		end
 
 		local mob_total_close = mob_counts_close[mob_type]
@@ -891,11 +777,12 @@ if mobs_spawn then
 			mob_total_close = 0
 		end
 
-		local cap_space_close = math.max(close_zone_cap - mob_total_close, 0)
+		local cap_space_close = close_zone_cap - mob_total_close
-		cap_space_available = math.min(cap_space_available, cap_space_close)
+		if cap_space_close < 1 then
+			cap_space_close = 0
+		end
 
-		mcl_log("cap_space_close", cap_space_close)
+		--mcl_log("spawn_class: " .. spawn_class)
-		mcl_log("cap_space_available", cap_space_available)
 
 		if false and mob_type == "water" then
 			mcl_log("mob_type: " .. mob_type .. " and pos: " .. minetest.pos_to_string(pos))
@@ -905,7 +792,7 @@ if mobs_spawn then
 			mcl_log("cap_space_close: " .. cap_space_close)
 		end
 
-		return cap_space_available
+		return cap_space_wide, cap_space_close
 	end
 
 	local function find_spawning_position(pos, max_times)
@@ -916,7 +803,7 @@ if mobs_spawn then
 
 		local y_min, y_max = decypher_limits(pos.y)
 
-		--mcl_log("mapgen_limit: " .. SPAWN_MAPGEN_LIMIT)
+		mcl_log("mapgen_limit: " .. SPAWN_MAPGEN_LIMIT)
 		local i = 0
 		repeat
 			local goal_pos = get_next_mob_spawn_pos(pos)
@@ -948,7 +835,7 @@ if mobs_spawn then
 		return spawning_position
 	end
 
-	local function spawn_a_mob(pos, cap_space_hostile, cap_space_non_hostile)
+	local function spawn_a_mob(pos)
 		--create a disconnected clone of the spawn dictionary, prevents memory leak
 		local mob_library_worker_table = table_copy(spawn_dictionary)
 
@@ -986,18 +873,22 @@ if mobs_spawn then
 			if mob_def and mob_def.name and minetest.registered_entities[mob_def.name] then
 
 				local mob_def_ent = minetest.registered_entities[mob_def.name]
+				--local mob_type = mob_def_ent.type
 				local mob_spawn_class = mob_def_ent.spawn_class
 
-				local cap_space_available = mob_cap_space (spawning_position, mob_spawn_class, mob_counts_close, mob_counts_wide, cap_space_hostile, cap_space_non_hostile)
+				--mcl_log("mob_spawn_class: " .. mob_spawn_class)
 
-				if cap_space_available > 0 then
+				local cap_space_wide, cap_space_close = mob_cap_space (spawning_position, mob_spawn_class, mob_counts_close, mob_counts_wide)
+
+
+				if cap_space_close > 0 and cap_space_wide > 0 then
 					--mcl_log("Cap space available")
 
 					-- Spawn caps for animals and water creatures fill up rapidly. Need to throttle this somewhat
 					-- for performance and for early game challenge. We don't want to reduce hostiles though.
 					local spawn_hostile = (mob_spawn_class == "hostile")
 					local spawn_passive = (mob_spawn_class ~= "hostile") and math.random(100) < peaceful_percentage_spawned
-
+					-- or not hostile
 					--mcl_log("Spawn_passive: " .. tostring(spawn_passive))
 					--mcl_log("Spawn_hostile: " .. tostring(spawn_hostile))
 
@@ -1028,10 +919,13 @@ if mobs_spawn then
 							local group_min = mob_def_ent.spawn_in_group_min or 1
 							if not group_min then group_min = 1 end
 
-							local amount_to_spawn = math.random(group_min, spawn_in_group)
+							local amount_to_spawn = math.random(group_min,spawn_in_group)
-							mcl_log("Spawning quantity: " .. amount_to_spawn)
+
-							amount_to_spawn = math.min(amount_to_spawn, cap_space_available)
+							if amount_to_spawn > cap_space_wide then
-							mcl_log("throttled spawning quantity: " .. amount_to_spawn)
+								mcl_log("Spawning quantity: " .. amount_to_spawn)
+								mcl_log("Throttle amount to cap space: " .. cap_space_wide)
+								amount_to_spawn = cap_space_wide
+							end
 
 							if logging then
 								minetest.log("action", "[mcl_mobs] A group of " ..amount_to_spawn .. " " .. mob_def.name .. " mob spawns on " ..minetest.get_node(vector.offset(spawning_position,0,-1,0)).name .." at " .. minetest.pos_to_string(spawning_position, 1))
@@ -1046,7 +940,7 @@ if mobs_spawn then
 
 						if spawned then
 							--mcl_log("We have spawned")
-							mob_counts_close, mob_counts_wide, total_mobs = count_mobs_all("spawn_class", pos)
+							mob_counts_close, mob_counts_wide, total_mobs = count_mobs_all("type", pos)
 							local new_spawning_position = find_spawning_position(pos, FIND_SPAWN_POS_RETRIES_SUCCESS_RESPIN)
 							if new_spawning_position then
 								mcl_log("Setting new spawning position")
@@ -1056,10 +950,10 @@ if mobs_spawn then
 							end
 						end
 					else
-						--mcl_log("Spawn check failed")
+						mcl_log("Spawn check failed")
 					end
 				else
-					--mcl_log("Cap space full")
+					mcl_log("Cap space full")
 				end
 
 			end
@@ -1079,13 +973,7 @@ if mobs_spawn then
 		timer = 0
 
 		local players = get_connected_players()
-		local total_mobs, total_non_hostile, total_hostile = count_mobs_total_cap()
+		local total_mobs = count_mobs_total_cap()
-
-		local cap_space_hostile = math.max(mob_cap.global_hostile - total_hostile, 0)
-		local cap_space_non_hostile =  math.max(mob_cap.global_non_hostile - total_non_hostile, 0)
-		mcl_log("global cap_space_hostile", cap_space_hostile)
-		mcl_log("global cap_space_non_hostile", cap_space_non_hostile)
-
 		if total_mobs > mob_cap.total or total_mobs > #players * mob_cap.player then
 			minetest.log("action","[mcl_mobs] global mob cap reached. no cycle spawning.")
 			return
@@ -1096,49 +984,21 @@ if mobs_spawn then
 			local dimension = mcl_worlds.pos_to_dimension(pos)
 			-- ignore void and unloaded area
 			if dimension ~= "void" and dimension ~= "default" then
-				spawn_a_mob(pos, cap_space_hostile, cap_space_non_hostile)
+				spawn_a_mob(pos)
 			end
 		end
 	end)
 end
 
-local function despawn_allowed(self)
-	local nametag = self.nametag and self.nametag ~= ""
-	local not_busy = self.state ~= "attack" and self.following == nil
-	if self.can_despawn == true then
-		if not nametag and not_busy and not self.tamed == true and not self.persistent == true then
-			return true
-		end
-	end
-	return false
-end
-
-function mob_class:despawn_allowed()
-	despawn_allowed(self)
-end
-
-
-assert(despawn_allowed({can_despawn=false}) == false, "despawn_allowed - can_despawn false failed")
-assert(despawn_allowed({can_despawn=true}) == true, "despawn_allowed - can_despawn true failed")
-
-assert(despawn_allowed({can_despawn=true, nametag=""}) == true, "despawn_allowed - blank nametag failed")
-assert(despawn_allowed({can_despawn=true, nametag=nil}) == true, "despawn_allowed - nil nametag failed")
-assert(despawn_allowed({can_despawn=true, nametag="bob"}) == false, "despawn_allowed - nametag failed")
-
-assert(despawn_allowed({can_despawn=true, state="attack"}) == false, "despawn_allowed - attack state failed")
-assert(despawn_allowed({can_despawn=true, following="blah"}) == false, "despawn_allowed - following state failed")
-
-assert(despawn_allowed({can_despawn=true, tamed=false}) == true, "despawn_allowed - not tamed")
-assert(despawn_allowed({can_despawn=true, tamed=true}) == false, "despawn_allowed - tamed")
-
-assert(despawn_allowed({can_despawn=true, persistent=true}) == false, "despawn_allowed - persistent")
-assert(despawn_allowed({can_despawn=true, persistent=false}) == true, "despawn_allowed - not persistent")
-
 function mob_class:check_despawn(pos, dtime)
 	self.lifetimer = self.lifetimer - dtime
 
 	-- Despawning: when lifetimer expires, remove mob
-	if remove_far and despawn_allowed(self) then
+	if remove_far
+	and self.can_despawn == true
+	and ((not self.nametag) or (self.nametag == ""))
+	and self.state ~= "attack"
+	and self.following == nil then
 		if self.despawn_immediately or self.lifetimer <= 0 then
 			if logging then
 				minetest.log("action", "[mcl_mobs] Mob "..self.name.." despawns at "..minetest.pos_to_string(pos, 1) .. " lifetimer ran out")

mods/ENTITIES/mcl_paintings/locale/mcl_paintings.dk.tr

@@ -1,2 +0,0 @@
-# textdomain:mcl_paintings
-Painting=Maleri

mods/ENTITIES/mcl_paintings/locale/mcl_paintings.es.tr

@@ -1,2 +0,0 @@
-# textdomain:mcl_paintings
-Painting=Cuadro

mods/ENTITIES/mobs_mc/README.md

@@ -39,7 +39,7 @@ This mod adds mobs which closely resemble the mobs from the game Minecraft, vers
 * Cave Spider
 * Enderman
 * Zombie Villager
-* Zombie Piglin
+* Zombie Pigman
 * Wither Skeleton
 * Magma Cube
 * Blaze

mods/ENTITIES/mobs_mc/axolotl.lua

@@ -2,7 +2,7 @@ local S = minetest.get_translator(minetest.get_current_modname())
 
 local axolotl = {
 	type = "animal",
-	spawn_class = "axolotl",
+	spawn_class = "water",
 	can_despawn = true,
 	passive = false,
 	hp_min = 14,

mods/ENTITIES/mobs_mc/blaze.lua

@@ -207,6 +207,5 @@ mcl_mobs.register_arrow("mobs_mc:blaze_fireball", {
 	end
 })
 
-mcl_mobs:non_spawn_specific("mobs_mc:blaze", "overworld", 0, 11)
+-- spawn eggs
--- spawn eggs.
 mcl_mobs.register_egg("mobs_mc:blaze", S("Blaze"), "#f6b201", "#fff87e", 0)

mods/ENTITIES/mobs_mc/cod.lua

@@ -31,7 +31,7 @@ local S = minetest.get_translator(minetest.get_current_modname())
 
 local cod = {
 	type = "animal",
-	spawn_class = "water_ambient",
+	spawn_class = "water",
 	can_despawn = true,
 	passive = true,
 	hp_min = 3,

mods/ENTITIES/mobs_mc/creeper.lua

@@ -10,7 +10,6 @@ local S = minetest.get_translator("mobs_mc")
 
 
 mcl_mobs.register_mob("mobs_mc:creeper", {
-	description = S("Creeper"),
 	type = "monster",
 	spawn_class = "hostile",
 	spawn_in_group = 1,
@@ -40,7 +39,7 @@ mcl_mobs.register_mob("mobs_mc:creeper", {
 	},
 	makes_footstep_sound = true,
 	walk_velocity = 1.05,
-	run_velocity = 2.0,
+	run_velocity = 2.1,
 	runaway_from = { "mobs_mc:ocelot", "mobs_mc:cat" },
 	attack_type = "explode",
 

mods/ENTITIES/mobs_mc/ender_dragon.lua

@@ -174,4 +174,3 @@ mcl_mobs.register_egg("mobs_mc:enderdragon", S("Ender Dragon"), "#252525", "#b31
 
 
 mcl_wip.register_wip_item("mobs_mc:enderdragon")
-mcl_mobs:non_spawn_specific("mobs_mc:enderdragon","overworld",0,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/enderman.lua

@@ -24,12 +24,6 @@
 -- added rain damage.
 -- fixed the grass_with_dirt issue.
 
--- How freqeuntly to take and place blocks, in seconds
-local take_frequency_min = 235
-local take_frequency_max = 245
-local place_frequency_min = 235
-local place_frequency_max = 245
-
 minetest.register_entity("mobs_mc:ender_eyes", {
 	visual = "mesh",
 	mesh = "mobs_mc_spider.b3d",
@@ -66,6 +60,13 @@ end
 
 local pr = PseudoRandom(os.time()*(-334))
 
+-- How freqeuntly to take and place blocks, in seconds
+local take_frequency_min = 235
+local take_frequency_max = 245
+local place_frequency_min = 235
+local place_frequency_max = 245
+
+
 -- Texuture overrides for enderman block. Required for cactus because it's original is a nodebox
 -- and the textures have tranparent pixels.
 local block_texture_overrides
@@ -344,8 +345,7 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 					self:teleport(nil)
 				end
 			end
-		end
+		else return end
-
 		-- AGRESSIVELY WARP/CHASE PLAYER BEHAVIOUR HERE.
 		if self.state == "attack" then
 			if self.attack then
@@ -358,11 +358,9 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 				end
 			end
 		else --if not attacking try to tp to the dark
-			if dim == 'overworld' then
+			local light = minetest.get_node_light(enderpos)
-				local light = minetest.get_node_light(enderpos)
+			if light and light > minetest.LIGHT_MAX then
-				if light and light > minetest.LIGHT_MAX then
+				self:teleport(nil)
-					self:teleport(nil)
-				end
 			end
 		end
 		-- ARROW / DAYTIME PEOPLE AVOIDANCE BEHAVIOUR HERE.
@@ -389,7 +387,6 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 				end
 			end
 		end
-
 		-- PROVOKED BEHAVIOUR HERE.
 		local enderpos = self.object:get_pos()
 		if self.provoked == "broke_contact" then
@@ -447,22 +444,6 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 			end
 		end
 
-		-- ATTACK ENDERMITE
-		local enderpos = self.object:get_pos()
-		if math.random(1,140) == 1 then
-			local mobsnear = minetest.get_objects_inside_radius(enderpos, 64)
-			for n=1, #mobsnear do
-				local mob = mobsnear[n]
-				if mob then
-					local entity = mob:get_luaentity()
-					if entity and entity.name == "mobs_mc:endermite" then
-						self.attack = mob
-						self.state = 'attack'
-					end
-				end
-			end
-		end
-
 		-- TAKE AND PLACE STUFF BEHAVIOUR BELOW.
 		if not mobs_griefing then
 			return
@@ -490,7 +471,7 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 					local dug = minetest.get_node_or_nil(take_pos)
 					if dug and dug.name == "air" then
 						self._taken_node = node.name
-						self.persistent = true
+						self.can_despawn = false
 						local def = minetest.registered_nodes[self._taken_node]
 						-- Update animation and texture accordingly (adds visibly carried block)
 						local block_type
@@ -541,7 +522,7 @@ mcl_mobs.register_mob("mobs_mc:enderman", {
 				if success then
 					local def = minetest.registered_nodes[self._taken_node]
 					-- Update animation accordingly (removes visible block)
-					self.persistent = false
+					self.can_despawn = true
 					self.animation = select_enderman_animation("normal")
 					self:set_animation(self.animation.current)
 					if def.sounds and def.sounds.place then

mods/ENTITIES/mobs_mc/endermite.lua

@@ -39,4 +39,3 @@ mcl_mobs.register_mob("mobs_mc:endermite", {
 })
 
 mcl_mobs.register_egg("mobs_mc:endermite", S("Endermite"), "#161616", "#6d6d6d", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:endermite","overworld",0,7)

mods/ENTITIES/mobs_mc/ghast.lua

@@ -139,6 +139,6 @@ mcl_mobs.register_arrow("mobs_mc:fireball", {
 
 
 
-mcl_mobs:non_spawn_specific("mobs_mc:ghast","overworld","0","7")
+
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:ghast", S("Ghast"), "#f9f9f9", "#bcbcbc", 0)

mods/ENTITIES/mobs_mc/glow_squid.lua

@@ -31,7 +31,7 @@ end
 
 mcl_mobs.register_mob("mobs_mc:glow_squid", {
 	type = "animal",
-	spawn_class = "water_underground",
+	spawn_class = "water",
 	can_despawn = true,
 	passive = true,
 	hp_min = 10,

mods/ENTITIES/mobs_mc/guardian.lua

@@ -102,6 +102,6 @@ mcl_mobs.register_mob("mobs_mc:guardian", {
 -- Spawning disabled due to size issues
 -- TODO: Re-enable spawning
 --mcl_mobs:spawn_specific("mobs_mc:guardian", { "mcl_core:water_source", "mclx_core:river_water_source" }, { "mcl_core:water_source", "mclx_core:river_water_source" }, 0, minetest.LIGHT_MAX+1, 30, 25000, 2, mcl_vars.mg_overworld_min, mobs_mc.water_level - 10)
-mcl_mobs:non_spawn_specific("mobs_mc:guardian","overworld",0,minetest.LIGHT_MAX+1)
+
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:guardian", S("Guardian"), "#5a8272", "#f17d31", 0)

mods/ENTITIES/mobs_mc/guardian_elder.lua

@@ -113,4 +113,5 @@ mcl_mobs.register_mob("mobs_mc:guardian_elder", {
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:guardian_elder", S("Elder Guardian"), "#ceccba", "#747693", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:guardian_elder","overworld",0,minetest.LIGHT_MAX+1)
+
+

mods/ENTITIES/mobs_mc/hoglin+zoglin.lua

@@ -10,7 +10,6 @@ local S = minetest.get_translator("mobs_mc")
 --###################
 
 local hoglin = {
-	description = S("Hoglin"),
 	type = "monster",
 	passive = false,
 	spawn_class = "hostile",
@@ -20,9 +19,8 @@ local hoglin = {
 	xp_max = 9,
 	armor = {fleshy = 90},
 	attack_type = "dogfight",
-	attack_frequency = 3;
 	damage = 4,
-	reach = 1.9,
+	reach = 3,
 	collisionbox = {-.6, -0.01, -.6, .6, 1.4, .6},
 	visual = "mesh",
 	mesh = "extra_mobs_hoglin.b3d",
@@ -38,7 +36,7 @@ local hoglin = {
 	jump = true,
 	makes_footstep_sound = true,
 	walk_velocity = 1,
-	run_velocity = 2.8,
+	run_velocity = 4,
 	drops = {
 		{name = "mobs_mcitems:leather",
 		chance = 1,
@@ -65,7 +63,7 @@ local hoglin = {
 		punch_end = 32,
 	},
 	fear_height = 4,
-	view_range = 16,
+	view_range = 32,
 	floats = 0,
 	custom_attack = function(self)
 		if self.state == "attack" and self.reach > vector.distance(self.object:get_pos(), self.attack:get_pos()) then
@@ -89,7 +87,6 @@ local hoglin = {
 mcl_mobs.register_mob("mobs_mc:hoglin", hoglin)
 
 local zoglin = table.copy(hoglin)
-zoglin.description = S("Zoglin")
 zoglin.fire_resistant = 1
 zoglin.textures = {"extra_mobs_zoglin.png"}
 zoglin.do_custom = function()
@@ -103,7 +100,6 @@ mcl_mobs.register_mob("mobs_mc:zoglin", zoglin)
 -- Baby hoglin.
 
 local baby_hoglin = table.copy(hoglin)
-baby_hoglin.description = S("Baby hoglin")
 baby_hoglin.collisionbox = {-.3, -0.01, -.3, .3, 0.94, .3}
 baby_hoglin.xp_min = 20
 baby_hoglin.xp_max = 20
@@ -124,6 +120,7 @@ mcl_mobs:spawn_specific(
 "nether",
 "ground",
 {
+"Nether",
 "CrimsonForest"
 },
 0,
@@ -134,7 +131,5 @@ minetest.LIGHT_MAX+1,
 mcl_vars.mg_nether_min,
 mcl_vars.mg_nether_max)
 
-mcl_mobs:non_spawn_specific("mobs_mc:hoglin","overworld",0,7)
-
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:hoglin", S("Hoglin"), "#85682e", "#2b2140", 0)

mods/ENTITIES/mobs_mc/horse.lua

@@ -646,5 +646,3 @@ mcl_mobs.register_egg("mobs_mc:skeleton_horse", S("Skeleton Horse"), "#68684f",
 --mobs:register_egg("mobs_mc:zombie_horse", S("Zombie Horse"), "#2a5a37", "#84d080", 0)
 mcl_mobs.register_egg("mobs_mc:donkey", S("Donkey"), "#534539", "#867566", 0)
 mcl_mobs.register_egg("mobs_mc:mule", S("Mule"), "#1b0200", "#51331d", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:mule","overworld",9,minetest.LIGHT_MAX+1)
-mcl_mobs:non_spawn_specific("mobs_mc:skeleton_horse","overworld",9,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/init.lua

@@ -139,6 +139,7 @@ dofile(path .. "/silverfish.lua") -- maikerumine Mesh and animation by toby109tt
 dofile(path .. "/skeleton+stray.lua") -- Mesh by Morn76 Animation by Pavel_S
 dofile(path .. "/skeleton_wither.lua") -- Mesh by Morn76 Animation by Pavel_S
 dofile(path .. "/zombie.lua") -- Mesh by Morn76 Animation by Pavel_S
+dofile(path .. "/zombiepig.lua") -- Mesh by Morn76 Animation by Pavel_S
 dofile(path .. "/slime+magma_cube.lua") -- Wuzzy
 dofile(path .. "/spider.lua") -- Spider by AspireMint (fishyWET (CC-BY-SA 3.0 license for texture)
 dofile(path .. "/vex.lua") -- KrupnoPavel
@@ -152,7 +153,7 @@ dofile(path .. "/dolphin.lua")
 
 dofile(path .. "/glow_squid.lua")
 
-dofile(path .. "/piglin.lua") -- "mobs_mc_zombie_pigman.b3d" Mesh by Morn76 Animation by Pavel_S
+dofile(path .. "/piglin.lua")
 dofile(path .. "/hoglin+zoglin.lua")
 
 dofile(path .. "/strider.lua")

mods/ENTITIES/mobs_mc/iron_golem.lua

@@ -206,4 +206,3 @@ function mobs_mc.check_iron_golem_summon(pos)
 		end
 	end
 end
-mcl_mobs:non_spawn_specific("mobs_mc:iron_golem","overworld",0,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/locale/mobs_mc.de.tr

@@ -48,7 +48,7 @@ Wither=Wither
 Wolf=Wolf
 Husk=Wüstenzombie
 Zombie=Zombie
-Zombie Piglin=Schweinezombie
+Zombie Pigman=Schweinezombie
 Farmer=Bauer
 Fisherman=Fischer
 Fletcher=Pfeilmacher

mods/ENTITIES/mobs_mc/locale/mobs_mc.dk.tr

@@ -1,70 +0,0 @@
-# textdomain: mobs_mc
-Agent=Agent
-Axolotl=Salamander
-Bat=Flagermus
-Blaze=Blaze
-Chicken=Kylling
-Cow=Ko
-Mooshroom=Svamp
-Creeper=Creeper
-Ender Dragon=Enderdrage
-Enderman=Enderman
-Endermite=Endermide
-Ghast=Ghast
-Elder Guardian=Gammel beskytter
-Guardian=Beskytter
-Horse=Hest
-Skeleton Horse=Skelethest
-Zombie Horse=Zombiehest
-Donkey=Æsel
-Mule=Muldyr
-Iron Golem=Jerngolem
-Llama=Lama
-Ocelot=Ozelot
-Parrot=Papegøje
-Pig=Gris
-Polar Bear=Isbjørn
-Rabbit=Kanin
-Killer Bunny=Dræberkanin
-Sheep=Får
-Shulker=Shulker
-Silverfish=Sølvfisk
-Skeleton=Skelet
-Stray=Omstrejfer
-Wither Skeleton=Wither-skelet
-Magma Cube=Magmakube
-Slime=Slimklump
-Snow Golem=Snegolem
-Spider=Edderkop
-Cave Spider=Huleedderkop
-Squid=Blæksprutte
-Vex=Vex
-Evoker=Fremkalder
-Illusioner=Illusionist
-Villager=Landsbyboer
-Vindicator=Hævner
-Zombie Villager=Zombie landsbyboer
-Witch=Heks
-Wither=Wither
-Wolf=Ulv
-Husk=Udtørretskal
-Zombie=Zombie
-Zombie Pigman=Zombificeret piglin
-Farmer=Landmand
-Fisherman=Fisker
-Fletcher=Pilemager
-Shepherd=Fårehyrde
-Librarian=Bibliotikar
-Cartographer=Kartograf
-Armorer=Rustningesmed
-Leatherworker=Læderarbejder
-Butcher=Slagter
-Weapon Smith=Våbensmed
-Tool Smith=Værktøjssmed
-Cleric=Gejstlig
-Nitwit=Tåbe
-Cod=Torsk
-Salmon=Laks
-Dolphin=Delfin
-Pillager=Plyndrer
-Tropical fish=Tropisk fisk

mods/ENTITIES/mobs_mc/locale/mobs_mc.es.tr

@@ -54,21 +54,5 @@ Baby Husk=Bebé Zombi Momificado
 Baby Zombie=Bebé Zombi
 Husk=Zombi Momificado
 Zombie=Zombi
-Baby Zombie Piglin=Bebé Hombrecerdo Zombi
+Baby Zombie Pigman=Bebé Hombrecerdo Zombi
-Zombie Piglin=Hombrecerdo Zombi
+Zombie Pigman=Hombrecerdo Zombi
-Cartographer=Cartógrafo
-Armorer=Escudero
-Leatherworker=Peletero
-Butcher=Carnicero
-Weapon Smith=Armero
-Tool Smith=Herrero
-Cleric=Clérigo
-Nitwit=Holgazán
-Cod=Bacalao
-Salmon=Salmón
-Dolphin=Delfín
-Pillager=Saqueador
-Tropical fish=Pez tropical
-Hoglin=Hoglin
-Strider=Lavagante
-Glow Squid=Calamar luminoso

mods/ENTITIES/mobs_mc/locale/mobs_mc.fr.tr

@@ -1,6 +1,5 @@
 # textdomain: mobs_mc
 Agent=Agent
-Axolotl=Axolotl
 Bat=Chauve-souris
 Blaze=Blaze
 Chicken=Poulet
@@ -49,7 +48,7 @@ Wither=Wither
 Wolf=Loup
 Husk=Zombie Momifié
 Zombie=Zombie
-Zombie Piglin=Zombie Cochon
+Zombie Pigman=Zombie Cochon
 Farmer=Fermier
 Fisherman=Pêcheur
 Fletcher=Archer
@@ -67,7 +66,4 @@ Cod=Morue
 Salmon=Saumon
 Dolphin=Dauphin
 Pillager=Pilleur
-Tropical fish=Poisson tropical
+Tropical fish=Poisson tropical
-Hoglin=Hoglin
-Strider=Arpenteur
-Glow Squid=Poulpe Brillant

mods/ENTITIES/mobs_mc/locale/mobs_mc.ja.tr

@@ -49,7 +49,7 @@ Wither=ウィザー
 Wolf=オオカミ
 Husk=ハスク
 Zombie=ゾンビ
-Zombie Piglin=ゾンビピッグマン
+Zombie Pigman=ゾンビピッグマン
 Farmer=農民
 Fisherman=漁師
 Fletcher=矢師

mods/ENTITIES/mobs_mc/locale/template.txt

@@ -49,12 +49,7 @@ Wither=
 Wolf=
 Husk=
 Zombie=
-Piglin=
+Zombie Pigman=
-Baby Piglin=
-Zombie Piglin=
-Baby Zombie Piglin=
-Sword Piglin=
-Piglin Brute=
 Farmer=
 Fisherman=
 Fletcher=
@@ -72,7 +67,4 @@ Cod=
 Salmon=
 Dolphin=
 Pillager=
-Tropical fish=
+Tropical fish=
-Hoglin=
-Strider=
-Glow Squid=

mods/ENTITIES/mobs_mc/pig.lua

@@ -82,7 +82,7 @@ mcl_mobs.register_mob("mobs_mc:pig", {
 		end
 
 		-- if driver present allow control of horse
-		if self.driver and self.driver:get_wielded_item():get_name() == "mcl_mobitems:carrot_on_a_stick" then
+		if self.driver then
 
 			mcl_mobs.drive(self, "walk", "stand", false, dtime)
 
@@ -149,14 +149,18 @@ mcl_mobs.register_mob("mobs_mc:pig", {
 			return
 		end
 
-			-- Should make pig go faster when right clicked with carrot on a stick.
+		-- Mount or detach player
-			-- FIXME: needs work on the going faster part.
+		local name = clicker:get_player_name()
-		--[[if self.driver and clicker == self.driver and self.driver:get_wielded_item():get_name() == "mcl_mobitems:carrot_on_a_stick" then
+		if self.driver and clicker == self.driver then
-			if not self.v3 then
+			-- Detach if already attached
-				self.v3 = 0
+			mcl_mobs.detach(clicker, {x=1, y=0, z=0})
-				self.max_speed_forward = 100
+			return
-				self.accel = 10
+
-			end
+		elseif not self.driver and self.saddle == "yes" and wielditem:get_name() == "mcl_mobitems:carrot_on_a_stick" then
+			-- Ride pig if it has a saddle and player uses a carrot on a stick
+
+			mcl_mobs.attach(self, clicker)
+
 			if not minetest.is_creative_enabled(clicker:get_player_name()) then
 
 				local inv = self.driver:get_inventory()
@@ -173,19 +177,6 @@ mcl_mobs.register_mob("mobs_mc:pig", {
 				end
 				inv:set_stack("main",self.driver:get_wield_index(), wielditem)
 			end
-		end]]
-
-		-- Mount or detach player
-		local name = clicker:get_player_name()
-		if self.driver and clicker == self.driver then -- and self.driver:get_wielded_item():get_name() ~= "mcl_mobitems:carrot_on_a_stick" then -- Note: This is for when the ability to make the pig go faster is implemented
-			-- Detach if already attached
-			mcl_mobs.detach(clicker, {x=1, y=0, z=0})
-			return
-
-		elseif not self.driver and self.saddle == "yes" then
-			-- Ride pig if it has a saddle
-
-			mcl_mobs.attach(self, clicker)
 			return
 
 		-- Capture pig
@@ -204,18 +195,6 @@ mcl_mobs.register_mob("mobs_mc:pig", {
 			return false
 		end
 	end,
-
-	after_activate = function(self, staticdata, def, dtime)
-		if self.saddle == "yes" then -- Make saddle load upon rejoin
-			self.base_texture = {
-				"mobs_mc_pig.png", -- base
-				"mobs_mc_pig_saddle.png", -- saddle
-			}
-			self.object:set_properties({
-				textures = self.base_texture
-			})
-		end
-	end,
 })
 
 mcl_mobs:spawn_specific(

mods/ENTITIES/mobs_mc/piglin.lua

@@ -25,10 +25,7 @@ function mobs_mc.player_wears_gold(player)
 	for i=1, 6 do
 		local stack = player:get_inventory():get_stack("armor", i)
 		local item = stack:get_name()
-		if string.find(item, "mcl_armor:chestplate_gold")
+		if item == "mcl_armor:chestplate_gold" or item == "mcl_armor:leggings_gold" or item == "mcl_armor:helmet_gold" or item == "mcl_armor:boots_gold" then
-				or string.find(item, "mcl_armor:leggings_gold")
-				or string.find(item, "mcl_armor:helmet_gold")
-				or string.find(item, "mcl_armor:boots_gold") then
 			return true
 		end
 	end
@@ -38,11 +35,9 @@ end
 --################### piglin
 --###################
 local piglin = {
-	description = S("Piglin"),
 	type = "monster",
 	passive = false,
 	spawn_class = "hostile",
-	group_attack = {"mobs_mc:piglin", "mobs_mc:sword_piglin", "mobs_mc:piglin_brute"},
 	hp_min = 16,
 	hp_max = 16,
 	xp_min = 9,
@@ -67,8 +62,8 @@ local piglin = {
 	},
 	jump = true,
 	makes_footstep_sound = true,
-	walk_velocity = 1.4,
+	walk_velocity = 4.317,
-	run_velocity = 2.8,
+	run_velocity = 5.6121,
 	drops = {
 		{name = "mcl_bows:crossbow",
 		chance = 10,
@@ -183,7 +178,6 @@ mcl_mobs.register_mob("mobs_mc:piglin", piglin)
 
 
 local sword_piglin = table.copy(piglin)
-sword_piglin.description = S("Sword Piglin")
 sword_piglin.mesh = "extra_mobs_sword_piglin.b3d"
 sword_piglin.textures = {"extra_mobs_piglin.png", "default_tool_goldsword.png"}
 sword_piglin.on_spawn = function(self)
@@ -212,127 +206,43 @@ sword_piglin.animation = {
 	punch_start = 189,
 	punch_end = 198,
 }
-
 mcl_mobs.register_mob("mobs_mc:sword_piglin", sword_piglin)
 
-
+local zombified_piglin = table.copy(piglin)
--- Zombified Piglin --
+zombified_piglin.fire_resistant = 1
-
+zombified_piglin.do_custom = function()
-
+	return
-local zombified_piglin = {
+end
-	description = S("Zombie Piglin"),
+zombified_piglin.on_spawn = function()
-	-- type="animal", passive=false: This combination is needed for a neutral mob which becomes hostile, if attacked
+	return
-	type = "animal",
+end
-	passive = false,
+zombified_piglin.on_rightclick = function()
-	spawn_class = "passive",
+	return
-	hp_min = 20,
+end
-	hp_max = 20,
+zombified_piglin.lava_damage = 0
-	xp_min = 6,
+zombified_piglin.fire_damage = 0
-	xp_max = 6,
+zombified_piglin.attack_animals = true
-	armor = {undead = 90, fleshy = 90},
+zombified_piglin.mesh = "extra_mobs_sword_piglin.b3d"
-	attack_type = "dogfight",
+zombified_piglin.textures = {"extra_mobs_zombified_piglin.png", "default_tool_goldsword.png", "extra_mobs_trans.png"}
-	group_attack = {"mobs_mc:zombified_piglin", "mobs_mc:baby_zombified_piglin"},
+zombified_piglin.attack_type = "dogfight"
-	damage = 9,
+zombified_piglin.animation = {
-	reach = 2,
+	stand_speed = 30,
-	head_swivel = "head.control",
+	walk_speed = 30,
-	bone_eye_height = 2.4,
+	punch_speed = 45,
-	head_eye_height = 1.4,
+	run_speed = 30,
-	curiosity = 15,
+	stand_start = 0,
-	collisionbox = {-0.3, -0.01, -0.3, 0.3, 1.94, 0.3}, -- same
+	stand_end = 79,
-	visual = "mesh",
+	walk_start = 168,
-	mesh = "mobs_mc_zombie_pigman.b3d",
+	walk_end = 187,
-	textures = { {
+	run_start = 440,
-					 "blank.png", --baby
+	run_end = 459,
-					 "default_tool_goldsword.png", --sword
+	punch_start = 189,
-					 "mobs_mc_zombie_pigman.png", --pigman
+	punch_end = 198,
-				 } },
-	visual_size = {x=3, y=3},
-	sounds = {
-		random = "mobs_mc_zombiepig_random",
-		war_cry = "mobs_mc_zombiepig_war_cry",
-		death = "mobs_mc_zombiepig_death",
-		damage = "mobs_mc_zombiepig_hurt",
-		distance = 16,
-	},
-	jump = true,
-	makes_footstep_sound = true,
-	walk_velocity = .8,
-	run_velocity = 2.6,
-	pathfinding = 1,
-	drops = {
-		{name = "mcl_mobitems:rotten_flesh",
-		 chance = 1,
-		 min = 1,
-		 max = 1,
-		 looting = "common"},
-		{name = "mcl_core:gold_nugget",
-		 chance = 1,
-		 min = 0,
-		 max = 1,
-		 looting = "common"},
-		{name = "mcl_core:gold_ingot",
-		 chance = 40, -- 2.5%
-		 min = 1,
-		 max = 1,
-		 looting = "rare"},
-		{name = "mcl_tools:sword_gold",
-		 chance = 100 / 8.5,
-		 min = 1,
-		 max = 1,
-		 looting = "rare"},
-	},
-	animation = {
-		stand_speed = 25,
-		walk_speed = 25,
-		run_speed = 50,
-		stand_start = 40,
-		stand_end = 80,
-		walk_start = 0,
-		walk_end = 40,
-		run_start = 0,
-		run_end = 40,
-		punch_start = 90,
-		punch_end = 130,
-	},
-	lava_damage = 0,
-	fire_damage = 0,
-	fear_height = 4,
-	view_range = 16,
-	harmed_by_heal = true,
-	fire_damage_resistant = true,
 }
-
 mcl_mobs.register_mob("mobs_mc:zombified_piglin", zombified_piglin)
 
-local baby_zombified_piglin = table.copy(zombified_piglin)
-baby_zombified_piglin.description = S("Baby Zombie Piglin")
-baby_zombified_piglin.collisionbox = {-0.25, -0.01, -0.25, 0.25, 0.94, 0.25}
-baby_zombified_piglin.xp_min = 13
-baby_zombified_piglin.xp_max = 13
-baby_zombified_piglin.textures = {
-	{
-	 "mobs_mc_zombie_pigman.png", --baby
-	 "default_tool_goldsword.png", --sword
-	 "mobs_mc_zombie_pigman.png", --pigman
-	}
-}
-baby_zombified_piglin.walk_velocity = 1.2
-baby_zombified_piglin.run_velocity = 2.4
-baby_zombified_piglin.light_damage = 0
-baby_zombified_piglin.child = 1
-
-mcl_mobs.register_mob("mobs_mc:baby_zombified_piglin", baby_zombified_piglin)
-
--- Compatibility code. These were removed, and now are called zombie piglins. They don't spawn.
--- This is only to catch old cases. Maybe could be an alias?
-mcl_mobs.register_mob("mobs_mc:pigman", zombified_piglin)
-mcl_mobs.register_mob("mobs_mc:baby_pigman", baby_zombified_piglin)
-
-
--- Piglin Brute --
 
 local piglin_brute = table.copy(piglin)
-piglin_brute.description = S("Piglin Brute")
 piglin_brute.xp_min = 20
 piglin_brute.xp_max = 20
 piglin_brute.hp_min = 50
@@ -369,17 +279,10 @@ piglin_brute.animation = {
 	punch_end = 198,
 }
 piglin_brute.can_despawn = false
-
+piglin_brute.group_attack = { "mobs_mc:piglin", "mobs_mc:piglin_brute" }
-piglin_brute.drops = {
-	{name = "mcl_tools:axe_gold",
-	chance = 8.5,
-	min = 1,
-	max = 1,},
-}
 mcl_mobs.register_mob("mobs_mc:piglin_brute", piglin_brute)
 
 
-
 -- Regular spawning in the Nether
 mcl_mobs:spawn_specific(
 "mobs_mc:piglin",
@@ -402,9 +305,9 @@ mcl_mobs:spawn_specific(
 "nether",
 "ground",
 {
-			"Nether",
+"Nether",
-			"CrimsonForest"
+"CrimsonForest"
-		},
+},
 0,
 minetest.LIGHT_MAX+1,
 30,
@@ -412,45 +315,6 @@ minetest.LIGHT_MAX+1,
 3,
 mcl_vars.mg_lava_nether_max,
 mcl_vars.mg_nether_max)
-
+-- spawn eggs
-mcl_mobs:spawn_specific(
-		"mobs_mc:zombified_piglin",
-		"nether",
-		"ground",
-		{
-			"Nether",
-			"CrimsonForest",
-		},
-		0,
-		minetest.LIGHT_MAX+1,
-		30,
-		6000,
-		3,
-		mcl_vars.mg_nether_min,
-		mcl_vars.mg_nether_max)
-
--- Baby zombie is 20 times less likely than regular zombies
-mcl_mobs:spawn_specific(
-		"mobs_mc:baby_zombified_piglin",
-		"nether",
-		"ground",
-		{
-			"Nether",
-			"CrimsonForest",
-		},
-		0,
-		minetest.LIGHT_MAX+1,
-		30,
-		100000,
-		4,
-		mcl_vars.mg_nether_min,
-		mcl_vars.mg_nether_max)
-
-mcl_mobs:non_spawn_specific("mobs_mc:piglin","overworld",0,7)
-mcl_mobs:non_spawn_specific("mobs_mc:sword_piglin","overworld",0,7)
-mcl_mobs:non_spawn_specific("mobs_mc:piglin_brute","overworld",0,7)
-mcl_mobs:non_spawn_specific("mobs_mc:zombified_piglin","overworld",0,minetest.LIGHT_MAX+1)
-
 mcl_mobs.register_egg("mobs_mc:piglin", S("Piglin"), "#7b4a17","#d5c381", 0)
 mcl_mobs.register_egg("mobs_mc:piglin_brute", S("Piglin Brute"), "#562b0c","#ddc89d", 0)
-mcl_mobs.register_egg("mobs_mc:zombified_piglin", S("Zombie Piglin"), "#ea9393", "#4c7129", 0)

mods/ENTITIES/mobs_mc/pillager.lua

@@ -122,4 +122,3 @@ pillager = {
 
 mcl_mobs.register_mob("mobs_mc:pillager", pillager)
 mcl_mobs.register_egg("mobs_mc:pillager", S("Pillager"), "#532f36", "#959b9b", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:pillager","overworld",0,7)

mods/ENTITIES/mobs_mc/rabbit.lua

@@ -217,4 +217,3 @@ mcl_mobs.register_egg("mobs_mc:rabbit", S("Rabbit"), "#995f40", "#734831", 0)
 
 -- Note: This spawn egg does not exist in Minecraft
 mcl_mobs.register_egg("mobs_mc:killer_bunny", S("Killer Bunny"), "#f2f2f2", "#ff0000", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:killer_bunny","overworld",9,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/salmon.lua

@@ -11,7 +11,7 @@ local S = minetest.get_translator(minetest.get_current_modname())
 
 local salmon = {
 	type = "animal",
-	spawn_class = "water_ambient",
+	spawn_class = "water",
 	can_despawn = true,
 	passive = true,
 	hp_min = 3,

mods/ENTITIES/mobs_mc/sheep.lua

@@ -6,8 +6,6 @@ local S = minetest.get_translator("mobs_mc")
 --################### SHEEP
 --###################
 
-local WOOL_REPLACE_RATE = 80
-
 local colors = {
 	-- group = { wool, textures }
 	unicolor_white = { "mcl_wool:white", "#FFFFFF00" },
@@ -103,20 +101,17 @@ mcl_mobs.register_mob("mobs_mc:sheep", {
 		stand_start = 0, stand_end = 0,
 		walk_start = 0, walk_end = 40, walk_speed = 30,
 		run_start = 0, run_end = 40, run_speed = 40,
-		eat_start = 40, eat_end = 80, eat_loop = false,
 	},
 	child_animations = {
 		stand_start = 81, stand_end = 81,
 		walk_start = 81, walk_end = 121, walk_speed = 45,
 		run_start = 81, run_end = 121, run_speed = 60,
-		eat_start = 121, eat_start = 161, eat_loop = false,
 	},
 	follow = { "mcl_farming:wheat_item" },
 	view_range = 12,
 
 	-- Eat grass
-	replace_rate = WOOL_REPLACE_RATE,
+	replace_rate = 20,
-	replace_delay = 1.3,
 	replace_what = {
 		{ "mcl_core:dirt_with_grass", "mcl_core:dirt", -1 },
 		{ "mcl_flowers:tallgrass", "air", 0 },
@@ -126,39 +121,19 @@ mcl_mobs.register_mob("mobs_mc:sheep", {
 		if not self.color or not colors[self.color] then
 			self.color = "unicolor_white"
 		end
+		self.gotten = false
 		self.base_texture = sheep_texture(self.color)
-
+		self.object:set_properties({ textures = self.base_texture })
 		self.drops = {
 			{name = "mcl_mobitems:mutton",
-			 chance = 1,
+			chance = 1,
-			 min = 1,
+			min = 1,
-			 max = 2,},
+			max = 2,},
 			{name = colors[self.color][1],
-			 chance = 1,
+			chance = 1,
-			 min = 1,
+			min = 1,
-			 max = 1,},
+			max = 1,},
 		}
-
-		self.state = "eat"
-		self:set_animation("eat")
-		self:set_velocity(0)
-
-
-
-		minetest.after(self.replace_delay, function()
-			if self and self.object and self.object:get_velocity() and self.health > 0 then
-				self.object:set_velocity(vector.zero())
-				self.gotten = false
-				self.object:set_properties({ textures = self.base_texture })
-			end
-		end)
-
-		minetest.after(2.5, function()
-			if self and self.object and self.state == 'eat' and self.health > 0 then
-				self.state = "walk"
-			end
-		end)
-
 	end,
 
 	-- Set random color on spawn
@@ -251,6 +226,12 @@ mcl_mobs.register_mob("mobs_mc:sheep", {
 				item:add_wear(mobs_mc.shears_wear)
 				clicker:get_inventory():set_stack("main", clicker:get_wield_index(), item)
 			end
+			self.drops = {
+				{name = "mcl_mobitems:mutton",
+				chance = 1,
+				min = 1,
+				max = 2,},
+			}
 			return
 		end
 		-- Dye sheep

mods/ENTITIES/mobs_mc/shulker.lua

@@ -178,7 +178,7 @@ mcl_mobs.register_arrow("mobs_mc:shulkerbullet", {
 
 
 mcl_mobs.register_egg("mobs_mc:shulker", S("Shulker"), "#946694", "#4d3852", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:shulker","overworld",0,minetest.LIGHT_MAX+1)
+
 --[[
 mcl_mobs:spawn_specific(
 "mobs_mc:shulker",

mods/ENTITIES/mobs_mc/silverfish.lua

@@ -56,4 +56,3 @@ mcl_mobs.register_mob("mobs_mc:silverfish", {
 })
 
 mcl_mobs.register_egg("mobs_mc:silverfish", S("Silverfish"), "#6d6d6d", "#313131", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:silverfish","overworld",0,11)

mods/ENTITIES/mobs_mc/skeleton+stray.lua

@@ -45,7 +45,7 @@ local skeleton = {
 		}
 	},
 	walk_velocity = 1.2,
-	run_velocity = 2.0,
+	run_velocity = 2.4,
 	damage = 2,
 	reach = 2,
 	drops = {
@@ -89,11 +89,23 @@ local skeleton = {
 		die_speed = 15,
 		die_loop = false,
 	},
+	jock = "mobs_mc:spider",
 	on_spawn = function(self)
-		if math.random(100) == 1 then
+		minetest.after(1,function()
-			self:jock_to("mobs_mc:spider", vector.zero(), vector.zero())
+			if self and self.object then
-		end
+				if math.random(100) == 1 or self.jockey == true then -- 1% like from MCwiki
-		return true
+					self.jockey = true
+					local jock = minetest.add_entity(self.object:get_pos(), "mobs_mc:spider")
+					jock:get_luaentity().docile_by_day = false
+					self.object:set_attach(jock, "", vector.new(0,0,0), vector.new(0,0,0))
+				end
+				self.jockey = false
+				return true
+			end
+		end)
+	end,
+	on_detach=function(self, parent)
+		self.jockey = false
 	end,
 	ignited_by_sunlight = true,
 	view_range = 16,
@@ -105,7 +117,8 @@ local skeleton = {
 			if self.attack then
 				self.object:set_yaw(minetest.dir_to_yaw(vector.direction(self.object:get_pos(), self.attack:get_pos())))
 			end
-			local dmg = math.random(2, 4)
+			-- 2-4 damage per arrow
+			local dmg = math.max(4, math.random(2, 8))
 			mcl_bows.shoot_arrow("mcl_bows:arrow", pos, dir, self.object:get_yaw(), self.object, nil, dmg)
 		end
 	end,

mods/ENTITIES/mobs_mc/skeleton_wither.lua

@@ -43,7 +43,7 @@ mcl_mobs.register_mob("mobs_mc:witherskeleton", {
 		distance = 16,
 	},
 	walk_velocity = 1.2,
-	run_velocity = 2.0,
+	run_velocity = 2.4,
 	damage = 7,
 	reach = 2,
 	drops = {
@@ -118,4 +118,3 @@ mcl_vars.mg_nether_max)
 --]]
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:witherskeleton", S("Wither Skeleton"), "#141414", "#474d4d", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:witherskeleton","overworld",0,7)

mods/ENTITIES/mobs_mc/slime+magma_cube.lua

@@ -1,111 +1,10 @@
 --License for code WTFPL and otherwise stated in readmes
+
+-- FIXME: Slimes should spawn only in "slime chunks" which make up only
+-- 10% of the map.
+--
 local S = minetest.get_translator("mobs_mc")
 
-local MAPBLOCK_SIZE = 16
-
-local seed = minetest.get_mapgen_setting("seed")
-
-local slime_chunk_match
-local x_modifier
-local z_modifier
-
-local function split_by_char (inputstr, sep, limit)
-	if sep == nil then
-		sep = "%d"
-	end
-	local t = {}
-
-	local i = 0
-	for str in string.gmatch(inputstr, "(["..sep.."])") do
-		i = i --+ 1
-		table.insert(t, tonumber(str))
-		if limit and i >= limit then
-			break
-		end
-	end
-	return t
-end
-
---Seed: "16002933932875202103" == random seed
---Seed: "1807191622654296300" == cheese
---Seed: "1" = 1
-local function process_seed (seed)
-	--minetest.log("seed: " .. seed)
-
-	local split_chars = split_by_char(tostring(seed), nil, 10)
-
-	slime_chunk_match = split_chars[1]
-	x_modifier = split_chars[2]
-	z_modifier = split_chars[3]
-
-	--minetest.log("x_modifier: " .. tostring(x_modifier))
-	--minetest.log("z_modifier: " .. tostring(z_modifier))
-	--minetest.log("slime_chunk_match: " .. tostring(slime_chunk_match))
-end
-
-local processed = process_seed (seed)
-
-
-local function convert_to_chunk_value (co_ord, modifier)
-	local converted = math.floor(math.abs(co_ord) / MAPBLOCK_SIZE)
-
-	if modifier then
-		converted = (converted + modifier)
-	end
-	converted = converted % 10
-
-	--minetest.log("co_ord: " .. co_ord)
-	--minetest.log("converted: " .. converted)
-	return converted
-end
-
-assert(convert_to_chunk_value(-16) == 1, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(-15) == 0, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(-1) == 0, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(0) == 0, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(1) == 0, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(15) == 0, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(16) == 1, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(31) == 1, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(32) == 2, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(1599) == 9, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(1600) == 0, "Incorrect convert_to_chunk_value result")
-
-assert(convert_to_chunk_value(0,9) == 9, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(16,5) == 6, "Incorrect convert_to_chunk_value result")
-assert(convert_to_chunk_value(1599,4) == 3, "Incorrect convert_to_chunk_value result")
-
-local function calculate_chunk_value (pos, x_mod, z_mod)
-	local chunk_val = math.abs(convert_to_chunk_value(pos.x, x_mod) - convert_to_chunk_value(pos.z, z_mod)) % 10
-	return chunk_val
-end
-
-assert(calculate_chunk_value(vector.new(0,0,0)) == 0, "calculate_chunk_value failed")
-assert(calculate_chunk_value(vector.new(0,0,0), 1, 1) == 0, "calculate_chunk_value failed")
-assert(calculate_chunk_value(vector.new(0,0,0), 2, 1) == 1, "calculate_chunk_value failed")
-assert(calculate_chunk_value(vector.new(64,0,16)) == (4-1), "calculate_chunk_value failed")
-assert(calculate_chunk_value(vector.new(16,0,64)) == (3), "calculate_chunk_value failed")
-assert(calculate_chunk_value(vector.new(-160,0,-160)) == 0, "calculate_chunk_value failed")
-
-local function is_slime_chunk(pos)
-	if not pos then return end
-
-	local chunk_val = calculate_chunk_value (pos, x_modifier, z_modifier)
-	local slime_chunk = chunk_val == slime_chunk_match
-
-	--minetest.log("x: " ..pos.x ..  ", z:" .. pos.z)
-
-	--minetest.log("seed slime_chunk_match: " .. tostring(slime_chunk_match))
-	--minetest.log("chunk_val: " .. tostring(chunk_val))
-	--minetest.log("Is slime chunk: " .. tostring(slime_chunk))
-	return slime_chunk
-end
-
-local check_position = function (pos)
-	return is_slime_chunk(pos)
-end
-
-
 -- Returns a function that spawns children in a circle around pos.
 -- To be used as on_die callback.
 -- self: mob reference
@@ -205,8 +104,8 @@ local slime_big = {
 	attack_type = "dogfight",
 	passive = false,
 	jump = true,
-	walk_velocity = 1.9,
+	walk_velocity = 2.5,
-	run_velocity = 1.9,
+	run_velocity = 2.5,
 	walk_chance = 0,
 	jump_height = 5.2,
 	fear_height = 0,
@@ -226,8 +125,8 @@ slime_small.collisionbox = {-0.51, -0.01, -0.51, 0.51, 1.00, 0.51}
 slime_small.visual_size = {x=6.25, y=6.25}
 slime_small.damage = 3
 slime_small.reach = 2.75
-slime_small.walk_velocity = 1.8
+slime_small.walk_velocity = 1.3
-slime_small.run_velocity = 1.8
+slime_small.run_velocity = 1.3
 slime_small.jump_height = 4.3
 slime_small.spawn_small_alternative = "mobs_mc:slime_tiny"
 slime_small.on_die = spawn_children_on_die("mobs_mc:slime_tiny", 0.6, 1.0)
@@ -250,8 +149,8 @@ slime_tiny.drops = {
 	min = 0,
 	max = 2,},
 }
-slime_tiny.walk_velocity = 1.7
+slime_tiny.walk_velocity = 0.7
-slime_tiny.run_velocity = 1.7
+slime_tiny.run_velocity = 0.7
 slime_tiny.jump_height = 3
 slime_tiny.spawn_small_alternative = nil
 slime_tiny.on_die = nil
@@ -269,6 +168,7 @@ local cave_biomes = {
 	"RoofedForest_underground",
 	"Jungle_underground",
 	"Swampland_underground",
+	"MushroomIsland_underground",
 	"BirchForest_underground",
 	"Plains_underground",
 	"MesaPlateauF_underground",
@@ -312,8 +212,7 @@ minetest.LIGHT_MAX+1,
 12000,
 4,
 cave_min,
-cave_max,
+cave_max)
-nil, nil, check_position)
 
 mcl_mobs:spawn_specific(
 "mobs_mc:slime_tiny",
@@ -339,8 +238,7 @@ minetest.LIGHT_MAX+1,
 8500,
 4,
 cave_min,
-cave_max,
+cave_max)
-nil, nil, check_position)
 
 mcl_mobs:spawn_specific(
 "mobs_mc:slime_small",
@@ -366,8 +264,7 @@ minetest.LIGHT_MAX+1,
 10000,
 4,
 cave_min,
-cave_max,
+cave_max)
-nil, nil, check_position)
 
 mcl_mobs:spawn_specific(
 "mobs_mc:slime_big",
@@ -403,8 +300,8 @@ local magma_cube_big = {
 		attack = "mobs_mc_magma_cube_attack",
 		distance = 16,
 	},
-	walk_velocity = 2.5,
+	walk_velocity = 4,
-	run_velocity = 2.5,
+	run_velocity = 4,
 	damage = 6,
 	reach = 3,
 	armor = 53,
@@ -456,7 +353,7 @@ magma_cube_small.visual_size = {x=6.25, y=6.25}
 magma_cube_small.damage = 3
 magma_cube_small.reach = 2.75
 magma_cube_small.walk_velocity = .8
-magma_cube_small.run_velocity = 2.0
+magma_cube_small.run_velocity = 2.6
 magma_cube_small.jump_height = 6
 magma_cube_small.damage = 4
 magma_cube_small.reach = 2.75
@@ -534,11 +431,6 @@ nether_max)
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:magma_cube_big", S("Magma Cube"), "#350000", "#fcfc00")
 
--- non_spawn_specific is typically for mobs who don't spawn in the overworld, or mobs that don't spawn
--- naturally. However, slimes are a particular case where they spawn under different conditions in the same
--- dimension.
-mcl_mobs:non_spawn_specific("mobs_mc:slime_big","overworld",0,minetest.LIGHT_MAX+1)
-mcl_mobs:non_spawn_specific("mobs_mc:magma_cube_big","overworld",0, minetest.LIGHT_MAX+1)
 mcl_mobs.register_egg("mobs_mc:slime_big", S("Slime"), "#52a03e", "#7ebf6d")
 
 -- FIXME: add spawn eggs for small and tiny slimes and magma cubes

mods/ENTITIES/mobs_mc/snowman.lua

@@ -197,4 +197,3 @@ end
 
 -- Spawn egg
 mcl_mobs.register_egg("mobs_mc:snowman", S("Snow Golem"), "#f2f2f2", "#fd8f47", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:snowman","overworld",0,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/spider.lua

@@ -62,6 +62,9 @@ local spider = {
 			self.object:get_children()[1]:set_detach()
 		end
 	end,
+	detach_child=function(self, child)
+		child:get_luaentity().jockey = false
+	end,
 	head_swivel = "Head_Control",
 	bone_eye_height = 1,
 	curiosity = 10,
@@ -83,7 +86,7 @@ local spider = {
 		distance = 16,
 	},
 	walk_velocity = 1.3,
-	run_velocity = 2.4,
+	run_velocity = 2.8,
 	jump = true,
 	jump_height = 4,
 	view_range = 16,
@@ -289,8 +292,6 @@ mcl_mobs:spawn_specific(
 mcl_vars.mg_overworld_min,
 mcl_vars.mg_overworld_max)
 
-mcl_mobs:non_spawn_specific("mobs_mc:cave_spider","overworld",0,7)
-
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:spider", S("Spider"), "#342d26", "#a80e0e", 0)
 mcl_mobs.register_egg("mobs_mc:cave_spider", S("Cave Spider"), "#0c424e", "#a80e0e", 0)

mods/ENTITIES/mobs_mc/strider.lua

@@ -246,4 +246,3 @@ mcl_mobs:spawn_setup({
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:strider", S("Strider"), "#000000", "#FF0000", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:strider","overworld",0,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/tropical_fish.lua

@@ -59,7 +59,7 @@ end
 
 local tropical_fish = {
 	type = "animal",
-	spawn_class = "water_ambient",
+	spawn_class = "water",
 	can_despawn = true,
 	passive = true,
 	hp_min = 3,

mods/ENTITIES/mobs_mc/vex.lua

@@ -95,4 +95,3 @@ mcl_mobs.register_mob("mobs_mc:vex", {
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:vex", S("Vex"), "#7a90a4", "#e8edf1", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:vex","overworld",0,7)

mods/ENTITIES/mobs_mc/villager.lua

@@ -122,7 +122,7 @@ local professions = {
 			},
 			{
 			{ E1, { "mcl_farming:carrot_item_gold", 3, 10 } },
-			{ E1, { "mcl_potions:speckled_melon", 1, 4 } },
+			{ E1, { "mcl_potions:speckled_melon", 4, 1 } },
 			TRADE_V6_BIRCH_SAPLING,
 			TRADE_V6_DARK_OAK_SAPLING,
 			TRADE_V6_ACACIA_SAPLING,
@@ -137,7 +137,7 @@ local professions = {
 			{
 			{ { "mcl_fishing:fish_raw", 6, 6, "mcl_core:emerald", 1, 1 },{ "mcl_fishing:fish_cooked", 6, 6 } },
 			{ { "mcl_mobitems:string", 15, 20 }, E1 },
-			{ { "mcl_core:coal_lump", 10, 15 }, E1 },
+			{ { "mcl_core:coal_lump", 15, 10 }, E1 },
 			-- FIXME missing: bucket of cod + fish should be cod.
 			},
 			{
@@ -2199,5 +2199,4 @@ mobs_mc.water_level+1,
 mcl_vars.mg_overworld_max)
 --]]
 -- spawn eggs
-mcl_mobs:non_spawn_specific("mobs_mc:villager","overworld", 0, minetest.LIGHT_MAX+1)
 mcl_mobs.register_egg("mobs_mc:villager", S("Villager"), "#563d33", "#bc8b72", 0)

mods/ENTITIES/mobs_mc/villager_evoker.lua

@@ -90,4 +90,3 @@ mcl_mobs.register_mob("mobs_mc:evoker", {
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:evoker", S("Evoker"), "#959b9b", "#1e1c1a", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:evoker","overworld",0,7)

mods/ENTITIES/mobs_mc/villager_illusioner.lua

@@ -66,4 +66,3 @@ mcl_mobs.register_mob("mobs_mc:illusioner", {
 })
 
 mcl_mobs.register_egg("mobs_mc:illusioner", S("Illusioner"), "#3f5cbb", "#8a8686", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:illusioner","overworld",0,7)

mods/ENTITIES/mobs_mc/villager_vindicator.lua

@@ -78,4 +78,3 @@ mcl_mobs.register_mob("mobs_mc:vindicator", {
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:vindicator", S("Vindicator"), "#959b9b", "#275e61", 0)
-mcl_mobs:non_spawn_specific("mobs_mc:vindicator","overworld",0,7)

mods/ENTITIES/mobs_mc/villager_zombie.lua

@@ -55,7 +55,7 @@ mcl_mobs.register_mob("mobs_mc:villager_zombie", {
 	damage = 3,
 	reach = 2,
 	walk_velocity = 1.2,
-	run_velocity = 1.8,
+	run_velocity = 2.4,
 	attack_type = "dogfight",
 	group_attack = true,
 	drops = {
@@ -109,7 +109,7 @@ mcl_mobs.register_mob("mobs_mc:villager_zombie", {
 				clicker:set_wielded_item(wielditem)
 				self._curing = math.random(3 * 60, 5 * 60)
 				self.shaking = true
-				self.persistent = true
+				self.can_despawn = false
 			end
 		end
 	end,

mods/ENTITIES/mobs_mc/witch.lua

@@ -106,5 +106,5 @@ mcl_mobs.register_arrow("mobs_mc:potion_arrow", {
 
 -- spawn eggs
 mcl_mobs.register_egg("mobs_mc:witch", S("Witch"), "#340000", "#51a03e", 0, true)
-mcl_mobs:non_spawn_specific("mobs_mc:witch","overworld",0,7)
+
 mcl_wip.register_wip_item("mobs_mc:witch")

mods/ENTITIES/mobs_mc/wither.lua

@@ -132,4 +132,3 @@ mcl_mobs.register_arrow("mobs_mc:wither_skull", {
 mcl_mobs.register_egg("mobs_mc:wither", S("Wither"), "#4f4f4f", "#4f4f4f", 0, true)
 
 mcl_wip.register_wip_item("mobs_mc:wither")
-mcl_mobs:non_spawn_specific("mobs_mc:wither","overworld",0,minetest.LIGHT_MAX+1)

mods/ENTITIES/mobs_mc/wolf.lua

@@ -216,6 +216,7 @@ mcl_mobs:spawn_specific(
 	"MegaTaiga",
 	"Forest",
 	"ColdTaiga",
+	"FlowerForest_beach",
 	"Forest_beach",
 	"ColdTaiga_beach_water",
 	"Taiga_beach",

mods/ENTITIES/mobs_mc/zombie.lua

@@ -79,7 +79,7 @@ local zombie = {
 		distance = 16,
 	},
 	walk_velocity = .8,
-	run_velocity = 1.8,
+	run_velocity = 1.6,
 	damage = 3,
 	reach = 2,
 	fear_height = 4,

mods/ENTITIES/mobs_mc/zombiepig.lua

@@ -0,0 +1,155 @@
+--MCmobs v0.4
+--maikerumine
+--made for MC like Survival game
+--License for code WTFPL and otherwise stated in readmes
+
+local S = minetest.get_translator("mobs_mc")
+
+--###################
+--################### ZOMBIE PIGMAN
+--###################
+
+
+local pigman = {
+	description = S("Zombie Pigman"),
+	-- type="animal", passive=false: This combination is needed for a neutral mob which becomes hostile, if attacked
+	type = "animal",
+	passive = false,
+	spawn_class = "passive",
+	hp_min = 20,
+	hp_max = 20,
+	xp_min = 6,
+	xp_max = 6,
+	armor = {undead = 90, fleshy = 90},
+	attack_type = "dogfight",
+	group_attack = { "mobs_mc:pigman", "mobs_mc:baby_pigman" },
+	damage = 9,
+	reach = 2,
+	head_swivel = "head.control",
+	bone_eye_height = 2.4,
+	head_eye_height = 1.4,
+	curiosity = 15,
+	collisionbox = {-0.3, -0.01, -0.3, 0.3, 1.94, 0.3},
+	visual = "mesh",
+	mesh = "mobs_mc_zombie_pigman.b3d",
+	textures = { {
+		"blank.png", --baby
+		"default_tool_goldsword.png", --sword
+		"mobs_mc_zombie_pigman.png", --pigman
+	} },
+	visual_size = {x=3, y=3},
+	sounds = {
+		random = "mobs_mc_zombiepig_random",
+		war_cry = "mobs_mc_zombiepig_war_cry",
+		death = "mobs_mc_zombiepig_death",
+		damage = "mobs_mc_zombiepig_hurt",
+		distance = 16,
+	},
+	jump = true,
+	makes_footstep_sound = true,
+	walk_velocity = .8,
+	run_velocity = 2.6,
+	pathfinding = 1,
+	drops = {
+		{name = "mcl_mobitems:rotten_flesh",
+		chance = 1,
+		min = 1,
+		max = 1,
+		looting = "common"},
+		{name = "mcl_core:gold_nugget",
+		chance = 1,
+		min = 0,
+		max = 1,
+		looting = "common"},
+		{name = "mcl_core:gold_ingot",
+		chance = 40, -- 2.5%
+		min = 1,
+		max = 1,
+		looting = "rare"},
+		{name = "mcl_tools:sword_gold",
+		chance = 100 / 8.5,
+		min = 1,
+		max = 1,
+		looting = "rare"},
+	},
+	animation = {
+		stand_speed = 25,
+		walk_speed = 25,
+		run_speed = 50,
+		stand_start = 40,
+		stand_end = 80,
+		walk_start = 0,
+		walk_end = 40,
+		run_start = 0,
+		run_end = 40,
+		punch_start = 90,
+		punch_end = 130,
+	},
+	lava_damage = 0,
+	fire_damage = 0,
+	fear_height = 4,
+	view_range = 16,
+	harmed_by_heal = true,
+	fire_damage_resistant = true,
+}
+
+mcl_mobs.register_mob("mobs_mc:pigman", pigman)
+
+-- Baby pigman.
+-- A smaller and more dangerous variant of the pigman
+
+local baby_pigman = table.copy(pigman)
+baby_pigman.description = S("Baby Zombie Pigman")
+baby_pigman.collisionbox = {-0.25, -0.01, -0.25, 0.25, 0.94, 0.25}
+baby_pigman.xp_min = 13
+baby_pigman.xp_max = 13
+baby_pigman.textures = { {
+	"mobs_mc_zombie_pigman.png", --baby
+	"default_tool_goldsword.png", --sword
+	"mobs_mc_zombie_pigman.png", --pigman
+} }
+baby_pigman.walk_velocity = 1.2
+baby_pigman.run_velocity = 2.4
+baby_pigman.light_damage = 0
+baby_pigman.child = 1
+
+mcl_mobs.register_mob("mobs_mc:baby_pigman", baby_pigman)
+
+-- Regular spawning in the Nether
+mcl_mobs:spawn_specific(
+"mobs_mc:pigman",
+"nether",
+"ground",
+{
+"Nether",
+"CrimsonForest",
+},
+0,
+minetest.LIGHT_MAX+1,
+30,
+6000,
+3,
+mcl_vars.mg_nether_min,
+mcl_vars.mg_nether_max)
+-- Baby zombie is 20 times less likely than regular zombies
+mcl_mobs:spawn_specific(
+"mobs_mc:baby_pigman",
+"nether",
+"ground",
+{
+"Nether",
+"CrimsonForest",
+},
+0,
+minetest.LIGHT_MAX+1,
+30,
+100000,
+4,
+mcl_vars.mg_nether_min,
+mcl_vars.mg_nether_max)
+
+-- Spawning in Nether portals in the Overworld
+--mobs:spawn_specific("mobs_mc:pigman", {"mcl_portals:portal"}, {"air"}, 0, minetest.LIGHT_MAX+1, 30, 500, 4, mcl_vars.mg_overworld_min, mcl_vars.mg_overworld_max)
+
+-- spawn eggs
+mcl_mobs.register_egg("mobs_mc:pigman", S("Zombie Pigman"), "#ea9393", "#4c7129", 0)

mods/ENVIRONMENT/lightning/locale/lightning.dk.tr

@@ -1,3 +0,0 @@
-# textdomain: lightning
-Let lightning strike at the specified position or player. No parameter will strike yourself.=Lad lynet slå ned på den angivne position eller spiller. Hvis du ikke skriver et parameter vil lynet ramme dig selv.
-No position specified and unknown player=Ingen position og ukendt spiller.