*** WIP ***
Committing to work on another ticket There are TODOs in study_service
This commit is contained in:
parent
b03e596dcf
commit
6d548c2a74
|
@ -41,13 +41,14 @@ class ProtocolBuilderStatus(enum.Enum):
|
||||||
class ProtocolBuilderStudy(object):
|
class ProtocolBuilderStudy(object):
|
||||||
def __init__(
|
def __init__(
|
||||||
self, STUDYID: int, TITLE: str, NETBADGEID: str,
|
self, STUDYID: int, TITLE: str, NETBADGEID: str,
|
||||||
DATE_MODIFIED: str, Q_COMPLETE: str=None
|
DATE_MODIFIED: str, Q_COMPLETE: str, HSRNUMBER: str
|
||||||
):
|
):
|
||||||
self.STUDYID = STUDYID
|
self.STUDYID = STUDYID
|
||||||
self.TITLE = TITLE
|
self.TITLE = TITLE
|
||||||
self.NETBADGEID = NETBADGEID
|
self.NETBADGEID = NETBADGEID
|
||||||
self.DATE_MODIFIED = DATE_MODIFIED
|
self.DATE_MODIFIED = DATE_MODIFIED
|
||||||
self.Q_COMPLETE = Q_COMPLETE
|
self.Q_COMPLETE = Q_COMPLETE
|
||||||
|
self.HSRNUMBER = HSRNUMBER
|
||||||
|
|
||||||
class ProtocolBuilderStudySchema(ma.Schema):
|
class ProtocolBuilderStudySchema(ma.Schema):
|
||||||
class Meta:
|
class Meta:
|
||||||
|
|
|
@ -30,10 +30,8 @@ class ProtocolBuilderService(object):
|
||||||
ProtocolBuilderService.__enabled_or_raise()
|
ProtocolBuilderService.__enabled_or_raise()
|
||||||
if not isinstance(user_id, str):
|
if not isinstance(user_id, str):
|
||||||
raise ApiError("protocol_builder_error", "This user id is invalid: " + str(user_id))
|
raise ApiError("protocol_builder_error", "This user id is invalid: " + str(user_id))
|
||||||
user_id = 'cah3us'
|
|
||||||
url = ProtocolBuilderService.STUDY_URL % user_id
|
url = ProtocolBuilderService.STUDY_URL % user_id
|
||||||
logging.info(f'get_studies: url: {url}')
|
logging.info(f'get_studies: url: {url}')
|
||||||
logging.debug(f'get_studies: url: {url}')
|
|
||||||
response = requests.get(url)
|
response = requests.get(url)
|
||||||
if response.ok and response.text:
|
if response.ok and response.text:
|
||||||
logging.info(f'get_studies: response.text: {response.text}')
|
logging.info(f'get_studies: response.text: {response.text}')
|
||||||
|
|
|
@ -37,9 +37,13 @@ class StudyService(object):
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _is_valid_study(study_id):
|
def _is_valid_study(study_id):
|
||||||
|
# TODO: Determine if this is always a list now
|
||||||
|
# TODO: If so, just grab the first item, and get rid of this `if` statement
|
||||||
study_info = ProtocolBuilderService().get_study_details(study_id)
|
study_info = ProtocolBuilderService().get_study_details(study_id)
|
||||||
if 'REVIEW_TYPE' in study_info.keys() and study_info['REVIEW_TYPE'] in [2, 3, 23, 24]:
|
if isinstance(study_info, list) and len(study_info) > 0:
|
||||||
return True
|
study_info = study_info[0]
|
||||||
|
if isinstance(study_info, dict) and 'REVIEW_TYPE' in study_info.keys() and study_info['REVIEW_TYPE'] in [2, 3, 23, 24]:
|
||||||
|
return True
|
||||||
return False
|
return False
|
||||||
|
|
||||||
def get_studies_for_user(self, user, include_invalid=False):
|
def get_studies_for_user(self, user, include_invalid=False):
|
||||||
|
@ -271,7 +275,8 @@ class StudyService(object):
|
||||||
for code, doc in doc_dictionary.items():
|
for code, doc in doc_dictionary.items():
|
||||||
|
|
||||||
doc['required'] = False
|
doc['required'] = False
|
||||||
if ProtocolBuilderService.is_enabled() and doc['id']:
|
# TODO: figure out the correct test here
|
||||||
|
if ProtocolBuilderService.is_enabled() and doc['id'] != '':
|
||||||
pb_data = next((item for item in pb_docs if int(item['AUXDOCID']) == int(doc['id'])), None)
|
pb_data = next((item for item in pb_docs if int(item['AUXDOCID']) == int(doc['id'])), None)
|
||||||
if pb_data:
|
if pb_data:
|
||||||
doc['required'] = True
|
doc['required'] = True
|
||||||
|
|
Loading…
Reference in New Issue