Add luacheck

.luacheckrc

@@ -0,0 +1,14 @@
+unused_args = false
+allow_defined_top = true
+
+globals = {
+	"minetest", "awards",
+}
+
+read_globals = {
+	string = {fields = {"split"}},
+	table = {fields = {"copy", "getn"}},
+	"vector", "default", "ItemStack",
+	"dump", "sfinv", "intllib",
+	"unified_inventory",
+}

.travis.yml

@@ -0,0 +1,12 @@
+language: generic
+sudo: false
+addons:
+  apt:
+    packages:
+    - luarocks
+before_install:
+  - luarocks install --local luacheck
+script:
+- $HOME/.luarocks/bin/luacheck --no-color .
+notifications:
+  email: false

api.lua

@@ -46,8 +46,8 @@ function awards.load()
 	return {}
 end
 
-function awards.register_trigger(name, func)
+function awards.register_trigger(name, tfunc)
-	awards.trigger_types[name] = func
+	awards.trigger_types[name] = tfunc
 	awards.on[name] = {}
 	awards['register_on_'..name] = function(func)
 		table.insert(awards.on[name], func)

triggers.lua

@@ -233,9 +233,9 @@ minetest.register_on_dignode(function(pos, oldnode, digger)
 				local tnodedug = string.split(entry.node, ":")
 				local tmod = tnodedug[1]
 				local titem = tnodedug[2]
-				if not tmod or not titem or not data.count[tmod] or not data.count[tmod][titem] then
+				if not (not tmod or not titem or not data.count[tmod] or
-					-- table running failed!
+							not data.count[tmod][titem]) and
-				elseif data.count[tmod][titem] > entry.target-1 then
+						data.count[tmod][titem] > entry.target-1 then
 					return entry.award
 				end
 			elseif awards.get_total_item_count(data, "count") > entry.target-1 then
@@ -260,9 +260,9 @@ minetest.register_on_placenode(function(pos, node, digger)
 				local tnodedug = string.split(entry.node, ":")
 				local tmod = tnodedug[1]
 				local titem = tnodedug[2]
-				if not tmod or not titem or not data.place[tmod] or not data.place[tmod][titem] then
+				if not (not tmod or not titem or not data.place[tmod] or
-					-- table running failed!
+							not data.place[tmod][titem]) and
-				elseif data.place[tmod][titem] > entry.target-1 then
+						data.place[tmod][titem] > entry.target-1 then
 					return entry.award
 				end
 			elseif awards.get_total_item_count(data, "place") > entry.target-1 then
@@ -286,9 +286,9 @@ minetest.register_on_item_eat(function(hp_change, replace_with_item, itemstack,
 				local titemstring = string.split(entry.item, ":")
 				local tmod = titemstring[1]
 				local titem = titemstring[2]
-				if not tmod or not titem or not data.eat[tmod] or not data.eat[tmod][titem] then
+				if not (not tmod or not titem or not data.eat[tmod] or
-					-- table running failed!
+							not data.eat[tmod][titem]) and
-				elseif data.eat[tmod][titem] > entry.target-1 then
+						data.eat[tmod][titem] > entry.target-1 then
 					return entry.award
 				end
 			elseif awards.get_total_item_count(data, "eat") > entry.target-1 then
@@ -314,9 +314,9 @@ minetest.register_on_craft(function(itemstack, player, old_craft_grid, craft_inv
 				local titemcrafted = string.split(entry.item, ":")
 				local tmod = titemcrafted[1]
 				local titem = titemcrafted[2]
-				if not tmod or not titem or not data.craft[tmod] or not data.craft[tmod][titem] then
+				if not (not tmod or not titem or not data.craft[tmod] or
-					-- table running failed!
+							not data.craft[tmod][titem]) and
-				elseif data.craft[tmod][titem] > entry.target-1 then
+						data.craft[tmod][titem] > entry.target-1 then
 					return entry.award
 				end
 			elseif awards.get_total_item_count(data, "craft") > entry.target-1 then