aboutsummaryrefslogtreecommitdiff
path: root/app/tasks/minetestcheck/tree.py
diff options
context:
space:
mode:
authorrubenwardy <rw@rubenwardy.com>2020-08-18 18:10:42 +0100
committerrubenwardy <rw@rubenwardy.com>2020-08-18 18:10:42 +0100
commite85d1755f058e661e451f0ab77401f820d0c836e (patch)
tree179894573c8a3a05ab701cc310b025336c392aee /app/tasks/minetestcheck/tree.py
parent1c4fe1b80c6bf6f0634e80e73403ac5f6c900170 (diff)
downloadcheatdb-e85d1755f058e661e451f0ab77401f820d0c836e.tar.xz
Increase thread/comment ratelimiting based on rank
Diffstat (limited to 'app/tasks/minetestcheck/tree.py')
-rw-r--r--app/tasks/minetestcheck/tree.py21
1 files changed, 12 insertions, 9 deletions
diff --git a/app/tasks/minetestcheck/tree.py b/app/tasks/minetestcheck/tree.py
index 07e67f6..57a769f 100644
--- a/app/tasks/minetestcheck/tree.py
+++ b/app/tasks/minetestcheck/tree.py
@@ -128,17 +128,20 @@ class PackageTreeNode:
result["optional_depends"] = []
- # Check dependencies
- for dep in result["depends"]:
- if not basenamePattern.match(dep):
- raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \
- .format(dep, self.relative))
+ def checkDependencies(deps):
+ for dep in result["depends"]:
+ if not basenamePattern.match(dep):
+ if " " in dep:
+ raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, did you forget a comma?") \
+ .format(dep, self.relative))
+ else:
+ raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \
+ .format(dep, self.relative))
- for dep in result["optional_depends"]:
- if not basenamePattern.match(dep):
- raise MinetestCheckError(("Invalid dependency name '{}' for mod at {}, names must only contain a-z0-9_.") \
- .format(dep, self.relative))
+ # Check dependencies
+ checkDependencies(result["depends"])
+ checkDependencies(result["optional_depends"])
# Fix games using "name" as "title"
if self.type == ContentType.GAME: