Clean code

Author: tevincent

app/src/main/java/com/infomaniak/drive/ui/fileList/preview/PreviewPDFActivity.kt

@@ -31,8 +31,7 @@ class PreviewPDFActivity : AppCompatActivity() {
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         setContentView(binding.root)
-        val navController = setupNavController()
-        navController.navigate(R.id.previewPDFFragment)
+        setupNavController().navigate(R.id.previewPDFFragment)
     }
 
     private fun getNavHostFragment() = supportFragmentManager.findFragmentById(R.id.hostFragment) as NavHostFragment

app/src/main/java/com/infomaniak/drive/ui/fileList/preview/PreviewPDFFragment.kt

@@ -180,9 +180,7 @@ class PreviewPDFFragment : PreviewFragment() {
     }
 
     private fun getConfigurator(uriString: String?, pdfFile: IOFile?): PDFView.Configurator {
-        return uriString?.let {
-            binding.pdfView.fromUri(Uri.parse(uriString))
-        } ?: binding.pdfView.fromFile(pdfFile)
+        return uriString?.let { binding.pdfView.fromUri(Uri.parse(uriString)) } ?: binding.pdfView.fromFile(pdfFile)
     }
 
     private fun onPdfPasswordError() {

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

@@ -15,20 +15,10 @@
   ~ You should have received a copy of the GNU General Public License
   ~ along with this program.  If not, see <http://www.gnu.org/licenses/>.
   -->
-<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<androidx.fragment.app.FragmentContainerView xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:id="@+id/hostFragment"
+    android:name="androidx.navigation.fragment.NavHostFragment"
     android:layout_width="match_parent"
-    android:layout_height="match_parent">
-
-    <androidx.fragment.app.FragmentContainerView
-        android:id="@+id/hostFragment"
-        android:name="androidx.navigation.fragment.NavHostFragment"
-        android:layout_width="match_parent"
-        android:layout_height="0dp"
-        app:defaultNavHost="true"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
-
-</androidx.constraintlayout.widget.ConstraintLayout>
+    android:layout_height="match_parent"
+    app:defaultNavHost="true" />