Skip to content

Commit b65dbb0

Browse files
committed
Code refactoring.
1 parent 22ffe84 commit b65dbb0

File tree

6 files changed

+28
-6
lines changed

6 files changed

+28
-6
lines changed

app/src/main/java/com/nextcloud/client/preferences/AppPreferences.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -350,6 +350,15 @@ default void onDarkThemeModeChanged(DarkMode mode) {
350350
void setDataAnalysis(boolean enableDataAnalysis);
351351
boolean isDataAnalysisEnabled();
352352

353+
/**
354+
* Saves the privacy policy action taken by user
355+
* this will maintain the state of current privacy policy action taken
356+
* @see com.nmc.android.ui.LoginPrivacySettingsActivity for actions
357+
* @param userAction taken by user
358+
*/
359+
void setPrivacyPolicyAction(int userAction);
360+
int getPrivacyPolicyAction();
361+
353362
/**
354363
* Gets status of migration to user id, default false
355364
*

app/src/main/java/com/nextcloud/client/preferences/AppPreferencesImpl.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import com.nextcloud.client.account.User;
3131
import com.nextcloud.client.account.UserAccountManager;
3232
import com.nextcloud.client.account.UserAccountManagerImpl;
33+
import com.nmc.android.ui.LoginPrivacySettingsActivity;
3334
import com.owncloud.android.datamodel.ArbitraryDataProvider;
3435
import com.owncloud.android.datamodel.ArbitraryDataProviderImpl;
3536
import com.owncloud.android.datamodel.FileDataStorageManager;
@@ -104,6 +105,7 @@ public final class AppPreferencesImpl implements AppPreferences {
104105
private static final String PREF__IN_APP_REVIEW_DATA = "in_app_review_data";
105106

106107
private static final String PREF__DATA_ANALYSIS = "data_analysis";
108+
private static final String PREF__PRIVACY_POLICY_ACTION = "privacy_policy_action";
107109

108110
private final Context context;
109111
private final SharedPreferences preferences;
@@ -584,6 +586,16 @@ public boolean isDataAnalysisEnabled() {
584586
return preferences.getBoolean(PREF__DATA_ANALYSIS, true);
585587
}
586588

589+
@Override
590+
public void setPrivacyPolicyAction(int userAction) {
591+
preferences.edit().putInt(PREF__PRIVACY_POLICY_ACTION, userAction).apply();
592+
}
593+
594+
@Override
595+
public int getPrivacyPolicyAction() {
596+
return preferences.getInt(PREF__PRIVACY_POLICY_ACTION, LoginPrivacySettingsActivity.NO_ACTION);
597+
}
598+
587599
@Override
588600
public boolean isUserIdMigrated() {
589601
return preferences.getBoolean(PREF__MIGRATED_USER_ID, false);

app/src/main/java/com/nmc/android/marketTracking/MoEngageSdkUtils.kt

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,13 @@ import com.owncloud.android.BuildConfig
66
import com.owncloud.android.R
77

88
object MoEngageSdkUtils {
9+
10+
//enable/disable moengage as we are not using it right now due to no proper firebase api key
11+
private const val MOENGAGE_ENABLED = false
12+
913
@JvmStatic
1014
fun initMoEngageSDK(application: Application) {
11-
if (application.resources.getBoolean(R.bool.moengage_enabled)) {
15+
if (MOENGAGE_ENABLED) {
1216
val moEngage = MoEngage.Builder(application, BuildConfig.MOENGAGE_APP_ID)
1317
.build()
1418
MoEngage.initialise(moEngage)

app/src/main/java/com/owncloud/android/ui/activity/SettingsActivity.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,9 @@
5757
import com.nextcloud.client.logger.ui.LogsActivity;
5858
import com.nextcloud.client.network.ClientFactory;
5959
import com.nextcloud.client.preferences.AppPreferences;
60+
import com.nmc.android.marketTracking.TealiumSdkUtils;
6061
import com.nextcloud.client.preferences.AppPreferencesImpl;
6162
import com.nextcloud.client.preferences.DarkMode;
62-
import com.nmc.android.marketTracking.TealiumSdkUtils;
6363
import com.owncloud.android.BuildConfig;
6464
import com.owncloud.android.MainApp;
6565
import com.owncloud.android.R;

app/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,9 @@
6464
import com.nextcloud.ui.fileactions.FileActionsBottomSheet;
6565
import com.nextcloud.utils.EditorUtils;
6666
import com.nextcloud.utils.ShortcutUtil;
67+
import com.nmc.android.marketTracking.TrackingScanInterface;
6768
import com.nextcloud.utils.view.FastScrollUtils;
6869
import com.nmc.android.marketTracking.AdjustSdkUtils;
69-
import com.nmc.android.marketTracking.TrackingScanInterface;
7070
import com.nmc.android.marketTracking.TrackingScanInterfaceImpl;
7171
import com.nmc.android.marketTracking.TealiumSdkUtils;
7272
import com.owncloud.android.MainApp;

app/src/main/res/values/setup.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,6 @@
122122
<string name="dev_changelog">https://github.com/nextcloud/android/raw/dev/CHANGELOG.md</string>
123123

124124
<string name="buildNumber" translatable="false"></string>
125-
126-
<!-- NMC Note: will be used to enable/disable moengage as we are not using it right now due to no proper api key -->
127-
<bool name="moengage_enabled">false</bool>
128125
</resources>
129126

130127

0 commit comments

Comments
 (0)