Finished refactoring.

api/__init__.py

@@ -0,0 +1,6 @@
+from flask import Blueprint
+
+api = Blueprint('api', __name__)
+
+from .controller import *
+

api/controller/__init__.py

@@ -0,0 +1,9 @@
+import pkgutil
+
+__all__ = []
+
+for loader, module_name, is_pkg in pkgutil.walk_packages(__path__):
+    __all__.append(module_name)
+
+from . import *
+

api/controller/accounts.py

@@ -14,18 +14,15 @@
     You should have received a copy of the GNU Affero General Public License
     along with Accountant.  If not, see <http://www.gnu.org/licenses/>.
 """
-from app import app
-from app import db
-
-from api.model.accounts import Account
-from api.model.entries import Entry
-from api.model.operations import Operation
-
+from .. import api
+from ..model import db
+from ..model.accounts import Account
+from ..model.entries import Entry
+from ..model.operations import Operation
 from flask import json, request
-
 from sqlalchemy import func, case, cast, extract, distinct
 
-@app.route("/api/accounts", methods=["GET"])
+@api.route("/accounts", methods=["GET"])
 def get_accounts():
     """
     Returns accounts with their solds.
@@ -50,7 +47,7 @@ def get_accounts():
         "future": str(i.future)
     } for i in query.all()])
 
-@app.route("/api/accounts/<account_id>/<year>/<month>/")
+@api.route("/accounts/<account_id>/<year>/<month>/")
 def get_account_status(account_id, year, month):
     session = db.session
 
@@ -80,7 +77,7 @@ def get_account_status(account_id, year, month):
         "balance": str(balance)
     })
 
-@app.route("/api/accounts/<account_id>/months")
+@api.route("/accounts/<account_id>/months")
 def get_months(account_id):
     session = db.session
 
@@ -94,7 +91,7 @@ def get_months(account_id):
         "month": i.month.rjust(2, '0')
     } for i in query.all()])
 
-@app.route("/api/accounts", methods=["PUT"])
+@api.route("/accounts", methods=["PUT"])
 def add_account():
     session = db.session
 
@@ -110,7 +107,7 @@ def add_account():
         raise
 
 
-@app.route("/api/accounts/<account_id>", methods=["PUT"])
+@api.route("/accounts/<account_id>", methods=["PUT"])
 def update_account(account_id):
     session = db.session
 
@@ -128,16 +125,16 @@ def update_account(account_id):
         session.rollback()
         raise
 
-@app.route("/api/accounts/<account_id>", methods=["DELETE"])
+@api.route("/accounts/<account_id>", methods=["DELETE"])
 def delete_account(account_id):
     session = db.session
 
     try:
         account = session.query(Account).filter(Account.id == account_id).first()
-    
+
         session.delete(account)
         session.commit()
-    
+
         return json.dumps("Account #%s deleted." % account_id)
     except:
         session.rollback()

api/controller/entries.py

@@ -14,23 +14,18 @@
     You should have received a copy of the GNU Affero General Public License
     along with Accountant.  If not, see <http://www.gnu.org/licenses/>.
 """
-from app import app
-from app import db
-
-from api.model.entries import Entry
-from api.model.operations import Operation
-from api.model.scheduled_operations import ScheduledOperation
-
+from .. import api
+from ..model import db
+from ..model.entries import Entry
+from ..model.operations import Operation
+from ..model.scheduled_operations import ScheduledOperation
+from flask import json, request
 from sqlalchemy import func, desc
 from sqlalchemy.ext.hybrid import hybrid_property, hybrid_method
 from sqlalchemy.orm import sessionmaker, column_property, aliased
 from sqlalchemy.sql import func, select, case
 
-#from sqlalchemy import *
-
-from flask import json, request
-
-@app.route("/api/entries/<account_id>/<year>/<month>")
+@api.route("/entries/<account_id>/<year>/<month>")
 def get_entries(account_id, year, month):
     """
     Return entries for an account, year, and month.
@@ -61,7 +56,7 @@ def get_entries(account_id, year, month):
          "scheduled_operation_id": i.scheduled_operation_id
     } for i in query.all()])
 
-@app.route("/api/entries", methods=["PUT"])
+@api.route("/entries", methods=["PUT"])
 def add_entry():
     session = db.session
 
@@ -75,22 +70,22 @@ def add_entry():
           account_id = request.json['account_id'],
           scheduled_operation_id = request.json['scheduled_operation_id']
         )
-    
+
         session.add(entry)
         session.commit()
-    
+
         return json.dumps("Entry added.")
     except:
         session.rollback()
         raise
 
-@app.route("/api/entries/<entry_id>", methods=["PUT"])
+@api.route("/entries/<entry_id>", methods=["PUT"])
 def update_entry(entry_id):
     session = db.session
 
     try:
         entry = session.query(Entry).filter(Entry.id == entry_id).first()
-    
+
         entry.id = entry_id
         entry.operation_date = request.json['operation_date']
         entry.pointed = request.json['pointed']
@@ -99,25 +94,25 @@ def update_entry(entry_id):
         entry.category = request.json['category']
         entry.account_id = request.json['account_id']
         entry.scheduled_operation_id = request.json['scheduled_operation_id']
-    
+
         session.merge(entry)
         session.commit()
-    
+
         return json.dumps("Entry #%s updated." % entry_id)
     except:
         session.rollback()
         raise
 
-@app.route("/api/entries/<entry_id>", methods=["DELETE"])
+@api.route("/entries/<entry_id>", methods=["DELETE"])
 def delete_entry(entry_id):
     session = db.session
 
     try:
         entry = session.query(Entry).filter(Entry.id == entry_id).first()
-    
+
         session.delete(entry)
         session.commit()
-    
+
         return json.dumps("Entry #%s deleted." % entry_id)
     except:
         session.rollback()

api/controller/scheduled_operations.py

@@ -1,16 +1,13 @@
-from app import app
-from app import db
-
-from api.model.scheduled_operations import ScheduledOperation
-
+from .. import api
+from ..model import db
+from ..model.scheduled_operations import ScheduledOperation
+from flask import json, request
 from sqlalchemy import func, desc
 from sqlalchemy.ext.hybrid import hybrid_property, hybrid_method
 from sqlalchemy.orm import sessionmaker, column_property
 from sqlalchemy.sql import func, select
 
-from flask import json, request
-
-@app.route("/api/scheduled_operations/<account_id>")
+@api.route("/scheduled_operations/<account_id>")
 def get_scheduled_operations(account_id):
     """
     Return entries for an account, year, and month.
@@ -41,7 +38,7 @@ def get_scheduled_operations(account_id):
          "account_id": i.account_id
     } for i in query.all()])
 
-@app.route("/api/scheduled_operations", methods=["PUT"])
+@api.route("/scheduled_operations", methods=["PUT"])
 def add_scheduled_operation():
     session = db.session
 
@@ -56,22 +53,22 @@ def add_scheduled_operation():
           category = request.json['category'],
           account_id = request.json['account_id']
         )
-    
+
         session.add(scheduledOperation)
         session.commit()
-    
+
         return json.dumps("Scheduled operation added.")
     except:
         session.rollback()
         raise
 
-@app.route("/api/scheduled_operations/<scheduled_operation_id>", methods=["PUT"])
+@api.route("/scheduled_operations/<scheduled_operation_id>", methods=["PUT"])
 def update_scheduled_operation(scheduled_operation_id):
     session = db.session
 
     try:
         scheduledOperation = session.query(ScheduledOperation).filter(ScheduledOperation.id == scheduled_operation_id).first()
-    
+
         scheduledOperation.id = scheduled_operation_id
         scheduledOperation.start_date = request.json['start_date'],
         scheduledOperation.stop_date = request.json['stop_date'],
@@ -81,25 +78,25 @@ def update_scheduled_operation(scheduled_operation_id):
         scheduledOperation.value = request.json['value']
         scheduledOperation.category = request.json['category']
         scheduledOperation.account_id = request.json['account_id']
-    
+
         session.merge(scheduledOperation)
         session.commit()
-    
+
         return json.dumps("Scheduled operation #%s updated." % scheduled_operation_id)
     except:
         session.rollback()
         raise
 
-@app.route("/api/scheduled_operations/<scheduled_operation_id>", methods=["DELETE"])
+@api.route("/scheduled_operations/<scheduled_operation_id>", methods=["DELETE"])
 def delete_scheduled_operation(scheduled_operation_id):
     session = db.session
 
     try:
         scheduledOperation = session.query(ScheduledOperation).filter(ScheduledOperation.id == scheduled_operation_id).first()
-    
+
         session.delete(scheduledOperation)
         session.commit()
-    
+
         return json.dumps("Scheduled operation #%s deleted." % scheduled_operation_id)
     except:
         session.rollback()

api/model/__init__.py

@@ -0,0 +1,31 @@
+from contextlib import contextmanager
+from flask.ext.sqlalchemy import SQLAlchemy
+from sqlalchemy.ext.declarative import declarative_base
+from sqlalchemy.orm import scoped_session, sessionmaker
+
+db = SQLAlchemy()
+
+@contextmanager
+def session_scope(engine):
+    if engine:
+        session = scoped_session(sessionmaker(autocommit = False, autoflush = False, bind = engine))
+        #Base.query = session.query_property()
+
+        try:
+            yield session
+            session.commit()
+        except:
+            session.rollback()
+            raise
+        finally:
+            session.close()
+
+import pkgutil
+
+__all__ = []
+
+for loader, module_name, is_pkg in pkgutil.walk_packages(__path__):
+    __all__.append(module_name)
+
+from . import *
+

api/model/accounts.py

@@ -14,8 +14,7 @@
     You should have received a copy of the GNU Affero General Public License
     along with Accountant.  If not, see <http://www.gnu.org/licenses/>.
 """
-from app import app
-from app import db
+from . import db
 
 class Account(db.Model):
     id = db.Column(db.Integer, primary_key = True)

api/model/entries.py

@@ -14,12 +14,9 @@
     You should have received a copy of the GNU Affero General Public License
     along with Accountant.  If not, see <http://www.gnu.org/licenses/>.
 """
-from app import app
-from app import db
-
-from api.model.accounts import Account
-from api.model.scheduled_operations import ScheduledOperation
-
+from . import db
+from .accounts import Account
+from .scheduled_operations import ScheduledOperation
 from sqlalchemy import func, desc
 from sqlalchemy.orm import column_property
 from sqlalchemy.sql import func, select

api/model/operations.py

@@ -14,11 +14,10 @@
     You should have received a copy of the GNU Affero General Public License
     along with Accountant.  If not, see <http://www.gnu.org/licenses/>.
 """
-from app import app
-from app import db
+from . import db
 
-from api.model.accounts import Account
-from api.model.scheduled_operations import ScheduledOperation
+from .accounts import Account
+from .scheduled_operations import ScheduledOperation
 
 from sqlalchemy import func, desc
 from sqlalchemy.orm import column_property

api/model/scheduled_operations.py

@@ -1,8 +1,5 @@
-from app import app
-from app import db
-
-from api.model.accounts import Account
-
+from . import db
+from .accounts import Account
 from sqlalchemy import func, desc
 from sqlalchemy.orm import column_property
 from sqlalchemy.sql import func, select