From 1214a752518efae12fbeed5c7f2ab4a8ef335895 Mon Sep 17 00:00:00 2001 From: Maksims Mihejevs Date: Mon, 24 Aug 2015 15:43:36 +0100 Subject: [PATCH] fix: texture/cubemap patch data can be optional --- CHANGES.md | 5 +++++ src/resources/resources_cubemap.js | 8 ++++---- src/resources/resources_texture.js | 12 ++++++------ 3 files changed, 15 insertions(+), 10 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index 61d2ae1d09e..ed54dd0f04d 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -1,5 +1,10 @@ # PlayCanvas Engine Changes +### 0.173.0 +* Added tags (`pc.Tags`) interface to pc.Asset +* Added findByTag method to pc.AssetRegistry +* [FIX] Texture and CubeMap data in patch can be optional + ### 0.172.0 * Added 'curve' script attributes * Added 'colorcurve' script attributes diff --git a/src/resources/resources_cubemap.js b/src/resources/resources_cubemap.js index 5f05bcd2b9e..120fd10c38a 100644 --- a/src/resources/resources_cubemap.js +++ b/src/resources/resources_cubemap.js @@ -91,18 +91,18 @@ pc.extend(pc, function () { if (cubemap.name !== assetCubeMap.name) cubemap.name = assetCubeMap.name; - if (cubemap.rgbm !== !! assetCubeMap.data.rgbm) + if (assetCubeMap.hasOwnProperty('rgbm') && cubemap.rgbm !== !! assetCubeMap.data.rgbm) cubemap.rgbm = !! assetCubeMap.data.rgbm; cubemap.fixCubemapSeams = !! assetCubeMap._dds; - if (cubemap.minFilter !== assetCubeMap.data.minFilter) + if (assetCubeMap.hasOwnProperty('minFilter') && cubemap.minFilter !== assetCubeMap.data.minFilter) cubemap.minFilter = assetCubeMap.data.minFilter; - if (cubemap.magFilter !== assetCubeMap.data.magFilter) + if (assetCubeMap.hasOwnProperty('magFilter') && cubemap.magFilter !== assetCubeMap.data.magFilter) cubemap.magFilter = assetCubeMap.data.magFilter; - if (cubemap.anisotropy !== assetCubeMap.data.anisotropy) + if (assetCubeMap.hasOwnProperty('anisotropy') && cubemap.anisotropy !== assetCubeMap.data.anisotropy) cubemap.anisotropy = assetCubeMap.data.anisotropy; if (cubemap.addressU !== pc.ADDRESS_CLAMP_TO_EDGE) diff --git a/src/resources/resources_texture.js b/src/resources/resources_texture.js index c17c2cc4d0a..5421272b458 100644 --- a/src/resources/resources_texture.js +++ b/src/resources/resources_texture.js @@ -226,22 +226,22 @@ pc.extend(pc, function () { if (texture.name !== asset.name) texture.name = asset.name; - if (asset.data.minfilter && texture.minFilter !== JSON_FILTER_MODE[asset.data.minfilter]) + if (asset.data.hasOwnProperty('minfilter') && texture.minFilter !== JSON_FILTER_MODE[asset.data.minfilter]) texture.minFilter = JSON_FILTER_MODE[asset.data.minfilter]; - if (asset.data.magfilter && texture.magFilter !== JSON_FILTER_MODE[asset.data.magfilter]) + if (asset.data.hasOwnProperty('magfilter') && texture.magFilter !== JSON_FILTER_MODE[asset.data.magfilter]) texture.magFilter = JSON_FILTER_MODE[asset.data.magfilter]; - if (asset.data.addressu && texture.addressU !== JSON_ADDRESS_MODE[asset.data.addressu]) + if (asset.data.hasOwnProperty('addressu') && texture.addressU !== JSON_ADDRESS_MODE[asset.data.addressu]) texture.addressU = JSON_ADDRESS_MODE[asset.data.addressu]; - if (asset.data.addressv && texture.addressV !== JSON_ADDRESS_MODE[asset.data.addressv]) + if (asset.data.hasOwnProperty('addressv') && texture.addressV !== JSON_ADDRESS_MODE[asset.data.addressv]) texture.addressV = JSON_ADDRESS_MODE[asset.data.addressv]; - if (texture.anisotropy !== asset.data.anisotropy) + if (asset.data.hasOwnProperty('anisotropy') && texture.anisotropy !== asset.data.anisotropy) texture.anisotropy = asset.data.anisotropy; - if (texture.rgbm !== !! asset.data.rgbm) + if (asset.data.hasOwnProperty('rgbm') && texture.rgbm !== !! asset.data.rgbm) texture.rgbm = !! asset.data.rgbm; } };