Skip to content

Commit 9eba975

Browse files
author
sreeder
committed
fix order of filtering objects
1 parent c646fb2 commit 9eba975

2 files changed

Lines changed: 11 additions & 9 deletions

File tree

Examples/Sample.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717
#connect to database
1818
# createconnection (dbtype, servername, dbname, username, password)
1919
# session_factory = dbconnection.createConnection('connection type: sqlite|mysql|mssql|postgresql', '/your/path/to/db/goes/here', 2.0)#sqlite
20-
session_factory = dbconnection.createConnection('mysql', 'localhost', 'odm2', 'ODM', 'odm')#mysql
21-
# session_factory= dbconnection.createConnection('mssql', "(local)", "ODM2", "ODM", "odm")#win MSSQL
20+
# session_factory = dbconnection.createConnection('mysql', 'localhost', 'odm2', 'ODM', 'odm')#mysql
21+
session_factory= dbconnection.createConnection('mssql', "(local)", "ODM2", "ODM", "odm")#win MSSQL
2222
# session_factory= dbconnection.createConnection('mssql', "arroyoodm2", "", "ODM", "odm")#mac/linux MSSQL
2323

2424

@@ -32,7 +32,7 @@
3232

3333
#_session = session_factory.getSession()
3434
read = ReadODM2(session_factory)
35-
create =CreateODM2(session_factory)
35+
create = CreateODM2(session_factory)
3636

3737

3838

odm2api/ODM2/services/readService.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -514,7 +514,7 @@ def getAffiliations(self, ids=None, personfirst=None, personlast=None, orgcode=N
514514
Results
515515
"""
516516

517-
def getResults(self, ids=None, type= None, uuids= None, actionid=None, simulationid = None, sfid= None, variableid = None):
517+
def getResults(self, ids=None, type=None, uuids=None, actionid=None, simulationid = None, sfid= None, variableid = None):
518518

519519
# TODO what if user sends in both type and actionid vs just actionid
520520
"""
@@ -529,16 +529,18 @@ def getResults(self, ids=None, type= None, uuids= None, actionid=None, simulatio
529529

530530
query = self._session.query(Results)
531531

532-
if actionid: query = query.join(FeatureActions).filter_by(ActionID=actionid)
533-
if type: query = query.filter_by(ResultTypeCV= type)
532+
533+
if type: query = query.filter_by(ResultTypeCV=type)
534+
if variableid: query = query.filter_by(VariableID=variableid)
535+
if ids: query = query.filter(Results.ResultID.in_(ids))
536+
if uuids: query = query.filter(Results.ResultUUID.in_(uuids))
534537
if simulationid: query = query.join(FeatureActions)\
535538
.join(Actions)\
536539
.join(Simulations)\
537540
.filter_by(SimulationID=simulationid)
538-
if ids: query = query.filter(Results.ResultID.in_(ids))
539-
if uuids: query =query.filter(Results.ResultUUID.in_(uuids))
541+
if actionid: query = query.join(FeatureActions).filter_by(ActionID=actionid)
540542
if sfid: query = query.join(FeatureActions).filter_by(SamplingFeatureID=sfid)
541-
if variableid: query = query.filter_by(VariableID=variableid)
543+
542544

543545
try:
544546
return query.all()

0 commit comments

Comments
 (0)