-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OTC-10: Capitation Payment Report use data from tblCapitationPayment #87
Conversation
I will wait for the call of the batch to modular before merging to develop. |
@dragos-dobre I've added calling modular batch process, can you review? |
IMIS/ProcessBatches.aspx.vb
Outdated
If dt Is Nothing Then Exit Sub | ||
Session("Report") = dt | ||
|
||
Response.Redirect("Report.aspx?r=ca&tid=18") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tid is used for getting back to reports.aspx. As the capitation report is not in the report list, I don't think we need to add it here. Please remove.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tid removed from redireciton.
IMIS/Reports.aspx.vb
Outdated
If Not getCapitationPayment() Then Exit Sub | ||
Session("Report") = dt | ||
url = "Report.aspx?r=ca&tid=18" | ||
ElseIf SelectedValueID = 19 Then | ||
If Not getRejectedPhoto() Then Exit Sub | ||
Session("Report") = dt | ||
url = "Report.aspx?r=rp&tid=19" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you removed the cap rep, shouldn't you change the tid to 18 to match SelectedValueID? I don't understand why Exact didn't concat url with SelectedValueID.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, I changed tids for every report in which SelectedValueID has changed.
Part of OTC-10
Procedure uspSSRSCapitationPayment is replaced with uspSSRSRetrieveCapitationPaymentReportData. Before report is generated, it is checked whether the batch run has been executed for given period and location.