Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Client with copied pad receives a "pad deleted" message #3183

Closed
lpagliari opened this issue May 4, 2017 · 2 comments
Closed

Client with copied pad receives a "pad deleted" message #3183

lpagliari opened this issue May 4, 2017 · 2 comments

Comments

@lpagliari
Copy link
Contributor

How to reproduce:

  • open a browser with a pad
  • call the api to copy the pad
  • result: a message is displayed on the browser saying the pad was removed

Code that makes this happen:

Pad.prototype.copy = function copy(destinationID, force, callback) {
(...)
  //kick everyone from this pad
  // TODO: this presents a message on the client saying that the pad was 'deleted'. Fix this?
  padMessageHandler.kickSessionsFromPad(sourceID);

Does anybody know if there was a reason for Pad.prototype.copy to call kickSessionsFromPad? It does not make sense to me at all. I suspect that Pad.prototype.copy code was based on Pad.prototype.remove, and in this case it makes total sense to tell clients the pad is no longer available. But for copy...

I'll implement a fix for this, unless someone gives a good reason to kick everyone from the copied pad.

lpagliari added a commit to storytouch/etherpad-lite that referenced this issue May 5, 2017
Message should be shown when pad is deleted or moved, not when it is copied.

Fix ether#3183
@renebrain
Copy link

#3125 was excactly this problem, solved it back then, by ignoring that message.

@lpagliari
Copy link
Contributor Author

Great, @renebrain , I'll close #3125 then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants