search

situ2001 / opensumi-core

🚀 A framework helps you quickly build Cloud or Desktop IDE products.
https://opensumi.com/zh
MIT License
0 stars 0 forks source link

[Enhancement] CollaborationService should listen to proper events #5

Closed situ2001 closed 2 years ago

situ2001 commented 2 years ago

目前创建TextModel与YText的binding的方法如下

通过监听用户切换EditorGroup的标签或者切换EditorGroup，即监听事件EditorActiveResourceStateChangedEvent，该事件发生后就把两者给绑定起来，顺手把当前的Editor给加进到这个binding上。销毁binding的方法就是监听EditorGroupCloseEvent，当对应的uri没有被打开，就会销毁掉这个binding...

虽然能工作，但有可能会出bug

今天在读源码的时候，看到了事件EditorDocumentModelCreationEvent，对应的是TextModel的创建

https://github.com/situ2001/opensumi-core/blob/7f598a64f264300a1712a996f31be72039cc916e/packages/editor/src/browser/doc-model/editor-document-model-service.ts#L84-L96

读的时候，还找到了这个事件，对应的是TextModel的销毁

https://github.com/situ2001/opensumi-core/blob/7f598a64f264300a1712a996f31be72039cc916e/packages/editor/src/browser/doc-model/editor-document-model.ts#L143-L145

由于TextModel与YText是绑定在一起的，为何不在EditorDocumentModel创建之后就马上绑上YText，然后直到EditorDocumentModel销毁的时候再销毁这个binding？

TODO

situ2001 commented 2 years ago

这个改造应该可行，并且比较重要，先把 #3 的事情延后一下

situ2001 commented 2 years ago

实际用起来是可以的，过一会把测试写上去

situ2001 commented 2 years ago

但是又有一个新问题了，这会在删除文件的时候，有可能不起作用（发现默认情况下，DocModel不会立即被移除）

situ2001 commented 2 years ago

查看源码得知，DocModelRef的移除是做了Debounce的，默认3s，但即使设置为0s也无法避免删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件，无法正常创建binding的问题。

因为该ref的移除会在引用数为0且当前执行栈空后（即使setTimeout超时为0），从task queue中取出来执行，无法干预“删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件”的这个行为，他用回的依旧是同一个ref

situ2001 commented 2 years ago

当然代码还有一处地方用到了ref但没有把它dispose掉，这也会使ref无法被移除（removal）

不过，即使改回来，也没有太大的作用，见上条comment

situ2001 commented 2 years ago

查看源码得知，DocModelRef的移除是做了Debounce的，默认3s，但即使设置为0s也无法避免删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件，无法正常创建binding的问题。

因为该ref的移除会在引用数为0且当前执行栈空后（即使setTimeout超时为0），从task queue中取出来执行，无法干预“删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件”的这个行为，他用回的依旧是同一个ref

哦，之前的那种方法也会引起这样的bug...

situ2001 commented 2 years ago

查看源码得知，DocModelRef的移除是做了Debounce的，默认3s，但即使设置为0s也无法避免删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件，无法正常创建binding的问题。

因为该ref的移除会在引用数为0且当前执行栈空后（即使setTimeout超时为0），从task queue中取出来执行，无法干预“删除文件后不关闭打开的显示为(已删除)的tab的同时再次创建同名文件”的这个行为，他用回的依旧是同一个ref

这个也许可以把防抖时间设为0。然后加入行为：当文件被删除后，强行干掉这个已删除文件的tab？（这么做，有点诡异）

situ2001 commented 2 years ago

既然DocModel在本地是引用计数的，删除文件这类操作，在该模块上的设定是：服务端监听删除事件，然后从yMap中删掉这个YText，新建文件的时候，客户端再去请求新建YText，最后客户端通过监听YMap的event来获取这个新建的YText

那么好办了，本地留好binding map，当action === add出现，要么是第一次绑上YText和TextModel。也有可能是TextModel和binding还在，只是YText发生了改变。那么我们只要把binding上的YText给换掉不就行了？