From aad000e85e83fd297efec70c187b1d2246c145b3 Mon Sep 17 00:00:00 2001
From: Samuel Fadel <samuelfadel@gmail.com>
Date: Thu, 28 Jan 2016 11:20:13 +0100
Subject: BarChart: bugfix related to brushing & no. of elems.

---
 barchart.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'barchart.cpp')

diff --git a/barchart.cpp b/barchart.cpp
index 195b706..1aacfdb 100644
--- a/barchart.cpp
+++ b/barchart.cpp
@@ -37,7 +37,6 @@ BarChart::BarChart(QQuickItem *parent)
     setClip(true);
     setFlag(QQuickItem::ItemHasContents);
     //setAcceptedMouseButtons(Qt::LeftButton);
-    setAcceptHoverEvents(true);
 }
 
 BarChart::~BarChart()
@@ -55,6 +54,7 @@ void BarChart::setValues(const arma::vec &values)
 
     m_originalIndices.resize(m_values.n_elem);
     m_currentIndices.resize(m_values.n_elem);
+    setAcceptHoverEvents(m_values.n_elem > 0);
     if (m_values.n_elem > 0) {
         m_scale.setDomain(m_values.min(), m_values.max());
         m_colorScale.setExtents(m_values.min(), m_values.max());
@@ -105,6 +105,10 @@ void BarChart::brushItem(int item)
         m_brushedItem = item;
         emit itemBrushed(m_brushedItem);
     } else {
+        if (m_values.n_elem == 0 || item > m_values.n_elem - 1) {
+            return;
+        }
+
         m_brushedItem = m_currentIndices[item];
         emit itemBrushed(m_originalIndices[m_brushedItem]);
     }
-- 
cgit v1.2.3