Removed some todo's

This commit is contained in:
James Cole 2014-11-21 19:18:44 +01:00
parent 36f6bda525
commit 13c2db5378
2 changed files with 4 additions and 7 deletions

View File

@ -351,9 +351,6 @@ class TransactionController extends BaseController
*/
public function update(TransactionJournal $journal)
{
/*
* TODO if the piggybank_id is set to 0, does this undo all piggy bank event(s)?
*/
/** @var \FireflyIII\Database\TransactionJournal $repos */
$repos = App::make('FireflyIII\Database\TransactionJournal');

View File

@ -111,8 +111,8 @@ Event::subscribe('FireflyIII\Event\TransactionJournal');
// direct calls to models are BAD
// cleanup everything related to reminders because it still feels a bit sloppy.
// use a Database\Reminder thing instead of self-made ORM.
// TODO create static calls instead of all the App::make() things.
// TODO see if the various has-many-throughs actually get used.
// TODO set very tight rules on all models
// TODO create custom uniquely rules.
// create static calls instead of all the App::make() things.
// see if the various has-many-throughs actually get used.
// set very tight rules on all models
// create custom uniquely rules.
return $app;