Skip to content

Commit

Permalink
refactor: rename assetsHandled
Browse files Browse the repository at this point in the history
  • Loading branch information
HayesGordon committed Feb 4, 2025
1 parent 0af9ec1 commit 56d2832
Show file tree
Hide file tree
Showing 6 changed files with 21 additions and 21 deletions.
14 changes: 7 additions & 7 deletions android/src/main/java/com/rivereactnative/RiveReactNativeView.kt
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ class RiveReactNativeView(private val context: ThemedReactContext) : FrameLayout
private var layoutScaleFactor: Float? = null
private var alignment: Alignment = Alignment.CENTER
private var autoplay: Boolean = false
private var assetsHandled: ReadableMap? = null
private var referencedAssets: ReadableMap? = null
private var shouldBeReloaded = true
private var exceptionManager: ExceptionsManagerModule? = null
private var isUserHandlingErrors = false
Expand Down Expand Up @@ -190,7 +190,7 @@ class RiveReactNativeView(private val context: ThemedReactContext) : FrameLayout
private fun clearReferences() {
riveAnimationView = null
exceptionManager = null
assetsHandled = null
referencedAssets = null
}

fun onPlay(animationName: String, isStateMachine: Boolean = false) {
Expand Down Expand Up @@ -476,7 +476,7 @@ class RiveReactNativeView(private val context: ThemedReactContext) : FrameLayout

private fun reloadIfNeeded() {
if (shouldBeReloaded) {
val assetStore = assetsHandled?.let {
val assetStore = referencedAssets?.let {
RiveReactNativeAssetStore(
it, loadAssetHandler = ::loadAsset
)
Expand Down Expand Up @@ -549,8 +549,8 @@ class RiveReactNativeView(private val context: ThemedReactContext) : FrameLayout
shouldBeReloaded = true
}

fun setAssetsHandled(assetsHandled: ReadableMap?) {
this.assetsHandled = assetsHandled;
fun setReferencedAssets(referencedAssets: ReadableMap?) {
this.referencedAssets = referencedAssets;
shouldBeReloaded = true;
}

Expand Down Expand Up @@ -777,10 +777,10 @@ class RiveReactNativeView(private val context: ThemedReactContext) : FrameLayout
typealias LoadAssetHandler = (source: ReadableMap, asset: FileAsset) -> Unit

private class RiveReactNativeAssetStore(
private val assetsHandled: ReadableMap, private val loadAssetHandler: LoadAssetHandler
private val referencedAssets: ReadableMap, private val loadAssetHandler: LoadAssetHandler
) : FileAssetLoader() {
override fun loadContents(asset: FileAsset, inBandBytes: ByteArray): Boolean {
val assetData = assetsHandled.getMap(asset.uniqueFilename.substringBeforeLast("."))
val assetData = referencedAssets.getMap(asset.uniqueFilename.substringBeforeLast("."))

val source = assetData?.getMap("source") ?: return false // Do not handle the asset.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -219,9 +219,9 @@ class RiveReactNativeViewManager : SimpleViewManager<RiveReactNativeView>() {
view.setAnimationName(animationName)
}

@ReactProp(name = "assetsHandled")
fun setAssetsHandled(view: RiveReactNativeView, source: ReadableMap?) {
view.setAssetsHandled(source)
@ReactProp(name = "referencedAssets")
fun setReferencedAssets(view: RiveReactNativeView, source: ReadableMap?) {
view.setReferencedAssets(source)
}

@ReactProp(name = "stateMachineName")
Expand Down
4 changes: 2 additions & 2 deletions example/src/OutOfBandAssets.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export default function StateMachine() {
fit={Fit.Contain}
style={styles.box}
stateMachineName="State Machine 1"
// You can use the `assetsHandled` prop to load in external assets from a URI
// You can use the `referencedAssets` prop to load in external assets from a URI
// or bundled asset on the native platform (iOS and Android)
// or as a source loaded directly from JavaScript.
//
Expand All @@ -22,7 +22,7 @@ export default function StateMachine() {
//
// The key of the map is the unique asset identifier (as exported in the Editor),
// which is a combination of the asset name and its unique identifier.
assetsHandled={{
referencedAssets={{
'Inter-594377': {
source: require('./assets/Inter-594377.ttf'),
// source: {
Expand Down
6 changes: 3 additions & 3 deletions ios/RiveReactNativeView.swift
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class RiveReactNativeView: RCTView, RivePlayerDelegate, RiveStateMachineDelegate

@objc var artboardName: String?

@objc var assetsHandled: NSDictionary?
@objc var referencedAssets: NSDictionary?
{
didSet {
requiresLocalResourceReconfigure = true;
Expand Down Expand Up @@ -108,7 +108,7 @@ class RiveReactNativeView: RCTView, RivePlayerDelegate, RiveStateMachineDelegate
}

override func didSetProps(_ changedProps: [String]!) {
if (changedProps.contains("url") || changedProps.contains("resourceName") || changedProps.contains("artboardName") || changedProps.contains("animationName") || changedProps.contains("stateMachineName") || changedProps.contains("assetsHandled")) {
if (changedProps.contains("url") || changedProps.contains("resourceName") || changedProps.contains("artboardName") || changedProps.contains("animationName") || changedProps.contains("stateMachineName") || changedProps.contains("referencedAssets")) {
reloadView()
}

Expand Down Expand Up @@ -213,7 +213,7 @@ class RiveReactNativeView: RCTView, RivePlayerDelegate, RiveStateMachineDelegate
}

private func customLoader(asset: RiveFileAsset, data: Data, factory: RiveFactory) -> Bool {
guard let assetData = assetsHandled?[asset.uniqueName()] as? NSDictionary else {
guard let assetData = referencedAssets?[asset.uniqueName()] as? NSDictionary else {
return false
}

Expand Down
2 changes: 1 addition & 1 deletion ios/RiveReactNativeViewManager.m
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ @interface RCT_EXTERN_MODULE(RiveReactNativeViewManager, RCTViewManager)
RCT_EXPORT_VIEW_PROPERTY(alignment, NSString)
RCT_EXPORT_VIEW_PROPERTY(autoplay, BOOL)
RCT_EXPORT_VIEW_PROPERTY(artboardName, NSString)
RCT_EXPORT_VIEW_PROPERTY(assetsHandled, NSDictionary)
RCT_EXPORT_VIEW_PROPERTY(referencedAssets, NSDictionary)
RCT_EXPORT_VIEW_PROPERTY(animationName, NSString)
RCT_EXPORT_VIEW_PROPERTY(stateMachineName, NSString)
RCT_EXPORT_VIEW_PROPERTY(isUserHandlingErrors, BOOL)
Expand Down
10 changes: 5 additions & 5 deletions src/Rive.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ type RiveProps = {
layoutScaleFactor?: number;
alignment: Alignment;
artboardName?: string;
assetsHandled?: FilesHandledMapping;
referencedAssets?: FilesHandledMapping;
animationName?: string;
stateMachineName?: string;
ref: any;
Expand Down Expand Up @@ -111,7 +111,7 @@ type Props = {
/**
* @experimental This is an experimental feature and may change without a major version update (breaking change).
*/
assetsHandled?: FilesHandledMapping;
referencedAssets?: FilesHandledMapping;
animationName?: string;
stateMachineName?: string;
autoplay?: boolean;
Expand Down Expand Up @@ -139,7 +139,7 @@ const RiveContainer = React.forwardRef<RiveRef, Props>(
fit = Fit.Contain,
layoutScaleFactor,
artboardName,
assetsHandled: assetsHandled,
referencedAssets: referencedAssets,
animationName,
stateMachineName,
testID,
Expand Down Expand Up @@ -445,7 +445,7 @@ const RiveContainer = React.forwardRef<RiveRef, Props>(
}

const convertedAssetHandledSources =
transformFilesHandledMapping(assetsHandled);
transformFilesHandledMapping(referencedAssets);

return (
<View style={[styles.container, style]} ref={ref as any} testID={testID}>
Expand Down Expand Up @@ -476,7 +476,7 @@ const RiveContainer = React.forwardRef<RiveRef, Props>(
onError={onErrorHandler}
alignment={alignment}
artboardName={artboardName}
assetsHandled={convertedAssetHandledSources}
referencedAssets={convertedAssetHandledSources}
animationName={animationName}
stateMachineName={stateMachineName}
/>
Expand Down

0 comments on commit 56d2832

Please sign in to comment.