From 0c70405407a8c8e6de9ae0e2a289067df1446a6c Mon Sep 17 00:00:00 2001 From: Russ Allbery Date: Wed, 5 Dec 2007 23:29:42 +0000 Subject: Further bits from Simon's code review. --- TODO | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/TODO b/TODO index 44ffb15..2d888fa 100644 --- a/TODO +++ b/TODO @@ -163,6 +163,15 @@ Future work: * The Wallet::Config class is very ugly and could use some better internal API to reference the variables in it. +* Consider using Class::Accessor to get rid of the scaffolding code to + access object data, and a Wallet::Base class to handle things like the + error() method common to many classes. + +* Use Class::DBI and Class::Trigger to handle the data access layer rather + than writing SQL directly, and implement the logging requirements with + triggers rather than explicit SQL. This may also replace + Wallet::Schema. + May or may not be good ideas: * Remove the hard-coded ADMIN ACL in the server with something more -- cgit v1.2.3