swingtix/bookkeeper

A double-entry bookkeeping system developed for use at SwingTix.ca.

bookkeeper's Issues

"RuntimeWarning: DateTimeField Transaction.t_stamp received a naive datetime"

I haven't had time to investigate what this means yet but I thought I would just ticket it quickly before I forget.

I'm just working my way through the demo transactions like this

>>> book = BookSet.objects.get(description="my book")
>>> revenue = book.get_account("revenue")
>>> bank    = book.get_account("bank")
>>> expense = book.get_account("expense")

#Someone pays you can advance.  Yay!
>>> bank.debit(120.00, revenue, "our first sale")
(<AccountEntry: 120 >, <AccountEntry: -120 >)

And when I get to that last bit I get a RuntimeWarning

>>> bank.debit(120.00, revenue, "our first sale")
/home/rshea/.virtualenvs/bookkpdemo/local/lib/python2.7/site-packages/django/db/models/fields/__init__.py:1278: RuntimeWarning: DateTimeField Transaction.t_stamp received a naive datetime (2014-11-22 04:23:43.055385) while time zone support is active.
  RuntimeWarning)

(<AccountEntry: 120 >, <AccountEntry: -120 >)
>>>

This is on Django 1.7.1 and bookkeeper 0.0.1.1 .

settings.TIME_ZONE is set to 'UTC' and the machine it's running on is set to NZDT

Django 1.7 System Check throws "(1_6.W001) Some project unittests may not execute as expected"

The system checker in 1.7 hightlights the fact that test discovery has altered in 1.7.

The message goes away if you add the following to settings:

TEST_RUNNER = 'django.test.runner.DiscoverRunner'

It seems to me that tests are running OK in 1.7 (which is what the warning message is asking you to check) so I'm going to do the above.

makemigrations/runserver on Django 1.7 generate warning messages

I will fix this myself if I can but there are a couple of things in the current code which causes the Django system check to throw warnings.

Here's what it looks like in makemigrations during the initial installation .

(bookkpdemo)rshea@jaffa:~/dev/bookkpdmo/mysite> python manage.py makemigrations
/home/rshea/.virtualenvs/bookkpdemo/local/lib/python2.7/site-packages/swingtix/bookkeeper/account_api.py:155: RemovedInDjango18Warning: commit_on_success is deprecated in favor of atomic.
  def post(self, amount, other_account, description, self_memo="", other_memo="", datetime=None):

System check identified some issues:

WARNINGS:
bookkeeper.Account.positive_credit: (1_6.W002) BooleanField does not have a default value.
        HINT: Django 1.6 changed the default value of BooleanField from False to None. See https://docs.djangoproject.com/en/1.6/ref/models/fields/#booleanfield for more information.
No changes detected

It looks like both of those could be fixed in a manner which would be compatible with <= 1.6

implementing debit/credit methods in Admin

Hi Geoff,
Are you planning any documentation about bookkeper?
It is not clear for me how to add the credit debit methods into admin interface.
From admin I can make accountEntry , creates accounts ,
but this method below i do not know where to put it .

bank.credit(2.20, expense, "coffee")

thanks for showing me the direction
Peter from Slovakia