[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-2-1' - loleaflet/src

Pranav Kant pranavk at collabora.co.uk
Tue May 16 15:15:02 UTC 2017


 loleaflet/src/layer/AnnotationManager.js |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit 05e6700004231530b87fdc5bc3a35a89b6a1c53c
Author: Pranav Kant <pranavk at collabora.co.uk>
Date:   Tue Apr 25 21:04:46 2017 +0530

    loleaflet: Warn, and don't crash, if we don't get expected data
    
    It is possible that backend sends comment for which there is no parent
    comment sent yet, as presently is the case as I am writing this. Warn
    nicely in this case.
    
    Change-Id: I2609dfdcdac69380b824d444ba8f64803996286c

diff --git a/loleaflet/src/layer/AnnotationManager.js b/loleaflet/src/layer/AnnotationManager.js
index 7fee34a6..cd55ae15 100644
--- a/loleaflet/src/layer/AnnotationManager.js
+++ b/loleaflet/src/layer/AnnotationManager.js
@@ -476,6 +476,11 @@ L.AnnotationManager = L.Class.extend({
 	adjustParentAdd: function(comment) {
 		if (comment.parent && comment.parent > '0') {
 			var parentIdx = this.getIndexOf(comment.parent);
+			if (parentIdx === -1) {
+				console.warn('adjustParentAdd: No parent comment to attach received comment to. ' +
+				             'Parent comment ID sought is :' + comment.parent + ' for current comment with ID : ' + comment.id);
+				return;
+			}
 			if (this._items[parentIdx + 1] && this._items[parentIdx + 1]._data.parent === this._items[parentIdx]._data.id) {
 				this._items[parentIdx + 1]._data.parent = comment.id;
 			}


More information about the Libreoffice-commits mailing list