y-websocket icon indicating copy to clipboard operation
y-websocket copied to clipboard

The `getContent` method references the method `getXmlElement` which does not appear to exist.

Open chanced opened this issue 4 years ago • 1 comments

Checklist

  • [X] Are you reporting a bug? Use github issues for bug reports and feature requests. For general questions, please use https://discuss.yjs.dev/

Describe the bug

the function getContent references getXmlElement on a Y.Doc which does not appear to exist.

https://github.com/yjs/y-websocket/blob/caa6304ba30c0e6d5921e735470f6470b045cb6a/bin/callback.js#L73

https://github.com/yjs/yjs/blob/dc9717ecd0650315d87129cff795dc5116f8bcf2/src/utils/Doc.js

To Reproduce I'm not sure that code branch can ever be reached but I wanted to let you know regardless.

Environment Information

  • y.js 13.5.6 per y-websocket's dependency
  • y-websocket - latest

Huly®: YJS-512

chanced avatar Oct 23 '21 15:10 chanced

Right, thanks! We should adopt this to ydoc.get(objName, Y.XmlElement).

dmonad avatar Nov 22 '21 10:11 dmonad