[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Target Milestone|VE-deploy-2012-12-10|VE-deploy-2012-11-26 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 Roan Kattouw roan.katt...@gmail.com changed: What|Removed |Added CC||roan.katt...@gmail.com --- Comment #3 from Roan Kattouw roan.katt...@gmail.com 2012-11-21 20:35:27 UTC --- There were actually two cases, with two different root causes. If the last node in the document was not a paragraph, comment #2 applies: the document is empty and the pre-annotations code was asking for annotations at offset 0. This was fixed a while ago. If the last node in the document was a paragraph (this is common), the document wouldn't actually be emptied. Instead, all nodes would be removed except the last paragraph, which gets stripped. So after removing everything, we are left with a document that only contains an empty paragraph. However, in the model tree, this empty paragraph is truly empty and does not contain a zero-length text node (most other empty paragraphs do have one). Once you type a character, CE pawns it first, processing a transaction that inserts a pawn in the paragraph. The transaction processor gets confused by the lack of any content nodes in the paragraph, and issues a rebuild for the contents of the paragraph, which fails spectacularly. The model tree ends up in an inconsistent state with a text node being a direct child of the document node and the paragraph node being gone. This then caused offset computations to be off by one, which caused other parts of CE to do strange things, culminating in an exception in openAnnotations(). When the exception occurs, the pawn hasn't been unpawned yet, so there's pawn leakage. Trevor has a commit queued up that will fix this. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 --- Comment #4 from Roan Kattouw roan.katt...@gmail.com 2012-11-21 22:03:45 UTC --- https://gerrit.wikimedia.org/r/#/c/34597/1 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED --- Comment #5 from James Forrester jforres...@wikimedia.org 2012-11-21 23:50:40 UTC --- Confirmed fixed, will be pushed live on Monday. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 James Forrester jforres...@wikimedia.org changed: What|Removed |Added Priority|Unprioritized |High Status|NEW |ASSIGNED AssignedTo|i...@wikia-inc.com |roan.katt...@gmail.com Target Milestone|--- |VE-deploy-2012-12-10 Severity|normal |critical -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 --- Comment #1 from Inez Korczyński i...@wikia-inc.com 2012-11-17 07:46:44 UTC --- *** Bug 42218 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 42219] VisualEditor: Trivial way to get a lot of errors
https://bugzilla.wikimedia.org/show_bug.cgi?id=42219 --- Comment #2 from Inez Korczyński i...@wikia-inc.com 2012-11-17 07:49:50 UTC --- After deleting all the content document if completely empty (data.length = 0), however view still askes what are the annotations at offset 0 - and that's what fails. Possible fix it to make method getAnnotationsFromOffset return empty set if given offset does not exists. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l