search

omcljs / om

ClojureScript interface to Facebook's React
6.66k stars 362 forks source link

Fix bind-query to work with MapEntry #900

Closed thomasmulvaney closed 6 years ago

thomasmulvaney commented 6 years ago

This patch fixes om.next under recent versions of ClojureScript which use MapEntry rather than PersistentVector for map entries.

I've tested it under the 1.10.126 and 1.9.908 with the boot test suite and a om.next project.

The patch uses vector? rather than map-entry? which would have made the intentions clearer but is not available in older versions of CLJS where as this patch should be backwards compatible.

thomasmulvaney commented 6 years ago

This patch attempts to address #901

thomasmulvaney commented 6 years ago

Fixed on master (cb4c439f9afc7e1296ce3d2cad016ee8a0e10b67)