-
Notifications
You must be signed in to change notification settings - Fork 14
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
retrieve dynamic search for query request #1358
Changes from all commits
955eb3a
992d0e6
8bfbc6f
89ad7cc
d35d47f
6c1c042
e0e60cd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,6 +30,7 @@ public class RemoteQueryDatum extends SessionDatum { | |
private OrderedHashtable<String, QueryPrompt> userQueryPrompts; | ||
private boolean useCaseTemplate; | ||
private boolean defaultSearch; | ||
private boolean dynamicSearch; | ||
private Text title; | ||
private Text description; | ||
|
||
|
@@ -45,12 +46,13 @@ public RemoteQueryDatum() { | |
public RemoteQueryDatum(URL url, String storageInstance, | ||
List<QueryData> hiddenQueryValues, | ||
OrderedHashtable<String, QueryPrompt> userQueryPrompts, | ||
boolean useCaseTemplate, boolean defaultSearch, Text title, Text description) { | ||
boolean useCaseTemplate, boolean defaultSearch, boolean dynamicSearch, Text title, Text description) { | ||
super(storageInstance, url.toString()); | ||
this.hiddenQueryValues = hiddenQueryValues; | ||
this.userQueryPrompts = userQueryPrompts; | ||
this.useCaseTemplate = useCaseTemplate; | ||
this.defaultSearch = defaultSearch; | ||
this.dynamicSearch = dynamicSearch; | ||
this.title = title; | ||
this.description = description; | ||
} | ||
|
@@ -81,6 +83,10 @@ public boolean doDefaultSearch() { | |
return defaultSearch; | ||
} | ||
|
||
public boolean getDynamicSearch() { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: rename boolean getters in this PR to |
||
return dynamicSearch; | ||
} | ||
|
||
public Text getTitleText() { | ||
return title; | ||
} | ||
|
@@ -102,7 +108,7 @@ public void readExternal(DataInputStream in, PrototypeFactory pf) | |
description = (Text) ExtUtil.read(in, new ExtWrapNullable(Text.class), pf); | ||
useCaseTemplate = ExtUtil.readBool(in); | ||
defaultSearch = ExtUtil.readBool(in); | ||
|
||
dynamicSearch = ExtUtil.readBool(in); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you need to add the corresponding There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
} | ||
|
||
@Override | ||
|
@@ -114,6 +120,7 @@ public void writeExternal(DataOutputStream out) throws IOException { | |
ExtUtil.write(out, new ExtWrapNullable(description)); | ||
ExtUtil.writeBool(out, useCaseTemplate); | ||
ExtUtil.writeBool(out, defaultSearch); | ||
ExtUtil.writeBool(out, dynamicSearch); | ||
|
||
} | ||
} |
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.
How does this get set for
QueryScreen
?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.
resolved 8bfbc6f