Remove unused get_by_request helper
This commit is contained in:
parent
1ef16c74b9
commit
b778942dd7
@ -66,10 +66,6 @@ class Portfolios(object):
|
|||||||
|
|
||||||
return portfolio
|
return portfolio
|
||||||
|
|
||||||
@classmethod
|
|
||||||
def get_by_request(cls, request):
|
|
||||||
return PortfoliosQuery.get_by_request(request)
|
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def get_with_members(cls, user, portfolio_id):
|
def get_with_members(cls, user, portfolio_id):
|
||||||
portfolio = PortfoliosQuery.get(portfolio_id)
|
portfolio = PortfoliosQuery.get(portfolio_id)
|
||||||
|
@ -1,8 +1,5 @@
|
|||||||
from sqlalchemy.orm.exc import NoResultFound
|
|
||||||
|
|
||||||
from atst.database import db
|
from atst.database import db
|
||||||
from atst.domain.common import Query
|
from atst.domain.common import Query
|
||||||
from atst.domain.exceptions import NotFoundError
|
|
||||||
from atst.models.portfolio import Portfolio
|
from atst.models.portfolio import Portfolio
|
||||||
from atst.models.portfolio_role import PortfolioRole, Status as PortfolioRoleStatus
|
from atst.models.portfolio_role import PortfolioRole, Status as PortfolioRoleStatus
|
||||||
|
|
||||||
@ -10,15 +7,6 @@ from atst.models.portfolio_role import PortfolioRole, Status as PortfolioRoleSta
|
|||||||
class PortfoliosQuery(Query):
|
class PortfoliosQuery(Query):
|
||||||
model = Portfolio
|
model = Portfolio
|
||||||
|
|
||||||
@classmethod
|
|
||||||
def get_by_request(cls, request):
|
|
||||||
try:
|
|
||||||
portfolio = db.session.query(Portfolio).filter_by(request=request).one()
|
|
||||||
except NoResultFound:
|
|
||||||
raise NotFoundError("portfolio")
|
|
||||||
|
|
||||||
return portfolio
|
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def get_for_user(cls, user):
|
def get_for_user(cls, user):
|
||||||
return (
|
return (
|
||||||
|
@ -1,4 +1,4 @@
|
|||||||
from sqlalchemy import Column, ForeignKey, String
|
from sqlalchemy import Column, String
|
||||||
from sqlalchemy.orm import relationship
|
from sqlalchemy.orm import relationship
|
||||||
from itertools import chain
|
from itertools import chain
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user