Fix render bug causing changes to the app state not being reflected

[alessandrod]

Avoid potentially nesting -set-property! calls for the :skip-render-root root
property. Makes it so that changes to the root state done while the root is
being rendered get applied immediately (in the next raf callback).

[alessandrod]

Here's an example demonstrating the issue. It should render at every rAF callback but it only renders twice.

(defn c [props owner]
  (reify
    om/IRender
    (render [_]
      (om/transact! props :i inc)
      (dom/span nil (:i props)))))

[alessandrod]

Ping?

[swannodette]

@alessandrod thanks for the reminder. Merging.

[lenaschoenburg]

Would you consider releasing a version with this fix? Unfortunately this is blocking for us.

[bensu]

@dignati for what it's worth, I just published a version of the library that contains the unpublished fixes (including this one).

[lenaschoenburg]

Thanks, I actually did the same. I think it's really unfortunate that this
is still an open issue. Most newcomers to Om/Clojurescript would probably
have a hard time figuring out that it's a bug in Om, not their mistake.

Sebastian Bensusan notifications@github.com schrieb am Di., 30. Juni 2015
15:59:

@dignati https://github.com/dignati for what it's worth, I just
published https://clojars.org/bensu/om a version of the library that
contains the unpublished fixes (including this one).

—
Reply to this email directly or view it on GitHub
#342 (comment).