libretic/etherpad-lite
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix clearing authorship colors which was broken by #1609

Browse source 
Fixes #1620
This commit is contained in:
Marcel Klehr 2013-03-15 18:08:51 +01:00
parent f0dee2d2ce
commit de552df6db
1 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/node/handler/PadMessageHandler.js
View file

@ -551,11 +551,20 @@ function handleUserChanges(client, message)
} }
}); });
// Validate all 'author' attribs to be the same value as the current user // Validate all added 'author' attribs to be the same value as the current user
wireApool.eachAttrib(function(type, value) { var iterator = Changeset.opIterator(Changeset.unpack(changeset).ops)
if('author' == type && value != thisSession.author) throw "Trying to submit changes as another author" , op
while(iterator.hasNext()) {
op = iterator.next()
if(op.opcode != '+') continue;
op.attribs.split('*').forEach(function(attr) {
if(!attr) return
attr = wireApool.getAttrib(attr)
if(!attr) return
if('author' == attr[0] && attr[1] != thisSession.author) throw "Trying to submit changes as another author"
}) })
} }
}
catch(e) catch(e)
{ {
// There is an error in this changeset, so just refuse it // There is an error in this changeset, so just refuse it