diff options
author | rubenwardy <rw@rubenwardy.com> | 2020-08-18 18:10:42 +0100 |
---|---|---|
committer | rubenwardy <rw@rubenwardy.com> | 2020-08-18 18:10:42 +0100 |
commit | e85d1755f058e661e451f0ab77401f820d0c836e (patch) | |
tree | 179894573c8a3a05ab701cc310b025336c392aee /app/tasks/minetestcheck/tree.py | |
parent | 1c4fe1b80c6bf6f0634e80e73403ac5f6c900170 (diff) | |
download | cheatdb-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.py | 21 |
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: |