Merge pull request #70 from duckietm/Dev

Dev

src/api/avatar/AvatarEditorThumbnailsHelper.ts

@@ -150,39 +150,34 @@ export class AvatarEditorThumbnailsHelper
 
         if(cached) return cached;
 
-        const avatarImage = GetAvatarRenderManager().createAvatarImage(figureString, AvatarScaleType.LARGE, null, null);
-
-        if(avatarImage.isPlaceholder())
+        return new Promise(async (resolve, reject) =>
         {
-            avatarImage.dispose();
+            const resetFigure = async (figure: string) =>
+            {
+                const avatarImage = GetAvatarRenderManager().createAvatarImage(figure, AvatarScaleType.LARGE, null, { resetFigure, dispose: null, disposed: false });
 
-            return null;
-        }
+                if(avatarImage.isPlaceholder()) return;
 
-        const texture = avatarImage.processAsTexture(AvatarSetType.HEAD, false);
+                const texture = avatarImage.processAsTexture(AvatarSetType.HEAD, false);
+                const sprite = new NitroSprite(texture);
 
-        if(!texture)
-        {
-            avatarImage.dispose();
+                if(isDisabled) sprite.filters = [ AvatarEditorThumbnailsHelper.ALPHA_FILTER ];
 
-            return null;
-        }
+                const imageUrl = await TextureUtils.generateImageUrl({
+                    target: sprite,
+                    frame: new NitroRectangle(0, 0, texture.width, texture.height)
+                });
 
-        const sprite = new NitroSprite(texture);
+                sprite.destroy();
+                avatarImage.dispose();
 
-        if(isDisabled) sprite.filters = [ AvatarEditorThumbnailsHelper.ALPHA_FILTER ];
+                AvatarEditorThumbnailsHelper.THUMBNAIL_CACHE.set(thumbnailKey, imageUrl);
 
-        const imageUrl = await TextureUtils.generateImageUrl({
-            target: sprite,
-            frame: new NitroRectangle(0, 0, texture.width, texture.height)
+                resolve(imageUrl);
+            };
+
+            resetFigure(figureString);
         });
-
-        sprite.destroy();
-        avatarImage.dispose();
-
-        if(imageUrl) AvatarEditorThumbnailsHelper.THUMBNAIL_CACHE.set(thumbnailKey, imageUrl);
-
-        return imageUrl;
     }
 
     private static sortByDrawOrder(a: IFigurePart, b: IFigurePart): number

src/api/catalog/CatalogUtilities.ts

@@ -13,6 +13,8 @@ export const GetSubscriptionProductIcon = (id: number) =>
 
 export const GetOfferNodes = (offerNodes: Map<number, ICatalogNode[]>, offerId: number) =>
 {
+    if(!offerNodes) return [];
+
     const nodes = offerNodes.get(offerId);
     const allowedNodes: ICatalogNode[] = [];
 
@@ -20,7 +22,7 @@ export const GetOfferNodes = (offerNodes: Map<number, ICatalogNode[]>, offerId:
     {
         for(const node of nodes)
         {
-            if(!node.isVisible) continue;
+            if(!node || !node.isVisible) continue;
 
             allowedNodes.push(node);
         }
@@ -31,6 +33,8 @@ export const GetOfferNodes = (offerNodes: Map<number, ICatalogNode[]>, offerId:
 
 export const FilterCatalogNode = (search: string, furniLines: string[], node: ICatalogNode, nodes: ICatalogNode[]) =>
 {
+    if(!node) return;
+
     if(node.isVisible && (node.pageId > 0))
     {
         let nodeAdded = false;

src/api/catalog/FurnitureOffer.ts

@@ -105,12 +105,12 @@ export class FurnitureOffer implements IPurchasableOffer
 
     public get localizationName(): string
     {
-        return this._furniData.name;
+        return this._furniData?.name ?? '';
     }
 
     public get localizationDescription(): string
     {
-        return this._furniData.description;
+        return this._furniData?.description ?? '';
     }
 
     public get isLazy(): boolean

src/common/layout/LayoutAvatarImageView.tsx

@@ -21,11 +21,15 @@ export const LayoutAvatarImageView: FC<LayoutAvatarImageViewProps> = props =>
     const [ isReady, setIsReady ] = useState<boolean>(false);
     const [ updateId, setUpdateId ] = useState<number>(0);
     const isDisposed = useRef(false);
-    const isPlaceholderRef = useRef(false);
+    const figureKeyRef = useRef<string>(null);
 
     useNitroEvent(NitroEventType.AVATAR_ASSET_LOADED, () =>
     {
-        if(isPlaceholderRef.current) setUpdateId(prev => prev + 1);
+        if(figureKeyRef.current)
+        {
+            AVATAR_IMAGE_CACHE.delete(figureKeyRef.current);
+            setUpdateId(prev => prev + 1);
+        }
     });
 
     const getClassNames = useMemo(() =>
@@ -61,9 +65,10 @@ export const LayoutAvatarImageView: FC<LayoutAvatarImageViewProps> = props =>
 
         const figureKey = [ figure, gender, direction, headOnly ].join('-');
 
+        figureKeyRef.current = figureKey;
+
         if(AVATAR_IMAGE_CACHE.has(figureKey))
         {
-            isPlaceholderRef.current = false;
             setAvatarUrl(AVATAR_IMAGE_CACHE.get(figureKey));
         }
         else
@@ -73,7 +78,7 @@ export const LayoutAvatarImageView: FC<LayoutAvatarImageViewProps> = props =>
                 {
                     if(isDisposed.current) return;
 
-                    isPlaceholderRef.current = false;
+                    AVATAR_IMAGE_CACHE.delete(figureKey);
                     setUpdateId(prev => prev + 1);
                 },
                 dispose: null,
@@ -90,15 +95,7 @@ export const LayoutAvatarImageView: FC<LayoutAvatarImageViewProps> = props =>
 
             if(imageUrl && !isDisposed.current)
             {
-                if(!avatarImage.isPlaceholder())
-                {
-                    AVATAR_IMAGE_CACHE.set(figureKey, imageUrl);
-                    isPlaceholderRef.current = false;
-                }
-                else
-                {
-                    isPlaceholderRef.current = true;
-                }
+                if(!avatarImage.isPlaceholder()) AVATAR_IMAGE_CACHE.set(figureKey, imageUrl);
 
                 setAvatarUrl(imageUrl);
             }

src/components/avatar-editor/figure-set/AvatarEditorFigureSetItemView.tsx

@@ -26,7 +26,11 @@ export const AvatarEditorFigureSetItemView: FC<{
 
     useNitroEvent(NitroEventType.AVATAR_ASSET_LOADED, () =>
     {
-        if(!assetUrl || !assetUrl.length) setRetryId(prev => prev + 1);
+        if(!assetUrl || !assetUrl.length)
+        {
+            AvatarEditorThumbnailsHelper.clearCache();
+            setRetryId(prev => prev + 1);
+        }
     });
 
     useEffect(() =>

src/components/catalog/views/page/common/CatalogGridOfferView.tsx

@@ -18,16 +18,18 @@ export const CatalogGridOfferView: FC<CatalogGridOfferViewProps> = props =>
     const { requestOfferToMover = null } = useCatalog();
     const { isVisible = false } = useInventoryFurni();
     const { isFavoriteOffer, toggleFavoriteOffer } = useCatalogFavorites();
-    const isFav = isFavoriteOffer(offer.offerId);
+    const isFav = offer ? isFavoriteOffer(offer.offerId) : false;
 
     const iconUrl = useMemo(() =>
     {
+        if(!offer) return null;
+
         if(offer.pricingModel === Offer.PRICING_MODEL_BUNDLE)
         {
             return null;
         }
 
-        return offer.product.getIconUrl(offer);
+        return offer.product?.getIconUrl(offer) ?? null;
     }, [ offer ]);
 
     const onMouseEvent = (event: MouseEvent) =>
@@ -49,6 +51,8 @@ export const CatalogGridOfferView: FC<CatalogGridOfferViewProps> = props =>
         }
     };
 
+    if(!offer) return null;
+
     const product = offer.product;
 
     if(!product) return null;

src/components/catalog/views/page/common/CatalogSearchView.tsx

@@ -22,6 +22,8 @@ export const CatalogSearchView: FC<{}> = () =>
 
         const timeout = setTimeout(() =>
         {
+            if(!offersToNodes || !rootNode) return;
+
             const furnitureDatas = GetSessionDataManager().getAllFurnitureData();
 
             if(!furnitureDatas || !furnitureDatas.length) return;
@@ -31,11 +33,13 @@ export const CatalogSearchView: FC<{}> = () =>
 
             for(const furniture of furnitureDatas)
             {
+                if(!furniture) continue;
+
                 if((currentType === CatalogType.BUILDER) && !furniture.availableForBuildersClub) continue;
 
                 if((currentType === CatalogType.NORMAL) && furniture.excludeDynamic) continue;
 
-                const searchValues = [ furniture.className, furniture.name, furniture.description ].join(' ').replace(/ /gi, '').toLowerCase();
+                const searchValues = [ furniture.className || '', furniture.name || '', furniture.description || '' ].join(' ').replace(/ /gi, '').toLowerCase();
 
                 if((currentType === CatalogType.BUILDER) && (furniture.purchaseOfferId === -1) && (furniture.rentOfferId === -1))
                 {