upstream/wesnoth
1
0
Fork 0
mirror of https://github.com/wesnoth/wesnoth synced 2025-04-30 00:58:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix some issues with wmlvalidator

Browse Source
This commit is contained in:
Alexander van Gessel 2010-02-14 23:53:01 +01:00
parent 5950a82a68
commit 30c33ab654
3 changed files with 30 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
data/schema.cfg
View File

@ -2,10 +2,10 @@
identifier="re ^[a-zA-Z0-9_ ]+$"
identifierlist="re ^([a-zA-Z0-9_ ]+,)*[a-zA-Z0-9_ ]+$"
# slash-separated filenames
path="re ^([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP)\(.*\))*$"
path="re ^([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP|RIGHT)\(.*\))*$"
#TODO: imagepath, remove imagepathfunctions from path
# comma-separated paths
pathlist="re ^(([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP)\(.*\))*,)*([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP)\(.*\))*$"
pathlist="re ^(([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP|RIGHT)\(.*\))*,)*([a-zA-Z0-9_\-.+]+/)*[a-zA-Z0-9_\-.+]+(~(TC|RC|PAL|FL|GS|CS|CROP|SCALE|BL|O|R|G|B|NOP|RIGHT)\(.*\))*$"
#TODO: imagepathlist, same as above
[root]
# All possible root elements
@ -95,6 +95,7 @@
wikiuser="optional string"
[/entry]
[female:unit_type]
id="forbidden identifier"
inherit="optional boolean"
[/female:unit_type]
[font]
@ -181,6 +182,7 @@
code="required string"
[/lua]
[male:unit_type]
id="forbidden identifier"
inherit="optional boolean"
[/male:unit_type]
[movetype]
@ -316,7 +318,7 @@
experience="optional integer" # required
flag_rgb="optional string" # list of integers
gender="optional string" # enum male,female
halo="optional path"
halo="optional string" # should be animlist: list of imagepaths with animation length
hide_help="optional boolean"
hitpoints="optional integer" # required
id="required identifier"
@ -333,6 +335,7 @@
zoc="optional boolean"
[/unit_type]
[variation:unit_type]
id="forbidden identifier"
inherit="optional boolean"
variation_name="required identifier"
[/variation:unit_type]

33
data/tools/wesnoth/wmlgrammar.py
View File

@ -6,8 +6,9 @@ import re
REQUIRED = 1
OPTIONAL = 2
REPEATED = 3
FORBIDDEN = 4
class Grammar:
class Grammar(object):
def __init__(self, schema):
schema = schema.get_first("schema")
self.datatypes = {
@ -34,21 +35,21 @@ class Grammar:
def get_datatype(self, name):
return self.datatypes[name]
class Node:
class Node(object):
def __init__(self, schema, datatypes):
self.name = schema.name
self.elements = []
self.elements = set([])
self.ext_elements = [] #Ugh, do we really want to do this?
self.attributes = []
self.attributes = set([])
self.parent = None
for item in schema.get_all_text():
if item.name[0] == '_':
self.elements.append( Element(item) )
self.elements.add( Element(item) )
else:
self.attributes.append( Attribute(item, datatypes) )
self.attributes.add( Attribute(item, datatypes) )
for item in schema.get_all_subs():
if item.name == "element":
print "[element] found, not parsing yet"
print "[element] found in schema, not parsing yet"
#self.ext_elements...
else:
raise Exception( "Unknown element [%s] encountered in grammar for [%s]" % (item.name, self.name) )
@ -56,8 +57,8 @@ class Node:
self.name, self.parent = self.name.split(':',1)
def inherit(self, other):
assert self.parent == other.name
self.elements += other.elements
self.attributes += other.attributes
self.elements.update( other.elements )
self.attributes.update( other.attributes )
self.parent = None
def get_attributes(self):
return self.attributes
@ -65,7 +66,7 @@ class Node:
return self.elements
class Element:
class Element(object):
def __init__(self, schema):
first, second = schema.data.split(" ",1)
self.name = schema.name[1:]
@ -73,6 +74,10 @@ class Element:
self.subname = second
def match(self, name):
return self.name == name
def __hash__(self):
return hash(self.name)
def __cmp__(self, other):
return (isinstance(other, type(self)) or isinstance(self, type(other))) and cmp(self.name, other.name)
class ExtElement(Element):
def __init__(self, schema):
@ -82,7 +87,7 @@ class ExtElement(Element):
def match(self, name):
return bool(self.re.match(name))
class Attribute:
class Attribute(object):
def __init__(self, schema, datatypes):
first, second = schema.data.split(" ",1)
if not second in datatypes:
@ -95,6 +100,10 @@ class Attribute:
return self.name == name
def validate(self, value):
return bool(self.re.match(value))
def __hash__(self):
return hash(self.name)
def __cmp__(self, other):
return (isinstance(other, type(self)) or isinstance(self, type(other))) and cmp(self.name, other.name)
def parse_frequency(string):
if string == "required":
@ -103,6 +112,8 @@ def parse_frequency(string):
return OPTIONAL
elif string == "repeated":
return REPEATED
elif string == "forbidden":
return FORBIDDEN
else:
raise Exception( "Unknown frequency '%s'" % (string,) )

2
data/tools/wmlvalidator
View File

@ -55,6 +55,8 @@ class Validator:
print "Attribute '[%s] %s' should appear exactly once, not %d times" % (verbosename, attribute.name, nummatches)
elif attribute.freq == wmlgrammar.OPTIONAL and nummatches > 1:
print "Attribute '[%s] %s' should appear at most once, not %d times" % (verbosename, attribute.name, nummatches)
elif attribute.freq == wmlgrammar.FORBIDDEN and nummatches > 0:
print "Attribute '[%s] %s' should not appear. It appears %d times" % (verbosename, attribute.name, nummatches)
for match in matches:
if not attribute.validate(match.data):
print "Attribute '[%s] %s's value should be %s, found: %s" % (verbosename, attribute.name, attribute.type, match.data)