|
Description
Jonathan Druart
2024-10-23 14:47:49 UTC
Created attachment 173221 [details] [review] Bug 38243: Remove DT's header_filter custom param It seems to be a leftover from bug 32401, the code does not make sense: 569 if(options.header_filter) { 570 options.query_parameters = query_parameters; 571 } else { 572 dataSet.q = query_parameters; 573 delete options.query_parameters; 574 } Then options.query_parameters is never used. Created attachment 173222 [details] [review] Bug 38243: Remove options.query_parameters This is never passed: `git grep query_parameters` Created attachment 173234 [details] [review] Bug 38243: Remove DT's header_filter custom param It seems to be a leftover from bug 32401, the code does not make sense: 569 if(options.header_filter) { 570 options.query_parameters = query_parameters; 571 } else { 572 dataSet.q = query_parameters; 573 delete options.query_parameters; 574 } Then options.query_parameters is never used. Signed-off-by: David Nind <david@davidnind.com> Created attachment 173235 [details] [review] Bug 38243: Remove options.query_parameters This is never passed: `git grep query_parameters` Signed-off-by: David Nind <david@davidnind.com> Not sure exactly how to test, but after the patch is applied a 'git grep options.query_parameters' returns no results. Feel free to change back to 'Needs Signoff' if that is not sufficient. Created attachment 173513 [details] [review] Bug 38243: Remove DT's header_filter custom param It seems to be a leftover from bug 32401, the code does not make sense: 569 if(options.header_filter) { 570 options.query_parameters = query_parameters; 571 } else { 572 dataSet.q = query_parameters; 573 delete options.query_parameters; 574 } Then options.query_parameters is never used. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 173514 [details] [review] Bug 38243: Remove options.query_parameters This is never passed: `git grep query_parameters` Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Confirmed, working, Passing QA Pushed for 24.11! Well done everyone, thank you! |