Merge pull request #156 from yevhenii-kanivets/152_crash_when_adding_event_to_google_calendar_if_there_is_no_google_calendar_on_device

#152. Fix crash when adding event to Google Calendar if there is no G…

Author: bogdan-evtushenko

app/src/main/java/com/bogdan/codeforceswatcher/adapter/ContestAdapter.kt

@@ -1,5 +1,6 @@
 package com.bogdan.codeforceswatcher.adapter
 
+import android.content.ActivityNotFoundException
 import android.content.Context
 import android.content.Intent
 import android.net.Uri
@@ -8,6 +9,7 @@ import android.view.View
 import android.view.ViewGroup
 import android.widget.ImageView
 import android.widget.TextView
+import android.widget.Toast
 import androidx.recyclerview.widget.RecyclerView
 import com.bogdan.codeforceswatcher.R
 import com.bogdan.codeforceswatcher.model.Contest
@@ -17,12 +19,12 @@ import java.net.URLEncoder
 import java.text.SimpleDateFormat
 import java.util.*
 
-class ContestAdapter(private var items: List<Contest>, private val ctx: Context) : RecyclerView.Adapter<ContestAdapter.ViewHolder>() {
+class ContestAdapter(private var items: List<Contest>, private val context: Context) : RecyclerView.Adapter<ContestAdapter.ViewHolder>() {
 
     override fun getItemCount() = items.size
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
-        return ViewHolder(LayoutInflater.from(ctx).inflate(R.layout.contests_list_view, parent, false))
+        return ViewHolder(LayoutInflater.from(context).inflate(R.layout.contests_list_view, parent, false))
     }
 
     override fun onBindViewHolder(holder: ViewHolder, position: Int) {
@@ -38,8 +40,11 @@ class ContestAdapter(private var items: List<Contest>, private val ctx: Context)
         val encodeName = URLEncoder.encode(contest.name)
         val calendarEventLink = "$CALENDAR_LINK?action=TEMPLATE&text=$encodeName&dates=$timeStart/$timeEnd&details=$CODEFORCES_LINK"
         val intent = Intent(Intent.ACTION_VIEW).setData(Uri.parse(calendarEventLink))
-        ctx.startActivity(intent)
-
+        try {
+            context.startActivity(intent)
+        } catch (error: ActivityNotFoundException) {
+            Toast.makeText(context, context.resources.getString(R.string.google_calendar_not_found), Toast.LENGTH_SHORT).show()
+        }
         Analytics.logAddContestToCalendarEvent(contest.name)
     }
 

app/src/main/java/com/bogdan/codeforceswatcher/adapter/UserAdapter.kt

@@ -17,7 +17,7 @@ import kotlinx.android.synthetic.main.users_list_view.view.*
 import java.text.SimpleDateFormat
 import java.util.*
 
-class UserAdapter(private var items: List<User>, private val ctx: Context) : RecyclerView.Adapter<UserAdapter.ViewHolder>() {
+class UserAdapter(private var items: List<User>, private val context: Context) : RecyclerView.Adapter<UserAdapter.ViewHolder>() {
 
     override fun getItemCount(): Int {
         return items.size
@@ -29,7 +29,7 @@ class UserAdapter(private var items: List<User>, private val ctx: Context) : Rec
     }
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
-        return ViewHolder(LayoutInflater.from(ctx).inflate(R.layout.users_list_view, parent, false))
+        return ViewHolder(LayoutInflater.from(context).inflate(R.layout.users_list_view, parent, false))
     }
 
     override fun onBindViewHolder(holder: ViewHolder, position: Int) {
@@ -39,42 +39,42 @@ class UserAdapter(private var items: List<User>, private val ctx: Context) : Rec
             holder.tvRating.text = null
         } else holder.tvRating.text = user.rating.toString()
         if (user.rank == null) {
-            holder.tvHandle.setTextColor(ContextCompat.getColor(ctx, grey))
-            holder.tvRating.setTextColor(ContextCompat.getColor(ctx, grey))
+            holder.tvHandle.setTextColor(ContextCompat.getColor(context, grey))
+            holder.tvRating.setTextColor(ContextCompat.getColor(context, grey))
         } else {
             if (user.rank == "legendary grandmaster") {
                 val text = "<font color=black>${user.handle[0]}</font><font color=red>${user.handle.subSequence(1, user.handle.lastIndex + 1)}</font>"
                 holder.tvHandle.text = HtmlCompat.fromHtml(text, HtmlCompat.FROM_HTML_MODE_LEGACY);
             } else {
-                holder.tvHandle.setTextColor(ContextCompat.getColor(ctx, getColor(user.rank)))
+                holder.tvHandle.setTextColor(ContextCompat.getColor(context, getColor(user.rank)))
             }
-            holder.tvRating.setTextColor(ContextCompat.getColor(ctx, getColor(user.rank)))
+            holder.tvRating.setTextColor(ContextCompat.getColor(context, getColor(user.rank)))
 
         }
         val lastRatingChange = user.ratingChanges.lastOrNull()
         if (lastRatingChange != null) {
             val ratingDelta = lastRatingChange.newRating - lastRatingChange.oldRating
-            holder.tvLastRatingUpdate.text = ctx.resources.getString(
+            holder.tvLastRatingUpdate.text = context.resources.getString(
                     R.string.last_rating_update,
                     getDateTime(lastRatingChange.ratingUpdateTimeSeconds)
             )
             if (ratingDelta >= 0) {
                 holder.ivDelta.setImageResource(R.drawable.ic_rating_up)
                 holder.tvRatingChange.text = ratingDelta.toString()
-                holder.tvRatingChange.setTextColor(ContextCompat.getColor(ctx, bright_green))
+                holder.tvRatingChange.setTextColor(ContextCompat.getColor(context, bright_green))
             } else {
                 holder.ivDelta.setImageResource(R.drawable.ic_rating_down)
                 holder.tvRatingChange.text = (-ratingDelta).toString()
-                holder.tvRatingChange.setTextColor(ContextCompat.getColor(ctx, red))
+                holder.tvRatingChange.setTextColor(ContextCompat.getColor(context, red))
             }
         } else {
-            holder.tvLastRatingUpdate.text = ctx.resources.getString(R.string.no_rating_update)
+            holder.tvLastRatingUpdate.text = context.resources.getString(R.string.no_rating_update)
             holder.ivDelta.setImageResource(0)
             holder.tvRatingChange.text = null
         }
 
         holder.itemView.setOnClickListener {
-            ctx.startActivity(TryActivity.newIntent(ctx, user.id))
+            context.startActivity(TryActivity.newIntent(context, user.id))
         }
     }
 

app/src/main/res/layout/fragment_contests.xml

@@ -10,5 +10,6 @@
     <androidx.recyclerview.widget.RecyclerView
         android:id="@+id/recyclerView"
         android:layout_width="wrap_content"
-        android:layout_height="wrap_content" />
+        android:layout_height="wrap_content"
+        android:background="@color/white" />
 </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>

app/src/main/res/layout/fragment_users.xml

@@ -14,7 +14,8 @@
         <androidx.recyclerview.widget.RecyclerView
             android:id="@+id/recyclerView"
             android:layout_width="wrap_content"
-            android:layout_height="wrap_content" />
+            android:layout_height="wrap_content"
+            android:background="@color/white" />
 
     </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>
 

app/src/main/res/values-ru-rRU/strings.xml

@@ -34,4 +34,6 @@
 
     <string name="users">Пользователи</string>
     <string name="contests">Контесты</string>
+
+    <string name="google_calendar_not_found">Извините, но на этом устройстве не найден Google Calendar</string>
 </resources>

app/src/main/res/values-uk-rUA/strings.xml

@@ -34,4 +34,6 @@
 
     <string name="users">Користувачі</string>
     <string name="contests">Контести</string>
+
+    <string name="google_calendar_not_found">На жаль, на цьому пристрої не знайдено Google Calendar</string>
 </resources>

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

@@ -30,4 +30,5 @@
     <string name="google_play_not_found">Google Play not found</string>
     <string name="users">Users</string>
     <string name="contests">Contests</string>
+    <string name="google_calendar_not_found">Sorry, but no Google Calendar found on this device</string>
 </resources>