From 8ee3363dbcc4d2e9e9f89b2737d26fcea55653a7 Mon Sep 17 00:00:00 2001 From: Peter Penz Date: Sun, 18 Nov 2007 13:24:43 +0000 Subject: fix crash for the following use case: - open a view with no selection - press SHIFT + right mouse-click on an item - a context menu will be opened although no item has been selected -> assertion gets triggered later svn path=/trunk/KDE/kdebase/apps/; revision=738255 --- src/dolphincolumnwidget.cpp | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'src/dolphincolumnwidget.cpp') diff --git a/src/dolphincolumnwidget.cpp b/src/dolphincolumnwidget.cpp index 2dc3bc153..7681b8838 100644 --- a/src/dolphincolumnwidget.cpp +++ b/src/dolphincolumnwidget.cpp @@ -309,12 +309,14 @@ void DolphinColumnWidget::contextMenuEvent(QContextMenuEvent* event) QListView::contextMenuEvent(event); - const QModelIndex index = indexAt(event->pos()); - if (index.isValid() || m_active) { - // Only open a context menu above an item or if the mouse is above - // the active column. - const QPoint pos = m_view->viewport()->mapFromGlobal(event->globalPos()); - m_view->m_controller->triggerContextMenuRequest(pos); + if (selectionModel()->hasSelection()) { + const QModelIndex index = indexAt(event->pos()); + if (index.isValid() || m_active) { + // Only open a context menu above an item or if the mouse is above + // the active column. + const QPoint pos = m_view->viewport()->mapFromGlobal(event->globalPos()); + m_view->m_controller->triggerContextMenuRequest(pos); + } } } -- cgit v1.3