Don't raise ValueError when creating non-creatable sections, return None instead.

This commit is contained in:
David Vierra 2015-10-07 01:30:55 -10:00
parent 83a882e7d0
commit bb05c9bb19

View File

@ -282,16 +282,13 @@ class AnvilChunkData(object):
"""
:param cy: Section number (y coordinate >> 4)
:param create: If False, returns None if the section is not present, otherwise creates the section.
:param create: If False, returns None if the section is not present or could not
be created, otherwise creates the section.
:returns: The requested section, or None if it wasn't created.
:rtype: AnvilSection
:raises: ValueError if create is True and the requested section can't be stored in this chunk.
"""
if (cy << 4) > self.adapter.maxHeight or cy < 0:
if create:
raise ValueError("Requested section %s exceeds world height" % cy)
else:
return None
return None
section = self._sections.get(cy)
if not section: