From 887c2cd696c54af0cd6fdd54950d006626afeacc Mon Sep 17 00:00:00 2001 From: Arno Date: Sun, 30 Sep 2012 04:34:22 +0200 Subject: Foremost a fix for SmTreeModel Not working again, but I eventually have to commit the changes. Fixes to SmTreeModel: * Fix SmTreeModel::index(). The previous comment was quite valid. I'm surprised that it worked at all. I have no clue why to return an invalid QModelIndex if the column isn't 0. Now an index with any valid column number can be created. * Fix SmTreeModel::parent(). Again, why shouldn't we create a parent index with a column other than 0? No idea... * Fix SmTreeModel::headerData(). Add some sanity checks. * Fix SmTreeModel::findRecursive(). Well, what is there to say. It never worked for models with a depth > 1, but obviously it didn't really matter until now. To make it work I had to change SmTreeItem as well. SmTreeItem::next() returns the next valid parent/sibling, or 0 if there isn't one. There may be some fallout from these changes, but they're yet to be seen. Changes to PictureView: * fix selecting an item according to the new datasbase layout * same goes for editing items. If an update actually works has to be checked. Overall, it's an intermediate commit that should have been a sane series of commits. Can't be changed now... --- smtreeitem.cpp | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'smtreeitem.cpp') diff --git a/smtreeitem.cpp b/smtreeitem.cpp index ed9f73e..ab5e636 100644 --- a/smtreeitem.cpp +++ b/smtreeitem.cpp @@ -42,6 +42,17 @@ SmTreeItem *SmTreeItem::child(int row) const{ return 0; } +SmTreeItem *SmTreeItem::next() const{ + int pos = row(); + if(pos + 1 < parent()->childCount()){ + return parent()->child(pos + 1); + } + if(parent() == 0){ + return 0; + } + return parent()->next(); +} + int SmTreeItem::childCount() const{ return mChildren.count(); } -- cgit v1.2.3-70-g09d2