Skip to content

Commit f464dad

Browse files
Freeman Latiffacebook-github-bot
Freeman Latif
authored andcommitted
Fix stylesheet validation of functions with additional prototype methods (#27264)
Summary: Some babel plugins add additional methods to `Function.prototype` (see https://github.com/MatAtBread/fast-async): ```js Function.prototype.$asyncbind = function $asyncbind(self, catcher) { ... ``` Although undocumented, React Native allows functions to be passed to `StyleSheet.create()` for dynamic styling: ```js const styles = StyleSheet.create({ animalItem: height => ({ height }) }); ``` If there are additional custom methods on `Function.prototype`, React Native's `StyleSheetValidation.validateStyle` will loop through these properties and raise an error because those properties are not valid style keys, because it loops through _all_ properties, including inherited ones. ![Simulator Screen Shot - iPhone 11 Pro Max - 2019-11-19 at 12 04 49](https://user-images.githubusercontent.com/4031216/69145112-ce589100-0ac4-11ea-80d7-e93d59b935a8.png) This PR modifies `StyleSheetValidation.validateStyle` to only loop through the style's own properties, instead of including inherited ones. ## Changelog [General] [Fixed] - Fix stylesheet validation for functions with custom prototype methods. Pull Request resolved: #27264 Test Plan: - Tested that non-function style properties are still validated Differential Revision: D18694895 Pulled By: hramos fbshipit-source-id: b36f4a62a435e7b6a689398de3bcc06d6bb14293
1 parent f21fa4e commit f464dad

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

Libraries/StyleSheet/StyleSheetValidation.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ class StyleSheetValidation {
5151
if (!__DEV__ || global.__RCTProfileIsProfiling) {
5252
return;
5353
}
54-
for (const prop in styles[name]) {
54+
const styleProps = Object.keys(styles[name]);
55+
for (const prop of styleProps) {
5556
StyleSheetValidation.validateStyleProp(
5657
prop,
5758
styles[name],

0 commit comments

Comments
 (0)