ResultRenderProvider returns list instead of optional URL #2018
Conversation
| @@ -22,7 +22,7 @@ | |||
| * @param allProviders A flag that should override internal "hide-this-url" flags. | |||
| * @return An Optional with the url or an empty optional. | |||
| */ | |||
| Optional<URL> generateResultURL(ManagedExecution<?> exec, UriBuilder uriBuilder, boolean allProviders); | |||
| Collection<URL> generateResultURLs(ManagedExecution<?> exec, UriBuilder uriBuilder, boolean allProviders); | |||
There was a problem hiding this comment.
public/private/protected?
There was a problem hiding this comment.
Ist ein public Interface, also automatisch public ;)
| @@ -22,6 +24,11 @@ | |||
| @Slf4j | |||
| public class ResultUtil { | |||
|
|
|||
| public static enum ContentDispositionOption { | |||
| inline, | |||
There was a problem hiding this comment.
warum klein geschrieben?
There was a problem hiding this comment.
Ich glaube im HTTP Header ist die Groß-/Kleinschreibung egal, ich schaue das nochmal nach. Ich habs klein geschrieben, weil der Parameter immer klein
There was a problem hiding this comment.
Laut https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html gilt das case-insensitive nur für den Feldnamen, nicht für das Feld selbst. Aber ich werde es mal groß schreiben und dann lowerCase draufaufrufen.
There was a problem hiding this comment.
ja wäre mir auch lieber, sonst geht was kaputt sobald jemand das aufräumt. Du könntest das auch machen mit einem value feld. Natürlich nicht super elegant aber dafür sicher.
There was a problem hiding this comment.
Ich habe dem enum eine Methode und etwas Doku gegeben das mit das deutlicher wird
delete-merged-branch
bot
deleted the
feature/allow-multiple-result-urls-per-provider
branch
No description provided.