Skip to content

Commit

Permalink
Modified TODO comments related to ODS-6432 to refer to ODS-6510, whic…
Browse files Browse the repository at this point in the history
…h is a new ticket/branch to address the necessary refactoring/consolidation.
  • Loading branch information
gmcelhanon committed Sep 21, 2024
1 parent 6c6ef9f commit 08e9d84
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ public QueryBuilder GetQueryBuilder(
IQueryParameters queryParameters,
IDictionary<string, string> additionalParameters)
{
// TODO: ODS-6432 - This needs to be invokes an authorization decorator of some sort -- copied from the data management controller pipeline. Also, look for approach to DRY here.
// TODO: ODS-6510 - This needs to be invokes an authorization decorator of some sort -- copied from the data management controller pipeline. Also, look for approach to DRY here.
EstablishAuthorizationFilteringContext(entity);

var rowNumbersQueryBuilder = new QueryBuilder(_dialect);
Expand Down Expand Up @@ -120,7 +120,7 @@ public QueryBuilder GetQueryBuilder(
return rowNumbersQueryBuilder;
}

// TODO: ODS-6432 - ALL OF THIS NEEDS TO REFACTORED OUT INTO A SECURITY COMPONENT SOMEWHERE - Pay attention to DRY
// TODO: ODS-6510 - THIS NEEDS TO REFACTORED OUT INTO A SECURITY COMPONENT SOMEWHERE - Pay attention to DRY
private void EstablishAuthorizationFilteringContext(dynamic aggregateRootEntity)
{
// Establish the authorization context -- currently done in SetAuthorizationContext pipeline step, not accessible here
Expand Down

0 comments on commit 08e9d84

Please sign in to comment.