Simplify catalog and product

1 files changed, 18 insertions, 37 deletions

diff --git a/catalog.cxx b/catalog.cxx
index 271dedd..e978858 100644
--- a/catalog.cxx
+++ b/catalog.cxx
@@ -5,11 +5,15 @@
 namespace QICClient {
 
 	Catalog::Catalog(icclient_catalog* catalog, QObject* parent) :
-		m_data{catalog},
-		QAbstractListModel{parent}
+		QAbstractListModel{parent},
+		m_data{catalog}
 	{
-		for (size_t i = 0; i < catalog->length; i++)
-			addProduct(Product{catalog->products[i]});
+		for (size_t i = 0; i < catalog->length; i++) addProduct(Product{catalog->products[i]});
+	}
+
+	Catalog::~Catalog()
+	{
+		icclient_free_catalog(m_data);
 	}
 
 	int Catalog::rowCount(QModelIndex const& parent) const
@@ -21,9 +25,7 @@ namespace QICClient {
 	QVariant Catalog::data(QModelIndex const& index, int role) const
 	{
 		auto row = index.row();
-
 		if (row < 0 || row >= products.count()) return QVariant();
-
 		auto product = products[row];
 		switch (role) {
 			case Product::SkuRole:
@@ -54,16 +56,16 @@ namespace QICClient {
 	QHash<int, QByteArray> Catalog::roleNames() const
 	{
 		return QHash<int, QByteArray>{
-			{Product::SkuRole, "sku"}
-			, {Product::DescriptionRole, "description"}
-			, {Product::CommentRole, "comment"}
-			, {Product::ThumbRole, "thumb"}
-			, {Product::ImageRole, "image"}
-			, {Product::PriceRole, "price"}
-			, {Product::ProdGroupRole, "prodGroup"}
-			, {Product::WeightRole, "weight"}
-			, {Product::AuthorRole, "author"}
-			, {Product::CrossSellRole, "crossSell"}
+			{ Product::SkuRole, "sku" },
+			{ Product::DescriptionRole, "description" },
+			{ Product::CommentRole, "comment" },
+			{ Product::ThumbRole, "thumb" },
+			{ Product::ImageRole, "image" },
+			{ Product::PriceRole, "price" },
+			{ Product::ProdGroupRole, "prodGroup" },
+			{ Product::WeightRole, "weight" },
+			{ Product::AuthorRole, "author" },
+			{ Product::CrossSellRole, "crossSell" }
 		};
 	}
 
@@ -74,25 +76,4 @@ namespace QICClient {
 		endInsertRows();
 	}
 
-	void Catalog::update(Catalog* catalog)
-	{
-		for (int i = 0; i < catalog->rowCount(); i++) {
-			auto index = catalog->index(i, 0);
-			Product product;
-			product.sku = catalog->data(index, Product::SkuRole).toString();
-			product.description = catalog->data(index, Product::DescriptionRole).toString();
-			product.comment = catalog->data(index, Product::CommentRole).toString();
-			product.thumb = catalog->data(index, Product::ThumbRole).toString();
-			product.image = catalog->data(index, Product::ImageRole).toString();
-			product.price = catalog->data(index, Product::PriceRole).toDouble();
-			product.prodGroup = catalog->data(index, Product::ProdGroupRole).toString();
-			product.weight = catalog->data(index, Product::WeightRole).toDouble();
-			product.author = catalog->data(index, Product::AuthorRole).toString();
-			product.crossSell = catalog->data(index, Product::CrossSellRole).toStringList();
-			addProduct(product);
-		}
-		this->m_data = const_cast<icclient_catalog*>(catalog->constData());
-		emit updated();
-	}
-
 }