5monkeys / djedi-cms / 328
97%
master: 96%

LAST BUILD BRANCH: ci/djedi-react-release-workflow
DEFAULT BRANCH: master
Ran
Jobs 1
Files 44
Run time 4s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
328

push

travis-ci

lydell 
Fix race condition when clicking outlines to edit

Sometimes when clicking an outline for a node to edit it, the node is
blanked out. It still works to fill in some new text and save, but the
initial node text failed to load. This is due to a race condition.

Normally, these are the logs from the cms.js and editor.js:

```
cms.js?v=1.2.1:159 select node
cms.js?v=1.2.1:349 Plugin.connect()
cms.js?v=1.2.1:357 Plugin.connect().loaded
cms.js?v=1.2.1:378 Plugin.resize()
editor.js?v=1.2.1:232 Editor.onLoad() i18n://sv-se@home/test.md
editor.js?v=1.2.1:219 Editor.trigger page:node:fetch
editor.js?v=1.2.1:236 Editor.inititial data {data: "I like djedi!", content: "<p>I like djedi!</p>"}
editor.js?v=1.2.1:266 Editor.setNode()
editor.js?v=1.2.1:290 Editor.setState() new
editor.js?v=1.2.1:219 Editor.trigger page:node:fetch
editor.js?v=1.2.1:428 Editor.renderContent()
editor.js?v=1.2.1:362 Editor.renderRevisions()
editor.js?v=1.2.1:219 Editor.trigger plugin:loaded
editor.js?v=1.2.1:395 Editor.render()
editor.js?v=1.2.1:248 content <p>I like djedi!</p>
editor.js?v=1.2.1:219 Editor.trigger node:render
cms.js?v=1.2.1:16 Event node:render
cms.js?v=1.2.1:98 Node.render
```

But when the bug occurs, the logs are:

```
cms.js?v=1.2.1:159 select node
editor.js?v=1.2.1:232 Editor.onLoad() i18n://sv-se@home/test.md
editor.js?v=1.2.1:219 Editor.trigger page:node:fetch
editor.js?v=1.2.1:266 Editor.setNode()
editor.js?v=1.2.1:290 Editor.setState() new
editor.js?v=1.2.1:219 Editor.trigger page:node:fetch
editor.js?v=1.2.1:362 Editor.renderRevisions()
editor.js?v=1.2.1:219 Editor.trigger plugin:loaded
editor.js?v=1.2.1:395 Editor.render()
editor.js?v=1.2.1:248 content
cms.js?v=1.2.1:349 Plugin.connect()
cms.js?v=1.2.1:357 Plugin.connect().loaded
cms.js?v=1.2.1:378 Plugin.resize()
editor.js?v=1.2.1:219 Editor.trigger node:render
cms.js?v=1.2.1:16 Event node:render
```

Notice how `Editor.onLoad()` is called _before_ `Plugin.connect()`. This
means that when Editor (in editor.js) ... (continued)

1248 of 1282 relevant lines covered (97.35%)

0.97 hits per line

Jobs
ID Job ID Ran Files Coverage
20 328.20 (TOXENV=coverage) 0
97.35
 Travis Job 328.20
Source Files on build 328
Detailed source file information is not available for this build.