Merge branch 'release/3.0.x' into feature/13208_refactor_datastore_3

Author: mtwalli

Corona-Warn-App/src/main/java/de/rki/coronawarnapp/covidcertificate/booster/BoosterNotificationService.kt

@@ -7,6 +7,7 @@ import de.rki.coronawarnapp.covidcertificate.notification.PersonNotificationSend
 import de.rki.coronawarnapp.covidcertificate.person.core.PersonCertificatesSettings
 import de.rki.coronawarnapp.tag
 import de.rki.coronawarnapp.util.TimeStamper
+import kotlinx.coroutines.flow.first
 import timber.log.Timber
 import javax.inject.Inject
 import javax.inject.Singleton
@@ -55,7 +56,18 @@ class BoosterNotificationService @Inject constructor(
             personCertificatesSettings.setBoosterNotifiedAt(personIdentifier, timeStamper.nowUTC)
             Timber.tag(TAG).d("Person %s notified about booster rule change", codeSHA256)
         } else {
-            personCertificatesSettings.acknowledgeBoosterRule(personIdentifier, newRuleId)
+            val lastNotifiedRule = personCertificatesSettings.personsSettings.first().entries.firstOrNull {
+                it.key.belongsToSamePerson(personIdentifier)
+            }?.value?.lastSeenBoosterRuleIdentifier
+            Timber.tag(TAG).d("lastNotifiedRule=%s, newRuleId=%s", lastNotifiedRule, newRuleId)
+            if (lastNotifiedRule != null && lastNotifiedRule == newRuleId) {
+                Timber.tag(TAG).d(
+                    "Dismissing badge because lastNotifiedRule did not change lastNotifiedRule=%s ",
+                    lastNotifiedRule
+                )
+                personCertificatesSettings.acknowledgeBoosterRule(personIdentifier, newRuleId)
+            }
+
             Timber.tag(TAG).d("Person %s shouldn't be notified about booster rule=%s", codeSHA256, newRuleId)
         }
 

Corona-Warn-App/src/main/java/de/rki/coronawarnapp/srs/ui/consent/SrsSubmissionConsentFragment.kt

@@ -70,7 +70,7 @@ class SrsSubmissionConsentFragment : Fragment(R.layout.fragment_srs_submission_c
         viewModel.event.observe2(this) {
             when (it) {
                 SrsSubmissionConsentNavigationEvents.NavigateToDataPrivacy ->
-                    findNavController().navigate(R.id.informationPrivacyFragment)
+                    findNavController().navigate(R.id.surveyConsentDetailFragment)
 
                 SrsSubmissionConsentNavigationEvents.NavigateToMainScreen ->
                     findNavController().navigate(

Corona-Warn-App/src/main/java/de/rki/coronawarnapp/srs/ui/done/SrsSubmissionDoneFragment.kt

@@ -5,7 +5,6 @@ import android.view.View
 import android.view.accessibility.AccessibilityEvent
 import androidx.core.view.isVisible
 import androidx.fragment.app.Fragment
-import androidx.navigation.fragment.navArgs
 import de.rki.coronawarnapp.R
 import de.rki.coronawarnapp.databinding.FragmentSubmissionDoneBinding
 import de.rki.coronawarnapp.util.ContextExtensions.getDrawableCompat
@@ -14,8 +13,6 @@ import de.rki.coronawarnapp.util.ui.viewBinding
 
 class SrsSubmissionDoneFragment : Fragment(R.layout.fragment_submission_done) {
 
-    private val args by navArgs<SrsSubmissionDoneFragmentArgs>()
-
     private val binding: FragmentSubmissionDoneBinding by viewBinding()
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
@@ -28,8 +25,8 @@ class SrsSubmissionDoneFragment : Fragment(R.layout.fragment_submission_done) {
             }
 
             submissionDoneButtonDone.setOnClickListener { popBackStack() }
-            submissionDoneContent.submissionDoneContent.submissionDonePcrValidation.root.isVisible = true // TODO
-            submissionDoneContent.submissionDoneContent.submissionDoneIllness.root.isVisible = true // TODO
+            // Regardless of the test type we show the same screen
+            submissionDoneContent.submissionDoneContent.submissionDoneIllness.root.isVisible = false
         }
     }
 

Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/submission/qrcode/consent/SubmissionConsentFragment.kt

@@ -64,6 +64,7 @@ class SubmissionConsentFragment : Fragment(R.layout.fragment_submission_consent)
                 is SubmissionNavigationEvents.NavigateToDataPrivacy -> findNavController().navigate(
                     SubmissionConsentFragmentDirections.actionSubmissionConsentFragmentToInformationPrivacyFragment()
                 )
+
                 is SubmissionNavigationEvents.ResolvePlayServicesException ->
                     it.exception.status.startResolutionForResult(
                         requireActivity(),
@@ -79,19 +80,22 @@ class SubmissionConsentFragment : Fragment(R.layout.fragment_submission_consent)
                     ),
                     navOptions
                 )
+
                 is SubmissionNavigationEvents.NavigateClose -> {
                     if (navArgs.comesFromDispatcherFragment) {
                         findNavController().navigate(
                             SubmissionConsentFragmentDirections.actionSubmissionConsentFragmentToHomeFragment()
                         )
                     } else popBackStack()
                 }
+
                 is SubmissionNavigationEvents.NavigateBackToTestRegistration -> findNavController().navigate(
                     SubmissionConsentFragmentDirections
                         .actionSubmissionConsentFragmentToTestRegistrationSelectionFragment(
                             navArgs.coronaTestQrCode
                         )
                 )
+
                 else -> Unit
             }
         }
@@ -113,17 +117,20 @@ class SubmissionConsentFragment : Fragment(R.layout.fragment_submission_consent)
                 State.Working -> {
                     // Handled above
                 }
+
                 is State.Error -> state.showExceptionDialog(this) { popBackStack() }
                 is State.TestRegistered -> when {
                     state.test.isPositive ->
                         NavGraphDirections.actionToSubmissionTestResultAvailableFragment(
-                            testIdentifier = state.test.identifier
+                            testIdentifier = state.test.identifier,
+                            comesFromDispatcherFragment = navArgs.comesFromDispatcherFragment
                         )
                             .run { findNavController().navigate(this, navOptions) }
 
                     else ->
                         NavGraphDirections.actionSubmissionTestResultPendingFragment(
-                            testIdentifier = state.test.identifier
+                            testIdentifier = state.test.identifier,
+                            comesFromDispatcherFragment = navArgs.comesFromDispatcherFragment
                         )
                             .run { findNavController().navigate(this, navOptions) }
                 }

Corona-Warn-App/src/main/java/de/rki/coronawarnapp/util/CertificateStateHelper.kt

@@ -57,7 +57,7 @@ fun IncludeCertificateQrcodeCardBinding.bindValidityViews(
     when (certificate.displayedState()) {
         is ExpiringSoon -> {
             setQrTitle(certificate, qrTitle, context)
-            notificationBadge.isVisible = certificate.hasNotificationBadge
+            notificationBadge.isVisible = false
             statusIcon.constraintLayoutParams.verticalBias = 0f
             statusIcon.setImageDrawable(context.getDrawableCompat(R.drawable.ic_av_timer))
             statusTitle.text = context.getString(
@@ -72,7 +72,7 @@ fun IncludeCertificateQrcodeCardBinding.bindValidityViews(
 
         is Expired -> {
             setQrTitle(certificate, qrTitle, context)
-            notificationBadge.isVisible = certificate.hasNotificationBadge
+            notificationBadge.isVisible = false
             statusIcon.constraintLayoutParams.verticalBias = 1.0f
             statusIcon.setImageDrawable(context.getDrawableCompat(R.drawable.ic_error_outline))
             statusTitle.text = context.getText(R.string.certificate_qr_expired)

Corona-Warn-App/src/main/res/layout/fragment_srs_submission_consent.xml

@@ -205,7 +205,7 @@
                     app:layout_constraintEnd_toEndOf="@id/guideline_end"
                     app:layout_constraintStart_toStartOf="@id/guideline_start"
                     app:layout_constraintTop_toBottomOf="@id/srs_next_step_submission"
-                    app:titleText="@string/submission_consent_main_bottom_body" />
+                    app:titleText="@string/srs_submission_consent_survey_button" />
 
                 <androidx.constraintlayout.widget.Guideline
                     android:id="@+id/guideline_start"

Corona-Warn-App/src/main/res/layout/include_srs_privacy_card.xml

@@ -47,10 +47,11 @@
             app:layout_constraintEnd_toEndOf="@id/guideline_end"
             app:layout_constraintStart_toStartOf="@id/guideline_start">
 
-            <include layout="@layout/include_bullet_point"
+            <include
+                layout="@layout/include_bullet_point"
                 android:layout_width="@dimen/bullet_point_size"
                 android:layout_height="@dimen/bullet_point_size"
-                app:layout_constraintBaseline_toBaselineOf="@id/first_bulletpoint_title"/>
+                app:layout_constraintBaseline_toBaselineOf="@id/first_bulletpoint_title" />
 
             <TextView
                 android:id="@+id/first_bulletpoint_title"
@@ -74,10 +75,11 @@
             app:layout_constraintEnd_toEndOf="@id/guideline_end"
             app:layout_constraintStart_toStartOf="@id/guideline_start">
 
-            <include layout="@layout/include_bullet_point"
+            <include
+                layout="@layout/include_bullet_point"
                 android:layout_width="@dimen/bullet_point_size"
                 android:layout_height="@dimen/bullet_point_size"
-                app:layout_constraintBaseline_toBaselineOf="@id/second_bulletpoint_title"/>
+                app:layout_constraintBaseline_toBaselineOf="@id/second_bulletpoint_title" />
 
             <TextView
                 android:id="@+id/second_bulletpoint_title"
@@ -101,10 +103,11 @@
             app:layout_constraintEnd_toEndOf="@id/guideline_end"
             app:layout_constraintStart_toStartOf="@id/guideline_start">
 
-            <include layout="@layout/include_bullet_point"
+            <include
+                layout="@layout/include_bullet_point"
                 android:layout_width="@dimen/bullet_point_size"
                 android:layout_height="@dimen/bullet_point_size"
-                app:layout_constraintBaseline_toBaselineOf="@id/third_bulletpoint_title"/>
+                app:layout_constraintBaseline_toBaselineOf="@id/third_bulletpoint_title" />
 
             <TextView
                 android:id="@+id/third_bulletpoint_title"
@@ -135,7 +138,7 @@
             layout="@layout/view_bullet_point_text"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginTop="@dimen/spacing_tiny"
+            android:layout_marginTop="@dimen/spacing_normal"
             app:itemText="@{@string/srs_submission_consent_second_bulletpoint_text}"
             app:layout_constraintEnd_toEndOf="@id/guideline_end"
             app:layout_constraintStart_toStartOf="@id/guideline_start"
@@ -152,6 +155,28 @@
             app:layout_constraintStart_toStartOf="@id/guideline_start"
             app:layout_constraintTop_toBottomOf="@id/srs_submission_consent_main_second_point" />
 
+        <include
+            android:id="@+id/srs_submission_consent_main_fourth_point"
+            layout="@layout/view_bullet_point_text"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="@dimen/spacing_tiny"
+            app:itemText="@{@string/srs_submission_consent_fourth_bulletpoint_text}"
+            app:layout_constraintEnd_toEndOf="@id/guideline_end"
+            app:layout_constraintStart_toStartOf="@id/guideline_start"
+            app:layout_constraintTop_toBottomOf="@id/srs_submission_consent_main_third_point" />
+
+        <include
+            android:id="@+id/srs_submission_consent_main_fifth_point"
+            layout="@layout/view_bullet_point_text"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="@dimen/spacing_tiny"
+            app:itemText="@{@string/srs_submission_consent_fifth_bulletpoint_text}"
+            app:layout_constraintEnd_toEndOf="@id/guideline_end"
+            app:layout_constraintStart_toStartOf="@id/guideline_start"
+            app:layout_constraintTop_toBottomOf="@id/srs_submission_consent_main_fourth_point" />
+
         <include layout="@layout/merge_guidelines_side" />
 
     </androidx.constraintlayout.widget.ConstraintLayout>

Corona-Warn-App/src/main/res/layout/survey_consent_detail_fragment.xml

@@ -50,10 +50,12 @@
                     android:layout_width="0dp"
                     android:layout_height="wrap_content"
                     android:layout_marginTop="45dp"
+                    android:layout_marginBottom="45dp"
                     android:orientation="vertical"
                     app:layout_constraintEnd_toStartOf="@+id/guideline_card_end"
                     app:layout_constraintStart_toStartOf="@+id/guideline_card_start"
-                    app:layout_constraintTop_toBottomOf="@+id/survey_details_headline">
+                    app:layout_constraintTop_toBottomOf="@+id/survey_details_headline"
+                    app:layout_constraintBottom_toBottomOf="parent">
 
                     <TextView
                         android:id="@+id/legal_title"
@@ -81,30 +83,6 @@
 
                 </androidx.constraintlayout.widget.ConstraintLayout>
 
-                <TextView
-                    android:id="@+id/survey_details_bottom_info_title"
-                    style="@style/subtitle"
-                    android:layout_width="0dp"
-                    android:layout_height="wrap_content"
-                    android:layout_marginTop="33dp"
-                    android:focusable="true"
-                    android:text="@string/datadonation_survey_consent_details_title_below"
-                    app:layout_constraintEnd_toEndOf="@id/guideline_end"
-                    app:layout_constraintStart_toStartOf="@id/guideline_start"
-                    app:layout_constraintTop_toBottomOf="@id/consent_layout" />
-
-                <TextView
-                    android:id="@+id/survey_details_bottom_info_body"
-                    style="@style/subtitle"
-                    android:layout_width="0dp"
-                    android:layout_height="wrap_content"
-                    android:layout_marginTop="@dimen/spacing_small"
-                    android:focusable="true"
-                    android:text="@string/datadonation_survey_consent_details_text_below"
-                    app:layout_constraintEnd_toEndOf="@id/guideline_end"
-                    app:layout_constraintStart_toStartOf="@id/guideline_start"
-                    app:layout_constraintTop_toBottomOf="@id/survey_details_bottom_info_title" />
-
                 <androidx.constraintlayout.widget.Guideline
                     android:id="@+id/guideline_start"
                     android:layout_width="wrap_content"

Corona-Warn-App/src/main/res/values-de/legal_strings.xml

@@ -67,7 +67,7 @@
     <!-- XHED: Title for the information box in the survey consent detail screen -->
     <string name="datadonation_survey_consent_details_title">"Prüfung der Echtheit und Drittlandsübermittlung"</string>
     <!-- XTXT: Text for the information box in the survey consent detail screen -->
-    <string name="datadonation_survey_consent_details_text">"Um die Echtheit Ihrer App zu bestätigen, erzeugt Ihr Smartphone eine eindeutige Kennung, die Informationen über die Version Ihres Smartphones und der App enthält. Das ist erforderlich, um zu verhindern, dass Nutzer mehrfach an der Befragung teilnehmen und so die Ergebnisse der Befragung verfälschen. Die Kennung wird hier einmalig an Google übermittelt. Dabei kann es auch zu einer Datenübermittlung in die USA oder andere Drittländer kommen. Dort besteht möglicherweise kein dem europäischen Recht entsprechendes Datenschutzniveau und Ihre europäischen Datenschutzrechte können eventuell nicht durchgesetzt werden. Insbesondere besteht die Möglichkeit, dass Sicherheitsbehörden im Drittland, auch ohne einen konkreten Verdacht, auf die übermittelten Daten bei Google zugreifen und diese auswerten, beispielsweise indem sie Daten mit anderen Informationen verknüpfen. Dies betrifft nur die an Google übermittelte Kennung. Die weiteren Angaben über Ihre Teilnahme an der Befragung erhält Google nicht. Möglicherweise kann Google jedoch anhand der Kennung auf Ihre Identität schließen und nachvollziehen, dass die Echtheitsprüfung Ihres Smartphones stattgefunden hat.\n\nWenn Sie mit der Drittlandsübermittlung nicht einverstanden sind, tippen Sie bitte nicht „Einverstanden“ an. Sie können die App weiterhin nutzen, eine Teilnahme an dieser Befragung ist dann jedoch nicht möglich."</string>
+    <string name="datadonation_survey_consent_details_text">"Um die Echtheit Ihrer App zu bestätigen, erzeugt Ihr Smartphone eine eindeutige Kennung, die Informationen über die Version Ihres Smartphones und der App enthält. Das ist erforderlich, um sicherzustellen, dass nur Nutzer eine Warnung nach einem positiven Selbsttest oder mit einem Testergebnis, das nicht in der App vorliegt, auslösen, die tatsächlich die Corona-Warn-App nutzen und nicht manipulierte Warnungen bereitstellen. Das Verfahren dient außerdem dazu, sicherzustellen, dass Warnungen nach einem positiven Selbsttest oder mit einem Testergebnis, das nicht in der App vorliegt, nur alle drei Monate möglich sind.\n\n\Die bei der Echtheitsprüfung erzeugte eindeutige Kennung wird beim Auslösen der Warnung einmalig an Apple übermittelt. Dabei kann es auch zu einer Datenübermittlung in die USA oder andere Drittländer kommen. Dort besteht möglicherweise kein dem europäischen Recht entsprechendes Datenschutzniveau und Ihre europäischen Datenschutzrechte können eventuell nicht durchgesetzt werden. Insbesondere besteht die Möglichkeit, dass Sicherheitsbehörden im Drittland, auch ohne einen konkreten Verdacht, auf die übermittelten Daten bei Apple zugreifen und diese auswerten, beispielsweise indem sie Daten mit anderen Informationen verknüpfen. Dies betrifft nur die an Apple übermittelte Kennung. Möglicherweise kann Apple jedoch anhand der Kennung auf Ihre Identität schließen und nachvollziehen, dass die Echtheitsprüfung Ihres Smartphones im Rahmen einer Warnung anderer stattgefunden hat.\n\n\Wenn Sie mit der Drittlandsübermittlung nicht einverstanden sind, tippen Sie bitte nicht „Einverstanden“ an. Sie können die App weiterhin nutzen, eine Warnung anderer nach einem positiven Selbsttest oder mit einem Testergebnis, das nicht in der App vorliegt, ist dann jedoch nicht möglich."</string>
 
     <!-- XTXT: onboarding privacy preserving analytics (ppa) - consent title -->
     <string name="ppa_onboarding_consent_title">"Ihr Einverständnis"</string>