Skip to content

Commit

Permalink
Don't filter ages with no owners, since Dirtsand always returns zero
Browse files Browse the repository at this point in the history
  • Loading branch information
@boq
boq committed Feb 5, 2012
1 parent 35ed62a commit 9864ecb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Python/nxusBookMachine.py
View file
Expand Up @@ -495,8 +495,8 @@ def gotPublicAgeList(self, ages):
ageFilename = age[0].getAgeFilename()
if ageFilename == "Neighborhood":
# if the current population and number of owners is zero then don't display it
#looks like it doesn't work (at least on Dirtsand)
if age[2] != 0 or age[1] != 0:
#HACK: looks like it doesn't work on Dirtsand, since owner count is always 0
#if age[2] != 0 or age[1] != 0:
hoods.append(AgeInstance(age))
else:
PtDebugPrint("nxusBookMachine.gotPublicAgeList() - got the list of %s instances" % ageFilename)
Expand Down

2 comments on commit 9864ecb

@NadnerbD
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should really never hack around a lack of functionality in dirtsand in the python. That just enables bad serving. :P

@boq
Copy link
Owner Author

@boq boq commented on 9864ecb Feb 5, 2012

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not making PR out of this right now. It's just emergency hax, until it's fixed proper way. Maintainers of repo can apply it temporarily to keep players happy, if they want.

Please sign in to comment.