From 11b13edac90733d53c605125416a265f30474c0a Mon Sep 17 00:00:00 2001 From: Christopher Baines Date: Wed, 3 Jun 2020 20:02:39 +0100 Subject: Populate package derivation outputs select options always Even when the query parameters are invalid, as this is more useful. Thanks to nckx for reporting. --- guix-data-service/web/revision/controller.scm | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/guix-data-service/web/revision/controller.scm b/guix-data-service/web/revision/controller.scm index f5ed8f0..d723c16 100644 --- a/guix-data-service/web/revision/controller.scm +++ b/guix-data-service/web/revision/controller.scm @@ -850,6 +850,9 @@ `("Revision " (samp ,commit-hash))) (header-link (string-append "/revision/" commit-hash))) + (define build-server-urls + (select-build-server-urls-by-id conn)) + (if (any-invalid-query-parameters? query-parameters) (case (most-appropriate-mime-type '(application/json text/html) @@ -863,9 +866,10 @@ commit-hash query-parameters '() - '() - '() - '() + build-server-urls + (valid-systems conn) + (valid-targets->options + (valid-targets conn)) #f #:path-base path-base #:header-text header-text @@ -889,8 +893,6 @@ #:target (assq-ref query-parameters 'target) #:limit-results limit-results #:after-path (assq-ref query-parameters 'after_path))) - (build-server-urls - (select-build-server-urls-by-id conn)) (show-next-page? (if all-results #f -- cgit v1.2.3