search

iamolegga / redux-translations

translations utils for react-redux apps
MIT License
3 stars 1 forks source link

исправил баг со статичными свойствами реакта #22

Closed nikitagolubitskiy closed 5 years ago

nikitagolubitskiy commented 5 years ago

HOC withTranslations копирует себе статичные методы реакта из-за чего возникают ошибки. например, если оборачивать в withTranslations компоненты, которые имеют статичный метод getDerivedStateFromProps, будет возникать ошибка из-за того что нет стейта в withTranslations(если бы был стейт, то getDerivedStateFromProps отрабатывал не очевидно, меняя стейт HOC-компонента).

iamolegga commented 5 years ago

https://github.com/iamolegga/redux-translations#withtranslationscomponent-copystaticmethods

это не то?

iamolegga commented 5 years ago

@nikitagolubitskiy я имею в виду, что если ты передашь вторым аргументом false, то оно не решит проблему?

iamolegga commented 5 years ago

Я так понимаю, это фикс для мест, где необходимо пробрасывать статичный метод родителю, но статичные реактовские свойства не нужны, правильно? Иначе можно просто отказаться от пробрасывания

iamolegga commented 5 years ago

ПОправь ошибку плиз https://travis-ci.org/iamolegga/redux-translations/builds/521979245

nikitagolubitskiy commented 5 years ago

@nikitagolubitskiy я имею в виду, что если ты передашь вторым аргументом false, то оно не решит проблему?

да это частично решает проблему, когда не нужно поднимать статичные свойства и методы родителю.

Я так понимаю, это фикс для мест, где необходимо пробрасывать статичный метод родителю, но статичные реактовские свойства не нужны, правильно? Иначе можно просто отказаться от пробрасывания

да, всё верно. моё мнение такое, что лучше не копировать реактовские статичные методы, потому что это может привести к сайд эффектам и ошибкам .