-
Notifications
You must be signed in to change notification settings - Fork 30
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
allow getting forms from db without appending locale #586
Conversation
allow configurability of different strategies of form identity naming
fix codacy util class naming
android-json-form-wizard/src/main/java/com/vijay/jsonwizard/utils/FormUtils.java
Outdated
Show resolved
Hide resolved
...-wizard/src/main/java/com/vijay/jsonwizard/activities/FormConfigurationJsonFormActivity.java
Outdated
Show resolved
Hide resolved
...-wizard/src/main/java/com/vijay/jsonwizard/activities/FormConfigurationJsonFormActivity.java
Outdated
Show resolved
Hide resolved
@Override | ||
public void writeValue(String stepName, String key, String value, String openMrsEntityParent, String openMrsEntity, String openMrsEntityId) throws JSONException { | ||
callSuperWriteValue(stepName, key, value, openMrsEntityParent, openMrsEntity, openMrsEntityId); | ||
} | ||
|
||
@Override | ||
public void onFormFinish() { | ||
callSuperFinish(); | ||
} | ||
|
||
protected void callSuperFinish(){ | ||
super.onFormFinish(); | ||
} | ||
|
||
protected void callSuperWriteValue(String stepName, String key, String value, String openMrsEntityParent, String openMrsEntity, String openMrsEntityId) throws JSONException { | ||
super.writeValue(stepName, key, value, openMrsEntityParent, openMrsEntity, openMrsEntityId); | ||
} |
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.
I thought that these four methods were redundant? What do you think?
I think that they each call the super and therefore no need to override and only call super with the exactly same params
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.
For these I wasn't sure, may be other libraries using native-forms may have had them overridden, because they'd been marked as protected.
...d/src/main/java/com/vijay/jsonwizard/activities/FormConfigurationJsonWizardFormActivity.java
Outdated
Show resolved
Hide resolved
and other formatting improvements
for testing purposes while awaiting opensrp/opensrp-client-native-form#586
for testing purposes while awaiting opensrp/opensrp-client-native-form#586
while awaiting review for opensrp/opensrp-client-native-form#586
allow configurability of different strategies of form identity naming
The Reveal client app saves forms and form rules to db, with just the filename and no locale appended. This would allow for similar apps to be able to retrieve forms and form rules from db, with no locale appended.
resolving #583
resolving #585