-
Notifications
You must be signed in to change notification settings - Fork 831
Conversation
@@ -165,7 +166,7 @@ public boolean onQueryTextChange(String query) { | |||
return true; | |||
} | |||
|
|||
private boolean haveNetworkConnection() { | |||
public boolean haveNetworkConnection() { |
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.
Why public?
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.
it may be private also.not needed
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.
We should try to keep private methods only. Please fix it.
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.
ok...suerly I will make it private
Please squash. Always!! |
@mananwason Squashed it.Please review it. |
@mananwason is there any other issues with the code? |
No @opticod Merging. |
In api/processor/*, for every success we are clearing the respective tables of database and adding new values.So, for every onRefresh(),its will clear the tables only when we have completed fetching that data else retain it. #88 #117