fix parcelize crash and sharing not attaching files due to permissions

Author: barnhill

README.md

@@ -7,13 +7,13 @@ This library provides an easy to consume Android Composable that will display PD
 ## Add the dependency
 
 ```Gradle
-implementation 'com.pnuema.android:pdfviewer:1.3.3'
+implementation 'com.pnuema.android:pdfviewer:1.3.4'
 ```
 ```Kotlin(KTS)
-implementation("com.pnuema.android:pdfviewer:1.3.3")
+implementation("com.pnuema.android:pdfviewer:1.3.4")
 ```
 ```TOML
-pdfviewer = { module = "com.pnuema.android:pdfviewer", version.ref = "1.3.3" }
+pdfviewer = { module = "com.pnuema.android:pdfviewer", version.ref = "1.3.4" }
 ```
 
 ## Usage

gradle.properties

@@ -28,7 +28,7 @@ org.jetbrains.dokka.experimental.gradle.pluginMode=V2Enabled
 org.jetbrains.dokka.experimental.gradle.pluginMode.noWarn=true
 
 GROUP=com.pnuema.android
-VERSION_NAME=1.3.3
+VERSION_NAME=1.3.4
 
 POM_NAME=PdfViewer
 POM_ARTIFACT_ID=pdfviewer

pdfviewer/src/main/kotlin/com/pnuema/android/pdfviewer/PdfOptions.kt

@@ -11,7 +11,6 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.flow.update
 import kotlinx.parcelize.Parcelize
-import kotlinx.parcelize.RawValue
 
 data class PdfOptions(
     val maxScale: Float = 5f,
@@ -51,7 +50,7 @@ data class PdfOptions(
                     allowSharing = state.allowSharing,
                     removeFileWhenFinished = state.removeFileWhenFinished,
                     backgroundColor = state.backgroundColor.value,
-                    spacingBetweenPages = state.spacingBetweenPages
+                    spacingBetweenPages = state.spacingBetweenPages.value
                 )
             },
             restore = { saved ->
@@ -62,7 +61,7 @@ data class PdfOptions(
                     allowSharing = saved.allowSharing,
                     removeFileWhenFinished = saved.removeFileWhenFinished,
                     backgroundColor = Color(saved.backgroundColor),
-                    spacingBetweenPages = saved.spacingBetweenPages
+                    spacingBetweenPages = saved.spacingBetweenPages.dp
                 )
             },
         )
@@ -96,5 +95,5 @@ data class PdfOptionsSavedState(
     val allowSharing: Boolean,
     val removeFileWhenFinished: Boolean,
     val backgroundColor: ULong,
-    val spacingBetweenPages: @RawValue Dp
+    val spacingBetweenPages: Float
     ) : Parcelable

pdfviewer/src/main/kotlin/com/pnuema/android/pdfviewer/actions/PdfActionUtil.kt

@@ -5,11 +5,12 @@ import android.content.Context.PRINT_SERVICE
 import android.content.Intent
 import android.print.PrintManager
 import androidx.compose.ui.unit.IntSize
-import androidx.core.net.toUri
+import androidx.core.content.FileProvider
 import com.pnuema.android.pdfviewer.generator.PdfBitmapGenerator
 import com.pnuema.android.pdfviewer.print.PdfPrintDocumentAdapter
 import java.io.File
 
+
 object PdfActionUtil {
     fun print(
         context: Context,
@@ -31,11 +32,15 @@ object PdfActionUtil {
     }
 
     fun share(context: Context, file: File) {
+        val uri = FileProvider.getUriForFile(
+            context,
+            context.applicationContext.packageName + ".provider", file
+        )
         context.startActivity(
             Intent(Intent.ACTION_SEND).apply {
                 setFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION)
                 setType("application/pdf")
-                putExtra(Intent.EXTRA_STREAM, file.path.toUri())
+                putExtra(Intent.EXTRA_STREAM, uri)
             }
         )
     }