Skip to content

Commit 26a8166

Browse files
liveHarshitiamareebjamal
authored andcommitted
fix: Use event's time zone as default (#2128) (#2132)
1 parent 0dfa2cd commit 26a8166

File tree

3 files changed

+12
-10
lines changed

3 files changed

+12
-10
lines changed

app/src/main/java/org/fossasia/openevent/general/event/EventUtils.kt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import androidx.preference.PreferenceManager
66
import org.fossasia.openevent.general.OpenEventGeneral
77
import org.fossasia.openevent.general.R
88
import org.fossasia.openevent.general.data.Resource
9+
import org.fossasia.openevent.general.settings.LOCAL_TIMEZONE
910
import org.threeten.bp.ZoneId
1011
import org.threeten.bp.ZonedDateTime
1112
import org.threeten.bp.format.DateTimeFormatter
@@ -23,14 +24,14 @@ object EventUtils {
2324
fun getEventDateTime(dateString: String, timeZone: String?): ZonedDateTime {
2425
try {
2526
return when (PreferenceManager.getDefaultSharedPreferences(OpenEventGeneral.appContext)
26-
.getBoolean("useEventTimeZone", false) && !timeZone.isNullOrBlank()) {
27+
.getBoolean(LOCAL_TIMEZONE, false) && !timeZone.isNullOrBlank()) {
2728

2829
true -> ZonedDateTime.parse(dateString)
2930
.toOffsetDateTime()
30-
.atZoneSameInstant(ZoneId.of(timeZone))
31+
.atZoneSameInstant(ZoneId.systemDefault())
3132
false -> ZonedDateTime.parse(dateString)
3233
.toOffsetDateTime()
33-
.atZoneSameInstant(ZoneId.systemDefault())
34+
.atZoneSameInstant(ZoneId.of(timeZone))
3435
}
3536
} catch (e: NullPointerException) {
3637
return ZonedDateTime.parse(dateString)

app/src/main/java/org/fossasia/openevent/general/settings/SettingsFragment.kt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ import java.util.prefs.PreferenceChangeListener
3636
import org.fossasia.openevent.general.utils.Utils.setToolbar
3737
import org.fossasia.openevent.general.utils.extensions.nonNull
3838

39+
const val LOCAL_TIMEZONE = "localTimeZone"
40+
3941
class SettingsFragment : PreferenceFragmentCompat(), PreferenceChangeListener {
4042
private val FORM_LINK: String = "https://docs.google.com/forms/d/e/" +
4143
"1FAIpQLSd7Y1T1xoXeYaAG_b6Tu1YYK-jZssoC5ltmQbkUX0kmDZaKYw/viewform"
@@ -69,7 +71,7 @@ class SettingsFragment : PreferenceFragmentCompat(), PreferenceChangeListener {
6971
"Version " + BuildConfig.VERSION_NAME
7072

7173
preferenceScreen.findPreference<Preference>(getString(R.string.key_timezone_switch))?.setDefaultValue(
72-
timeZonePreference.getBoolean("useEventTimeZone", false)
74+
timeZonePreference.getBoolean(LOCAL_TIMEZONE, false)
7375
)
7476

7577
preferenceScreen.findPreference<Preference>(getString(R.string.key_profile))?.isVisible =
@@ -122,10 +124,9 @@ class SettingsFragment : PreferenceFragmentCompat(), PreferenceChangeListener {
122124
}
123125
if (preference?.key == getString(R.string.key_timezone_switch)) {
124126
val timeZonePreference = PreferenceManager.getDefaultSharedPreferences(context)
125-
val timeZonePreferenceKey = "useEventTimeZone"
126-
when (timeZonePreference.getBoolean(timeZonePreferenceKey, false)) {
127-
true -> timeZonePreference.edit().putBoolean(timeZonePreferenceKey, false).apply()
128-
false -> timeZonePreference.edit().putBoolean(timeZonePreferenceKey, true).apply()
127+
when (timeZonePreference.getBoolean(LOCAL_TIMEZONE, false)) {
128+
true -> timeZonePreference.edit().putBoolean(LOCAL_TIMEZONE, false).apply()
129+
false -> timeZonePreference.edit().putBoolean(LOCAL_TIMEZONE, true).apply()
129130
}
130131
return true
131132
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,8 +186,8 @@
186186
<string name="key_profile">profile</string>
187187
<string name="logged_in_account">Account</string>
188188
<string name="key_timezone_switch" translatable="false">timeZoneSwitch</string>
189-
<string name="timeZone_title">Use event\'s timezone</string>
190-
<string name="timeZone_summary">Your timezone is used when disabled.</string>
189+
<string name="timeZone_title">Use local timezone</string>
190+
<string name="timeZone_summary">Event\'s timezone is used when disabled.</string>
191191
<string name="key_change_password">changePassword</string>
192192
<string name="title_change_password">Change Password</string>
193193
<string name="change">Change</string>

0 commit comments

Comments
 (0)