libretic
/
etherpad-lite
5
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[fix] Consider line attribs of plugins when building changesets 

When checking the places on text that are identical between content
before and after a changeset, we were considering only the standard
(Etherpad-core) attributes, and not taking into account attributes
created by the plugins. One consequence was that the '*' marker of lines
with line attribs were being kept, even when the new lines have
different line attribs. See #3118 for more details.

Fix #3118.
This commit is contained in:
Luiza Pagliari 2017-01-12 12:31:42 -02:00
parent deed74c8c8
commit b2d9f57338
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/static/js/ace2_inner.js
View File

@ -1782,9 +1782,9 @@ function Ace2Inner(){
return !!STYLE_ATTRIBS[aname];
}
function isIncorpedAttribute(aname)
function isOtherIncorpedAttribute(aname)
{
return ( !! STYLE_ATTRIBS[aname]) || ( !! OTHER_INCORPED_ATTRIBS[aname]);
return !!OTHER_INCORPED_ATTRIBS[aname];
}
function insertDomLines(nodeToAddAfter, infoStructs, isTimeUp)
@ -2526,7 +2526,6 @@ function Ace2Inner(){
function doIncorpLineSplice(startLine, deleteCount, newLineEntries, lineAttribs, hints)
{
var startOldChar = rep.lines.offsetOfIndex(startLine);
var endOldChar = rep.lines.offsetOfIndex(startLine + deleteCount);
@ -2760,7 +2759,7 @@ function Ace2Inner(){
{
function incorpedAttribFilter(anum)
{
return isStyleAttribute(rep.apool.getAttribKey(anum));
return !isOtherIncorpedAttribute(rep.apool.getAttribKey(anum));
}
function attribRuns(attribs)