diff --git a/components/impl/PresentationImpl.cpp b/components/impl/PresentationImpl.cpp
index 714740f367c..3ca29fb24c2 100644
--- a/components/impl/PresentationImpl.cpp
+++ b/components/impl/PresentationImpl.cpp
@@ -1,262 +1,264 @@
 /*
  * This file is part of the KDE project
  *
  * Copyright (C) 2013 Arjen Hiemstra <ahiemstra@heimr.nl>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "PresentationImpl.h"
 #include "PresentationKoPAView.h"
 
 #include <QGraphicsWidget>
 #include <QMimeDatabase>
 #include <QTextDocument>
 #include <QTextFrame>
 #include <QTextLayout>
 #include <QDebug>
 
 #include <stage/part/KPrPart.h>
 #include <stage/part/KPrDocument.h>
 #include <libs/textlayout/KoTextShapeData.h>
 #include <KoFindText.h>
 #include <KoPACanvasItem.h>
 #include <KoPAPageBase.h>
 #include <KoShape.h>
 #include <KoZoomController.h>
 
 using namespace Calligra::Components;
 
 class PresentationImpl::Private
 {
 public:
     Private() : part{nullptr}, document{nullptr}
     { }
 
     KPrPart* part;
     KPrDocument* document;
 
     PresentationKoPAView* koPaView;
 
     QList< QPair< QRectF, QUrl > > links;
 
     QList<KoShape*> deepShapeFind(QList<KoShape*> shapes)
     {
         QList<KoShape*> allShapes;
         foreach(KoShape* shape, shapes) {
             allShapes.append(shape);
             KoShapeContainer *container = dynamic_cast<KoShapeContainer*>(shape);
             if(container) {
                 allShapes.append(deepShapeFind(container->shapes()));
             }
         }
         return allShapes;
     }
 
     void updateLinkTargets()
     {
         links.clear();
 
         if(!koPaView || !koPaView->activePage())
             return;
 
         foreach(const KoShape* shape, koPaView->activePage()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
             }
         }
 
         QList<QTextDocument*> texts;
         KoFindText::findTextInShapes(koPaView->activePage()->shapes(), texts);
         QList<KoShape*> allShapes = deepShapeFind(koPaView->activePage()->shapes());
         foreach(QTextDocument* text, texts) {
             QTextBlock block = text->rootFrame()->firstCursorPosition().block();
             for (; block.isValid(); block = block.next()) {
                 block.begin();
                 QTextBlock::iterator it;
                 for (it = block.begin(); !(it.atEnd()); ++it) {
                     QTextFragment fragment = it.fragment();
                     if (fragment.isValid()) {
                         QTextCharFormat format = fragment.charFormat();
                         if(format.isAnchor()) {
                             // This is an anchor, store target and position...
                             QRectF rect = getFragmentPosition(block, fragment);
                             foreach(KoShape* shape, allShapes) {
                                 KoTextShapeData *shapeData = dynamic_cast<KoTextShapeData*>(shape->userData());
                                 if (!shapeData)
                                     continue;
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;
                                 }
                             }
                             links.append(QPair<QRectF, QUrl>(koPaView->kopaCanvas()->viewConverter()->documentToView(rect), QUrl(format.anchorHref())));
                         }
                     }
                 }
             }
         }
         qDebug() << "Discovered the following links in the slide:" << links;
     }
 
     QRectF getFragmentPosition(QTextBlock block, QTextFragment fragment)
     {
         // TODO this only produces a position for the first part, if the link spans more than one line...
         // Need to sort that somehow, unfortunately probably by slapping this code into the above function.
         // For now leave it like this, more important things are needed.
         QTextLayout* layout = block.layout();
         QTextLine line = layout->lineForTextPosition(fragment.position() - block.position());
         if(!line.isValid())
         {
             // fragment has no valid position and consequently no line...
             return QRectF();
         }
         qreal top = line.position().y() + (line.height() / 2);
         qreal bottom = top + line.height();
         qreal left = line.cursorToX(fragment.position() - block.position());
         qreal right = line.cursorToX((fragment.position() - block.position()) + fragment.length());
         QRectF fragmentPosition(QPointF(left, top), QPointF(right, bottom));
         return fragmentPosition.adjusted(layout->position().x(), layout->position().y(), 0, 0);
     }
 
     static const float wiggleFactor;
 };
 
 const float Calligra::Components::PresentationImpl::Private::wiggleFactor{ 4.f };
 
 PresentationImpl::PresentationImpl(QObject* parent)
     : DocumentImpl{parent}, d{new Private}
 {
     setDocumentType(DocumentType::Presentation);
 }
 
 PresentationImpl::~PresentationImpl()
 {
     delete d;
 }
 
 bool PresentationImpl::load(const QUrl& url)
 {
     delete d->part;
     delete d->document;
 
     d->part = new KPrPart{this};
     d->document = new KPrDocument{d->part};
     setKoDocument(d->document);
     d->part->setDocument(d->document);
 
     bool retval = false;
     if (url.scheme() == QStringLiteral("template")) {
         bool ok = d->document->loadNativeFormat(url.toString().mid(11));
         d->document->setModified(false);
         d->document->undoStack()->clear();
 
         if (ok) {
             QString mimeType = QMimeDatabase().mimeTypeForUrl(url).name();
             // in case this is a open document template remove the -template from the end
             mimeType.remove( QRegExp( "-template$" ) );
             d->document->setMimeTypeAfterLoading(mimeType);
             d->document->resetURL();
             d->document->setEmpty();
         } else {
             // some kind of error reporting thing here... failed to load template, tell the user
             // why their canvas is so terribly empty.
             d->document->initEmpty();
         }
         d->document->setModified(true);
         retval = true;
     } else {
         retval = d->document->openUrl(url);
     }
 
     auto canvas = static_cast<KoPACanvasItem*>(d->part->canvasItem(d->document));
 
     createAndSetCanvasController(canvas);
 
     d->koPaView = new PresentationKoPAView(canvasController(), canvas, d->document);
     canvas->setView(d->koPaView);
 
     createAndSetZoomController(canvas);
     d->koPaView->setZoomController(zoomController());
     d->koPaView->connectToZoomController();
 
     KoPAPageBase* page = d->document->pageByIndex(0, false);
     if(page) {
         d->koPaView->doUpdateActivePage(page);
     }
     d->updateLinkTargets();
 
     setCanvas(canvas);
 
     return retval;
 }
 
 int PresentationImpl::currentIndex()
 {
     if (d->document && d->koPaView && d->koPaView->activePage()) {
         return d->document->pageIndex(d->koPaView->activePage());
     } else {
         return -1;
     }
 }
 
 void PresentationImpl::setCurrentIndex(int newValue)
 {
     if(newValue != currentIndex()) {
         d->koPaView->doUpdateActivePage(d->document->pageByIndex(newValue, false));
         d->updateLinkTargets();
         emit requestViewUpdate();
         emit currentIndexChanged();
     }
 }
 
 int PresentationImpl::indexCount() const
 {
     return d->document->pageCount();
 }
 
 QUrl PresentationImpl::urlAtPoint(QPoint point)
 {
     for( const QPair< QRectF, QUrl >& link : d->links )
     {
         QRectF hitTarget{
             link.first.x() - Private::wiggleFactor,
             link.first.y() - Private::wiggleFactor,
             link.first.width() + Private::wiggleFactor * 2,
             link.first.height() + Private::wiggleFactor * 2
         };
 
         if( hitTarget.contains( point ) )
         {
             return link.second;
         }
     }
     return QUrl();
 }
 
 QObject* PresentationImpl::part() const
 {
     return d->part;
 }
diff --git a/components/impl/SpreadsheetImpl.cpp b/components/impl/SpreadsheetImpl.cpp
index 9512918fe45..0ab71e1e8d2 100644
--- a/components/impl/SpreadsheetImpl.cpp
+++ b/components/impl/SpreadsheetImpl.cpp
@@ -1,243 +1,245 @@
 /*
  * This file is part of the KDE project
  *
  * Copyright (C) 2013 Arjen Hiemstra <ahiemstra@heimr.nl>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "SpreadsheetImpl.h"
 
 #include <QGraphicsWidget>
 #include <QTextDocument>
 #include <QTextFrame>
 #include <QTextLayout>
 
 #include <KoToolManager.h>
 
 #include <KoFindText.h>
 #include <KoShape.h>
 #include <KoShapeContainer.h>
 #include <KoZoomController.h>
 #include <KoViewConverter.h>
 #include <sheets/part/Part.h>
 #include <sheets/part/Doc.h>
 #include <sheets/part/CanvasItem.h>
 #include <sheets/Map.h>
 #include <sheets/Sheet.h>
 #include <libs/textlayout/KoTextShapeData.h>
 #include <Damages.h>
 
 using namespace Calligra::Components;
 
 class SpreadsheetImpl::Private
 {
 public:
     Private() : part{nullptr}, document{nullptr}
     { }
 
     Calligra::Sheets::Part* part;
     Calligra::Sheets::Doc* document;
     Calligra::Sheets::CanvasItem* canvas;
     int currentSheet;
 
     QList< QPair< QRectF, QUrl > > links;
 
     QList<KoShape*> deepShapeFind(QList<KoShape*> shapes)
     {
         QList<KoShape*> allShapes;
         foreach(KoShape* shape, shapes) {
             allShapes.append(shape);
             KoShapeContainer *container = dynamic_cast<KoShapeContainer*>(shape);
             if(container) {
                 allShapes.append(deepShapeFind(container->shapes()));
             }
         }
         return allShapes;
     }
 
     void updateLinkTargets()
     {
         links.clear();
 
         if(!canvas)
             return;
         foreach(const KoShape* shape, canvas->activeSheet()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
             }
         }
 
         QList<QTextDocument*> texts;
         KoFindText::findTextInShapes(canvas->activeSheet()->shapes(), texts);
         QList<KoShape*> allShapes = deepShapeFind(canvas->activeSheet()->shapes());
         foreach(QTextDocument* text, texts) {
             QTextBlock block = text->rootFrame()->firstCursorPosition().block();
             for (; block.isValid(); block = block.next()) {
                 block.begin();
                 QTextBlock::iterator it;
                 for (it = block.begin(); !(it.atEnd()); ++it) {
                     QTextFragment fragment = it.fragment();
                     if (fragment.isValid()) {
                         QTextCharFormat format = fragment.charFormat();
                         if(format.isAnchor()) {
                             // This is an anchor, store target and position...
                             QRectF rect = getFragmentPosition(block, fragment);
                             foreach(KoShape* shape, allShapes) {
                                 KoTextShapeData *shapeData = dynamic_cast<KoTextShapeData*>(shape->userData());
                                 if (!shapeData)
                                     continue;
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;
                                 }
                             }
                             links.append(QPair<QRectF, QUrl>(canvas->viewConverter()->documentToView(rect), QUrl(format.anchorHref())));
                         }
                     }
                 }
             }
         }
     }
 
     QRectF getFragmentPosition(QTextBlock block, QTextFragment fragment)
     {
         // TODO this only produces a position for the first part, if the link spans more than one line...
         // Need to sort that somehow, unfortunately probably by slapping this code into the above function.
         // For now leave it like this, more important things are needed.
         QTextLayout* layout = block.layout();
         QTextLine line = layout->lineForTextPosition(fragment.position() - block.position());
         if(!line.isValid())
         {
             // fragment has no valid position and consequently no line...
             return QRectF();
         }
         qreal top = line.position().y() + (line.height() / 2);
         qreal bottom = top + line.height();
         qreal left = line.cursorToX(fragment.position() - block.position());
         qreal right = line.cursorToX((fragment.position() - block.position()) + fragment.length());
         QRectF fragmentPosition(QPointF(left, top), QPointF(right, bottom));
         return fragmentPosition.adjusted(layout->position().x(), layout->position().y(), 0, 0);
     }
 
     static const float wiggleFactor;
 };
 
 const float Calligra::Components::SpreadsheetImpl::Private::wiggleFactor{ 4.f };
 
 SpreadsheetImpl::SpreadsheetImpl(QObject* parent)
     : DocumentImpl{parent}, d{new Private}
 {
     setDocumentType(DocumentType::Spreadsheet);
 }
 
 SpreadsheetImpl::~SpreadsheetImpl()
 {
     delete d;
 }
 
 bool SpreadsheetImpl::load(const QUrl& url)
 {
     delete d->part;
     delete d->document;
 
     d->part = new Calligra::Sheets::Part{this};
     d->document = new Calligra::Sheets::Doc{d->part};
     setKoDocument(d->document);
     d->part->setDocument(d->document);
 
     bool retval = d->document->openUrl(url);
 
     d->canvas = static_cast<Calligra::Sheets::CanvasItem*>(d->part->canvasItem(d->document));
 
     createAndSetCanvasController(d->canvas);
     createAndSetZoomController(d->canvas);
     connect(d->canvas, &Calligra::Sheets::CanvasItem::documentSizeChanged, this, &SpreadsheetImpl::updateDocumentSize);
 
     auto sheet = d->document->map()->sheet(0);
     if(sheet) {
         updateDocumentSize(sheet->documentSize().toSize());
     }
 
     setCanvas(d->canvas);
 
     d->updateLinkTargets();
 
     return retval;
 }
 
 int SpreadsheetImpl::currentIndex()
 {
     if (d->document && d->document->map() && d->canvas->activeSheet()) {
         return d->document->map()->indexOf(d->canvas->activeSheet());
     } else {
         return -1;
     }
 }
 
 void SpreadsheetImpl::setCurrentIndex(int newValue)
 {
     if(newValue != currentIndex()) {
         d->canvas->setActiveSheet(d->document->map()->sheet(newValue));
         d->updateLinkTargets();
         emit currentIndexChanged();
     }
 }
 
 void SpreadsheetImpl::updateDocumentSize(const QSize& size)
 {
     QRectF activeRect = d->canvas->viewConverter()->documentToView(d->canvas->activeSheet()->cellCoordinatesToDocument(d->canvas->activeSheet()->usedArea(true)));
     zoomController()->setDocumentSize(activeRect.size(), false);
     setDocumentSize(activeRect.size().toSize());
 }
 
 int SpreadsheetImpl::indexCount() const
 {
     return d->document->map()->count();
 }
 
 QUrl SpreadsheetImpl::urlAtPoint(QPoint point)
 {
     for( const QPair< QRectF, QUrl >& link : d->links )
     {
         QRectF hitTarget{
             link.first.x() - Private::wiggleFactor,
             link.first.y() - Private::wiggleFactor,
             link.first.width() + Private::wiggleFactor * 2,
             link.first.height() + Private::wiggleFactor * 2
         };
 
         if( hitTarget.contains( point ) )
         {
             return link.second;
         }
     }
     return QUrl();
 }
 
 QObject* SpreadsheetImpl::part() const
 {
     return d->part;
 }
diff --git a/components/impl/TextDocumentImpl.cpp b/components/impl/TextDocumentImpl.cpp
index 5dd7f881aff..63cd5a9b03a 100644
--- a/components/impl/TextDocumentImpl.cpp
+++ b/components/impl/TextDocumentImpl.cpp
@@ -1,311 +1,313 @@
 /*
  * This file is part of the KDE project
  *
  * Copyright (C) 2013 Arjen Hiemstra <ahiemstra@heimr.nl>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "TextDocumentImpl.h"
 
 #include <QGraphicsWidget>
 
 #include <kactioncollection.h>
 
 #include <KWPart.h>
 #include <KWDocument.h>
 #include <KWCanvasItem.h>
 #include <KoColumns.h>
 #include <KoFindText.h>
 #include <KoShape.h>
 #include <KoShapeContainer.h>
 #include <KoToolManager.h>
 #include <KoUnit.h>
 #include <KoZoomController.h>
 #include <KoZoomHandler.h>
 
 #include <QTextDocument>
 #include <QTextFrame>
 #include <QTextLayout>
 #include <QDebug>
 #include <QPointer>
 #include <QUrlQuery>
 
 #include "ComponentsKoCanvasController.h"
 #include <libs/textlayout/KoTextShapeData.h>
 
 using namespace Calligra::Components;
 
 class TextDocumentImpl::Private
 {
 public:
     Private() : part{nullptr}, document{nullptr}
     { }
 
     QPointer<KWPart> part;
     QPointer<KWDocument> document;
     QPointer<KWCanvasItem> canvas;
     QTimer indexChangedDelay;
 
     QList< QPair< QRectF, QUrl > > links;
 
     QList<KoShape*> deepShapeFind(QList<KoShape*> shapes)
     {
         QList<KoShape*> allShapes;
         foreach(KoShape* shape, shapes) {
             allShapes.append(shape);
             KoShapeContainer *container = dynamic_cast<KoShapeContainer*>(shape);
             if(container) {
                 allShapes.append(deepShapeFind(container->shapes()));
             }
         }
         return allShapes;
     }
 
     void updateLinkTargets()
     {
         links.clear();
 
         if(!canvas || !canvas->shapeManager())
             return;
 
         foreach(const KoShape* shape, canvas->shapeManager()->shapes()) {
             if(!shape->hyperLink().isEmpty()) {
                 QRectF rect = shape->boundingRect();
-                while(KoShapeContainer* parent = shape->parent()) {
+                for (KoShapeContainer* parent = shape->parent();
+                     parent; parent = parent->parent()) {
                     rect.translate(parent->position());
                 }
                 links.append(QPair<QRectF, QUrl>(rect, QUrl(shape->hyperLink())));
             }
         }
 
         QList<QTextDocument*> texts;
         KoFindText::findTextInShapes(canvas->shapeManager()->shapes(), texts);
         QList<KoShape*> allShapes = deepShapeFind(canvas->shapeManager()->shapes());
         foreach(QTextDocument* text, texts) {
             QTextBlock block = text->rootFrame()->firstCursorPosition().block();
             for (; block.isValid(); block = block.next()) {
                 block.begin();
                 QTextBlock::iterator it;
                 for (it = block.begin(); !(it.atEnd()); ++it) {
                     QTextFragment fragment = it.fragment();
                     if (fragment.isValid()) {
                         QTextCharFormat format = fragment.charFormat();
                         if(format.isAnchor()) {
                             // This is an anchor, store target and position...
                             QRectF rect = getFragmentPosition(block, fragment);
                             foreach(KoShape* shape, allShapes) {
                                 KoTextShapeData *shapeData = dynamic_cast<KoTextShapeData*>(shape->userData());
                                 if (!shapeData)
                                     continue;
                                 if(shapeData->document() == text)
                                 {
                                     rect.translate(shape->position());
-                                    while(KoShapeContainer* parent = shape->parent()) {
+                                    for (KoShapeContainer* parent = shape->parent();
+                                         parent; parent = parent->parent()) {
                                         rect.translate(parent->position());
                                     }
                                     break;
                                 }
                             }
                             KWPage page = document->pageManager()->page(rect.top());
                             //rect.translate(page.rightMargin(), page.topMargin());
                             //rect = canvas->viewConverter()->documentToView(rect);
                             //rect.translate(0, page.pageNumber() * (page.topMargin() + page.bottomMargin()) + 20);
                             rect.translate(0, (page.pageNumber() - 1) * (page.topMargin() + 20));
                             links.append(QPair<QRectF, QUrl>(canvas->viewConverter()->documentToView(rect), QUrl(format.anchorHref())));
                         }
                     }
                 }
             }
         }
         qDebug() << links;
     }
 
     QRectF getFragmentPosition(QTextBlock block, QTextFragment fragment)
     {
         // TODO this only produces a position for the first part, if the link spans more than one line...
         // Need to sort that somehow, unfortunately probably by slapping this code into the above function.
         // For now leave it like this, more important things are needed.
         QTextLayout* layout = block.layout();
         QTextLine line = layout->lineForTextPosition(fragment.position() - block.position());
         if(!line.isValid())
         {
             // fragment has no valid position and consequently no line...
             return QRectF();
         }
         qreal top = line.position().y() + (line.height() / 2);
         qreal bottom = top + line.height();
         qreal left = line.cursorToX(fragment.position() - block.position());
         qreal right = line.cursorToX((fragment.position() - block.position()) + fragment.length());
         QRectF fragmentPosition(QPointF(left, top), QPointF(right, bottom));
         return fragmentPosition.adjusted(layout->position().x(), layout->position().y(), 0, 0);
     }
 
     static const float wiggleFactor;
 };
 
 const float Calligra::Components::TextDocumentImpl::Private::wiggleFactor{ 4.f };
 
 TextDocumentImpl::TextDocumentImpl(QObject* parent)
     : DocumentImpl{parent}, d{new Private}
 {
     setDocumentType(DocumentType::TextDocument);
     d->indexChangedDelay.setInterval(0);
     connect(&d->indexChangedDelay, SIGNAL(timeout()), this, SIGNAL(currentIndexChanged()));
 }
 
 TextDocumentImpl::~TextDocumentImpl()
 {
     delete d;
 }
 
 bool TextDocumentImpl::load(const QUrl& url)
 {
     delete d->part;
     delete d->document;
 
     d->part = new KWPart{this};
     d->document = new KWDocument{d->part};
     setKoDocument(d->document);
     d->part->setDocument(d->document);
 
     d->document->setAutoSave(0);
     d->document->setCheckAutoSaveFile(false);
 
     bool retval = false;
     if (url.scheme() == QStringLiteral("newfile")) {
         QUrlQuery query(url);
 
         d->document->initEmpty();
         KWPageStyle style = d->document->pageManager()->defaultPageStyle();
         Q_ASSERT(style.isValid());
 
         KoColumns columns;
         columns.count = query.queryItemValue("columncount").toInt();
         columns.gapWidth = query.queryItemValue("columngap").toDouble();
         style.setColumns(columns);
 
         KoPageLayout layout = style.pageLayout();
         layout.format = KoPageFormat::formatFromString(query.queryItemValue("pageformat"));
         layout.orientation = (KoPageFormat::Orientation)query.queryItemValue("pageorientation").toInt();
         layout.height = MM_TO_POINT(query.queryItemValue("height").toDouble());
         layout.width = MM_TO_POINT(query.queryItemValue("width").toDouble());
         if (query.queryItemValue("facingpages").toInt() == 1) {
             layout.bindingSide = MM_TO_POINT(query.queryItemValue("leftmargin").toDouble());
             layout.pageEdge = MM_TO_POINT(query.queryItemValue("rightmargin").toDouble());
             layout.leftMargin = layout.rightMargin = -1;
         }
         else {
             layout.bindingSide = layout.pageEdge = -1;
             layout.leftMargin = MM_TO_POINT(query.queryItemValue("leftmargin").toDouble());
             layout.rightMargin = MM_TO_POINT(query.queryItemValue("rightmargin").toDouble());
         }
         layout.topMargin = MM_TO_POINT(query.queryItemValue("topmargin").toDouble());
         layout.bottomMargin = MM_TO_POINT(query.queryItemValue("bottommargin").toDouble());
         style.setPageLayout(layout);
 
         d->document->setUnit(KoUnit::fromSymbol(query.queryItemValue("unit")));
         d->document->relayout();
         retval = true;
     }
     else if (url.scheme() == QStringLiteral("template")) {
         // Nip away the manually added template:// bit of the uri passed from the caller
         bool ok = d->document->loadNativeFormat(url.toString().mid(11));
         d->document->setModified(false);
         d->document->undoStack()->clear();
 
         if (ok) {
             QString mimeType = QMimeDatabase().mimeTypeForUrl(url).name();
             // in case this is a open document template remove the -template from the end
             mimeType.remove( QRegExp( "-template$" ) );
             d->document->setMimeTypeAfterLoading(mimeType);
             d->document->resetURL();
             d->document->setEmpty();
         } else {
             d->document->showLoadingErrorDialog();
             d->document->initEmpty();
         }
         retval = true;
     }
     else {
         retval = d->document->openUrl(url);
     }
     qDebug() << "Attempting to open" << url << "and our success was" << retval;
 
     d->canvas = static_cast<KWCanvasItem*>(d->part->canvasItem(d->document));
 
     createAndSetCanvasController(d->canvas);
     createAndSetZoomController(d->canvas);
     zoomController()->setPageSize(d->document->pageManager()->begin().rect().size());
     connect(d->canvas, SIGNAL(documentSize(QSizeF)), zoomController(), SLOT(setDocumentSize(QSizeF)));
 
     d->canvas->updateSize();
 
     setCanvas(d->canvas);
     connect(canvasController()->proxyObject, SIGNAL(moveDocumentOffset(QPoint)), &d->indexChangedDelay, SLOT(start()));
 
     d->updateLinkTargets();
 
     return retval;
 }
 
 int TextDocumentImpl::currentIndex()
 {
     if(!d->canvas || !d->canvas->viewConverter()) {
         return 0;
     }
     QPointF newPoint = d->canvas->viewConverter()->viewToDocument(canvasController()->documentOffset());
     KWPage page = d->document->pageManager()->page(newPoint.y());
     return page.pageNumber();
 }
 
 void TextDocumentImpl::setCurrentIndex(int newValue)
 {
     KWPage newPage = d->document->pageManager()->page(newValue + 1);
     QRectF newRect = d->canvas->viewConverter()->documentToView(newPage.rect());
     canvasController()->setScrollBarValue(newRect.topLeft().toPoint());
     emit requestViewUpdate();
     emit currentIndexChanged();
 }
 
 int TextDocumentImpl::indexCount() const
 {
     return d->document->pageCount();
 }
 
 QUrl TextDocumentImpl::urlAtPoint(QPoint point)
 {
     qDebug() << Q_FUNC_INFO << point + (d->canvas->documentOffset() / zoomController()->zoomAction()->effectiveZoom());
     for( const QPair< QRectF, QUrl >& link : d->links )
     {
         QRectF hitTarget{
             link.first.x() - Private::wiggleFactor,
             link.first.y() - Private::wiggleFactor,
             link.first.width() + Private::wiggleFactor * 2,
             link.first.height() + Private::wiggleFactor * 2
         };
 
         if( hitTarget.contains( point + (d->canvas->documentOffset() / zoomController()->zoomAction()->effectiveZoom()) ) )
             return link.second;
     }
     return QUrl();
 }
 
 QObject* TextDocumentImpl::part() const
 {
     return d->part;
 }