From 83a0174ef20d054eaed7e06f4e41788e0866839d Mon Sep 17 00:00:00 2001 From: frosch Date: Sun, 30 Jun 2013 14:37:50 +0000 Subject: (svn r25540) -Fix: Unify selecting a new object class in the object GUI. (sbr) --- src/object_gui.cpp | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/src/object_gui.cpp b/src/object_gui.cpp index 4c1a47aa6..4efca0eaa 100644 --- a/src/object_gui.cpp +++ b/src/object_gui.cpp @@ -83,9 +83,9 @@ public: NWidgetMatrix *matrix = this->GetWidget(WID_BO_SELECT_MATRIX); matrix->SetScrollbar(this->GetScrollbar(WID_BO_SELECT_SCROLL)); - matrix->SetCount(ObjectClass::Get(_selected_object_class)->GetUISpecCount()); if (this->CanRestoreSelectedObject()) { + this->SelectOtherClass(_selected_object_class), this->SelectOtherObject(_selected_object_index); } else { this->SelectFirstAvailableObject(true); @@ -328,6 +328,16 @@ public: } } + /** + * Select the specified object class. + * @param object_class_index Object class index to select. + */ + void SelectOtherClass(ObjectClassID object_class_index) + { + _selected_object_class = object_class_index; + this->GetWidget(WID_BO_SELECT_MATRIX)->SetCount(ObjectClass::Get(_selected_object_class)->GetUISpecCount()); + } + /** * Select the specified object in #_selected_object_class class. * @param object_index Object index to select, \c -1 means select nothing. @@ -373,8 +383,7 @@ public: int num_clicked = this->vscroll->GetPosition() + (pt.y - this->nested_array[widget]->pos_y) / this->line_height; if (num_clicked >= (int)ObjectClass::GetUIClassCount()) break; - _selected_object_class = ObjectClass::GetUIClass(num_clicked); - this->GetWidget(WID_BO_SELECT_MATRIX)->SetCount(ObjectClass::Get(_selected_object_class)->GetUISpecCount()); + this->SelectOtherClass(ObjectClass::GetUIClass(num_clicked)); this->SelectFirstAvailableObject(false); break; } @@ -421,8 +430,7 @@ public: for (uint i = 0; i < objclass->GetSpecCount(); i++) { const ObjectSpec *spec = objclass->GetSpec(i); if (spec->IsAvailable()) { - _selected_object_class = j; - this->GetWidget(WID_BO_SELECT_MATRIX)->SetCount(ObjectClass::Get(_selected_object_class)->GetUISpecCount()); + this->SelectOtherClass(j); this->SelectOtherObject(i); return; } @@ -434,7 +442,7 @@ public: /* ... but make sure that the class is not empty. */ for (ObjectClassID j = OBJECT_CLASS_BEGIN; j < OBJECT_CLASS_MAX; j++) { if (ObjectClass::Get(j)->GetUISpecCount() > 0) { - _selected_object_class = j; + this->SelectOtherClass(j); break; } } -- cgit v1.2.3-70-g09d2