X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=blobdiff_plain;f=bluechips%2Fcontrollers%2Fspend.py;h=cc6466fed32f95ef15365e9bc86ec53079773dc2;hb=afa1282a519cc7bda8e11082914ef4045139f113;hp=749d73039a4c34550d5baaadf607315f098b65b8;hpb=b57666fb3255e06ec0d55cabd26a01c83d214326;p=bluechips.git diff --git a/bluechips/controllers/spend.py b/bluechips/controllers/spend.py index 749d730..cc6466f 100644 --- a/bluechips/controllers/spend.py +++ b/bluechips/controllers/spend.py @@ -8,13 +8,17 @@ from decimal import Decimal, InvalidOperation from bluechips.lib.base import * -from pylons import request +from pylons import request, app_globals as g from pylons.decorators.rest import dispatch_on from pylons.decorators import validate +from pylons.controllers.util import abort from formencode import validators, Schema from formencode.foreach import ForEach from formencode.variabledecode import NestedVariables +from formencode.schema import SimpleFormValidator + +from mailer import Message log = logging.getLogger(__name__) @@ -26,6 +30,12 @@ class ShareSchema(Schema): amount = validators.Number(not_empty=True) +def validate_state(value_dict, state, validator): + if all(s['amount'] == 0 for s in value_dict['shares']): + return {'shares-0.amount': 'Need at least one non-zero share'} +ValidateNotAllZero = SimpleFormValidator(validate_state) + + class ExpenditureSchema(Schema): "Validate an expenditure." allow_extra_fields = False @@ -35,6 +45,7 @@ class ExpenditureSchema(Schema): description = validators.UnicodeString() date = validators.DateConverter() shares = ForEach(ShareSchema) + chained_validators = [ValidateNotAllZero] class SpendController(BaseController): @@ -62,6 +73,24 @@ class SpendController(BaseController): else: c.title = 'Edit an Expenditure' c.expenditure = meta.Session.query(model.Expenditure).get(id) + if c.expenditure is None: + abort(404) + c.values = {} + for ii, user_row in enumerate(c.users): + user_id, user = user_row + try: + share = [s.share for s in c.expenditure.splits + if s.user == user][0] + if c.expenditure.amount == 0: + percent = 0 + else: + percent = (Decimal(100) * Decimal(int(share)) / + Decimal(int(c.expenditure.amount))).\ + quantize(Decimal("0.001")) + except IndexError: + percent = 0 + c.values['shares-%d.amount' % ii] = percent + return render('/spend/index.mako') @validate(schema=ExpenditureSchema(), form='edit', variable_decode=True) @@ -71,27 +100,36 @@ class SpendController(BaseController): if id is None: e = model.Expenditure() meta.Session.add(e) + op = 'created' else: e = meta.Session.query(model.Expenditure).get(id) + if e is None: + abort(404) + op = 'updated' # Set the fields that were submitted shares = self.form_result.pop('shares') update_sar(e, self.form_result) - if e.id is not None: - e.update_split() users = dict(meta.Session.query(model.User.id, model.User).all()) split_dict = {} for share_params in shares: user = users[share_params['user_id']] - split_dict[user] = Decimal(share_params['amount']) + split_dict[user] = Decimal(str(share_params['amount'])) e.split(split_dict) meta.Session.commit() - - if id is None: - h.flash("Expenditure created.") - else: - h.flash('Expenditure updated.') + show = ("Expenditure of %s paid for by %s %s." % + (e.amount, e.spender, op)) + h.flash(show) + + # Send email notification to involved users if they have an email set. + involved_users = set(sp.user for sp in e.splits if sp.share != 0) + involved_users.add(e.spender) + body = render('/emails/expenditure.txt', + extra_vars={'expenditure': e, + 'op': op}) + g.handle_notification(involved_users, show, body) + return h.redirect_to('/')