Merge pull request draft-js-plugins#596 from sugarshin/patch-1

nikgraf · web-flow · commit 479f6c9a1f1a · 2016-12-09T14:03:56.000+01:00
Patch blockStyleFn
diff --git a/draft-js-plugins-editor/CHANGELOG.md b/draft-js-plugins-editor/CHANGELOG.md
@@ -11,6 +11,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - `defaultBlockRenderMap` option, by default is set to true. If set to false the defaultBlockRenderMap from Draft.js is not used as base for the generated blockRenderMap.
 - `decorators` option now allows custom implementations the [DraftDecoratorType](https://github.com/facebook/draft-js/blob/master/src/model/decorators/DraftDecoratorType.js) interface to be passed into the array along with the traditional CompositeDecorator objects
 - added the utility function `composeDecorators` as a named export.
+- Ignore handle `blockStyleFn` in case its value is `null` . [#596](https://github.com/draft-js-plugins/draft-js-plugins/pull/596)
 
 ### Removed
 
diff --git a/draft-js-plugins-editor/src/Editor/index.js b/draft-js-plugins-editor/src/Editor/index.js
@@ -170,7 +170,7 @@ class PluginEditor extends Component {
       for (const plugin of plugins) {
         if (typeof plugin[methodName] !== 'function') continue;
         const result = plugin[methodName](...newArgs);
-        if (result !== undefined) {
+        if (result !== undefined && result !== null) {
           styles = (styles ? (`${styles} `) : '') + result;
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ class PluginEditor extends Component {`
`170`	`170`	`for (const plugin of plugins) {`
`171`	`171`	`if (typeof plugin[methodName] !== 'function') continue;`
`172`	`172`	`const result = plugin[methodName](...newArgs);`
`173`		`- if (result !== undefined) {`
	`173`	`+ if (result !== undefined && result !== null) {`
`174`	`174`	styles = (styles ? (`${styles} `) : '') + result;
`175`	`175`	`}`
`176`	`176`	`}`