Merge pull request #839 from vector-im/feature/developer

Show all hidden event in the timeline when the developer settings is ON

vector/src/main/java/im/vector/riotx/features/home/room/detail/RoomDetailViewModel.kt

@@ -90,8 +90,7 @@ class RoomDetailViewModel @AssistedInject constructor(@Assisted initialState: Ro
     private val timelineSettings = if (userPreferencesProvider.shouldShowHiddenEvents()) {
         TimelineSettings(30,
                 filterEdits = false,
-                filterTypes = true,
-                allowedTypes = TimelineDisplayableEvents.DEBUG_DISPLAYABLE_TYPES,
+                filterTypes = false,
                 buildReadReceipts = userPreferencesProvider.shouldShowReadReceipts())
     } else {
         TimelineSettings(30,

vector/src/main/java/im/vector/riotx/features/home/room/detail/timeline/factory/TimelineItemFactory.kt

@@ -68,12 +68,12 @@ class TimelineItemFactory @Inject constructor(
                         encryptedItemFactory.create(event, nextEvent, highlight, callback)
                     }
                 }
-
                 // Unhandled event types (yet)
                 EventType.STATE_ROOM_THIRD_PARTY_INVITE -> defaultItemFactory.create(event, highlight, callback)
                 else                                    -> {
+                    // Should only happen when shouldShowHiddenEvents() settings is ON
                     Timber.v("Type ${event.root.getClearType()} not handled")
-                    null
+                    defaultItemFactory.create(event, highlight, callback)
                 }
             }
         } catch (throwable: Throwable) {

vector/src/main/java/im/vector/riotx/features/home/room/detail/timeline/helper/TimelineDisplayableEvents.kt

@@ -42,11 +42,6 @@ object TimelineDisplayableEvents {
             EventType.STATE_ROOM_TOMBSTONE,
             EventType.STATE_ROOM_JOIN_RULES
     )
-
-    val DEBUG_DISPLAYABLE_TYPES = DISPLAYABLE_TYPES + listOf(
-            EventType.REDACTION,
-            EventType.REACTION
-    )
 }
 
 fun TimelineEvent.canBeMerged(): Boolean {

vector/src/main/res/xml/vector_settings_advanced_settings.xml

@@ -30,8 +30,6 @@
             android:summary="@string/settings_developer_mode_fail_fast_summary"
             android:title="@string/settings_developer_mode_fail_fast_title" />
 
-        <!-- TODO Display unsupported events -->
-
     </im.vector.riotx.core.preference.VectorPreferenceCategory>
 
     <im.vector.riotx.core.preference.VectorPreferenceCategory